On 1 February 2012 16:06, Kenneth Graunke <span dir="ltr">&lt;<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If you drop a new .vpfp file into tests/shaders/generic, presumably<br>
it&#39;s a new vpfpgeneric test you&#39;d like tested.  Eliminates the fuss<br>
of having to enumerate each new test in all.tests explicitly.<br>
<br>
Signed-off-by: Kenneth Graunke &lt;<a href="mailto:kenneth@whitecape.org">kenneth@whitecape.org</a>&gt;<br>
---<br>
 tests/all.tests |   51 ++++++---------------------------------------------<br>
 1 files changed, 6 insertions(+), 45 deletions(-)<br>
<br>
diff --git a/tests/all.tests b/tests/all.tests<br>
index 5843080..c468abd 100644<br>
--- a/tests/all.tests<br>
+++ b/tests/all.tests<br>
@@ -502,51 +502,12 @@ add_plain_test(shaders, &#39;useshaderprogram-bad-type&#39;)<br>
 add_plain_test(shaders, &#39;useshaderprogram-bad-program&#39;)<br>
 add_plain_test(shaders, &#39;useshaderprogram-flushverts-1&#39;)<br>
<br>
-vpfpgeneric = Group()<br>
-def add_vpfpgeneric(name):<br>
-       vpfpgeneric[name] = PlainExecTest([&#39;vpfp-generic&#39;, &#39;-auto&#39;, testsDir + &#39;/shaders/generic/&#39; + name + &#39;.vpfp&#39;])<br>
-add_vpfpgeneric(&#39;arl&#39;)<br>
-add_vpfpgeneric(&#39;big-param&#39;)<br>
-add_vpfpgeneric(&#39;dataflow-bug&#39;)<br>
-add_vpfpgeneric(&#39;fdo30337a&#39;)<br>
-add_vpfpgeneric(&#39;fdo30337b&#39;)<br>
-add_vpfpgeneric(&#39;fdo38145&#39;)<br>
-add_vpfpgeneric(&#39;fogcoord-dp3&#39;)<br>
-add_vpfpgeneric(&#39;fogcoord-dph&#39;)<br>
-add_vpfpgeneric(&#39;fogcoord-dp4&#39;)<br>
-add_vpfpgeneric(&#39;fp-arb-fragment-coord-conventions-none&#39;)<br>
-add_vpfpgeneric(&#39;fp-arb-fragment-coord-conventions-integer&#39;)<br>
-add_vpfpgeneric(&#39;fp-cmp&#39;)<br>
-add_vpfpgeneric(&#39;fp-dst-aliasing-1&#39;)<br>
-add_vpfpgeneric(&#39;fp-dst-aliasing-2&#39;)<br>
-add_vpfpgeneric(&#39;fp-ex2-sat&#39;)<br>
-add_vpfpgeneric(&#39;fp-two-constants&#39;)<br>
-add_vpfpgeneric(&#39;nv-mov&#39;)<br>
-add_vpfpgeneric(&#39;nv-add&#39;)<br>
-add_vpfpgeneric(&#39;nv-arl&#39;)<br>
-add_vpfpgeneric(&#39;nv-init-zero-reg&#39;)<br>
-add_vpfpgeneric(&#39;nv-init-zero-addr&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array-huge&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array-huge-varying&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array-huge-offset&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array-huge-offset-neg&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array-huge-overwritten&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array-huge-relative-offset&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-constant-array-varying&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-env-array&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-local-array&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-neg-array&#39;)<br>
-add_vpfpgeneric(&#39;vp-arl-neg-array-2&#39;)<br>
-add_vpfpgeneric(&#39;vp-constant-array&#39;)<br>
-add_vpfpgeneric(&#39;vp-constant-array-huge&#39;)<br>
-add_vpfpgeneric(&#39;vp-constant-negate&#39;)<br>
-add_vpfpgeneric(&#39;vp-exp-alias&#39;)<br>
-add_vpfpgeneric(&#39;vp-max&#39;)<br>
-add_vpfpgeneric(&#39;vp-min&#39;)<br>
-add_vpfpgeneric(&#39;vp-sge-alias&#39;)<br>
-add_vpfpgeneric(&#39;vp-two-constants&#39;)<br>
-shaders[&#39;vpfp-generic&#39;] = vpfpgeneric<br>
+# Add the vpfpgeneric tests:<br>
+for shader in glob(path.join(&#39;tests&#39;, &#39;shaders&#39;, &#39;generic&#39;, &#39;*.vpfp&#39;)):<br>
+       test = PlainExecTest(&#39;vpfp-generic -auto &#39; + shader)<br>
+       test.runConcurrent = True<br>
+       name = path.splitext(path.relpath(shader, &#39;tests&#39;).replace(&#39;/generic&#39;, &#39;/vpfp-generic&#39;))[0]<br></blockquote><div><br>These two uses of &#39;tests&#39; should also be changed to testsDir.<br>
<br>With that change:<br><br>Reviewed-by: Paul Berry &lt;<a href="mailto:stereotype441@gmail.com">stereotype441@gmail.com</a>&gt;<br> </div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">

+       profile.test_list[name] = test<br>
<br>
 bugs = Group()<br>
 add_plain_test(bugs, &#39;crash-cubemap-order&#39;)<br>
<span class="HOEnZb"><font color="#888888">--<br>
1.7.7.6<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>
</font></span></blockquote></div><br>