605640 - Minor changes to test framework for ATS test export.

Status: VERIFIED FIXED

(Tamarin Graveyard :: Tools, defect)

Description

[Chris Peyer]

- runtests needs to be run with only 1 thread when export ATS swfs
- runtests doesn't always read in the testconfig ats_skip directive
- update testconfig for new tests
- rename tests that start with numbers: ATS needs tests to start with letters
- shell.as needs to have the print function defined

Comment 1

[Chris Peyer]

Attachment: ats changes

Hodgepodge of fixes for issues listed above.  Also has whitespace fixes as I mistakenly had my editor set to clear all trailing whitespace.

Comment 2

[Brent Baker]

Comment on attachment 487026 [details] [diff] [review]
ats changes

r+ with nits

Please do not commmit the whitespace change and your code change in one patch. Whitespace changes should be committed on there own and not with other code changes. I end up having to parse the entire patch (90% whitespace change) just to see what I should be ignoring as not a code change.

Create and reference a bug (watson or bugzilla) in the testconfig where tests are being skipped in ATS because there is currently no solution for testing versioning in the player. Also please change "soln" to "solution" :)

Comment 3

[Chris Peyer]

Pushed to redux with suggested changes:

http://hg.mozilla.org/tamarin-redux/rev/612dfaf51cce