diff options
author | David Fugate <dfugate@microsoft.com> | 2010-11-16 15:43:29 -0800 |
---|---|---|
committer | David Fugate <dfugate@microsoft.com> | 2010-11-16 15:43:29 -0800 |
commit | 42b884b7baa17d2413515a8ad2cae0e672d33f43 (patch) | |
tree | e2daa34fea6272672daae1d0eac31bbacf003282 /tools | |
parent | 3fedec41306680e2b38c588b4e42a44b59d53faf (diff) |
TestCasePackager.ps1 had problems dealing with test case files that had valid test code
prior to the usage of "ES5Harness". Specifically in cases like:
var abc = ...;
ES5Harness.registerTest( {
//...abc gets used here
the declaration of "abc" would've been ripped out thereby corrupting the test.
This change fixes the issue for all existing IE Test Center and Sputnik test cases. Note
however that it could potentially fail for lines like this: "/*abc*/ /*". If we ever add
cases including comments like this, the implementation of isTestStarted will need to be
revisited.
There were five such test cases that were being affected by this issue in IE Test
Center's chapter 10 and 15 tests. These have all been fixed via the updated script.
Diffstat (limited to 'tools')
-rw-r--r-- | tools/TestCaseHTMLPackager/TestCasePackager.ps1 | bin | 12718 -> 14644 bytes |
1 files changed, 0 insertions, 0 deletions
diff --git a/tools/TestCaseHTMLPackager/TestCasePackager.ps1 b/tools/TestCaseHTMLPackager/TestCasePackager.ps1 Binary files differindex 0727636ac..9f03d8306 100644 --- a/tools/TestCaseHTMLPackager/TestCasePackager.ps1 +++ b/tools/TestCaseHTMLPackager/TestCasePackager.ps1 |