Skip to content

Conversation

@michaelyaakoby
Copy link
Member

@michaelyaakoby michaelyaakoby commented Jul 15, 2020

Solves #20, #22, #23, #24, #25, #28

See commit-comments for details.

michael.yak added 2 commits July 15, 2020 21:33
E2E's test_recurring_registration was extended to verify that
deregistration is working properly for all the web frameworks.

Note that the actual invocation of test_recurring_registration is
registered in `atexit` however in the test its too hard to invoke
`atexit` therefore it is called directly.

Solves #25
@michaelyaakoby michaelyaakoby changed the title README.md should link to supported web frameworks README.md should link to supported web frameworks and iprove coverage Jul 15, 2020
Add tests for verifying Pyctuator's support for basic-auth registration
with Spring Boot Admin.

Solves #28
@michaelyaakoby michaelyaakoby changed the title README.md should link to supported web frameworks and iprove coverage Fix issues #20, #25 and #28 Jul 16, 2020
@michaelyaakoby michaelyaakoby changed the title Fix issues #20, #25 and #28 Fix issues 20, 25 and 28 Jul 16, 2020
@codecov-commenter
Copy link

codecov-commenter commented Jul 16, 2020

Codecov Report

Merging #27 into master will increase coverage by 4.91%.
The diff coverage is 66.66%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #27      +/-   ##
==========================================
+ Coverage   88.35%   93.26%   +4.91%     
==========================================
  Files          25       25              
  Lines         936      936              
==========================================
+ Hits          827      873      +46     
+ Misses        109       63      -46     
Impacted Files Coverage Δ
pyctuator/pyctuator.py 92.10% <0.00%> (ø)
pyctuator/impl/aiohttp_pyctuator.py 97.43% <100.00%> (+1.28%) ⬆️
pyctuator/impl/spring_boot_admin_registration.py 92.40% <100.00%> (+24.05%) ⬆️
pyctuator/impl/fastapi_pyctuator.py 100.00% <0.00%> (+1.01%) ⬆️
pyctuator/impl/flask_pyctuator.py 98.96% <0.00%> (+1.03%) ⬆️
pyctuator/impl/pyctuator_impl.py 94.80% <0.00%> (+5.19%) ⬆️
pyctuator/health/diskspace_health_impl.py 100.00% <0.00%> (+19.23%) ⬆️
pyctuator/metrics/thread_metrics_impl.py 100.00% <0.00%> (+27.77%) ⬆️
... and 1 more

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 09d69bb...1314d02. Read the comment docs.

@michaelyaakoby michaelyaakoby changed the title Fix issues 20, 25 and 28 Fix issues 20, 23, 25 and 28 Jul 17, 2020
@michaelyaakoby michaelyaakoby changed the title Fix issues 20, 23, 25 and 28 Fix issues 20, 23, 24, 25 and 28 Jul 17, 2020
@michaelyaakoby michaelyaakoby force-pushed the michael/issues branch 2 times, most recently from 8da0488 to 7881856 Compare July 17, 2020 20:10
@michaelyaakoby michaelyaakoby changed the title Fix issues 20, 23, 24, 25 and 28 Fix issues 20, 22, 23, 24, 25 and 28 Jul 17, 2020
@michaelyaakoby
Copy link
Member Author

Not sure why codecov isn't updated, but the latest commit brings coverage to (+7.6%):
image

@michaelyaakoby michaelyaakoby merged commit 1d71706 into master Jul 21, 2020
@michaelyaakoby michaelyaakoby deleted the michael/issues branch October 3, 2020 12:04
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.

3 participants