Revert "fix(node): allow running in node environment (testing-library#114)"

This reverts commit 1b12385.

Author: Kent C. Dodds

.size-snapshot.json

@@ -1,7 +1,7 @@
 {
   "dist/dom-testing-library.umd.js": {
-    "bundled": 144335,
-    "minified": 52790,
-    "gzipped": 15639
+    "bundled": 115443,
+    "minified": 50960,
+    "gzipped": 15294
   }
 }

jest.config.js

@@ -1,5 +1,5 @@
 const jestConfig = require('kcd-scripts/jest')
 
 module.exports = Object.assign(jestConfig, {
-  testEnvironment: 'jest-environment-node',
+  testEnvironment: 'jest-environment-jsdom',
 })

package.json

@@ -27,9 +27,8 @@
     "build": "kcd-scripts build && kcd-scripts build --bundle umd --no-clean",
     "lint": "kcd-scripts lint",
     "test": "kcd-scripts test",
-    "test:all": "npm test && npm test -- --env jsdom",
     "test:update": "npm test -- --updateSnapshot --coverage",
-    "validate": "kcd-scripts validate build,lint,test:all",
+    "validate": "kcd-scripts validate",
     "setup": "npm install && npm run validate -s",
     "precommit": "kcd-scripts precommit",
     "dtslint": "dtslint typings"
@@ -39,15 +38,14 @@
     "typings"
   ],
   "dependencies": {
-    "@sheerun/mutationobserver-shim": "^0.3.2",
+    "mutationobserver-shim": "^0.3.2",
     "pretty-format": "^23.6.0",
     "wait-for-expect": "^1.0.0"
   },
   "devDependencies": {
     "dtslint": "^0.3.0",
-    "jest-dom": "^2.0.4",
+    "jest-dom": "^1.7.0",
     "jest-in-case": "^1.0.2",
-    "jsdom": "^12.2.0",
     "kcd-scripts": "^0.41.0",
     "microbundle": "^0.4.4"
   },

src/__tests__/element-queries.js

@@ -1,9 +1,8 @@
 import 'jest-dom/extend-expect'
 import {render} from './helpers/test-utils'
-import document from './helpers/document'
 
 beforeEach(() => {
-  document.defaultView.Cypress = null
+  window.Cypress = null
 })
 
 test('query can return null', () => {
@@ -149,7 +148,9 @@ test('get element by its alt text', () => {
       <img alt="finding nemo poster" src="/finding-nemo.png" />
     </div>,
   `)
-  expect(getByAltText(/fin.*nem.*poster$/i).src).toContain('/finding-nemo.png')
+  expect(getByAltText(/fin.*nem.*poster$/i).src).toBe(
+    'http://localhost/finding-nemo.png',
+  )
 })
 
 test('query/get element by its title', () => {
@@ -488,7 +489,7 @@ test('test the debug helper prints the dom state here', () => {
 })
 
 test('get throws a useful error message without DOM in Cypress', () => {
-  document.defaultView.Cypress = {}
+  window.Cypress = {}
   const {
     getByLabelText,
     getBySelectText,

src/__tests__/events.js

@@ -1,5 +1,4 @@
 import {fireEvent} from '..'
-import document from './helpers/document'
 
 const eventTypes = [
   {
@@ -172,7 +171,7 @@ test('assigning a value to a target that cannot have a value throws an error', (
 
 test('assigning the files property on an input', () => {
   const node = document.createElement('input')
-  const file = new document.defaultView.File(['(⌐□_□)'], 'chucknorris.png', {
+  const file = new File(['(⌐□_□)'], 'chucknorris.png', {
     type: 'image/png',
   })
   fireEvent.change(node, {target: {files: [file]}})

src/__tests__/example.js

@@ -2,7 +2,6 @@
 import {getByLabelText, getByText, getByTestId, queryByTestId, wait} from '../'
 // adds special assertions like toHaveTextContent
 import 'jest-dom/extend-expect'
-import document from './helpers/document'
 
 function getExampleDOM() {
   // This is just a raw example of setting up some DOM

src/__tests__/get-queries-for-element.js

@@ -1,6 +1,5 @@
 import {getQueriesForElement} from '../get-queries-for-element'
 import {queries} from '..'
-import document from './helpers/document'
 
 test('uses default queries', () => {
   const container = document.createElement('div')

src/__tests__/helpers/document.js

@@ -1,5 +1,4 @@
 import {getQueriesForElement} from '../../get-queries-for-element'
-import document from './document'
 
 function render(html) {
   const container = document.createElement('div')

src/__tests__/helpers/test-utils.js

@@ -1,6 +1,5 @@
 import {prettyDOM} from '../pretty-dom'
 import {render} from './helpers/test-utils'
-import document from './helpers/document'
 
 test('it prints out the given DOM element tree', () => {
   const {container} = render('<div>Hello World!</div>')

src/__tests__/pretty-dom.js

@@ -2,7 +2,6 @@ import {waitForElement, wait} from '../'
 // adds special assertions like toBeTruthy
 import 'jest-dom/extend-expect'
 import {render} from './helpers/test-utils'
-import document from './helpers/document'
 
 async function skipSomeTime(delayMs) {
   await new Promise(resolve => setTimeout(resolve, delayMs))
@@ -105,16 +104,7 @@ test('it waits for the next DOM mutation with default callback', async () => {
   const successHandler = jest.fn().mockName('successHandler')
   const errorHandler = jest.fn().mockName('errorHandler')
 
-  let promise
-
-  if (typeof window !== 'undefined' && typeof window.document !== 'undefined') {
-    promise = waitForElement().then(successHandler, errorHandler)
-  } else {
-    promise = waitForElement(undefined, {container: document}).then(
-      successHandler,
-      errorHandler,
-    )
-  }
+  const promise = waitForElement().then(successHandler, errorHandler)
 
   // Promise callbacks are always asynchronous.
   expect(successHandler).toHaveBeenCalledTimes(0)