Added .spec.js extension to test files

Author: JamesMessinger

test/mocha.opts

@@ -1,3 +1,3 @@
 --bail
 test/fixtures/*.js
-test/specs/*.js
+test/specs/*.spec.js

test/specs/best-practices.js renamed to test/specs/best-practices.spec.js

@@ -24,10 +24,9 @@ describe("best-practices", function () {
   });
 
   it('should require the "use strict" pragma at the global level for CommonJS', function () {
-    let results = ESLint.run(["modular/best-practices", "modular/common-js"], "function foo () {} foo();");
+    let results = ESLint.run(["modular/best-practices", "modular/modules/cjs"], "function foo () {} foo();");
     results.errorCount.should.equal(1);
     results.rules.should.deep.equal(["strict"]);
     results.messages[0].message.should.equal("Use the global form of 'use strict'.");
   });
 });
-

test/specs/browser.js renamed to test/specs/browser.spec.js

@@ -24,7 +24,7 @@ describe("browser", function () {
   });
 
   it('should require the "use strict" pragma at the global level if the CommonJS module is loaded after', function () {
-    let results = ESLint.run(["modular/browser", "modular/common-js"], "function foo () {} foo();");
+    let results = ESLint.run(["modular/browser", "modular/modules/cjs"], "function foo () {} foo();");
     results.errorCount.should.equal(1);
     results.rules.should.deep.equal(["strict"]);
     results.messages[0].message.should.equal("Use the global form of 'use strict'.");

test/specs/common-js.js renamed to test/specs/common-js.spec.js

@@ -4,24 +4,23 @@ const ESLint = require("../fixtures/eslint");
 const chai = require("chai");
 chai.should();
 
-describe("common-js", function () {
+describe("modules/cjs", function () {
   it("should not be enforced if module is not used", function () {
     let results = ESLint.run("modular/style", "foo(); function foo () {}\n");
     results.errorCount.should.equal(0);
   });
 
   it('should require the "use strict" pragma at the global level', function () {
-    let results = ESLint.run("modular/common-js", "function foo () {} foo();");
+    let results = ESLint.run("modular/modules/cjs", "function foo () {} foo();");
     results.errorCount.should.equal(1);
     results.rules.should.deep.equal(["strict"]);
     results.messages[0].message.should.equal("Use the global form of 'use strict'.");
   });
 
   it("should not allow ES6 module syntax", function () {
-    let results = ESLint.run("modular/common-js", "import foo from 'bar';");
+    let results = ESLint.run("modular/modules/cjs", "import foo from 'bar';");
     results.errorCount.should.equal(1);
     results.messages.should.have.lengthOf(1);
     results.messages[0].message.should.equal("Parsing error: The keyword 'import' is reserved");
   });
 });
-

test/specs/es5.js renamed to test/specs/es5.spec.js

@@ -4,7 +4,7 @@ const ESLint = require("../fixtures/eslint");
 const chai = require("chai");
 chai.should();
 
-describe("es6-modules", function () {
+describe("modules/esm", function () {
   it("should not be enforced if module is not used", function () {
     let results = ESLint.run("modular/es6",
       "const foo = 5;\n" +
@@ -14,28 +14,27 @@ describe("es6-modules", function () {
   });
 
   it('should not allow the "use strict" pragma', function () {
-    let results = ESLint.run("modular/es6-modules", '"use strict";');
+    let results = ESLint.run("modular/modules/esm", '"use strict";');
     results.errorCount.should.equal(1);
     results.rules.should.deep.equal(["strict"]);
     results.messages[0].message.should.equal("'use strict' is unnecessary inside of modules.");
   });
 
   it("should allow ES6 module syntax", function () {
     let results = ESLint.run(
-      ["modular/es6", "modular/es6-modules"],
+      ["modular/es6", "modular/modules/esm"],
       "import foo from 'bar';"
     );
     results.errorCount.should.equal(0);
   });
 
-  it('should not allow ES6 module syntax if followed by the "common-js" module', function () {
+  it('should not allow ES6 module syntax if followed by the "modules/cjs" module', function () {
     let results = ESLint.run(
-      ["modular/es6", "modular/es6-modules", "modular/common-js"],
+      ["modular/es6", "modular/modules/esm", "modular/modules/cjs"],
       "import foo from 'bar';"
     );
     results.errorCount.should.equal(1);
     results.messages.should.have.lengthOf(1);
     results.messages[0].message.should.equal("Parsing error: 'import' and 'export' may appear only with 'sourceType: module'");
   });
 });
-

test/specs/es6-modules.js renamed to test/specs/es6-modules.spec.js

@@ -13,7 +13,7 @@ describe("es6", function () {
     results.errorCount.should.equal(0);
   });
 
-  it('should not allow ES6 module syntax unless the "es6-modules" module is also used', function () {
+  it('should not allow ES6 module syntax unless the "modules/esm" module is also used', function () {
     let results = ESLint.run("modular/es6", "import foo from 'bar';");
     results.errorCount.should.equal(1);
     results.messages.should.have.lengthOf(1);
@@ -36,4 +36,3 @@ describe("es6", function () {
     results.rules.should.deep.equal(["object-shorthand"]);
   });
 });
-

test/specs/es6.js renamed to test/specs/es6.spec.js

@@ -14,11 +14,10 @@ describe("jsx", function () {
   });
 
   it("should require double quotes for JSX attributes", function () {
-    let results = ESLint.run(["modular/es6", "modular/jsx"],
+    let results = ESLint.run(["modular/es6", "modular/browser/jsx"],
       "let foo = <div className='some-class'>hello, world</div>;"
     );
     results.errorCount.should.equal(1);
     results.rules.should.deep.equal(["jsx-quotes"]);
   });
 });
-