Skip to content

Fix of test_host CLI test #18967

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Jul 9, 2025
Merged

Fix of test_host CLI test #18967

merged 2 commits into from
Jul 9, 2025
+13 −15

Conversation

LadislavVasina1
Copy link
Contributor

@LadislavVasina1 LadislavVasina1 commented Jul 9, 2025 •
edited
Loading

Problem Statement

The test test_positive_host_list_with_cv_and_lce was failing for a long time because it was using just CV and now we use content view environments.

Solution

Rename the test to test_positive_host_list_with_cve so it fits the test content properly and refactor it so it tests the content view environment.

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py -k 'test_positive_host_list_with_cve'

@LadislavVasina1 LadislavVasina1 self-assigned this Jul 9, 2025
@LadislavVasina1 LadislavVasina1 requested a review from a team as a code owner July 9, 2025 07:04
@LadislavVasina1 LadislavVasina1 added CherryPick PR needs CherryPick to previous branches 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.17.z labels Jul 9, 2025
@LadislavVasina1
Copy link
Contributor Author 

trigger: test-robottelo
pytest: tests/foreman/cli/test_host.py -k 'test_positive_host_list_with_cve'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 12023
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_host.py -k test_positive_host_list_with_cve --external-logging
Test Result : ========== 1 passed, 105 deselected, 7 warnings in 945.14s (0:15:45) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jul 9, 2025
@LadislavVasina1 LadislavVasina1 requested a review from a team July 9, 2025 07:41
@LadislavVasina1 LadislavVasina1 requested a review from pnovotny July 9, 2025 08:40
@pondrejk pondrejk merged commit 3529cef into SatelliteQE:master Jul 9, 2025
10 checks passed
github-actions bot pushed a commit that referenced this pull request Jul 9, 2025
@LadislavVasina1 @web-flow
Fix of test_host CLI test (#18967)
051337c 
* test_host_cli_test_fix

* ChangeName

(cherry picked from commit 3529cef)
@Satellite-QE Satellite-QE mentioned this pull request Jul 9, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lhellebr lhellebr lhellebr approved these changes

@pondrejk pondrejk pondrejk approved these changes

@pnovotny pnovotny Awaiting requested review from pnovotny

Assignees

@LadislavVasina1 LadislavVasina1

Labels
6.17.z AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@LadislavVasina1 @Satellite-QE @lhellebr @pondrejk