Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .cursor/commands/generate-regression-test.md
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ Generate automated regression tests from test documentation in [`docs/incident_d
Verifies: OU-XXX
*/

import { incidentsPage } from "../../../views/incidents-page";
import { incidentsPage } from "../../../views/incidents/incidents-page";

describe("Regression: [Section Name]", () => {
before(() => {
Expand Down Expand Up @@ -524,7 +524,7 @@ Tests both incidents chart and alerts chart tooltip behavior.
Verifies: OU-XXX
*/

import { incidentsPage } from "../../../views/incidents-page";
import { incidentsPage } from "../../../views/incidents/incidents-page";

describe("Regression: Tooltip Positioning", () => {
before(() => {
Expand Down
6 changes: 3 additions & 3 deletions .cursor/rules/incidents-testing-guidelines.mdc
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ alwaysApply: true
description: "Development guidelines for Incidents page Cypress tests"
globs:
- "web/cypress/e2e/incidents/**/*.cy.ts"
- "web/cypress/views/incidents-page.ts"
- "web/cypress/views/incidents/incidents-page.ts"
- "web/cypress/fixtures/incidents/scenarios/**/*.yaml"
---

Expand Down Expand Up @@ -99,7 +99,7 @@ Additional context about the bug or behavior.
Verifies: OU-XXX
*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

describe('Regression: <Section Name>', () => {
before(() => {
Expand Down Expand Up @@ -294,7 +294,7 @@ cy.mockIncidents([]);
### Imports
```typescript
// Always import from relative path
import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

// Import types when needed
import { IncidentDefinition } from '../../../support/incidents';
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ Shelved due to Cypress DOM snapshot accumulation causing ~10x degradation over 1
**To re-enable:** save the code block below as `web/cypress/e2e/incidents/performance/03.performance_endurance.cy.ts`.

```typescript
import { incidentsPage } from "../../../views/incidents-page";
import { incidentsPage } from "../../../views/incidents/incidents-page";

const INCIDENT_COUNT = 20;
const CYCLES_PER_INCIDENT = 5;
Expand Down
17 changes: 8 additions & 9 deletions web/cypress/e2e/alerts/alerts_acm.cy.ts
Original file line number Diff line number Diff line change
@@ -1,18 +1,17 @@
// E2E test for validating ACM Alerting UI integration with Cluster Observability Operator (COO)
import '../../support/shared/commands/auth-commands';
import { commonPages } from '../../views/common';
import { nav } from '../../views/nav';
import { acmAlertingPage } from '../../views/acm-alerting-page';

import { troubleshootingPanelPage } from 'cypress/views/troubleshooting-panel';
import { incidentsPage } from 'cypress/views/incidents-page';
import { troubleshootingPanelPage } from '../../views/shared/troubleshooting-panel';
import { incidentsPage } from '../../views/incidents/incidents-page';
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { commonPages } from '../../views/shared/common';
import { nav } from '../../views/shared/nav';
import { acmAlertingPage } from '../../views/alerts/acm-alerting-page';
import {
testAlertsFleetManagementRegression,
testAlertsRegression,
} from 'cypress/support/alerts/alerts_regressions.cy';
import { listPage } from 'cypress/views/list-page';
} from '../../support/alerts/alerts_regressions.cy';
import { listPage } from '../../views/shared/list-page';
import { CLUSTER_MONITORING_OPERATOR } from '../../support/shared/operators';
import { CustomerPerspectiveName } from '@/shared/constants/perspective';

const expectedAlerts = ['Watchdog', 'Watchdog-spoke', 'ClusterCPUHealth-jb'];

Expand Down
4 changes: 2 additions & 2 deletions web/cypress/e2e/alerts/alerts_bvt.cy.ts
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';
import { alerts } from '../../fixtures/alerts/interceptWatchdogAlert';
import { testBVTMonitoringTestsNamespace } from '../../support/alerts/alerts_bvt_namespaced.cy';
import { commonPages } from '../../views/common';
import { commonPages } from '../../views/shared/common';
import { CLUSTER_MONITORING_OPERATOR } from '../../support/shared/operators';

describe('BVT: Monitoring - Namespaced', { tags: ['@alerting', '@metrics'] }, () => {
Expand Down
6 changes: 3 additions & 3 deletions web/cypress/e2e/alerts/alerts_ivt.cy.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { alerts } from '../../fixtures/alerts/interceptWatchdogAlert';
import { testAlertsRegression } from '../../support/alerts/alerts_regressions.cy';
import { commonPages } from '../../views/common';
import { nav } from '../../views/nav';
import { guidedTour } from '../../views/tour';
import { commonPages } from '../../views/shared/common';
import { nav } from '../../views/shared/nav';
import { guidedTour } from '../../views/shared/tour';

describe(
'Regression: Monitoring - Alerts (Virtualization)',
Expand Down
4 changes: 2 additions & 2 deletions web/cypress/e2e/alerts/alerts_regression.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5,8 +5,8 @@ import {
} from '../../support/alerts/alerts_regressions.cy';
import { alerts } from '../../fixtures/alerts/interceptWatchdogAlert';
import { testAlertsRegressionNamespace } from '../../support/alerts/alerts_regressions_namespaced.cy';
import { commonPages } from '../../views/common';
import { nav } from '../../views/nav';
import { commonPages } from '../../views/shared/common';
import { nav } from '../../views/shared/nav';
import { CLUSTER_MONITORING_OPERATOR } from '../../support/shared/operators';

// Test suite for Core platform perspective
Expand Down
8 changes: 4 additions & 4 deletions web/cypress/e2e/alerts/alerts_virtualization_bvt.cy.ts
Original file line number Diff line number Diff line change
@@ -1,10 +1,10 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { testBVTAlerts } from '../../support/alerts/alerts_bvt.cy';
import { guidedTour } from '../../views/tour';
import { guidedTour } from '../../views/shared/tour';
import { alerts } from '../../fixtures/alerts/interceptWatchdogAlert';
import { nav } from '../../views/nav';
import { commonPages } from '../../views/common';
import { troubleshootingPanelPage } from '../../views/troubleshooting-panel';
import { nav } from '../../views/shared/nav';
import { commonPages } from '../../views/shared/common';
import { troubleshootingPanelPage } from '../../views/shared/troubleshooting-panel';

describe(
'IVT: Monitoring + Virtualization',
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/incidents_bvt.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ is irrelevant for toolbar/filter verification). Tests 4-5 switch mocks
mid-test for empty state and traversal scenarios.
*/

import { incidentsPage } from '../../views/incidents-page';
import { incidentsPage } from '../../views/incidents/incidents-page';

describe('BVT: Incidents - UI', { tags: ['@cluster-health-analyzer', '@coo'] }, () => {
before(() => {
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/incidents_e2e.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
The test verifies the whole lifecycle of the Incident feature, without any external dependencies.
The run time can be 15 - 20 minutes. (Waiting untill the incident detection captures the new alert)
*/
import { incidentsPage } from '../../views/incidents-page';
import { incidentsPage } from '../../views/incidents/incidents-page';

describe('BVT: Incidents - e2e', { tags: ['@cluster-health-analyzer', '@coo', '@slow'] }, () => {
let currentAlertName: string;
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/incidents_mocking_example.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ This test showcases different mocking approaches:
3. Demonstrating empty state handling
*/

import { incidentsPage } from '../../views/incidents-page';
import { incidentsPage } from '../../views/incidents/incidents-page';

describe('Incidents - Mocking Examples', { tags: ['@cluster-health-analyzer', '@coo'] }, () => {
before(() => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ median observed time.
Verifies: OBSINTA-1006
*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';
import { BenchmarkCollector } from '../../../support/shared/commands/benchmark-utils';

// Wall-clock thresholds in ms. Includes Cypress overhead (navigation, intercept
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ this test measures incremental re-render cost during a realistic user session.
Verifies: OBSINTA-1006
*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';
import { BenchmarkCollector } from '../../../support/shared/commands/benchmark-utils';

const THRESHOLDS = {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ Both tests require continuously firing alerts and cannot be tested with mocked d
Verifies: OU-XXX (time-based resolution bugs)
*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

describe(
'Regression: Time-Based Alert Resolution (E2E with Firing Alerts)',
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/regression/filtering.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ Uses elements defined in incidents-page.ts for all interactions.
Verifies: OU-727
*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

describe('Regression: Incidents Filtering', { tags: ['@cluster-health-analyzer', '@coo'] }, () => {
before(() => {
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/regression/interval.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ with no 5-minute gap between End of one segment and Start of the next.
*/

/* eslint-disable @typescript-eslint/no-unused-expressions */
import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

describe(
'Regression: Mixed Severity Interval Boundary Times',
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ Tests:
Verifies: OU-1020, OU-706, OU-1213
*/

import { incidentsPage } from '../../../views/incidents-page';
import { nav } from '../../../views/nav';
import { incidentsPage } from '../../../views/incidents/incidents-page';
import { nav } from '../../../views/shared/nav';

describe(
'Regression: Silences Not Applied Correctly',
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/regression/stress_test_ui.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ test is independent of timeline length. See WIP.stress-test-1000-alerts.cy.ts to
the overflow with the original 5d timeline.
*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

const MAX_GAP_STANDARD = 250;
const MAX_GAP_RELAXED = 500;
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/regression/ui_interaction.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ Test cases:
Verifies: OU-1030
*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

describe(
'Regression: Redux State Management',
Expand Down
2 changes: 1 addition & 1 deletion web/cypress/e2e/incidents/regression/ui_regressions.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ This test loads comprehensive test data covering:

*/

import { incidentsPage } from '../../../views/incidents-page';
import { incidentsPage } from '../../../views/incidents/incidents-page';

const ARROW_HEIGHT = 12;
const ALLOWED_MARGIN = 8;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,8 @@ import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { CLUSTER_MONITORING_OPERATOR } from '../../support/shared/operators';
import { testLegacyDashboardsRegression } from '../../support/legacy-dashboards/legacy_dashboards_regressions.cy';
import { testLegacyDashboardsRegressionNamespace } from '../../support/legacy-dashboards/legacy_dashboards_regressions_namespaced.cy';
import { commonPages } from '../../views/common';
import { nav } from '../../views/nav';
import { commonPages } from '../../views/shared/common';
import { nav } from '../../views/shared/nav';

// Test suite for Administrator perspective
describe(
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,9 +2,9 @@ import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { CLUSTER_MONITORING_OPERATOR } from '../../support/shared/operators';
import { testLegacyDashboardsRegression } from '../../support/legacy-dashboards/legacy_dashboards_regressions.cy';
import { testLegacyDashboardsRegressionNamespace } from '../../support/legacy-dashboards/legacy_dashboards_regressions_namespaced.cy';
import { commonPages } from '../../views/common';
import { nav } from '../../views/nav';
import { guidedTour } from '../../views/tour';
import { commonPages } from '../../views/shared/common';
import { nav } from '../../views/shared/nav';
import { guidedTour } from '../../views/shared/tour';

describe(
'Regression: Monitoring - Legacy Dashboards (Virtualization)',
Expand Down
4 changes: 2 additions & 2 deletions web/cypress/e2e/metrics/metrics_regression.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -7,8 +7,8 @@ import {
testMetricsRegressionNamespace1,
testMetricsRegressionNamespace2,
} from '../../support/metrics/metrics_regressions_namespaced.cy';
import { commonPages } from '../../views/common';
import { nav } from '../../views/nav';
import { commonPages } from '../../views/shared/common';
import { nav } from '../../views/shared/nav';
import { CLUSTER_MONITORING_OPERATOR } from '../../support/shared/operators';

// Test suite for Administrator perspective
Expand Down
6 changes: 3 additions & 3 deletions web/cypress/e2e/metrics/metrics_virtualization_ivt.cy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,9 @@ import {
testMetricsRegressionNamespace1,
testMetricsRegressionNamespace2,
} from '../../support/metrics/metrics_regressions_namespaced.cy';
import { commonPages } from '../../views/common';
import { nav } from '../../views/nav';
import { guidedTour } from '../../views/tour';
import { commonPages } from '../../views/shared/common';
import { nav } from '../../views/shared/nav';
import { guidedTour } from '../../views/shared/tour';
import { CLUSTER_MONITORING_OPERATOR } from '../../support/shared/operators';

describe('Regression: Monitoring - Metrics (Virtualization)', () => {
Expand Down
10 changes: 5 additions & 5 deletions web/cypress/e2e/perses-dashboards/lightspeed_integration.cy.ts
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
import { operatorAuthUtils } from '../../support/shared/commands/auth-commands';
import { nav } from '../../views/nav';
import { commonPages } from '../../views/common';
import { listPersesDashboardsPage } from '../../views/perses-dashboards-list-dashboards';
import { persesCreateDashboardsPage } from '../../views/perses-dashboards-create-dashboard';
import { persesDashboardsPage } from '../../views/perses-dashboards';
import { nav } from '../../views/shared/nav';
import { commonPages } from '../../views/shared/common';
import { listPersesDashboardsPage } from '../../views/perses-dashboards/perses-dashboards-list-dashboards';
import { persesCreateDashboardsPage } from '../../views/perses-dashboards/perses-dashboards-create-dashboard';
import { persesDashboardsPage } from '../../views/perses-dashboards/perses-dashboards';
import { persesMUIDataTestIDs } from '@/shared/constants/data-test';

const PROMPT = 'visualize CPU consumption in the openshift-monitoring namespace in last 2 hours';
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
// 02.acm_perses.cy.ts
// E2E test for validating ACM Perses integration with Cluster Observability Operator (COO)
import { nav } from 'cypress/views/nav';
import { nav } from 'cypress/views/shared/nav';
import '../../support/shared/commands/auth-commands';
import { testBVTCOOPerses1 } from 'cypress/support/perses/perses_bvt_admin.cy';
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';
//TODO: rename after customizable-dashboards gets merged
import { testBVTCOOPerses1 } from '../../support/perses/perses_bvt_admin.cy';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';
import { operatorAuthUtils } from '../../support/shared/commands/auth-commands';
import { testCOOCreatePerses } from '../../support/perses/perses_create_admin.cy';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { testCOOCreateImportPerses } from '../../support/perses/perses_create_import_admin.cy';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';

describe(
'COO - Dashboards (Perses) - Perses Global Datasources with Tempo and Loki',
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';
import { operatorAuthUtils } from '../../support/shared/commands/auth-commands';
import { testCOOEditPerses } from '../../support/perses/perses_edit_admin.cy';
import { testCOOEditPerses1 } from '../../support/perses/perses_edit_admin_1.cy';
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';
import { operatorAuthUtils } from '../../support/shared/commands/auth-commands';
import { testCOOImportPerses } from '../../support/perses/perses_import_admin.cy';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';
import {
testCOOListPerses,
testCOOListPersesDuplicateDashboard,
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../views/nav';
import { nav } from '../../views/shared/nav';
import { testBVTCOOPerses1 } from '../../support/perses/perses_bvt_admin.cy';
import { guidedTour } from '../../views/tour';
import { commonPages } from '../../views/common';
import { guidedTour } from '../../views/shared/tour';
import { commonPages } from '../../views/shared/common';

describe(
'IVT: COO - Dashboards (Perses) - Virtualization perspective',
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../../views/nav';
import { nav } from '../../../views/shared/nav';
import { testCOORBACPersesTestsDevUser1 } from '../../../support/perses/rbac/perses_user1.cy';

describe(
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../../views/nav';
import { nav } from '../../../views/shared/nav';
import { operatorAuthUtils } from '../../../support/shared/commands/auth-commands';
import { testCOORBACPersesTestsDevUser2 } from '../../../support/perses/rbac/perses_user2.cy';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../../views/nav';
import { nav } from '../../../views/shared/nav';
import { operatorAuthUtils } from '../../../support/shared/commands/auth-commands';
import { testCOORBACPersesTestsDevUser3 } from '../../../support/perses/rbac/perses_user3.cy';

Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { CustomerPerspectiveName } from '@/shared/constants/perspective';
import { nav } from '../../../views/nav';
import { nav } from '../../../views/shared/nav';
import { operatorAuthUtils } from '../../../support/shared/commands/auth-commands';
import { testCOORBACPersesTestsDevUser4 } from '../../../support/perses/rbac/perses_user4.cy';

Expand Down
Loading