-
Notifications
You must be signed in to change notification settings - Fork 164
[1LP][RFR] - Test to check edit option for existing group #10120
Conversation
e7dbe78
to
954ae36
Compare
# Selecting first row from all group | ||
row = view.table[0] | ||
row[0].check() | ||
assert row[0].checked |
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.
Is this bit redundant? I think it is browser's job to keep the element checked once it was checked. We don't check these things and take them as granted working.
If there was a problem that this got unchecked by some javascript, then I believe we need some kind of wait + check or periodic checking of the checked state during some time interval anyway.
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.
Changes are done
954ae36
to
4e80b1c
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.
A few small changes, thanks for this PR!
failure_patterns=[msg]).waiting(timeout=120): | ||
|
||
# Selecting first row from all group | ||
row = view.table[0] |
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 docstring description and the testSteps and expectedResults sections indicate that the test should be for a custom group, and that the option is called 'Edit Group'. But the test clicks on the first group in the list of all groups, which is probably going to be the default group EvmGroup-administrator, not a custom group, and the option is 'Edit the selected Group'.
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
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 docstring description and expectedResults are still wrong:
Should be able to see "edit group" option for existing group after selecting the group
and
2. Able to see Edit Group option
The option is Edit the selected Group
, not edit group
or Edit Group
.
4e80b1c
to
541ccbe
Compare
541ccbe
to
d5292cb
Compare
d5292cb
to
f435fd2
Compare
f435fd2
to
9f4b951
Compare
9f4b951
to
3cf10c8
Compare
3cf10c8
to
2ca2b17
Compare
2ca2b17
to
d73ddb2
Compare
Purpose or Intent
PRT Run
{{pytest: cfme/tests/configure/test_access_control.py -k "test_select_edit_group" -v}}