-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add step one test for wpml #46
Conversation
Need #45 merged first |
@Khadreal Could you merge trunk into this PR branch to keep it updated. |
d0c1a72
to
ba888ba
Compare
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.
Everything seems to be in order. Both scenarios have passed on my end. The steps accurately perform or verify what's required. I have also validated that the steps correspond with those in TestRail.
Description
To run this test, run this command
npm run test:wpml
Fixes #23
Type of change
Is the solution different from the one proposed during the grooming?
Please describe in this section if there is any change to the groomed solution, and why.
Checklists
Generic development checklist
Test summary
If not, detail what you could not test.
Please describe any additional tests you performed.