Add operations for automation #4250
Conversation
pranavprakash20
requested review from
vdas-redhat,
sayaleeraut,
vinpapnoi,
vamahaja,
MohitBis,
pranavprakash20,
tintumathew10 and
adityaramteke
ceph/UI/helper.py
Outdated
| def take_ss(driver, filename: str): | ||
| """Takes a screenshot and saves it to the specified filename.""" | ||
| try: | ||
| driver.save_screenshot(filename) |
There was a problem hiding this comment.
Add logic on returning the screenshot path. The user should be able to get the file name upon calling this method
ceph/UI/helper.py
Outdated
| raise RuntimeError(f"Failed to take screenshot: {e}") | ||
|
|
||
|
|
||
| def wait_for_element(driver, element_id: str, timeout: int = 10): |
There was a problem hiding this comment.
Extend the logic for all element types like class, name, xpath, css selector etc and not just to id.
ceph/UI/operations.py
Outdated
| try: | ||
| username_field = self.driver.find_element_by_id("username") | ||
| password_field = self.driver.find_element_by_id("password") | ||
| login_button = self.driver.find_element_by_id("login") |
There was a problem hiding this comment.
Add logic to make it generic. Not always its the id, it can accept all other identifiers.
|
@NamrathaRao24 Please add a more meaningful commit message |
NamrathaRao24
force-pushed
the
add-operations-helper
branch
from
6592faa to
388b6e0
Compare
Signed-off-by: nrao <[email protected]>
|
This Pull request has been automatically marked as STALE due to inactivity for 15 days and will be CLOSED on further inactivity on the PR for another 15 days. |
Signed-off-by: nrao <[email protected]>
NamrathaRao24
force-pushed
the
add-operations-helper
branch
from
388b6e0 to
8237e60
Compare
|
I am unable to find |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: MohitBis, NamrathaRao24, pranavprakash20 The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
PRs under review, and the following points need to be addressed:
|
Description
Please include Automation development guidelines. Source of Test case - New Feature/Regression Test/Close loop of customer BZs
click to expand checklist