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

KOGITO-8949: Recreate TrySample test for Serverless Logic Web Tools #2027

Merged
merged 1 commit into from
Nov 28, 2023

Conversation

tomasdavidorg
Copy link
Contributor

https://issues.redhat.com/browse/KOGITO-8949

This PR recreates the e2e test for trying samples from the catalog.

The catalog downloads the samples from GitHub repository which causes that the API limit is exceeded and the samples are not loaded.
In this PR, the test expects GitHub token as CYPRESS_E2E_TEST_GITHUB_TOKEN env. variable and run the tests only if the token is present.
GitHub action will not run the tests because currently there is no test token. It is primary focused on QE Jenkins runs but if needed we can add the token to the GitHub actions credentials.

@tomasdavidorg tomasdavidorg temporarily deployed to ci October 25, 2023 13:00 — with GitHub Actions Inactive
@tomasdavidorg tomasdavidorg temporarily deployed to ci October 25, 2023 13:00 — with GitHub Actions Inactive
@tomasdavidorg tomasdavidorg added waiting-for-qe pr: waiting-for-review Waiting for peer reviews pr: waiting-for-ci CI is still running or broken. area:sonataflow labels Oct 25, 2023
@tomasdavidorg tomasdavidorg temporarily deployed to ci October 27, 2023 06:18 — with GitHub Actions Inactive
@tomasdavidorg tomasdavidorg temporarily deployed to ci October 27, 2023 06:18 — with GitHub Actions Inactive
@tomasdavidorg tomasdavidorg temporarily deployed to ci October 27, 2023 06:18 — with GitHub Actions Inactive
@tomasdavidorg tomasdavidorg removed the pr: waiting-for-ci CI is still running or broken. label Oct 27, 2023
@tomasdavidorg
Copy link
Contributor Author

Rebased with main branch because of the Cypress update.

Copy link
Contributor

@LuboTerifaj LuboTerifaj left a comment

Choose a reason for hiding this comment

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

It looks good to me. Thanks!

@caponetto
Copy link
Contributor

@tomasdavidorg Could you please resolve the conflict so I can merge this PR?

@tomasdavidorg
Copy link
Contributor Author

@caponetto Rebased and the checks are green.

@caponetto caponetto merged commit 380b51b into apache:main Nov 28, 2023
8 checks passed
@tomasdavidorg tomasdavidorg deleted the KOGITO-8949 branch November 28, 2023 14:15
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.

4 participants