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

Fix long failing AK CLI tests #17504

Merged

Conversation

LadislavVasina1
Copy link
Contributor

@LadislavVasina1 LadislavVasina1 commented Feb 5, 2025

Problem Statement

Some CLI AK tests did not work in the stream due to multi-CV changes.
The main goal is to get rid of Environment ID and content view ID.
Also some outdated asserts are fixed.

Solution

Mostly using module_promoted_cv fixture helps.

Related Issues

SAT-28579

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/cli/test_activationkey.py -k "test_positive_create_with_default_lce_by_id or test_positive_create_with_non_default_lce or test_positive_create_with_default_lce_by_name or test_positive_create_with_cv or test_positive_delete_with_lce or test_positive_update_lce or test_positive_remove_host_collection_by_id or test_positive_remove_host_collection_by_name or test_positive_update_autoattach_toggle"

@LadislavVasina1 LadislavVasina1 added No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Feb 5, 2025
@LadislavVasina1 LadislavVasina1 self-assigned this Feb 5, 2025
@LadislavVasina1 LadislavVasina1 requested a review from a team as a code owner February 5, 2025 07:30
@LadislavVasina1
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_activationkey.py -k "test_positive_create_with_default_lce_by_id or test_positive_create_with_non_default_lce or test_positive_create_with_default_lce_by_name or test_positive_create_with_cv or test_positive_delete_with_lce or test_positive_update_lce or test_positive_remove_host_collection_by_id or test_positive_remove_host_collection_by_name or test_positive_update_autoattach_toggle"

@LadislavVasina1 LadislavVasina1 added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 5, 2025
@jameerpathan111 jameerpathan111 merged commit 4e783c2 into SatelliteQE:master Feb 5, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need 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
Development

Successfully merging this pull request may close these issues.

3 participants