In which test suite does your new test belong?
- jit-tests are intended to test the implementation of the JIT. Only add tests to these suites which test jit correctness or functionality.
- jstests is intended for tests of language-visible functionality. Please put tests of functionality into jstests even if related tests are in jit-tests, since jstests are closer to (and more easily converted to) test262 tests. (In fact, the test262 test suite is run as part of jstests.)
Practical differences between the two test suites:
- New jstest files should be put the code in the appropriate subdirectory of js/src/tests/non262/, or, under some scenarios, contributed directly to the test262 repository.
- jstests run in both the shell and the browser (although you can specify that the test should be run in only one of the two locations).
- jstests automatically load js/src/tests/shell.js before they run, which creates a ton of functions.
- Read more advice on jstests here.
- New jit-test files should be put in js/src/jit-test/tests/basic or one of the other appropriate subdirectories of jit-test/tests.
- jit-tests run only in the shell.
- jit-tests do not load extra test functionality automatically.
Have a look at the existing files and follow what they do. All tests, in both suite, can use the assertEq function.
assertEq(v1, v2[, message])
Check that v1 and v2 are the same value. If they're not, throw an exception (which will cause the test to fail).
If you are writing a jstests, additional testing functionality is provided for you in shell.js files. You can read about them here.
Do not attempt to test the performance of engine features in the test suite.
Do not add performance tests to the test suite.
It is not generally even possible to tell if the speed of any particular feature is going to be important in the real world without running a real-world benchmark. It is very hard to write a good real-world benchmark. For this reason, the best place to find out if a change is performance sensitive is on arewefastyet.com.
Focus on writing fast, light tests that cover a single feature. There is basically no cost to adding a new test, so add as many feature tests as needed to cover each feature orthogonally. Remember that whenever a test fails, someone -- probably you -- is going to have to figure out what went wrong.
Run your new test locally before checking it in (or posting it for review). Nobody likes patches that include failing tests!
It's also a good sanity check to run each new test against an unpatched shell or browser. The test should fail if it's working properly.
Tests are usually reviewed and pushed just like any other code change. Just include the test in your patch.
Security-sensitive tests should not be committed until the corresponding bug has been made public. Instead, ask a SpiderMonkey peer how to proceed.
It is OK under certain circumstances to push new tests to certain repositories without a code review. Don't do this unless you know what you're doing. Ask a SpiderMonkey peer for details.