Merge pull request #398 from CBIIT/dx-fix-silent-warnings

Fix silent warnings allowing Jest to succeed
CBIIT · Jun 7, 2024 · 870a77b · 870a77b
2 parents 5944336 + 68c14e9
commit 870a77b
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 0 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -21,6 +21,7 @@
     "graphql": "^16.7.1",
     "graphql-tag": "^2.12.6",
     "jest-axe": "^8.0.0",
+    "jest-fail-on-console": "^3.3.0",
     "lodash": "^4.17.21",
     "notistack": "^3.0.1",
     "papaparse": "^5.4.1",

diff --git a/src/setupTests.tsx b/src/setupTests.tsx
@@ -1,6 +1,7 @@
 // eslint-disable-next-line max-classes-per-file
 import "@testing-library/jest-dom";
 import "jest-axe/extend-expect";
+import failOnConsole from "jest-fail-on-console";
 
 /**
  * Mocks the enqueueSnackbar function from notistack for testing
@@ -58,3 +59,12 @@ jest.mock("recharts", () => ({
   ...jest.requireActual("recharts"),
   ResponsiveContainer: MockResponsiveContainer,
 }));
+
+/**
+ * Prevents the console.error and console.warn from silently failing
+ * in tests by throwing an error when called
+ */
+failOnConsole({
+  shouldFailOnWarn: true,
+  shouldFailOnError: true,
+});