-
Notifications
You must be signed in to change notification settings - Fork 115
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
Test for the change host's content source feature #14004
Test for the change host's content source feature #14004
Conversation
trigger: test-robottelo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK, three non-blocking comments left.
Great work, @LadislavVasina1!
capsule = module_target_sat.cli.Capsule.update( | ||
{ | ||
'name': module_capsule_configured.hostname, | ||
'organization-ids': org.id, | ||
'location-ids': loc.id, | ||
} | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking at the usage of capsule
in the test, the value seems to be a list of capsules. Can you access the value on the first index and store it in capsule
?
capsule = module_target_sat.cli.Capsule.update( | |
{ | |
'name': module_capsule_configured.hostname, | |
'organization-ids': org.id, | |
'location-ids': loc.id, | |
} | |
) | |
capsule = module_target_sat.cli.Capsule.update( | |
{ | |
'name': module_capsule_configured.hostname, | |
'organization-ids': org.id, | |
'location-ids': loc.id, | |
} | |
)[0] |
ff08d2c
to
883eeae
Compare
|
PRT Result
|
* Test for change hosts content source * Fix comment * Set fixture to module scope * Fix comments * Add some asserts * Add imports * Remove comment (cherry picked from commit 57db84f)
* Test for change hosts content source * Fix comment * Set fixture to module scope * Fix comments * Add some asserts * Add imports * Remove comment (cherry picked from commit 57db84f)
* Test for change hosts content source * Fix comment * Set fixture to module scope * Fix comments * Add some asserts * Add imports * Remove comment
Problem Statement
Create a test for the change host's content source feature.
Solution
Fixture and test were created.
The fixture does the necessary setup on the satellite, host, and capsule.
Test exercises different ways how to change the host's content source.
Needs Airgun#1226.