Skip to content

fix verifyGuideSetRelatedElementsForPlots#1192

Open
ankurjuneja wants to merge 2 commits intorelease26.3-SNAPSHOTfrom
26.3_fb_fixFailingTests
Open

fix verifyGuideSetRelatedElementsForPlots#1192
ankurjuneja wants to merge 2 commits intorelease26.3-SNAPSHOTfrom
26.3_fb_fixFailingTests

Conversation

@ankurjuneja
Copy link
Copy Markdown
Contributor

@ankurjuneja ankurjuneja commented Apr 4, 2026

Rationale

Add cleanup in TargetedMSQCTest as shared annotations were getting displayed in other test containers causing failures. The failures are intermittent because of the ordering of tests execution in TC.

Related Pull Requests

Changes

@ankurjuneja ankurjuneja marked this pull request as ready for review April 6, 2026 17:02
protected void doCleanup(boolean afterTest) throws TestTimeoutException
{
// Use the API-based approach for deletion so that we don't trigger AJAX requests navigating to the delete page
// that may run in the background and cause SQL Server deadlock exceptions
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Panorama doesn't support SQL Server anymore so this comment can be updated or trimmed.

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