Skip to content

fixed cypress test container#12

Merged
stijnpotters1 merged 1 commit intomasterfrom
fix/cypress-e2e-container
Nov 5, 2025
Merged

fixed cypress test container#12
stijnpotters1 merged 1 commit intomasterfrom
fix/cypress-e2e-container

Conversation

@stijnpotters1
Copy link
Contributor

@stijnpotters1 stijnpotters1 commented Nov 5, 2025

When the Cypress test fails internally, the Cypress test container doesn't recognize the error. The unit test still passes, and the pipeline succeeds.
With this adjustment it is fixed.

[INFO] 
[INFO] Results:
[INFO]
[ERROR] Failures: 
[ERROR]   RunCypressE2eTest.runCypressTests:116 Cypress tests failed! 1 test(s) failed.

Cypress tests run: 67
Cypress tests passing: 66
Cypress tests failing: 1
[INFO]
[ERROR] Tests run: 253, Failures: 1, Errors: 0, Skipped: 0

Copy link
Contributor

@evandongen evandongen left a comment

Choose a reason for hiding this comment

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

Nice!

@stijnpotters1 stijnpotters1 merged commit 1d63f48 into master Nov 5, 2025
1 check passed
@stijnpotters1 stijnpotters1 deleted the fix/cypress-e2e-container branch November 5, 2025 22:13
@github-project-automation github-project-automation bot moved this from Review to Done in Frank!Framework Nov 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants