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 xdai plugin #1289

Merged

Conversation

elenaizaguirre
Copy link
Contributor

@elenaizaguirre elenaizaguirre commented Aug 31, 2021

Includes tests for endpoints deployContractV1,
invokeContractV1 and runTransactionV1, each of them
with test cases:

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

Closes: #1288

Relationed with #847

Signed-off-by: Elena Izaguirre [email protected]

@codecov-commenter
Copy link

codecov-commenter commented Aug 31, 2021

Codecov Report

Merging #1289 (5aa1fbb) into main (35b20ac) will increase coverage by 0.69%.
The diff coverage is n/a.

❗ Current head 5aa1fbb differs from pull request most recent head b824441. Consider uploading reports for the commit b824441 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1289      +/-   ##
==========================================
+ Coverage   67.93%   68.63%   +0.69%     
==========================================
  Files         301      301              
  Lines       10948    10948              
  Branches     1347     1347              
==========================================
+ Hits         7438     7514      +76     
+ Misses       2819     2733      -86     
- Partials      691      701      +10     
Impacted Files Coverage Δ
...main/typescript/rustc-container/rustc-container.ts 68.53% <0.00%> (+2.24%) ⬆️
...cript/generated/openapi/typescript-axios/common.ts 43.47% <0.00%> (+6.52%) ⬆️
...ypescript/openethereum/openethereum-test-ledger.ts 75.55% <0.00%> (+8.88%) ⬆️
...ypescript/web-services/invoke-contract-endpoint.ts 83.87% <0.00%> (+19.35%) ⬆️
...ypescript/web-services/run-transaction-endpoint.ts 83.87% <0.00%> (+19.35%) ⬆️
...ices/deploy-contract-solidity-bytecode-endpoint.ts 83.87% <0.00%> (+19.35%) ⬆️
...pescript/generated/openapi/typescript-axios/api.ts 84.94% <0.00%> (+48.38%) ⬆️

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 35b20ac...b824441. Read the comment docs.

@elenaizaguirre elenaizaguirre force-pushed the openapi-validation-xdai branch from 51d65a0 to 0fdae88 Compare September 1, 2021 06:57
@elenaizaguirre elenaizaguirre force-pushed the openapi-validation-xdai branch 2 times, most recently from d1c926d to 6f44052 Compare September 2, 2021 09:44
Includes tests for endpoints deployContractV1,
invokeContractV1 and runTransactionV1, each of them
with test cases:
  - Right request
  - Request including an invalid parameter
  - Request without a required parameter

Closes hyperledger-cacti#1288

Relationed with hyperledger-cacti#847

Signed-off-by: Elena Izaguirre <[email protected]>
@elenaizaguirre elenaizaguirre changed the title fix: openapi-validation for xdai plugin fix: openapi-validation test for xdai plugin Sep 14, 2021
@petermetz petermetz marked this pull request as ready for review September 14, 2021 19:25
@petermetz petermetz requested review from takeutak and petermetz and removed request for jonathan-m-hamilton September 14, 2021 19:25
@petermetz petermetz merged commit ba8a784 into hyperledger-cacti:main Sep 17, 2021
@petermetz petermetz deleted the openapi-validation-xdai branch September 17, 2021 18: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.

fix(plugin-ledger-connector-xdai): OpenAPI spec validation missing from plugin REST endpoints
4 participants