Add tests for non-positional outcomes

Author: wooorm

package.json

@@ -27,11 +27,14 @@
   "devDependencies": {
     "browserify": "^14.0.0",
     "esmangle": "^1.0.1",
+    "is-hidden": "^1.1.0",
+    "not": "^0.1.0",
     "nyc": "^11.0.0",
     "parse5": "^3.0.0",
     "remark-cli": "^3.0.0",
     "remark-preset-wooorm": "^3.0.0",
     "tape": "^4.0.0",
+    "unist-util-visit": "^1.1.3",
     "vfile": "^2.0.0",
     "xo": "^0.18.0"
   },

test/index.js

@@ -4,8 +4,11 @@
 var fs = require('fs');
 var path = require('path');
 var test = require('tape');
+var not = require('not');
+var hidden = require('is-hidden');
 var vfile = require('vfile');
 var parse5 = require('parse5');
+var visit = require('unist-util-visit');
 var fromParse5 = require('..');
 
 /* Methods. */
@@ -298,28 +301,92 @@ test('hast-util-from-parse5', function (t) {
 /* Fixtures. */
 test('fixtures', function (t) {
   var base = join(__dirname, 'fixtures');
+  var entries = dir(base);
 
-  dir(base)
-    .filter(function (fixture) {
-      return fixture.charAt(0) !== '.';
-    })
-    .forEach(function (fixture) {
-      var filePath = join(base, fixture, 'index.json');
-      var input = read(join(base, fixture, 'index.html'), 'utf8');
-      var ast = parse5.parse(input, {locationInfo: true});
-      var doc = fromParse5(ast, {file: vfile(input), verbose: true});
-      var output;
+  t.plan(entries.length);
+  entries.filter(not(hidden)).forEach(each);
 
-      try {
-        output = read(filePath, 'utf8');
-      } catch (err) {
-        /* New fixture. */
-        write(filePath, JSON.stringify(doc, 0, 2) + '\n');
-        return;
-      }
+  function each(fixture) {
+    t.test(fixture, function (st) {
+      var opts = {
+        file: vfile(read(join(base, fixture, 'index.html'), 'utf8')),
+        out: join(base, fixture, 'index.json')
+      };
+
+      st.plan(4);
 
-      t.deepEqual(doc, JSON.parse(output), fixture);
+      checkYesYes(st, fixture, opts);
+      checkNoYes(st, fixture, opts);
+      checkYesNo(st, fixture, opts);
+      checkNoNo(st, fixture, opts);
     });
+  }
 
-  t.end();
+  function checkYesYes(t, fixture, options) {
+    var input = parse5.parse(String(options.file), {locationInfo: true});
+    var actual = fromParse5(input, {file: options.file, verbose: true});
+    var expected;
+
+    try {
+      expected = JSON.parse(read(options.out));
+    } catch (err) {
+      /* New fixture. */
+      write(options.out, JSON.stringify(actual, 0, 2) + '\n');
+      return;
+    }
+
+    t.deepEqual(actual, expected, 'p5 w/ position, hast w/ intent of position');
+  }
+
+  function checkYesNo(t, fixture, options) {
+    var input = parse5.parse(String(options.file), {locationInfo: true});
+    var actual = fromParse5(input);
+    var expected = JSON.parse(read(options.out));
+
+    clean(expected);
+
+    t.deepEqual(actual, expected, 'p5 w/ position, hast w/o intent of position');
+  }
+
+  function checkNoYes(t, fixture, options) {
+    var input = parse5.parse(String(options.file));
+    var actual = fromParse5(input, {file: options.file, verbose: true});
+    var expected = JSON.parse(read(options.out));
+
+    clean(expected);
+
+    t.deepEqual(actual, expected, 'p5 w/o position, hast w/ intent of position');
+  }
+
+  function checkNoNo(t, fixture, options) {
+    var input = parse5.parse(String(options.file), {locationInfo: true});
+    var actual = fromParse5(input);
+    var expected = JSON.parse(read(options.out));
+
+    clean(expected);
+
+    try {
+      require('assert').deepEqual(actual, expected, 'w/o position');
+    } catch (err) {
+      console.log('actual: ');
+      console.dir(actual, {depth: null});
+      console.log('expected: ');
+      console.dir(expected, {depth: null});
+    }
+
+    t.deepEqual(actual, expected, 'p5 w/o position, hast w/o intent of position');
+  }
 });
+
+function clean(tree) {
+  visit(tree, cleaner);
+}
+
+function cleaner(node) {
+  delete node.position;
+
+  /* Remove verbose data */
+  if (node.type === 'element') {
+    delete node.data;
+  }
+}