Skip to content
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

Update e2e ui tests, harmonize the inputs and test running #77

Merged
merged 1 commit into from
Feb 6, 2025

Conversation

sjd78
Copy link
Member

@sjd78 sjd78 commented Feb 5, 2025

Bring the two global ci workflows in harmony with each other by:

  • Both workflows can specify inputs:

    • api_tests_tiers to define the API tests to run
    • ui_test_tags to define the UI tests to run
  • Both workflows will docker load all of the tar files present in their defined (artifact or component_name input props) downloaded artifacts

  • e2e UI tests do not need to run in parallel

  • Default e2e UI tests to run @tier0 tests

  • Bump to action UI tests to use cypress-io/github-action@v6

Bring the two global ci workflows in harmony with each other by:

  - Both workflows can specify inputs:
    - `api_tests_tiers` to define the API tests to run
    - `ui_test_tags` to define the UI tests to run

  - Both workflows will `docker load` all of the tar files
    present in their defined (`artifact` or `component_name`
    input props) downloaded artifacts

  - e2e UI tests do not need to run in parallel

  - Default e2e UI tests to run `@tier0` tests

  - Bump to action UI tests to use `cypress-io/github-action@v6`

Signed-off-by: Scott J Dickerson <[email protected]>
@sjd78 sjd78 requested a review from aufi February 5, 2025 22:51
Copy link
Member

@aufi aufi left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@sjd78 sjd78 merged commit 3b60dbd into konveyor:main Feb 6, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants