-
Notifications
You must be signed in to change notification settings - Fork 4
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
Stubbed out scan tests for CRUD #207
Conversation
Codecov Report
@@ Coverage Diff @@
## master #207 +/- ##
=======================================
Coverage 75.25% 75.25%
=======================================
Files 5 5
Lines 194 194
=======================================
Hits 146 146
Misses 48 48 Continue to review full report at Codecov.
|
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.
Left a few comments. Trust you to handle them appropriately.
One final request -- please squash these commits into one commit. Thanks!
@@ -344,6 +344,9 @@ | |||
QCS_SOURCE_TYPES = ('vcenter', 'network', 'satellite') | |||
"""Types of sources that the quipucords server supports.""" | |||
|
|||
QCS_SCAN_TYPES = ('inspect', 'connect') | |||
"""Types of scans that the quipucords server supports.""" | |||
|
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.
🎉
result = scan_add({ | ||
'name': scan_name, | ||
'sources': config_sources()[0]['name'], | ||
'sources': source_name |
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.
👍
'source_type': 'network'}]} | ||
|
||
assert expected_result == scan_show_result | ||
|
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.
If you wanted to be exhaustive you could somehow make this do more combinations by parametrizing the test ... but that gets done on API side too so may be redundant. Up to you
camayoc/tests/qcs/cli/test_scans.py
Outdated
:description: Perform a scan and disable an optional product. | ||
:steps: Run ``qpc scan start --sources <source> --disable-optional-products | ||
:steps: | ||
1) Run ``qpc scan start --sources <source> --disable-optional-products |
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.
I thought this module would not have any running scans?
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.
Copy/paste error. Fixed it now.
No description provided.