<div dir="ltr"><div style>While I'm not against renaming all.tests;</div><div><br></div>We already have include/exclude switches in piglit, what about loading the include/exclude switches with blacklists, like with a --profile switch or something similar?<div>
<br></div><div style>Also python standard very strongly suggests not using `'from module import *'</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 2, 2013 at 2:18 PM, Tom Gall <span dir="ltr"><<a href="mailto:tom.gall@linaro.org" target="_blank">tom.gall@linaro.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">+1<br>
<br>
I've had a all_gles2.tests I've been maintaining in my own git repo<br>
which could easily evolve to use this. I'm sure I'm not the only<br>
person that would like to make use of this.<br>
<div class="HOEnZb"><div class="h5"><br>
On Thu, May 2, 2013 at 4:03 PM, Brian Paul <<a href="mailto:brianp@vmware.com">brianp@vmware.com</a>> wrote:<br>
><br>
> I'd like to rename tests/all.tests (and the others) to something like<br>
> tests/all_tests.py.<br>
><br>
> The contents of all.tests is Python code so putting the .py suffix on it<br>
> makes sense.  Then, I'd like to be able to import that code into another<br>
> test spec file where I do blacklisting.<br>
><br>
> For example, I want to create a test spec file "vmware_tests.py" that<br>
> executes everything in all.tests except for stuff we don't want to test.  An<br>
> example "vmware_tests.py" file might be something like:<br>
><br>
> import sys<br>
> sys.path.append("tests")<br>
> from all_tests import *<br>
><br>
> black_list= [<br>
>    "spec/!OpenGL 3.1/*",<br>
>    "*glean*"<br>
> ]<br>
><br>
> # Remove the blacklisted tests from the test profile<br>
> for t in black_list:<br>
>    profile.remove_test(t)<br>
><br>
><br>
><br>
> As it is now we can't do "import all.tests" in Python.  Renaming to<br>
> all_tests.py fixes that problem.<br>
><br>
> -Brian<br>
><br>
> _______________________________________________<br>
> Piglit mailing list<br>
> <a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
> <a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
<br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
Regards,<br>
Tom<br>
<br>
"Where's the kaboom!? There was supposed to be an earth-shattering<br>
kaboom!" Marvin Martian<br>
Tech Lead, Graphics Working Group | Linaro.org │ Open source software<br>
for ARM SoCs<br>
w) tom.gall att <a href="http://linaro.org" target="_blank">linaro.org</a><br>
h) tom_gall att <a href="http://mac.com" target="_blank">mac.com</a><br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Piglit mailing list<br>
<a href="mailto:Piglit@lists.freedesktop.org">Piglit@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/piglit" target="_blank">http://lists.freedesktop.org/mailman/listinfo/piglit</a><br>
</div></div></blockquote></div><br></div>