-
Notifications
You must be signed in to change notification settings - Fork 1
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 POM and Update Tests Name #27
base: master
Are you sure you want to change the base?
Conversation
expect(await this.page.locator(selectors.pageCheckbox).isChecked()).toBeTruthy(); | ||
} | ||
// this function is used to Save settings and verify | ||
async saveSettings(){ |
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.
@alvitazwar Add assertion here and validate whether the settings are saved.
await this.page.locator(selectors.backendPopover).click(); | ||
} | ||
// this fucntion is used to Publish and verify page/post | ||
async publishPagePost(){ |
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.
Add assertion here as well to validate if the page is published.
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.
@alvitazwar Along with POM implementation, need to update the playwright version as well. Do update the playwright version in package.json.
yml
File (We can close this PR in this case)