KnitrReportTest: use checkExpectedErrors() instead of pause/resume JS…

… error checks
LabKey · Feb 25, 2025 · b95c150 · b95c150
1 parent 0d4dbdf
commit b95c150
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/src/org/labkey/test/tests/KnitrReportTest.java b/src/org/labkey/test/tests/KnitrReportTest.java
@@ -199,13 +199,10 @@ private void verifyAdhocReportDependencies(String viewName, String dependencies)
 
         createKnitrReport(rmdDependenciesReport, RReportHelper.ReportOption.knitrMarkdown);
 
-        pauseJsErrorChecker(); // Don't fail due to "$ is not a function"
-        {
-            _rReportHelper.clickReportTab();
-            waitForElement(Locator.id("mtcars_table"));
-            assertElementNotPresent(Locator.id("mtcars_table_wrapper")); // Created by jQuery
-        }
-        resumeJsErrorChecker();
+        _rReportHelper.clickReportTab();
+        waitForElement(Locator.id("mtcars_table"));
+        assertElementNotPresent(Locator.id("mtcars_table_wrapper")); // Created by jQuery
+        checkExpectedErrors(1); // JavaScript error: "$(...).dataTable is not a function"
 
         // now set the dependencies
         _rReportHelper.clickSourceTab();