Skip to content

test: improve reliability of status action logs tests - #1756

Open
simonfernandes wants to merge 3 commits into
developfrom
improve-status-action-logs-tests
Open

test: improve reliability of status action logs tests#1756
simonfernandes wants to merge 3 commits into
developfrom
improve-status-action-logs-tests

Conversation

@simonfernandes

Copy link
Copy Markdown
Contributor

Description

Hopefully fixes the flakiness we are seeing with the status action logs tests currrently.

  1. Instead of waiting for a predetermined amount of time for the status actions to complete, this will keep polling the status actions logs in the beforeEach() to ensure they have successfully completed.
  2. Also simplifies a test that was creating a new proposal and waiting unnecessarily.

Motivation and Context

Addressing comment from Yoga about test flakiness

How Has This Been Tested

Ran e2e locally

Fixes

n/a

Changes

  1. Updates beforeEach() with polling change.
  2. Updates a single test to simplify.

Depends on

Tests included/Docs Updated?

  • I have added tests to cover my changes.
  • All relevant doc has been updated

@simonfernandes
simonfernandes requested review from a team, DChambersSTFC and yoganandaness and removed request for a team August 27, 2026 12:26
@simonfernandes
simonfernandes requested a review from a team as a code owner August 27, 2026 12:26
@simonfernandes
simonfernandes requested review from William-Edwards-STFC and removed request for a team August 27, 2026 12:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants