Skip to content
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

Add open_url test framework #24

Merged
merged 14 commits into from Dec 11, 2020

Conversation

felixfontein
Copy link
Collaborator

SUMMARY

Analogue to the fetch_url module test framework.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

tests/unit/utils/open_url_framework.py

@codecov
Copy link

codecov bot commented Dec 7, 2020

Codecov Report

Merging #24 (42d3803) into main (5e2759a) will increase coverage by 16.14%.
The diff coverage is 91.78%.

Impacted file tree graph

@@             Coverage Diff             @@
##             main      #24       +/-   ##
===========================================
+ Coverage   55.91%   72.05%   +16.14%     
===========================================
  Files           6        9        +3     
  Lines         279      476      +197     
  Branches       33       55       +22     
===========================================
+ Hits          156      343      +187     
- Misses        113      117        +4     
- Partials       10       16        +6     
Impacted Files Coverage Δ
tests/unit/utils/fetch_url_module_framework.py 59.72% <12.50%> (+0.28%) ⬆️
tests/unit/utils/open_url_framework.py 92.00% <92.00%> (ø)
plugins/lookup/open_url_test_lookup.py 100.00% <100.00%> (ø)
plugins/modules/fetch_url_test_module.py 78.57% <100.00%> (ø)
...s/unit/plugins/lookup/test_open_url_test_lookup.py 100.00% <100.00%> (ø)
...unit/plugins/modules/test_fetch_url_test_module.py 100.00% <100.00%> (ø)

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 5e2759a...42d3803. Read the comment docs.

felixfontein added a commit to felixfontein/community.hrobot that referenced this pull request Dec 7, 2020
@felixfontein
Copy link
Collaborator Author

@felixfontein felixfontein changed the title [WIP] Add open_url test framework Add open_url test framework Dec 9, 2020
@felixfontein felixfontein merged commit 484e711 into ansible-collections:main Dec 11, 2020
@felixfontein felixfontein deleted the open-url-framework branch December 11, 2020 05:42
felixfontein added a commit to ansible-collections/community.hrobot that referenced this pull request Dec 11, 2020
* Move generic open_url code for plugins to module_utils.

* Move failover code from module_utils/robot to module_utils/failover.

* Add boilerplate.

* Add tests for plugin_open_url_json.

* Make it work with all Python versions.

* Use community.internal_test_tools from ansible-collections/community.internal_test_tools#24

* Fix error instantiation.

* Add more complete test.

* The branch has been merged, but the collection hasn't been released yet.

* Forgot to remove path to my repo.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant