-
Notifications
You must be signed in to change notification settings - Fork 167
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
Fix: Cypress Pipeline e2e test #3281
Fix: Cypress Pipeline e2e test #3281
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3281 +/- ##
==========================================
- Coverage 84.87% 84.86% -0.01%
==========================================
Files 1306 1306
Lines 29201 29201
Branches 7888 7888
==========================================
- Hits 24784 24782 -2
- Misses 4417 4419 +2 see 6 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
/lgtm |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks Fede!
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: christianvogt, manosnoam The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
New changes are detected. LGTM label has been removed. |
https://issues.redhat.com/browse/RHOAIENG-13384
Description
After some changes in the DSProjects view search toolbar, the Pipelines e2e test was failing.
Also I've updated the URL used to import the pipeline.
How Has This Been Tested?
cypress/job/dashboard-tests/18
After the PR is posted & before it merges:
main