Add testing (#12)

* Add inital mocha testing
* Update linter rules for mocha
* Add test to check for expected number of differences

Author: cshaver

.eslintrc

@@ -19,6 +19,7 @@
   "globals": {},
   "env":{
     "node": true,
-    "es6": true
+    "es6": true,
+    "mocha": true
   }
 }

lib/css-ast-diff.js

@@ -6,7 +6,8 @@ const parse = require('css').parse;
 const sort = require('./sorters/ast');
 const stringify = require('css').stringify;
 const fs = require('fs');
-const colors = require('colors');
+
+require('es6-object-assign').polyfill();
 
 module.exports = function cssAstDiff(diff, src, callback, options) {
   try {

package.json

@@ -2,9 +2,8 @@
   "name": "css-ast-diff",
   "version": "1.0.6",
   "description": "Diffs CSS files by parsing into AST, sorting, and finding functional differences",
-  "main": "index.js",
   "scripts": {
-    "test": "echo \"Error: no test specified\" && exit 1"
+    "test": "mocha --harmony"
   },
   "repository": {
     "type": "git",
@@ -39,6 +38,7 @@
     "css": "^2.2.1",
     "debug": "^2.2.0",
     "diff": "^2.2.1",
+    "es6-object-assign": "^1.0.3",
     "fs": "0.0.2",
     "node-diff": "^0.2.0",
     "nodegit": "^0.11.8",
@@ -47,5 +47,9 @@
     "svn": "^0.2.0",
     "svn-interface": "^0.5.0",
     "util": "^0.10.3"
+  },
+  "devDependencies": {
+    "chai": "^3.5.0",
+    "mocha": "^3.2.0"
   }
 }

test/cases.js

@@ -0,0 +1,65 @@
+'use strict';
+
+const path = require('path');
+const async = require('async');
+const fs = require('fs');
+
+const chai = require('chai');
+const expect = chai.expect;
+
+const cases = fs.readdirSync(path.join(__dirname, 'cases'));
+const files = {
+  src: 'src.css',
+  diff: 'diff.css',
+  stats: 'stats.json'
+};
+
+const cssAstDiff = require('../lib/api');
+const options = {
+  type: 'hunks'
+};
+
+cases.forEach(function(name) {
+  describe('cases/' + name, function() {
+    var data = {};
+
+    before(function(done) {
+      this.timeout(5000);
+      runDiff(name, data, done);
+    });
+
+    // TODO: create more useful tests
+
+    it('detects the expected number of changes', function(done) {
+      expect(data.changes).to.have.length(data.stats.changes);
+      done();
+    });
+  });
+});
+
+function runDiff(name, data, callback) {
+  var dir = path.join(__dirname, 'cases', name);
+  var src = path.join(dir, files.src);
+  var diff = path.join(dir, files.diff);
+  var stats = path.join(dir, files.stats);
+
+  async.parallel([
+      function readStats(callback) {
+        fs.readFile(stats, 'utf8', function(err, results) {
+          data.stats = JSON.parse(results);
+          callback();
+        });
+      },
+      function createDiff(callback) {
+        cssAstDiff.compareFiles(src, diff, function(err, results) {
+          if (err) {
+            throw err;
+          }
+          data.changes = results;
+          callback();
+        }, options);
+      }
+    ],
+    callback
+  );
+}

test/bootstrap/bootstrap-3.3.6.css renamed to test/cases/bootstrap/diff.css

@@ -0,0 +1,4 @@
+{
+  "ignore": true,
+  "changes": 47
+}

test/bootstrap/bootstrap-3.3.5.css renamed to test/cases/bootstrap/src.css

@@ -0,0 +1,3 @@
+{
+  "changes": 2
+}