Merge pull request #13 from testing-library/fix-sync-query-errors-producing-logs

fix: expected getBy query errors being logged

Author: olivierwilkinson

src/index.ts

@@ -108,26 +108,30 @@ function executeQuery(
 
   const [matcher, options, waitForOptions] = args.map(deserializeArg)
 
-  Promise.resolve(
-    window.TestingLibraryDom[query](
-      container,
-      matcher,
-      options,
-      waitForOptions,
-    ),
-  )
-    .then((result) => {
-      if (!result) {
-        return done(null)
-      }
-      if (Array.isArray(result)) {
-        return done(
-          result.map((element) => ({selector: window.Simmer(element)})),
-        )
-      }
-      return done({selector: window.Simmer(result)})
-    })
-    .catch((e) => done(e.message))
+  try {
+    Promise.resolve(
+      window.TestingLibraryDom[query](
+        container,
+        matcher,
+        options,
+        waitForOptions,
+      ),
+    )
+      .then((result) => {
+        if (!result) {
+          return done(null)
+        }
+        if (Array.isArray(result)) {
+          return done(
+            result.map((element) => ({selector: window.Simmer(element)})),
+          )
+        }
+        return done({selector: window.Simmer(result)})
+      })
+      .catch((e) => done(e.message))
+  } catch (e) {
+    done(e.message)
+  }
 }
 
 function createQuery(element: ElementBase, queryName: string) {

test/async/configure.e2e.ts

@@ -47,7 +47,7 @@ describe('configure', () => {
     await expect(() =>
       getByTestId('button-that-should-not-use-testid'),
     ).rejects.toThrowError(
-      'TestingLibraryElementError: A better query is available',
+      'A better query is available',
     )
   })
 

wdio.conf.js

@@ -77,7 +77,7 @@ exports.config = {
   // Define all options that are relevant for the WebdriverIO instance here
   //
   // Level of logging verbosity: trace | debug | info | warn | error | silent
-  logLevel: 'silent',
+  logLevel: 'warn',
   //
   // Set specific log levels per logger
   // loggers: