search

aws-cloudformation / aws-cloudformation-samples

Apache License 2.0
134 stars 55 forks source link

Add 3 additional resource targets. #105

Closed mrinaudo-aws closed 4 months ago

mrinaudo-aws commented 4 months ago

Issue #, if available:

Description of changes: Add 3 additional resource targets.

Unit tests excerpts

% pytest --cov
[...]
Name                                           Stmts   Miss Branch BrPart  Cover   Missing
------------------------------------------------------------------------------------------
src/awssamples_resourcetags_hook/handlers.py     317      0    170      0   100%
------------------------------------------------------------------------------------------
TOTAL                                            317      0    170      0   100%

Required test coverage of 85.0% reached. Total coverage: 100.00%
[...] 87 passed

Contract tests excerpts

% cfn test
[...]
collected 24 items / 17 deselected / 7 selected

[...]/hook/handler_pre_create.py::contract_pre_create_success PASSED                                                         [ 14%]
[...]/hook/handler_pre_create.py::contract_pre_create_failed PASSED                                                          [ 28%]
[...]/hook/handler_pre_create.py::contract_pre_create_failed_unsupported_target SKIPPED (No wildcard hook targets. Skipp...) [ 42%]
[...]/hook/handler_pre_update.py::contract_pre_update_success PASSED                                                         [ 57%]
[...]/hook/handler_pre_update.py::contract_pre_update_failed PASSED                                                          [ 71%]
[...]/hook/handler_pre_update.py::contract_pre_update_failed_unsupported_target SKIPPED (No wildcard hook targets. Skipp...) [ 85%]
[...]/resource/handler_misc.py::contract_check_asserts_work PASSED                                                           [100%]

[...[ 5 passed [...]

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.