Closed dfitchett closed 1 month ago
118 tests ±0 118 :white_check_mark: ±0 19s :stopwatch: -1s 34 suites ±0 0 :zzz: ±0 34 files ±0 0 :x: ±0
Results for commit e5529d77. ± Comparison against base commit 1a976cb0.
:recycle: This comment has been updated with latest results.
Overall Project | 70.98% | :x: |
---|
There is no coverage information present for the Files changed
What was the problem?
Currently, EP Merge does not abort jobs that should have been ineligible from the start.
Associated tickets or Slack threads:
3241
How does this fix it?
Adds an ABORTING and ABORTED state. See acceptance criteria in the ticket for when jobs should be aborted.
This is the old state machine:
And this is the new implementation:
How to test this PR
pytest
pytest ./integration
export BIP_CLAIM_URL=mock-bip-claims-api:20300
COMPOSE_PROFILES=“all” ./gradlew dockerComposeUp
COMPOSE_PROFILES="bip,bgs" ./gradlew -p mocks :dockerComposeUp
./gradlew :domain-ee:ee-ep-merge-app:endToEndTest