Re: [PATCH 02/12] test: Remove unnecessary sleeps from layout functions

On Sat, Sep 24, 2022 at 01:08:01AM +0200, Stefano Brivio wrote:

On Fri, 23 Sep 2022 17:20:28 +1000 David Gibson wrote:

...

These make sense for displaying in the demo, but not for automated testing.

Actually, they kind of do for the CI terminal captures: especially the sleep 3 before closing panes is helpful to easily pause on the results of the last test that was run there.

Also the sleep 1 after setting up the new layout gives me a bit more time to realise what's going on.

Hmm, I guess. I guess I find the idea of looking back at video of tests rather than logs a bit alien.

I know, perhaps it's not a very common use case, but on the other hand we're just saving 20 seconds altogether, it doesn't feel like it's that much right now.

I suppose. It is rather more significant when restricting the tests down to a smaller handful for debugging. -- David Gibson | I'll have my music baroque, and my code david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_ | _way_ _around_! http://www.ozlabs.org/~dgibson