Skip to content

Commit

Permalink
Don't assume client-side error will always happen
Browse files Browse the repository at this point in the history
  • Loading branch information
labkey-tchad committed Feb 27, 2025
1 parent 664b44f commit 1985839
Showing 1 changed file with 7 additions and 2 deletions.
9 changes: 7 additions & 2 deletions src/org/labkey/test/tests/KnitrReportTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -203,8 +203,13 @@ private void verifyAdhocReportDependencies(String viewName, String dependencies)
_rReportHelper.clickReportTab();
waitForElement(Locator.id("mtcars_table"));
assertElementNotPresent(Locator.id("mtcars_table_wrapper")); // Created by jQuery
Assertions.assertThat(getServerErrors()).as("Server errors").contains("$(...).dataTable is not a function");
checkExpectedErrors(1); // JavaScript error: "$(...).dataTable is not a function"
String serverErrors = getServerErrors();
if (!serverErrors.isEmpty())
{
// Client-side error doesn't always happen but, if it does, make sure it is the one we expect.
Assertions.assertThat(serverErrors).as("Server errors").contains("$(...).dataTable is not a function");
checkExpectedErrors(1); // JavaScript error: "$(...).dataTable is not a function"
}

// now set the dependencies
_rReportHelper.clickSourceTab();
Expand Down

0 comments on commit 1985839

Please sign in to comment.