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

unit tests for credential_schema::Schema #808

Merged
merged 3 commits into from
Apr 27, 2023

Conversation

nachiketkanore
Copy link
Contributor

@codecov-commenter
Copy link

codecov-commenter commented Apr 18, 2023

Codecov Report

Merging #808 (31abb4d) into main (ca7bb8f) will increase coverage by 0.08%.
The diff coverage is 61.53%.

@@            Coverage Diff             @@
##             main     #808      +/-   ##
==========================================
+ Coverage   48.95%   49.04%   +0.08%     
==========================================
  Files         410      410              
  Lines       33458    33497      +39     
  Branches     7426     7441      +15     
==========================================
+ Hits        16380    16428      +48     
+ Misses      11978    11952      -26     
- Partials     5100     5117      +17     
Flag Coverage Δ
unittests-aries-vcx 49.01% <61.53%> (+0.08%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ies_vcx/src/common/primitives/credential_schema.rs 46.45% <61.53%> (+14.63%) ⬆️

... and 5 files with indirect coverage changes

@Patrik-Stas
Copy link
Contributor

hi @nachiketkanore thank you for contribution! The CI looks green, but DCO check is currently failing. No worries is simple fix! Have a look here https://github.com/hyperledger/aries-vcx/pull/808/checks?check_run_id=12831789658 what DCO means
You can fix your existing commit by ammending signoff to it

git commit --amend --signoff

and then you'll have to push force again.

Thank you!

Patrik-Stas
Patrik-Stas previously approved these changes Apr 19, 2023
…schema_id (easy)

Signed-off-by: Nachiket Kanore <nachiket.kanore@gmail.com>
@nachiketkanore
Copy link
Contributor Author

Done

@Patrik-Stas Patrik-Stas merged commit 019e5c7 into hyperledger:main Apr 27, 2023
40 of 41 checks passed
tech-bash pushed a commit to tech-bash/aries-vcx that referenced this pull request Apr 30, 2023
…schema_id (easy) (hyperledger#808)

Signed-off-by: Nachiket Kanore <nachiket.kanore@gmail.com>
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.

Write unit tests for Schema
3 participants