Skip to content

Commit df39821

Browse files
Claude Agentclaude
andcommitted
fix(tests): add plugin warm-up to regression spec before() hooks
The monitoring-console-plugin registers the Incidents tab extension asynchronously after Cypress session restoration. Without warm-up, the tab may not be registered before beforeEach() tries to navigate, causing flaky failures in reg/02-05 specs. Uses warmUpForPlugin() (added in previous commit) which polls for the Incidents tab via jQuery check for up to 3 minutes — replacing the fragile goTo() that relied on the 80s default command timeout. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 8cd04b8 commit df39821

4 files changed

Lines changed: 4 additions & 0 deletions

File tree

web/cypress/e2e/incidents/regression/02.reg_ui_charts_comprehensive.cy.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -103,6 +103,7 @@ const MP = {
103103
describe('Regression: Charts UI - Comprehensive', { tags: ['@incidents'] }, () => {
104104
before(() => {
105105
cy.beforeBlockCOO(MCP, MP, { dashboards: false, troubleshootingPanel: false });
106+
incidentsPage.warmUpForPlugin();
106107
});
107108

108109
beforeEach(() => {

web/cypress/e2e/incidents/regression/03.reg_api_calls.cy.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ const MP = {
3434
describe('Regression: Silences Not Applied Correctly', { tags: ['@incidents'] }, () => {
3535
before(() => {
3636
cy.beforeBlockCOO(MCP, MP, { dashboards: false, troubleshootingPanel: false });
37+
incidentsPage.warmUpForPlugin();
3738
});
3839

3940
beforeEach(() => {

web/cypress/e2e/incidents/regression/04.reg_redux_effects.cy.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ const MP = {
3333
describe('Regression: Redux State Management', { tags: ['@incidents', '@incidents-redux'] }, () => {
3434
before(() => {
3535
cy.beforeBlockCOO(MCP, MP, { dashboards: false, troubleshootingPanel: false });
36+
incidentsPage.warmUpForPlugin();
3637
});
3738

3839
beforeEach(() => {

web/cypress/e2e/incidents/regression/05.reg_stress_testing_ui.cy.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ const MAX_GAP_RELAXED = 500;
3434
describe('Regression: Stress Testing UI', { tags: ['@incidents'] }, () => {
3535
before(() => {
3636
cy.beforeBlockCOO(MCP, MP, { dashboards: false, troubleshootingPanel: false });
37+
incidentsPage.warmUpForPlugin();
3738
});
3839

3940
it('5.1 No excessive padding between chart top and alert bars for 100, 200, and 500 alerts', () => {

0 commit comments

Comments
 (0)