Testing Object Storage Browser p2 [WIP] #11246
Conversation
AviadP
commented
Jan 28, 2025
•
AviadP
commented
- Moved folder generation file
- now uploading multiple files
- added new test for getting all buckets list
- Added list buckets test
- Added delete bucket test
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: AviadP 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 |
AviadP
changed the title
ocs_ci/ocs/ui/views.py
Outdated
| By.XPATH, | ||
| ), | ||
| "storage_class_noobaa_option": ( | ||
| "#openshift-storage\\.noobaa\\.io-link > a > div:nth-child(1) > span", |
There was a problem hiding this comment.
this will fail if storage namespace is not "openshift-storage"
we'll need to use config.ENV_DATA["cluster_namespace]
also: are you sure about index, that it will be always [1] ? here > div:nth-child(1)
ocs_ci/ocs/ui/views.py
Outdated
| By.XPATH, | ||
| ), | ||
| "create_bucket_button": ( | ||
| "//*[@id='yaml-create']", |
There was a problem hiding this comment.
this will perform better.
"create_resource_button": ("yaml-create", By.ID), we have it in "generic_locators" dict in views.py. You may remove this and use generic_locators["create_resource_button"]
| list: WebElement objects representing bucket links | ||
| """ | ||
| buckets = self.get_elements( | ||
| ("//a[starts-with(@href, '/odf/object-storage/buckets/')]", By.XPATH) |
There was a problem hiding this comment.
please move to views.py
| bucket_ui.nav_object_storage_page() | ||
| bucket_ui.create_folder_in_bucket() | ||
| bucket_ui.nav_buckets_page() | ||
| bucket_ui.get_buckets_list() |
There was a problem hiding this comment.
is it some kind of a libtest? do we want to add assertions to validate bucket list?
There was a problem hiding this comment.
its WIP. going to compare list by GUI to list by CLI
create bucket via OBC create bucket via s3 upload folder and file to bucket Signed-off-by: Aviadp <[email protected]>
Signed-off-by: Aviadp <[email protected]> Moved folder generation file, also, now uploading multiple files. added new test for getting all buckets list Signed-off-by: Aviadp <[email protected]> New test for couning buckets Signed-off-by: Aviadp <[email protected]>
