[Piglit] [PATCH 07/13] Make a plain_test() helper like concurrent_test() vs add_concurrent_test().

Eric Anholt eric at anholt.net
Tue Oct 18 14:15:25 PDT 2011


On Mon, 17 Oct 2011 10:15:42 -0700, Paul Berry <stereotype441 at gmail.com> wrote:
Non-text part: multipart/alternative
> On 15 October 2011 13:19, Eric Anholt <eric at anholt.net> wrote:
> > diff --git a/tests/all.tests b/tests/all.tests
> > index e095e2c..d7d1e11 100644
> > --- a/tests/all.tests
> > +++ b/tests/all.tests
> > @@ -25,6 +25,12 @@ generatedTestDir = os.path.join(
> >        'generated_tests')
> >
> >  # Quick wrapper for PlainExecTest for our usual concurrent args.
> > +def plain_test(name):
> > +       return PlainExecTest([name, '-auto'])
> > +
> > +def add_plain_test(group, name):
> > +       group[name] = PlainExecTest([name, '-auto'])
> > +
> >
> 
> Did you mean to do this?
> 
> def add_plain_test(group, name):
>     group[name] = plain_test(name)

Yeah, I moved it from framework where it was kind of lonely.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20111018/24e33756/attachment.pgp>


More information about the Piglit mailing list