From 203b234fc0959fd6ed408eb6e238c769b6c5e8fe Mon Sep 17 00:00:00 2001 From: Mike Pennisi <mike@mikepennisi.com> Date: Sat, 28 May 2016 15:42:19 -0400 Subject: [PATCH] Reformat negative SyntaxError tests (runtime) These tests specifically concern error produced from the global scope, precluding the use of the `assert.throws` helper function. --- .../eval-code/direct/var-env-global-lex-non-strict.js | 4 +++- test/language/eval-code/indirect/parse-failure-2.js | 4 +++- .../module-code/eval-export-dflt-expr-err-get-value.js | 4 +++- test/language/module-code/instn-resolve-empty-export.js | 4 +++- test/language/module-code/instn-resolve-empty-import.js | 4 +++- test/language/module-code/instn-resolve-err-reference.js | 4 +++- test/language/module-code/instn-resolve-order-depth.js | 4 +++- test/language/module-code/instn-resolve-order-src.js | 4 +++- test/language/module-code/parse-err-reference.js | 4 +++- 9 files changed, 27 insertions(+), 9 deletions(-) diff --git a/test/language/eval-code/direct/var-env-global-lex-non-strict.js b/test/language/eval-code/direct/var-env-global-lex-non-strict.js index 364d4c5d59..5b87d9ed55 100644 --- a/test/language/eval-code/direct/var-env-global-lex-non-strict.js +++ b/test/language/eval-code/direct/var-env-global-lex-non-strict.js @@ -13,7 +13,9 @@ info: | 2. NOTE: eval will not create a global var declaration that would be shadowed by a global lexical declaration. [...] -negative: SyntaxError +negative: + phase: runtime + type: SyntaxError flags: [noStrict] features: [let] ---*/ diff --git a/test/language/eval-code/indirect/parse-failure-2.js b/test/language/eval-code/indirect/parse-failure-2.js index d04397fa83..e15f39dce5 100644 --- a/test/language/eval-code/indirect/parse-failure-2.js +++ b/test/language/eval-code/indirect/parse-failure-2.js @@ -5,7 +5,9 @@ info: If the parse fails, throw a SyntaxError exception (but see also clause 16) esid: sec-performeval es5id: 15.1.2.1_A2_T2 description: Checking if execution of "(0,eval)("x = 1; x\u000A++")" fails -negative: SyntaxError +negative: + phase: runtime + type: SyntaxError ---*/ var x; diff --git a/test/language/module-code/eval-export-dflt-expr-err-get-value.js b/test/language/module-code/eval-export-dflt-expr-err-get-value.js index 8d292118e3..0c397452ff 100644 --- a/test/language/module-code/eval-export-dflt-expr-err-get-value.js +++ b/test/language/module-code/eval-export-dflt-expr-err-get-value.js @@ -24,7 +24,9 @@ info: | 2. If Type(V) is not Reference, return V. 3. Let base be GetBase(V). 4. If IsUnresolvableReference(V) is true, throw a ReferenceError exception. -negative: ReferenceError +negative: + phase: runtime + type: ReferenceError flags: [module] ---*/ diff --git a/test/language/module-code/instn-resolve-empty-export.js b/test/language/module-code/instn-resolve-empty-export.js index c8309579e0..b770b99a45 100644 --- a/test/language/module-code/instn-resolve-empty-export.js +++ b/test/language/module-code/instn-resolve-empty-export.js @@ -29,7 +29,9 @@ info: | { } { ExportsList } { ExportsList , } -negative: ReferenceError +negative: + phase: early + type: ReferenceError flags: [module] ---*/ diff --git a/test/language/module-code/instn-resolve-empty-import.js b/test/language/module-code/instn-resolve-empty-import.js index 82bc3a240f..7bf6151eae 100644 --- a/test/language/module-code/instn-resolve-empty-import.js +++ b/test/language/module-code/instn-resolve-empty-import.js @@ -35,7 +35,9 @@ info: | { } { ImportsList } { ImportsList , } -negative: ReferenceError +negative: + phase: early + type: ReferenceError flags: [module] ---*/ diff --git a/test/language/module-code/instn-resolve-err-reference.js b/test/language/module-code/instn-resolve-err-reference.js index 10649df49c..2e7b1e8479 100644 --- a/test/language/module-code/instn-resolve-err-reference.js +++ b/test/language/module-code/instn-resolve-err-reference.js @@ -10,7 +10,9 @@ info: | [...] b. Let requiredModule be ? HostResolveImportedModule(module, required). [...] -negative: ReferenceError +negative: + phase: early + type: ReferenceError flags: [module] ---*/ diff --git a/test/language/module-code/instn-resolve-order-depth.js b/test/language/module-code/instn-resolve-order-depth.js index 855b06432d..01e03bfd54 100644 --- a/test/language/module-code/instn-resolve-order-depth.js +++ b/test/language/module-code/instn-resolve-order-depth.js @@ -3,7 +3,9 @@ /*--- description: Module dependencies are resolved following a depth-first strategy esid: sec-moduledeclarationinstantiation -negative: ReferenceError +negative: + phase: early + type: ReferenceError flags: [module] ---*/ diff --git a/test/language/module-code/instn-resolve-order-src.js b/test/language/module-code/instn-resolve-order-src.js index 0b1482b477..a30556b404 100644 --- a/test/language/module-code/instn-resolve-order-src.js +++ b/test/language/module-code/instn-resolve-order-src.js @@ -3,7 +3,9 @@ /*--- description: Modules dependencies are resolved in source text order esid: sec-moduledeclarationinstantiation -negative: ReferenceError +negative: + phase: early + type: ReferenceError flags: [module] ---*/ diff --git a/test/language/module-code/parse-err-reference.js b/test/language/module-code/parse-err-reference.js index 8521599c0e..9e8c492baa 100644 --- a/test/language/module-code/parse-err-reference.js +++ b/test/language/module-code/parse-err-reference.js @@ -3,7 +3,9 @@ /*--- description: Early ReferenceError resulting from module parsing esid: sec-parsemodule -negative: ReferenceError +negative: + phase: early + type: ReferenceError info: | [...] 2. Parse sourceText using Module as the goal symbol and analyze the parse -- GitLab