quipucords / camayoc

Test automation framework that facilitates functional testing of quipucords.
https://camayoc.readthedocs.io/
GNU General Public License v3.0
5 stars 4 forks source link

Update error message for the exclude_hosts option when testing sources with CLI #384

Closed ruda closed 1 year ago

ruda commented 1 year ago

Pytest execution (snippet).

E               0: re.compile(b'exclude_hosts: The exclude_hosts option is not valid for source of type satellite.')

/Users/rmoura/Developer/camayoc/.venv/lib/python3.11/site-packages/pexpect/expect.py:122: EOF
=================================== short test summary info ====================================
FAILED camayoc/tests/qpc/cli/test_sources.py::test_add_exclude_hosts_negative[vcenter-192.168.0.42-192.168.0.43] - pexpect.exceptions.EOF: End Of File (EOF). Empty string style platform.
FAILED camayoc/tests/qpc/cli/test_sources.py::test_add_exclude_hosts_negative[satellite-192.168.0.42-192.168.0.43] - pexpect.exceptions.EOF: End Of File (EOF). Empty string style platform.

The qpc error message has changed for the exclude_hosts option.

$ qpc source add --name 54d0fac1-d178-4063-a5ec-f58ada0b2e56 --cred 948bf0dd-73ad-4190-b3d3-ff076da4da7b --hosts 192.168.0.42 --exclude-hosts 192.168.0.43 --type satellite
exclude_hosts: The exclude_hosts option is not valid for this source.

Related to #383.

codecov[bot] commented 1 year ago

Codecov Report

Merging #384 (cef2bec) into master (b0ef228) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #384   +/-   ##
=======================================
  Coverage   79.91%   79.91%           
=======================================
  Files           5        5           
  Lines         229      229           
=======================================
  Hits          183      183           
  Misses         46       46           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more