Throw error instead of logging

Author: mathiassoeholm

src/config.ts

@@ -32,10 +32,17 @@ let config: InternalConfig = {
 
   // called when getBy* queries fail. (message, container) => Error
   getElementError(message, container) {
-    const playgroundUrl =
-      message &&
-      getConfig().printPlaygroundLink &&
-      getPlaygroundUrl(container, false)
+    const playgroundUrl = (() => {
+      try {
+        return (
+          message &&
+          getConfig().printPlaygroundLink &&
+          getPlaygroundUrl(container)
+        )
+      } catch {
+        return null
+      }
+    })()
 
     const error = new Error(
       [

src/playground-helper.ts

@@ -10,19 +10,13 @@ function encode(value: string) {
   return compressToEncodedURIComponent(unindent(value))
 }
 
-function getPlaygroundUrl(element: Element | null, logErrors = true) {
+function getPlaygroundUrl(element: Element | null) {
   if (!element || !('innerHTML' in element)) {
-    if (logErrors) {
-      console.log(`The element you're providing isn't a valid DOM element.`)
-    }
-    return null
+    throw new Error(`The element you're providing isn't a valid DOM element.`)
   }
 
   if (!element.innerHTML) {
-    if (logErrors) {
-      console.log(`The provided element doesn't have any children.`)
-    }
-    return null
+    throw new Error(`The provided element doesn't have any children.`)
   }
 
   return `https://testing-playground.com/#markup=${encode(element.innerHTML)}`

src/screen.js

@@ -10,9 +10,11 @@ const debug = (element, maxLength, options) =>
     : logDOM(element, maxLength, options)
 
 const logTestingPlaygroundURL = (element = getDocument().body) => {
-  const url = getPlaygroundUrl(element)
-  if (url) {
+  try {
+    const url = getPlaygroundUrl(element)
     console.log(`Open this URL in your browser\n\n${url}`)
+  } catch (e) {
+    console.log(e.message)
   }
 }