Skip to content

Commit 3b53ef8

Browse files
authored
Fixes Cypress screenshots in buildkite (#243321)
### Problem After the Security Solution Cypress tests were migrated from `x-pack/test/security_solution_cypress` to `x-pack/solutions/security/test/security_solution_cypress`, seems like test failure screenshots are not being properly uploaded as artifacts in Buildkite CI pipelines. This is making debugging of test failures difficult as we couldn't access visual evidence of what went wrong during test execution. ### Root Cause The migration changed the relative path depth of the test files, which affected the screenshotsFolder configuration paths. ### Solution Verified screenshot folder paths: Confirmed that all Cypress configuration files correctly point to target/kibana-security-solution/cypress/screenshots, which matches the Buildkite artifact upload pattern (target/kibana-security-solution/**/*.png).
1 parent af0d5f3 commit 3b53ef8

File tree

5 files changed

+10
-10
lines changed

5 files changed

+10
-10
lines changed

x-pack/solutions/security/test/security_solution_cypress/cypress/cypress.config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,10 +19,10 @@ export default defineCypressConfig({
1919
execTimeout: 60000,
2020
pageLoadTimeout: 60000,
2121
responseTimeout: 60000,
22-
screenshotsFolder: '../../../../../../target/kibana-security-solution/cypress/screenshots',
22+
screenshotsFolder: '../../../../../target/kibana-security-solution/cypress/screenshots',
2323
trashAssetsBeforeRuns: false,
2424
video: false,
25-
videosFolder: '../../../../../../target/kibana-security-solution/cypress/videos',
25+
videosFolder: '../../../../../target/kibana-security-solution/cypress/videos',
2626
viewportHeight: 1200,
2727
viewportWidth: 1920,
2828
numTestsKeptInMemory: 10,

x-pack/solutions/security/test/security_solution_cypress/cypress/cypress_ci.config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,10 @@ export default defineCypressConfig({
2727
retries: {
2828
runMode: 1,
2929
},
30-
screenshotsFolder: '../../../../../../target/kibana-security-solution/cypress/screenshots',
30+
screenshotsFolder: '../../../../../target/kibana-security-solution/cypress/screenshots',
3131
trashAssetsBeforeRuns: false,
3232
video: false,
33-
videosFolder: '../../../../../../target/kibana-security-solution/cypress/videos',
33+
videosFolder: '../../../../../target/kibana-security-solution/cypress/videos',
3434
viewportHeight: 1200,
3535
viewportWidth: 1920,
3636
e2e: {

x-pack/solutions/security/test/security_solution_cypress/cypress/cypress_ci_serverless.config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,10 +28,10 @@ export default defineCypressConfig({
2828
retries: {
2929
runMode: 1,
3030
},
31-
screenshotsFolder: '../../../../../../target/kibana-security-solution/cypress/screenshots',
31+
screenshotsFolder: '../../../../../target/kibana-security-solution/cypress/screenshots',
3232
trashAssetsBeforeRuns: false,
3333
video: false,
34-
videosFolder: '../../../../../../target/kibana-security-solution/cypress/videos',
34+
videosFolder: '../../../../../target/kibana-security-solution/cypress/videos',
3535
viewportHeight: 1200,
3636
viewportWidth: 1920,
3737
e2e: {

x-pack/solutions/security/test/security_solution_cypress/cypress/cypress_ci_serverless_qa.config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,10 +30,10 @@ export default defineCypressConfig({
3030
retries: {
3131
runMode: 1,
3232
},
33-
screenshotsFolder: '../../../../../../target/kibana-security-solution/cypress/screenshots',
33+
screenshotsFolder: '../../../../../target/kibana-security-solution/cypress/screenshots',
3434
trashAssetsBeforeRuns: false,
3535
video: false,
36-
videosFolder: '../../../../../../target/kibana-security-solution/cypress/videos',
36+
videosFolder: '../../../../../target/kibana-security-solution/cypress/videos',
3737
viewportHeight: 1200,
3838
viewportWidth: 1920,
3939
e2e: {

x-pack/solutions/security/test/security_solution_cypress/cypress/cypress_serverless.config.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,10 @@ export default defineCypressConfig({
1616
execTimeout: 60000,
1717
pageLoadTimeout: 60000,
1818
responseTimeout: 60000,
19-
screenshotsFolder: '../../../../../../target/kibana-security-solution/cypress/screenshots',
19+
screenshotsFolder: '../../../../../target/kibana-security-solution/cypress/screenshots',
2020
trashAssetsBeforeRuns: false,
2121
video: false,
22-
videosFolder: '../../../../../../target/kibana-security-solution/cypress/videos',
22+
videosFolder: '../../../../../target/kibana-security-solution/cypress/videos',
2323
viewportHeight: 1200,
2424
viewportWidth: 1920,
2525
numTestsKeptInMemory: 10,

0 commit comments

Comments
 (0)