Feature/cypress test #148
Feature/cypress test #148
Conversation
github-actions
bot
added
the
needs:feedback
|
@sudip-md thanks for the PR! Could you please fill out the PR template with description, changelog, and credits information so that we can properly review and merge this? |
jeffpaul
requested review from
a team and
iamdharmesh
and removed request for
dkotter and
a team
There was a problem hiding this comment.
Hi @sudip-md, Thanks for adding additional tests and raising a PR. I have updated the initialize script to fix tests failing in the GH action environment. However, I have added 2 notes on the PR. Could you please help with checking on those? Once it is resolved, we will be able to merge this.
Thank you.
| it("Create Article Status", () => { | ||
| cy.visit(`wp-admin/edit-tags.php?taxonomy=pi-article-status&post_type=print-issue`); | ||
| cy.get('#tag-name').type('Active-01'); | ||
| cy.get('#submit').click(); |
There was a problem hiding this comment.
We might want to add a verification step here to confirm that the article status was created successfully.
There was a problem hiding this comment.
@iamdharmesh probably best if you help get things updated for this PR and merge in
| cy.get('[aria-label="“Active-01” (Edit)"]').click(); | ||
| cy.get('#name').clear(); | ||
| cy.get('#name').type('Active-02') | ||
| cy.get('.button').click(); |
There was a problem hiding this comment.
Same as above, Add a verification step to confirm status edited successfully. Could you please help to add verification steps for all the tests?
###Local Test Result
Description of the Change
Added and update test cases
Closes #
How to test the Change
Changelog Entry
Checklist: