[CV Eval] add test for customer case bz #15268
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sambible
added
CherryPick
|
trigger: test-robottelo |
|
PRT Result |
Satellite-QE
added
the
PRT-Failed
vijaysawant
reviewed
Jul 3, 2024
…to republish-metadata
|
trigger: test-robottelo |
|
PRT Result |
|
PRT Result |
|
trigger: test-robottelo |
|
PRT Result |
|
trigger: test-robottelo |
|
PRT Result |
|
trigger: test-robottelo |
|
PRT Result |
|
trigger: test-robottelo |
|
PRT Result |
vijaysawant
requested changes
Aug 12, 2024
Comment on lines
+150
to
+152
| target_sat.cli.ContentView.version_republish_repositories( | ||
| {'id': version.id, 'force': 'false'} | ||
| ) |
There was a problem hiding this comment.
Suggested change
| target_sat.cli.ContentView.version_republish_repositories( | |
| {'id': version.id, 'force': 'false'} | |
| ) | |
| target_sat.cli.ContentView.version_republish_repositories( | |
| {'id': version.id} | |
| ) |
Passing 'force': 'false' is optional I guess, could you please check it locally and update here
| ) | ||
| rh_repo = target_sat.api.Repository(id=rh_repo_id).read() | ||
| cv = target_sat.api.ContentView(organization=function_sca_manifest_org).create() | ||
| cv = target_sat.api.ContentView(id=cv.id, repository=[rh_repo]).update(["repository"]) |
There was a problem hiding this comment.
Suggested change
| cv = target_sat.api.ContentView(id=cv.id, repository=[rh_repo]).update(["repository"]) | |
| cv = target_sat.api.ContentView(id=cv.id, repository=[rh_repo]).update(['repository']) |
Comment on lines
+126
to
+128
| 5. Verify that you can't click the "republish metadata" option from the UI | ||
| 6. Verify that you can't republish metadata from the cli without the force option | ||
| 7. Verify that you can republish metadata from the CLI using the --force option |
There was a problem hiding this comment.
Suggested change
| 5. Verify that you can't click the "republish metadata" option from the UI | |
| 6. Verify that you can't republish metadata from the cli without the force option | |
| 7. Verify that you can republish metadata from the CLI using the --force option | |
| 5. Verify that the "republish metadata" option from the UI doesn't work | |
| 6. Verify that the "republish metadata" from the CLI without the force option doesn't work | |
| 7. Verify that the "republish metadata" from the CLI using the --force = true/yes/1 does work |
| @@ -110,6 +113,53 @@ def test_no_blank_page_on_language_switch(session, target_sat, module_org): | |||
| assert session.contentview_new.read_french_lang_cv() | |||
|
|
|||
|
|
|||
| def test_republish_metadata(session, function_sca_manifest_org, target_sat): | |||
| """Verify that you can't republish metadata from the UI, and you can from the CLI | |||
There was a problem hiding this comment.
Suggested change
| """Verify that you can't republish metadata from the UI, and you can from the CLI | |
| """Verify that republish metadata from the UI doesn't work and from the CLI it does work with --force true/yes/1 option |
| 6. Verify that you can't republish metadata from the cli without the force option | ||
| 7. Verify that you can republish metadata from the CLI using the --force option | ||
|
|
||
| :expectedresults: You can't republish RH Repo metadata from the UI, and can from the CLI with --force |
There was a problem hiding this comment.
Suggested change
| :expectedresults: You can't republish RH Repo metadata from the UI, and can from the CLI with --force | |
| :expectedresults: "republish RH Repo metadata" from UI not worked and it worked from the CLI with --force true/yes/1 option |
|
@sambible I'm converting this pr to draft for now, please mark it ready for review once you're done with changes/addressing the comments. |
|
trigger: test-robottelo |
|
PRT Result |
damoore044
reviewed
Oct 18, 2024
|
@sambible if you rebase the Airgun PR with changes we merged yesterday, prt may pass for you now as-is. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem Statement
Adding BZ coverage for https://bugzilla.redhat.com/show_bug.cgi?id=2227271
Requires: SatelliteQE/airgun#1411
trigger: test-robottelo
pytest: tests/foreman/ui/test_contentview.py -k 'test_republish_metadata'
airgun: 1411