Khronos public bugtracker – Bug 332
small WebGL test harness improvements
Last modified: 2010-07-30 15:10:10 PDT
Created attachment 46 [details]
some webgl test harness improvements
The attached patch does a couple of things:
* it makes the TestHarness constructor actually honor its filelistUrl parameter. Currently, this is ignored as the value 'test_list.xt' was hardcoded there (but was also passed by the test runner!)
* it renames test_list.txt to 00_test_list.txt as is already the filename used in the subdirectories. My motivation for that is that in Mozilla we are using Mochikit for our test framework, and it interpretes any file starting by test_ ... as a test. So please consider either this renaming or any other renaming of your choice, this would be very helpful to us.
* it adds the ability to configure the timeout delay for the tests, by calling setTimeoutDelay(). Currently, this is hard-coded to 3 seconds.
Is it OK to commit?
Created attachment 47 [details]
some webgl test harness improvements (second try)
Sorry, the first version of the patch was missing webgl-test-harness.js.
This patch still doesn't seem to show the file rename correctly, just trust me that it's renaming the file and not deleting it as it looks like :-)
Here's the output of svn st:
A + 00_test_list.txt
These changes look fine to me.
Thanks for the quick reply! committing.