Skip to content

Conversation

@PeterHamilton
Copy link
Contributor

This change adds payload request and response base classes to prepare for future simplification updates to the current client architecture. No new tests are required for this change.

This change adds payload request and response base classes to
prepare for future simplification updates to the current client
architecture. No new tests are required for this change.
@codecov-io
Copy link

codecov-io commented Nov 12, 2019

Codecov Report

Merging #599 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #599      +/-   ##
==========================================
+ Coverage   96.44%   96.45%   +<.01%     
==========================================
  Files          76       77       +1     
  Lines       16241    16272      +31     
==========================================
+ Hits        15664    15695      +31     
  Misses        577      577
Impacted Files Coverage Δ
kmip/core/messages/payloads/sign.py 99.34% <100%> (ø) ⬆️
kmip/core/messages/payloads/poll.py 100% <100%> (ø) ⬆️
kmip/core/messages/payloads/get.py 100% <100%> (ø) ⬆️
kmip/core/messages/payloads/decrypt.py 100% <100%> (ø) ⬆️
kmip/core/messages/payloads/base.py 100% <100%> (ø)
kmip/core/messages/payloads/discover_versions.py 100% <100%> (ø) ⬆️
kmip/core/messages/payloads/register.py 100% <100%> (ø) ⬆️
kmip/core/messages/payloads/cancel.py 100% <100%> (ø) ⬆️
kmip/core/messages/payloads/recover.py 100% <100%> (ø) ⬆️
kmip/core/messages/payloads/__init__.py 100% <100%> (ø) ⬆️
... and 21 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 676aaf5...4d651f5. Read the comment docs.

@PeterHamilton PeterHamilton merged commit 77d5b32 into master Nov 12, 2019
@PeterHamilton PeterHamilton deleted the feat/add-payload-base-classes branch November 12, 2019 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants