-
Notifications
You must be signed in to change notification settings - Fork 10
Open
Labels
Description
Description
In the current E2E test setup, four separate GitHub Actions are triggered to run tests for different categories:
@general
@cashapp
@sync
@giftcard
When the first E2E test completes successfully, the workflow adds the label status: e2e tests passing
to the PR. However, when subsequent E2E tests pass, the workflow removes and re-adds the same label, even though no change is necessary.
This behavior causes unnecessary label updates, leading to disturbances in the PR activity (e.g., redundant add/delete label).
Expected Behavior
- The workflow should only add/update labels if a change is required.
- If all E2E tests pass, the label
status: e2e tests passing
should be added once and not updated repeatedly. - If any E2E test fails, the workflow should:
- Remove the
status: e2e tests passing
label. - Add
status: e2e tests failing
to reflect the failure.
- Remove the
Steps to Reproduce
- Open a PR that triggers the E2E tests.
- Observe the labels being updated after each test completes.
- Notice that the same label is removed and re-added unnecessarily when multiple tests pass.
jeffpaul