fix(data-store-orm): remove double entries

cre8 commented 9 months ago

Signed-off-by: Mirko Mollik mirko.mollik@fit.fraunhofer.de

What issue is this PR fixing

fixes #1285

What is being changed

It will reduce the credentials in case there were multiuple claims pointing to the claim credential

Quality

Check all that apply:

[X] I want these changes to be integrated
[X] I successfully ran pnpm i, pnpm build, pnpm test, pnpm test:browser locally.
[X] I allow my PR to be updated by the reviewers (to speed up the review process).
[ ] I added unit tests.
[ ] I added integration tests.
[X] I did not add automated tests because there was no existing test for this function and I am aware that a PR without tests will likely get rejected.

codecov-commenter commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (462735d) 85.89% compared to head (c626cab) 85.46%. Report is 2 commits behind head on next.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## next #1299 +/- ## ========================================== - Coverage 85.89% 85.46% -0.44% ========================================== Files 170 170 Lines 18939 18939 Branches 2137 2115 -22 ========================================== - Hits 16268 16186 -82 - Misses 2671 2753 +82 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

cre8 commented 9 months ago

@mirceanis didn't find a way to run the test locally just for one package. Fixed it and it passed in the ci

decentralized-identity / veramo