From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefano Brivio To: passt-dev@passt.top Subject: Re: [PATCH 02/12] test: Remove unnecessary sleeps from layout functions Date: Sat, 24 Sep 2022 01:08:01 +0200 Message-ID: <20220924010801.255bbc43@elisabeth> In-Reply-To: <20220923072038.1667106-3-david@gibson.dropbear.id.au> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============6857856091375516485==" --===============6857856091375516485== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit 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. 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. -- Stefano --===============6857856091375516485==--