Skip to content

Commit 8ccba8b

Browse files
authored
[Profiling] Remove profiling Cypress tests in favor of the new Scout tests (#242255)
Closes [#210174](#210174) ## Summary After adding the e2e tests in #239964 , we are safe to remove the Cypress ones > [!IMPORTANT] > Please check also: #242030 It fixes the differential functions ones and it's good to be merged first :)
1 parent c577a5f commit 8ccba8b

37 files changed

+3
-28084
lines changed

.buildkite/ftr_oblt_stateful_configs.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ disabled:
66
- x-pack/solutions/observability/plugins/observability_onboarding/e2e/ftr_config.ts
77
- x-pack/solutions/observability/plugins/inventory/e2e/ftr_config_run.ts
88
- x-pack/solutions/observability/plugins/inventory/e2e/ftr_config.ts
9-
- x-pack/solutions/observability/test/profiling_cypress/cli_config.ts
109
#FTR configs
1110
- x-pack/solutions/observability/plugins/uptime/e2e/config.ts
1211
- x-pack/solutions/observability/test/api_integration/config.ts

.buildkite/pipelines/flaky_tests/groups.json

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -112,10 +112,6 @@
112112
"key": "cypress/apm_cypress",
113113
"name": "APM - Cypress"
114114
},
115-
{
116-
"key": "cypress/profiling_cypress",
117-
"name": "Profiling - Cypress"
118-
},
119115
{
120116
"key": "cypress/cloud_security_posture",
121117
"name": "Cloud Security Posture - Cypress"

.buildkite/pipelines/on_merge.yml

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -601,21 +601,6 @@ steps:
601601
- exit_status: '-1'
602602
limit: 1
603603

604-
- command: .buildkite/scripts/steps/functional/profiling_cypress.sh
605-
label: 'Profiling Cypress Tests'
606-
agents:
607-
image: family/kibana-ubuntu-2404
608-
imageProject: elastic-images-prod
609-
provider: gcp
610-
machineType: n2-standard-4
611-
preemptible: true
612-
timeout_in_minutes: 120
613-
parallelism: 3
614-
retry:
615-
automatic:
616-
- exit_status: '-1'
617-
limit: 1
618-
619604
- command: .buildkite/scripts/steps/functional/apm_cypress.sh
620605
label: 'APM Cypress Tests'
621606
agents:

.buildkite/pipelines/pull_request/profiling_cypress.yml

Lines changed: 0 additions & 22 deletions
This file was deleted.

.buildkite/scripts/pipelines/pull_request/pipeline.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -101,13 +101,6 @@ const getPipeline = (filename: string, removeSteps = true) => {
101101
pipeline.push(getPipeline('.buildkite/pipelines/pull_request/apm_cypress.yml'));
102102
}
103103

104-
if (
105-
(await doAnyChangesMatch([/^x-pack\/solutions\/observability\/plugins\/profiling/])) ||
106-
GITHUB_PR_LABELS.includes('ci:all-cypress-suites')
107-
) {
108-
pipeline.push(getPipeline('.buildkite/pipelines/pull_request/profiling_cypress.yml'));
109-
}
110-
111104
if (
112105
(await doAnyChangesMatch([
113106
/^x-pack\/platform\/plugins\/shared\/fleet/,

.buildkite/scripts/steps/functional/profiling_cypress.sh

Lines changed: 0 additions & 16 deletions
This file was deleted.

.github/CODEOWNERS

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2951,9 +2951,6 @@ x-pack/solutions/security/plugins/security_solution/server/lib/security_integrat
29512951
# Changes to translation files should not ping code reviewers
29522952
x-pack/platform/plugins/private/translations/translations
29532953

2954-
# Profiling cypress tests
2955-
x-pack/solutions/observability/test/profiling_cypress @elastic/obs-ux-infra_services-team
2956-
29572954
# Profiling api integration testing
29582955
x-pack/solutions/observability/test/api_integration/profiling @elastic/obs-ux-infra_services-team
29592956

x-pack/solutions/observability/plugins/profiling/README.md

Lines changed: 3 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -46,29 +46,11 @@ The authentication and authorization on this endpoint are provided as part of th
4646
node scripts/jest --config x-pack/solutions/observability/plugins/profiling/jest.config.js [--watchAll]
4747
```
4848

49-
## E2E Tests (Cypress)
50-
The E2E tests are located in [`x-pack/solutions/observability/plugins/profiling/e2e`](./e2e).
49+
## E2E Tests (Scout)
5150

52-
Universal Profiling uses [FTR](../../../../src/platform/packages/shared/kbn-test/README.mdx) (functional test runner) and [Cypress](https://www.cypress.io/) to run the e2e tests. The tests are located at `kibana/x-pack/solutions/observability/plugins/profiling/e2e/cypress/e2e`.
51+
The E2E tests are located in [`x-pack/solutions/observability/plugins/profiling/test/scout`](./test/scout).
5352

54-
### Start with Cypress Dashboard
55-
56-
```
57-
node x-pack/solutions/observability/plugins/profiling/scripts/test/e2e --open
58-
```
59-
60-
### Run tests
61-
Runs all tests in the terminal
62-
63-
```
64-
node x-pack/solutions/observability/plugins/profiling/scripts/test/e2e
65-
```
66-
67-
### Run tests in headed mode
68-
69-
```
70-
node x-pack/solutions/observability/plugins/profiling/scripts/test/e2e --headed
71-
```
53+
Universal Profiling uses [Scout](https://github.com/elastic/kibana/tree/main/x-pack/solutions/observability/packages/kbn-scout-oblt) (Playwright-based test framework) to run the e2e tests. For detailed instructions on how to run the tests, see the [Scout tests README](./test/scout/README.md).
7254

7355
## Other resources
7456
- [Official Profiling documentation](https://www.elastic.co/observability/universal-profiling)

x-pack/solutions/observability/plugins/profiling/e2e/README.md

Lines changed: 0 additions & 45 deletions
This file was deleted.

x-pack/solutions/observability/plugins/profiling/e2e/cypress.config.ts

Lines changed: 0 additions & 34 deletions
This file was deleted.

0 commit comments

Comments
 (0)