Merge pull request #52 from Turbo87/simplify-tests

Simplify test code

Author: rwjblue

package.json

@@ -5,7 +5,6 @@
     "babel-preset-latest": "^6.24.0",
     "babel-register": "^6.22.0",
     "chai": "^3.5.0",
-    "chai-files": "^1.4.0",
     "mocha": "^3.2.0"
   },
   "name": "babel-plugin-debug-macros",

src/tests/debug-tools-test.js

@@ -3,8 +3,7 @@
 const DebugToolsPlugin = require('../index');
 const transform = require('babel-core').transform;
 const expect = require('chai').expect;
-const file = require('chai-files').file;
-const lstatSync = require('fs').lstatSync;
+const fs = require('fs');
 
 const presets = [["latest", {
   "es2015": false,
@@ -435,50 +434,42 @@ let cases = {
   },
 }
 
-function compile(source, transformOptions) {
-  return transform(source, transformOptions);
-}
-
 Object.keys(cases).forEach(caseName => {
   describe(caseName, () => {
     let ep = 0;
 
-    cases[caseName].fixtures.forEach(assertionName => {
-      if (cases[caseName].only) {
+    let testcase = cases[caseName];
+
+    testcase.fixtures.forEach(assertionName => {
+      if (testcase.only) {
         it.only(assertionName, () => {
-          test(caseName, cases, assertionName, ep);
+          test(testcase, assertionName, ep);
         });
-      } else if (cases[caseName].skip) {
+      } else if (testcase.skip) {
         it.skip(assertionName, () => {});
       } else {
         it(assertionName, () => {
-          test(caseName, cases, assertionName, ep);
+          test(testcase, assertionName, ep);
         });
       }
     });
   });
 });
 
 
-function test(caseName, cases, assertionName, ep) {
-  let sample = file(`./fixtures/${assertionName}/sample.js`).content;
-  let options = cases[caseName].transformOptions;
+function test(testcase, assertionName, ep) {
+  let sample = fs.readFileSync(`./fixtures/${assertionName}/sample.js`, 'utf-8');
+  let options = testcase.transformOptions;
   let expectationPath = `./fixtures/${assertionName}/expectation.js`;
-  let expectationExists = true;
-
-  try {
-    lstatSync(expectationPath);
-  } catch (e) {
-    expectationExists = false
-  }
+  let expectationExists = fs.existsSync(expectationPath);
 
   if (expectationExists) {
-    let expectation = file(`./fixtures/${assertionName}/expectation.js`).content;
-    let compiled = compile(sample, options);
+    let expectation = fs.readFileSync(expectationPath, 'utf-8');
+    let compiled = transform(sample, options);
     expect(compiled.code).to.equal(expectation);
 
   } else {
-    let fn = () => compile(sample, options);
-    expect(fn).to.throw(new RegExp(cases[caseName].errors[ep++]));
+    let fn = () => transform(sample, options);
+    expect(fn).to.throw(new RegExp(testcase.errors[ep++]));
   }
 }

yarn.lock

@@ -594,12 +594,6 @@ caniuse-db@^1.0.30000639:
   version "1.0.30000640"
   resolved "https://registry.yarnpkg.com/caniuse-db/-/caniuse-db-1.0.30000640.tgz#7b7fd3cf13c0d9d41f8754b577b202113e2be7ca"
 
-chai-files@^1.4.0:
-  version "1.4.0"
-  resolved "https://registry.yarnpkg.com/chai-files/-/chai-files-1.4.0.tgz#0e25610fadc551b1eae79c2f4ee79faf2f842296"
-  dependencies:
-    assertion-error "^1.0.1"
-
 chai@^3.5.0:
   version "3.5.0"
   resolved "https://registry.yarnpkg.com/chai/-/chai-3.5.0.tgz#4d02637b067fe958bdbfdd3a40ec56fef7373247"