Skip to content
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

remove summary from module streams details section #15586

Conversation

vijaysawant
Copy link
Contributor

@vijaysawant vijaysawant commented Jul 5, 2024

Problem Statement

  • Test from foreman UI module stream is failing due to change in xpath from ModuleStreamView and ModuleStreamDetailsView classes when webdriver trying to access header breadcrumb and module stream details tab.
  • Summary from details tab is getting empty value as there is nothing on UI, also hammer cli doesn't have summary key/value
[root@satellite-ip ~]# hammer module-stream info --id 1
Id:                 1
Module Stream Name: walrus
Stream:             5.21
Uuid:               /pulp/api/v3/content/rpm/modulemds/01908233-44b2-7c7e-aa28-5a950aa04ae0/
Version:            20180704144203
Architecture:       x86_64
Context:            deadbeef
Repositories:       
 1) Id:   7
    Name: mQVxvxizcV

Solution

  • Update xpaths from airgun view modulestream classes
  • Summary section doesn't give any value so keeping it empty (added screenshot for reference)

Related Issues

Airgun PR: SatelliteQE/airgun#1456

PRT Comment

trigger: test-robottelo
pytest: tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo
airgun: 1456

Screenshot from 2024-07-05 18-12-01

@vijaysawant vijaysawant added UI Issues and PRs involving the UI TestFailure Issues and PRs related to a test failing in automation Easy Fix :) Easiest Fix to review and quick merge request. Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing Stream Introduced in or relating directly to Satellite Stream/Master 6.15.z Introduced in or relating directly to Satellite 6.15 labels Jul 5, 2024
@vijaysawant vijaysawant added CherryPick PR needs CherryPick to previous branches and removed Auto_Cherry_Picked Automatically cherrypicked PR using GHA labels Jul 5, 2024
@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7657
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo --external-logging
Test Result : ================== 1 failed, 10 warnings in 725.29s (0:12:05) ==================

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jul 5, 2024
@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo
airgun: 1456

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7659
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo --external-logging
Test Result : ================== 1 failed, 10 warnings in 704.98s (0:11:44) ==================

@vijaysawant
Copy link
Contributor Author

testing running fine locally

Testing started at 7:44 pm ...

Launching pytest with arguments -k test_positive_module_stream_details_search_in_repo -vvs /--path--/workspace/robottelo/tests/foreman/ui/test_modulestreams.py --no-header --no-summary -q in /--path--/workspace/robottelo

2024-07-05 19:45:10 - robottelo - WARNING - The snap version was not provided. Snap number will not be used in the URL.
2024-07-05 19:45:12 - robottelo - WARNING - The snap version was not provided. Snap number will not be used in the URL.
2024-07-05 19:45:17 - robottelo - WARNING - The snap version was not provided. Snap number will not be used in the URL.
2024-07-05 19:45:19 - robottelo - WARNING - The snap version was not provided. Snap number will not be used in the URL.
============================= test session starts ==============================
collecting ... 2024-07-05 19:46:46 - robottelo.collection - INFO - Processing test items to add testimony token markers
collected 1 item

tests/foreman/ui/test_modulestreams.py::test_positive_module_stream_details_search_in_repo [DEBUG 240705 19:51:03] Constructing host using kwargs={'hostname': '123infraasfesgridafdsgw.qwer23.2134.ytebc.com'}
[DEBUG 240705 19:51:03] 123infraasfesgridafdsgw.qwer23.2134.ytebc.com executing command: rm -rf /var/www/html/videos/f28b9e113c0c6277cef9
[DEBUG 240705 19:51:06] 123infraasfesgridafdsgw.qwer23.2134.ytebc.com command result:
    stdout:
    
    stderr:
    
    status: 0


================== 1 passed, 78 warnings in 260.01s (0:04:20) ==================

@vijaysawant
Copy link
Contributor Author

vijaysawant commented Jul 15, 2024

trigger: test-robottelo
pytest: tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo
airgun: 1456
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.15'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7748
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo --external-logging
Test Result : ======================== 6 warnings, 1 error in 51.35s =========================

@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo
airgun: 1456
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.15'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7899
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_modulestreams.py -k test_positive_module_stream_details_search_in_repo --external-logging
Test Result : =================== 6 warnings, 1 error in 66.91s (0:01:06) ====================

@jyejare jyejare added the 6.16.z Introduced in or relating directly to Satellite 6.16 label Aug 22, 2024
Copy link

github-actions bot commented Oct 7, 2024

This pull request has not been updated in the past 45 days.

@github-actions github-actions bot added the Stale Stale issue or Pull Request label Oct 7, 2024
Copy link

This pull request is now being closed after stale warnings.

@github-actions github-actions bot closed this Oct 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.15.z Introduced in or relating directly to Satellite 6.15 6.16.z Introduced in or relating directly to Satellite 6.16 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request. PRT-Failed Indicates that latest PRT run is failed for the PR Stale Stale issue or Pull Request Stream Introduced in or relating directly to Satellite Stream/Master TestFailure Issues and PRs related to a test failing in automation UI Issues and PRs involving the UI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants