-
Notifications
You must be signed in to change notification settings - Fork 180
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
chore(pd): move tests and actions changed into chore_release-pd #17588
chore(pd): move tests and actions changed into chore_release-pd #17588
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## chore_release-pd-8.4.3 #17588 +/- ##
=========================================================
Coverage ? 18.58%
=========================================================
Files ? 2740
Lines ? 211530
Branches ? 6522
=========================================================
Hits ? 39322
Misses ? 172208
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more.
|
/** | ||
* "Rename Mix step" | ||
*/ | ||
Rename: (): StepThunk => ({ | ||
call: () => { | ||
cy.log('Renaming Mix Step ---------------------------') | ||
cy.get(MixLocators.RenameBtn).should('exist').should('be.visible').click() |
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.
Having the docstring on top of the function lets you see it in your IDE.
cy.log
gets the description into they Cypress UI so you may easily read which function you are in when composing or fixing tests.
…#17588 chore(pd): to refactor again
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.
LGTM, thanks for the work on this. Going to much easier to use/more effective in catching issues when working on pre-release builds.
comment: | ||
# add flags to `layout` configuration to show up in the PR comment | ||
layout: 'reach, diff, flags, files' | ||
- '**/*.{mjs,css,cjs,mts}' |
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.
👍
Get tests running against current PD work
This pull request is to get our release branches for pd using the latest Cypress tests and to use the latest PD workflow and code coverage settings.