Restore prettyDOM logging for Cypress

Author: istateside

src/__tests__/element-queries.js

@@ -1,10 +1,6 @@
 import {configure} from '../config'
 import {render, renderIntoDocument} from './helpers/test-utils'
 
-beforeEach(() => {
-  document.defaultView.Cypress = null
-})
-
 test('query can return null', () => {
   const {
     queryByLabelText,
@@ -1061,46 +1057,6 @@ test('the debug helper prints the dom state here', () => {
   process.env.DEBUG_PRINT_LIMIT = originalDebugPrintLimit
 })
 
-test('get throws a useful error message without DOM in Cypress', () => {
-  document.defaultView.Cypress = {}
-  const {
-    getByLabelText,
-    getByPlaceholderText,
-    getByText,
-    getByTestId,
-    getByAltText,
-    getByTitle,
-    getByDisplayValue,
-  } = render('<div />')
-  expect(() =>
-    getByLabelText('LucyRicardo'),
-  ).toThrowErrorMatchingInlineSnapshot(
-    `Unable to find a label with the text of: LucyRicardo`,
-  )
-  expect(() =>
-    getByPlaceholderText('LucyRicardo'),
-  ).toThrowErrorMatchingInlineSnapshot(
-    `Unable to find an element with the placeholder text of: LucyRicardo`,
-  )
-  expect(() => getByText('LucyRicardo')).toThrowErrorMatchingInlineSnapshot(
-    `Unable to find an element with the text: LucyRicardo. This could be because the text is broken up by multiple elements. In this case, you can provide a function for your text matcher to make your matcher more flexible.`,
-  )
-  expect(() => getByTestId('LucyRicardo')).toThrowErrorMatchingInlineSnapshot(
-    `Unable to find an element by: [data-testid="LucyRicardo"]`,
-  )
-  expect(() => getByAltText('LucyRicardo')).toThrowErrorMatchingInlineSnapshot(
-    `Unable to find an element with the alt text: LucyRicardo`,
-  )
-  expect(() => getByTitle('LucyRicardo')).toThrowErrorMatchingInlineSnapshot(
-    `Unable to find an element with the title: LucyRicardo.`,
-  )
-  expect(() =>
-    getByDisplayValue('LucyRicardo'),
-  ).toThrowErrorMatchingInlineSnapshot(
-    `Unable to find an element with the display value: LucyRicardo.`,
-  )
-})
-
 test('getByText ignores script tags by default', () => {
   const {getAllByText} = render(
     '<script>Hello</script><div>Hello</div><style>.Hello{}</style>',

src/pretty-dom.js

@@ -4,25 +4,11 @@ import {getUserCodeFrame} from './get-user-code-frame'
 import {getDocument} from './helpers'
 import {DEFAULT_IGNORE_TAGS} from './shared'
 
-function inCypress(dom) {
-  const window =
-    (dom.ownerDocument && dom.ownerDocument.defaultView) || undefined
-  return (
-    (typeof global !== 'undefined' && global.Cypress) ||
-    (typeof window !== 'undefined' && window.Cypress)
-  )
-}
-
 const inNode = () =>
   typeof process !== 'undefined' &&
   process.versions !== undefined &&
   process.versions.node !== undefined
 
-const getMaxLength = dom =>
-  inCypress(dom)
-    ? 0
-    : (typeof process !== 'undefined' && process.env.DEBUG_PRINT_LIMIT) || 7000
-
 const {DOMCollection} = prettyFormat.plugins
 
 // https://developer.mozilla.org/en-US/docs/Web/API/Node/nodeType#node_type_constants
@@ -43,7 +29,8 @@ function prettyDOM(dom, maxLength, options = {}) {
     dom = getDocument().body
   }
   if (typeof maxLength !== 'number') {
-    maxLength = getMaxLength(dom)
+    maxLength =
+      (typeof process !== 'undefined' && process.env.DEBUG_PRINT_LIMIT) || 7000
   }
 
   if (maxLength === 0) {