[Fix]: test cases of no-amd and no-commonjs

import-js · Aug 16, 2021 · d4739ca · d4739ca
1 parent 8be2ec2
commit d4739ca
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/tests/src/rules/no-amd.js b/tests/src/rules/no-amd.js
@@ -2,7 +2,7 @@ import { RuleTester } from 'eslint';
 import eslintPkg from 'eslint/package.json';
 import semver from 'semver';
 
-const ruleTester = new RuleTester();
+const ruleTester = new RuleTester({ parserOptions: { ecmaVersion: 2015, sourceType: 'module' } });
 
 ruleTester.run('no-amd', require('rules/no-amd'), {
   valid: [
@@ -25,6 +25,9 @@ ruleTester.run('no-amd', require('rules/no-amd'), {
     // unmatched arg types/number
     'define(0, 1, 2)',
     'define("a")',
+
+    // sourceType: 'script'
+    { code: 'define([], function(a) {})', parserOptions: { ecmaVersion: 2015, sourceType: 'script' } },
   ],
 
   invalid: semver.satisfies(eslintPkg.version, '< 4.0.0') ? [] : [

diff --git a/tests/src/rules/no-commonjs.js b/tests/src/rules/no-commonjs.js
@@ -5,7 +5,7 @@ import semver from 'semver';
 const EXPORT_MESSAGE = 'Expected "export" or "export default"';
 const IMPORT_MESSAGE = 'Expected "import" instead of "require()"';
 
-const ruleTester = new RuleTester();
+const ruleTester = new RuleTester({ parserOptions: { ecmaVersion: 2015, sourceType: 'module' } });
 
 ruleTester.run('no-commonjs', require('rules/no-commonjs'), {
   valid: [
@@ -63,6 +63,9 @@ ruleTester.run('no-commonjs', require('rules/no-commonjs'), {
     { code: 'if (typeof window !== "undefined") { require("x") }', options: [{ allowRequire: false }] },
 
     { code: 'try { require("x") } catch (error) {}' },
+
+    // sourceType: 'script'
+    { code: 'var x = require("x")', parserOptions: { ecmaVersion: 2015, sourceType: 'script' } },
   ],
 
   invalid: [