Closed Bug 1203963 Opened 6 years ago Closed 6 years ago
Missing tests in syntax-error-illegal-character
js/src/jit-test/basic/syntax-error-illegal-character.js - There is no call to the test_syntax() function in this file. js/src/jit-test/asm.js/syntax-error-illegal-character.js - This file is empty except for the |jit-test| line. http://hg.mozilla.org/mozilla-central/rev/4f87d8225c7a
Thank you for catching this. js/src/jit-test/asm.js/syntax-error-illegal-character.js should be removed, all tests there were merged to js/src/jit-test/lib/syntax.js. I'll prepare patch shortly.
Now Reflect.parse requires module option to parse export/import, so added it to test for export/import.
Assignee: nobody → arai.unmht
Attachment #8660002 - Flags: review?(jwalden+bmo)
I overlooked those changes while rebasing. Moved to jit-test/tests/parser as others are there. Green on try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=07c0b5130452
Attachment #8660004 - Flags: review?(jwalden+bmo)
Removed test_no_* function and changed to pass options directly :)
Attachment #8661459 - Flags: review?(jwalden+bmo) → review+
Attachment #8660004 - Flags: review?(jwalden+bmo) → review+
You need to log in before you can comment on or make changes to this bug.