-
Notifications
You must be signed in to change notification settings - Fork 18
Read and used in page objects #75
Read and used in page objects #75
Conversation
dbab6ea
to
aaedc24
Compare
needs to be rebased because of conflict after merging #74 |
@@ -190,6 +192,7 @@ public void deselectByIndices(final Integer... indices) { | |||
String index = option.getAttribute("index"); | |||
allSelectedIndices.add(Integer.valueOf(index)); | |||
} | |||
this.markAsRead(); |
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.
all of these this.markAsRead()
could be replaced by adding the marking to the public List<WebElement> getAllSelectedOptions()
method
@@ -300,6 +307,7 @@ public Integer execute(PageObject pageObject) { | |||
|
|||
@Override | |||
public List<String> execute(PageObject pageObject) { | |||
pageObject.markAsRead(); |
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.
Most of the pageObject.markAsRead();
could be replaced by putting them into getAllSelectedOptions()
, getFirstSelectedOption()
and getAllOptions()
try { | ||
pageObject.markAsRead(); |
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.
can be moved to getSelectedOption()
…GenericTextField and MultiSelect
aaedc24
to
1816b38
Compare
…electedOption()`, `getAllOptions()` and `getSelectedOption()` for better understanding
Review status: 0 of 6 files reviewed at latest revision, all discussions resolved. Comments from Reviewable |
Reviewed 3 of 6 files at r1, 3 of 3 files at r2. Comments from Reviewable |
Branch where I added
markAsRead()
to relevant methods of multiple page objects#74 is required for this pull request
This change is