Skip to content

Update payloads for the Activate operation - #314

Open
PeterHamilton wants to merge 1 commit into
masterfrom
feat/update-activate-payloads
Open

Update payloads for the Activate operation#314
PeterHamilton wants to merge 1 commit into
masterfrom
feat/update-activate-payloads

Conversation

@PeterHamilton

Copy link
Copy Markdown
Contributor

This change updates the request and response payloads for the Activate operation, bringing them up-to-date with current coding style. The unit test suites for the payloads have been overhauled to also match current standards. Payload usage in the client and server also now comply with the updated implementations.

This change updates the request and response payloads for the
Activate operation, bringing them up-to-date with current coding
style. The unit test suites for the payloads have been overhauled
to also match current standards. Payload usage in the client and
server also now comply with the updated implementations.
@codecov-io

codecov-io commented Jul 30, 2017

Copy link
Copy Markdown

Codecov Report

Merging #314 into master will increase coverage by 0.06%.
The diff coverage is 98.92%.

Impacted file tree graph

@@ Coverage Diff @@## master #314 +/- ##
==========================================
+ Coverage 91.5% 91.56% +0.06% 
==========================================
Files 64 64 Lines 9871 9911 +40 ==========================================
+ Hits 9032 9075 +43 + Misses 839 836 -3
Impacted FilesCoverage Δ
kmip/services/kmip_client.py61.8% <0%> (+0.33%)⬆️
kmip/core/messages/payloads/activate.py100% <100%> (ø)⬆️
kmip/services/server/engine.py99.88% <100%> (ø)⬆️
kmip/core/messages/payloads/__init__.py100% <100%> (ø)⬆️

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 278a543...09473e6. Read the comment docs.

Sign up for freeto 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.

2 participants

@PeterHamilton@codecov-io