Skip to content

PB-1179: fix profile segment test not passing on CI #1336

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 12, 2025

Conversation

pakb
Copy link
Contributor

@pakb pakb commented May 12, 2025

adding some retry-ability to the tests so that it behaves better on the CI.

Test link

@github-actions github-actions bot added the bug label May 12, 2025
Copy link

cypress bot commented May 12, 2025

web-mapviewer    Run #5289

Run Properties:  status check passed Passed #5289  •  git commit e4084f94f4: PB-1179: fix profile segment test not passing on CI
Project web-mapviewer
Branch Review fix-PB-1179-add-retryability-to-tests
Run status status check passed Passed #5289
Run duration 05m 51s
Commit git commit e4084f94f4: PB-1179: fix profile segment test not passing on CI
Committer Pascal Barth
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 20
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 251
View all changes introduced in this branch ↗︎

@pakb pakb force-pushed the fix-PB-1179-add-retryability-to-tests branch from 2fe0e65 to 5b1c173 Compare May 12, 2025 06:36
@pakb pakb requested a review from sommerfe May 12, 2025 06:47
adding some retry-ability to the tests so that it behaves better on the CI.
@pakb pakb force-pushed the fix-PB-1179-add-retryability-to-tests branch from 5b1c173 to e4084f9 Compare May 12, 2025 06:49
@pakb pakb merged commit 7d38da9 into develop May 12, 2025
6 checks passed
@pakb pakb deleted the fix-PB-1179-add-retryability-to-tests branch May 12, 2025 07:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants