Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: openapi validation test for keychain-vault plugin #1330

Merged
merged 2 commits into from Sep 18, 2021

Conversation

elenaizaguirre
Copy link
Contributor

Includes tests for endpoints setKeychainEntryV1,
getKeychainEntryV1, deleteKeychainEntryV1 and
hasKeychainEntryV1, each of them, when appropiate,
with test cases:

  • Right request
  • Request including an invalid parameter
  • Request without a required parameter

Closes #1329

Relationed with #847

Signed-off-by: Elena Izaguirre e.izaguirre.equiza@accenture.com

@petermetz petermetz marked this pull request as ready for review September 14, 2021 20:15
@petermetz petermetz requested review from takeutak and removed request for jonathan-m-hamilton September 14, 2021 20:15
Includes tests for endpoints setKeychainEntryV1,
getKeychainEntryV1, deleteKeychainEntryV1 and
hasKeychainEntryV1, each of them, when appropiate,
with test cases:
  - Right request
  - Request including an invalid parameter
  - Request without a required parameter

Closes hyperledger#1329

Relationed with hyperledger#847

Signed-off-by: Elena Izaguirre <e.izaguirre.equiza@accenture.com>
@codecov-commenter
Copy link

codecov-commenter commented Sep 17, 2021

Codecov Report

Merging #1330 (04bea2c) into main (c568ad3) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1330   +/-   ##
=======================================
  Coverage   68.63%   68.63%           
=======================================
  Files         301      301           
  Lines       10948    10948           
  Branches     1347     1347           
=======================================
  Hits         7514     7514           
  Misses       2733     2733           
  Partials      701      701           
Impacted Files Coverage Δ
...pescript/generated/openapi/typescript-axios/api.ts 83.33% <ø> (ø)

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 c568ad3...04bea2c. Read the comment docs.

@petermetz petermetz merged commit 6630ebe into hyperledger:main Sep 18, 2021
@petermetz petermetz deleted the openapi-keychain-vault branch September 18, 2021 07:13
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.

fix(cactus-plugin-keychain-vault): Implement tests for openapi validation
4 participants