search

ngine-io / ansible-collection-cloudstack

CloudStack Ansible Collections
https://galaxy.ansible.com/ngine_io/cloudstack
GNU General Public License v3.0
21 stars 28 forks source link

fixes issue #51 #52

Closed rvalle closed 3 years ago

rvalle commented 3 years ago

Handling also SystemExit enables the module configuration to be reached.

codecov[bot] commented 3 years ago

Codecov Report

Merging #52 (54b2f83) into master (94653d3) will decrease coverage by 0.01%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #52      +/-   ##
==========================================
- Coverage   84.39%   84.37%   -0.02%     
==========================================
  Files          53       53              
  Lines        5512     5512              
  Branches     1247     1247              
==========================================
- Hits         4652     4651       -1     
- Misses        430      432       +2     
+ Partials      430      429       -1
Impacted Files Coverage Δ
plugins/module_utils/cloudstack.py 72.58% <0.00%> (-0.23%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 94653d3...269d75d. Read the comment docs.

rvalle commented 3 years ago

I see a problem with this. We loose quality in the error when the cloudstack.ini file is not found. bare with me...

rvalle commented 3 years ago

OK, I am refactoring this as part of #26