Re: [PATCH] test: Add rudimentary support to run selected tests only

[Stefano Brivio <sbrivio@redhat.com>]

[-- Attachment #1: Type: text/plain, Size: 1027 bytes --]

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