Skip to content

Check for errors after saving query definition#2978

Merged
labkey-tchad merged 1 commit intodevelopfrom
fb_catchSaveErrors
May 6, 2026
Merged

Check for errors after saving query definition#2978
labkey-tchad merged 1 commit intodevelopfrom
fb_catchSaveErrors

Conversation

@labkey-tchad
Copy link
Copy Markdown
Member

Rationale

Not detecting unexpected query errors can lead to hard to diagnose errors in subsequent tests.

java.lang.AssertionError: Did not find correct number of finished pipeline jobs. expected:<1> but was:<0>
  at org.junit.Assert.fail(Assert.java:89)
  at org.junit.Assert.failNotEquals(Assert.java:835)
  at org.junit.Assert.assertEquals(Assert.java:647)
  at org.labkey.test.LabKeySiteWrapper.waitForPipelineJobsToFinish(LabKeySiteWrapper.java:1641)
  at org.labkey.test.LabKeySiteWrapper.waitForPipelineJobsToComplete(LabKeySiteWrapper.java:1600)
  at org.labkey.test.LabKeySiteWrapper.waitForPipelineJobsToComplete(LabKeySiteWrapper.java:1587)
  at org.labkey.test.LabKeySiteWrapper.waitForPipelineJobsToComplete(LabKeySiteWrapper.java:1582)
  at org.labkey.test.tests.compliance.ComplianceQuerySnapshotTest.testPivotQueryETLToList(ComplianceQuerySnapshotTest.java:240)

Related Pull Requests

  • N/A

Changes

  • Check for errors after saving query definition

@labkey-tchad labkey-tchad self-assigned this May 5, 2026
@labkey-tchad labkey-tchad merged commit 56b2e5e into develop May 6, 2026
8 checks passed
@labkey-tchad labkey-tchad deleted the fb_catchSaveErrors branch May 6, 2026 21:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants