Replace test helper with node-browser-environment

9 years ago · d2f85a4abc
3 changed files with 7 additions and 11 deletions
--- a/package.json
+++ b/package.json
@ -21,11 +21,6 @@
      "add-module-exports"
    ]
  },
-  "ava": {
-    "require": [
-      "./test/helpers/setup-browser-env.js"
-    ]
-  },
  "repository": {
    "type": "git",
    "url": "git+https://github.com/lukechilds/create-node.git"
@ -50,6 +45,7 @@
    "babel-preset-es2015": "^6.9.0",
    "coveralls": "^2.11.9",
    "jsdom": "^9.1.0",
+    "node-browser-environment": "^1.0.2",
    "nyc": "^6.4.4"
  }
 }
--- a/test/helpers/setup-browser-env.js
+++ b/test/helpers/setup-browser-env.js
@ -1,3 +0,0 @@
-global.document = require('jsdom').jsdom('<body></body>');
-global.window = document.defaultView;
-global.navigator = window.navigator;
--- a/test/unit.js
+++ b/test/unit.js
@ -1,13 +1,16 @@
 import test from 'ava';
+import browserEnv from 'node-browser-environment';
 import createNode from '../dist/create-node';

+browserEnv();
+
 test('createNode should be a function', t => {
  t.is(typeof createNode, 'function');
 });

 test('createNode(markup) should create an HTMLElement DOM node', t => {
  const domNode = createNode('<div></div>');
-  t.true(domNode instanceof window.HTMLElement);
+  t.true(domNode instanceof HTMLElement);
 });

 test('DOM node should match markup', t => {
@ -20,10 +23,10 @@ test('DOM node should match markup', t => {
  const expectedText = ['hello', 'world'];
  const domNode = createNode(markup);

-  t.true(domNode instanceof window.HTMLDivElement);
+  t.true(domNode instanceof HTMLDivElement);
  t.is(domNode.children.length, 2);
  Array.from(domNode.children).forEach((childNode, i) => {
-    t.true(childNode instanceof window.HTMLSpanElement);
+    t.true(childNode instanceof HTMLSpanElement);
    t.is(childNode.textContent, expectedText[i]);
  });
 });