Skip to content
This repository was archived by the owner on Jan 23, 2026. It is now read-only.

Add POM Refactor#124

Open
alvitazwar wants to merge 1 commit intomasterfrom
add/pom
Open

Add POM Refactor#124
alvitazwar wants to merge 1 commit intomasterfrom
add/pom

Conversation

@alvitazwar
Copy link
Contributor

  • Refactor Test cases
  • Added New Test case
  • Added New Assertions

await expect(this.page.locator(selectors.leftAlignedValidate).first()).toBeVisible();
}
// this function is used to set Right aligned in top position with iconCount style Social Icons
async setRightAlignTopPlaceIcon(){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alvitazwar
Add assertion here to validate whether the element is a checkbox for the right alignment is checked.
expect(this.page.locator(selectors.rightAlignment)).toBeChecked();

}
// this function is used to set Right aligned in top position with iconCount style Social Icons
async setRightAlignTopPlaceIcon(){
await this.page.locator(selectors.topPlacement).check();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here as well

Copy link
Contributor

@pooja-muchandikar pooja-muchandikar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alvitazwar

  • Remove the artifacts folder.
  • Along with POM implementation, need to update the playwright version as well. Do update the playwright version in package.json.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants