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

[CC Automation] Add test covering interaction between File screen and published Content Views #15414

Merged
merged 5 commits into from
Jun 21, 2024

Conversation

sambible
Copy link
Contributor

Problem Statement

Automation for this BZ: https://bugzilla.redhat.com/show_bug.cgi?id=2026701
Requires airgun PR SatelliteQE/airgun#1426

Related Issues

This area of Robottelo actually has no tests, so in a separate PR I'm going to raise at least 1 or 2 small tests for the other functionality in this area, just to get some coverage.

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py -k 'test_file_cv_display'
airgun: 1426

@sambible sambible added CherryPick PR needs CherryPick to previous branches 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Jun 14, 2024
@sambible sambible requested a review from a team June 14, 2024 15:28
@sambible sambible self-assigned this Jun 14, 2024
@sambible sambible requested a review from a team as a code owner June 14, 2024 15:28
@LadislavVasina1
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py -k 'test_file_cv_display'
airgun: 1426

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7400
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contentview.py -k test_file_cv_display --external-logging
Test Result : ============ 3 deselected, 9 warnings, 1 error in 198.70s (0:03:18) ============

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jun 17, 2024
@sambible
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py -k 'test_file_cv_display'
airgun: 1426

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7412
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_contentview.py -k test_file_cv_display --external-logging
Test Result : =========== 1 passed, 3 deselected, 14 warnings in 662.09s (0:11:02) ===========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Jun 17, 2024
Copy link
Contributor

@vijaysawant vijaysawant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Trying to correct sentances (I might be wrong) from test doc string > steps & expected results
  • Suggestion to use Verifies field
  • Suggestion to use single quotes for strings using inside test module to make it consistent.

tests/foreman/ui/test_contentview.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_contentview.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_contentview.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_contentview.py Outdated Show resolved Hide resolved
tests/foreman/ui/test_contentview.py Show resolved Hide resolved
tests/foreman/ui/test_contentview.py Outdated Show resolved Hide resolved
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 18, 2024
Copy link
Contributor

@vijaysawant vijaysawant left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for work, looks good to me.
Pending PRT result.

@vijaysawant
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py -k 'test_file_cv_display'
airgun: 1426

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7440
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_contentview.py -k test_file_cv_display --external-logging
Test Result : =========== 1 passed, 3 deselected, 14 warnings in 681.55s (0:11:21) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 19, 2024
@damoore044 damoore044 merged commit 921df06 into SatelliteQE:master Jun 21, 2024
10 checks passed
github-actions bot pushed a commit that referenced this pull request Jun 21, 2024
… published Content Views (#15414)

* Add test for customer case BZ

* Update docstring

Co-authored-by: vijay sawant <[email protected]>

* Fix whitespace issue

---------

Co-authored-by: vijay sawant <[email protected]>
(cherry picked from commit 921df06)
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
… published Content Views (SatelliteQE#15414)

* Add test for customer case BZ

* Update docstring

Co-authored-by: vijay sawant <[email protected]>

* Fix whitespace issue

---------

Co-authored-by: vijay sawant <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants