Skip to content

Conversation

@PeterHamilton
Copy link
Contributor

This change adds integration tests that test registering, retrieving, and destroying SplitKey objects with the server. Minor updates are included for the client and server to ensure that SplitKey operations function as expected.

Partially implements #545

This change adds integration tests that test registering,
retrieving, and destroying SplitKey objects with the server.
Minor updates are included for the client and server to ensure
that SplitKey operations function as expected.

Partially implements #545
@codecov-io
Copy link

codecov-io commented Sep 20, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@a8713fc). Click here to learn what that means.
The diff coverage is 40%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #586   +/-   ##
=========================================
  Coverage          ?   96.28%           
=========================================
  Files             ?       75           
  Lines             ?    15808           
  Branches          ?        0           
=========================================
  Hits              ?    15221           
  Misses            ?      587           
  Partials          ?        0
Impacted Files Coverage Δ
kmip/core/factories/secrets.py 82.14% <20%> (ø)
kmip/services/server/engine.py 99.23% <50%> (ø)
kmip/pie/factory.py 98.33% <66.66%> (ø)

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 a8713fc...1eb9426. Read the comment docs.

@PeterHamilton PeterHamilton merged commit 29750cb into master Sep 20, 2019
@PeterHamilton PeterHamilton deleted the feat/add-split-key-integration-tests branch September 20, 2019 18:05
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