Add runtime check back in logTestingPlaygroundUrl

Author: mathiassoeholm

src/__tests__/screen.js

@@ -46,6 +46,12 @@ test('logs messsage when element is not a valid HTML', () => {
   expect(console.log.mock.calls[0][0]).toMatchInlineSnapshot(
     `"The element you're providing isn't a valid DOM element."`,
   )
+  console.log.mockClear()
+  screen.logTestingPlaygroundURL({})
+  expect(console.log).toHaveBeenCalledTimes(1)
+  expect(console.log.mock.calls[0][0]).toMatchInlineSnapshot(
+    `"The element you're providing isn't a valid DOM element."`,
+  )
 })
 
 test('logs Playground URL that are passed as element', () => {

src/playground-helper.ts

@@ -11,11 +11,6 @@ function encode(value: string) {
 }
 
 function getPlaygroundUrl(element: Element) {
-  // eslint-disable-next-line @typescript-eslint/no-unnecessary-condition
-  if (!element) {
-    throw new Error(`The element you're providing isn't a valid DOM element.`)
-  }
-
   if (!element.innerHTML) {
     throw new Error(`The provided element doesn't have any children.`)
   }

src/screen.js

@@ -10,6 +10,11 @@ const debug = (element, maxLength, options) =>
     : logDOM(element, maxLength, options)
 
 const logTestingPlaygroundURL = (element = getDocument().body) => {
+  if (!element || !('innerHTML' in element)) {
+    console.log(`The element you're providing isn't a valid DOM element.`)
+    return
+  }
+
   try {
     const url = getPlaygroundUrl(element)
     console.log(`Open this URL in your browser\n\n${url}`)

types/screen.d.ts

@@ -16,7 +16,7 @@ export type Screen<Q extends Queries = typeof queries> = BoundFunctions<Q> & {
    * Convenience function for `Testing Playground` which logs URL that
    * can be opened in a browser
    */
-  logTestingPlaygroundURL: (element?: Element) => void
+  logTestingPlaygroundURL: (element?: Element | HTMLDocument) => void
 }
 
 export const screen: Screen