-
Notifications
You must be signed in to change notification settings - Fork 2
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
Locators,Action and Stepdesign class for hub #12
Conversation
1eef033
to
107421d
Compare
*/ | ||
public class CdfHubLocators { | ||
|
||
@FindBy(how = How.XPATH, using = "//input[@class='search-input form-control'][@placeholder='Search by name']") |
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.
No need to write them here which is related to control center , how can we extract those locators from control center locator file ? so that duplicate code can be avoided ?
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.
removed.
} | ||
@Then("Verify user can click on the delete icon of the plugin and plugin gets deleted successfully") | ||
public void deleteControlCenterPlugin() { | ||
CdfHubActions.deletePluginControlCenter(); |
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.
Where are we verifying that the plugin is actualy deleted after successful delete operation . Ideally we should be adding some validationsteps to search for the deleted plugin post delete
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.
added the validation step
} | ||
|
||
public static WebElement deleteButton() { | ||
return SeleniumDriver.getDriver().findElement(By.xpath("//button[@class='btn btn-primary']" + |
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.
Change to //button[@data-cy='Delete'] or //button[@data-testid='Delete']
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.
done.
2282768
to
fdeef41
Compare
public static WebElement deleteButton; | ||
@FindBy(how = How.XPATH, using = "//button[@id='navbar-hub']") | ||
public static WebElement hubPage; | ||
@FindBy(how = How.XPATH, using = "//*[contains(text(),'No entities found in namespace \"default\"')]") |
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.
As discussed, check for 'No results found' text after deleting
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.
The text which we are getting there is No entities found.
@FindBy(how = How.XPATH, using = "//button[@id='navbar-hub']") | ||
public static WebElement hubPage; | ||
@FindBy(how = How.XPATH, using = "//*[contains(text(),'No entities found in namespace \"default\"')]") | ||
public static WebElement verifyDelete; |
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.
I think verifyDelete can be changed to something else specific to deletion of element like that?
741a14d
to
a147e95
Compare
a147e95
to
811d0dd
Compare
c0592d1
to
47fb50b
Compare
No description provided.