From mboxrd@z Thu Jan  1 00:00:00 1970
From: Stefano Brivio <sbrivio@redhat.com>
To: passt-dev@passt.top
Subject: Re: [PATCH] test: Add rudimentary support to run selected tests only
Date: Fri, 07 Oct 2022 09:37:29 +0200
Message-ID: <20221007093729.5e20b10a@elisabeth>
In-Reply-To: <Yz/DLKxyuIrWwWSn@yekko>
MIME-Version: 1.0
Content-Type: multipart/mixed; boundary="===============4859795758285657632=="

--===============4859795758285657632==
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit

On Fri, 7 Oct 2022 17:11:56 +1100
David Gibson <david(a)gibson.dropbear.id.au> wrote:

> On Fri, Oct 07, 2022 at 02:47:32AM +0200, Stefano Brivio wrote:
> > To keep this simple, only support tests that have corresponding setup
> > and teardown functions implied by their path. For example:
> > 
> >   ./run passt/ndp
> > 
> > will trigger the 'passt' setup and teardown functions.
> > 
> > This is not really elegant, but it looks robust, and while David is
> > considering proper alternatives, it should be quite useful.
> > 
> > Signed-off-by: Stefano Brivio <sbrivio(a)redhat.com>  
> 
> I'm not sure I'd go so far as to call it "robust".  AFAICT it will
> break for cases where the setup function isn't named after the test
> dir (e.g. for perf/* which need the passt_in_ns setup).  And, of
> course, it's still the user's responsibility to handle inter-test
> dependencies (dhcp needs ndp to run first , tcp & udp need dhcp to run
> first , etc.)

Right, well, but I mentioned both facts in the README... ;)

-- 
Stefano


--===============4859795758285657632==--