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

add integration test for aid to join multisig #255

Closed
wants to merge 25 commits into from

Conversation

daviddm
Copy link
Contributor

@daviddm daviddm commented Apr 30, 2024

This PR includes multisig test join and illustrates the problem in #227.

The integration test creates a multisig-group with 2 members and threshold 1.
Member 1 adds a new member(member3) to the group and does a double rotation for the new member to be able to join and add the end-role by himself.

Copy link

codecov bot commented Apr 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.76%. Comparing base (aadfce0) to head (317ec3f).
Report is 27 commits behind head on main.

❗ Current head 317ec3f differs from pull request most recent head 6b514c3. Consider uploading reports for the commit 6b514c3 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #255      +/-   ##
==========================================
- Coverage   82.85%   82.76%   -0.10%     
==========================================
  Files          46       46              
  Lines        4189     4218      +29     
  Branches     1043     1053      +10     
==========================================
+ Hits         3471     3491      +20     
- Misses        689      722      +33     
+ Partials       29        5      -24     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@lenkan
Copy link
Collaborator

lenkan commented May 6, 2024

I added the waitOperation call. Needs a new Keria image with WebOfTrust/keria#236 to make the tests pass.

github-actions bot and others added 24 commits May 8, 2024 14:16
- add method in renaming credential registries.
- update credential.test.ts

Signed-off-by: arshdeep singh <[email protected]>
…t#222)

* fix: cannot set different salts for credential

* fix types

* add comments

* fix dt for issuance event
* URL enconding of name in api call

* prettier

* content-lenght header not needed

* add test

* remove debris

* fix test

* pretty

* Update test/app/aiding.test.ts
maybe fix upload issue
* remove docs

* add docs publish

* fix permissions
@daviddm daviddm force-pushed the multisig-join-2 branch from be3cf62 to 6b514c3 Compare May 8, 2024 12:19
@daviddm
Copy link
Contributor Author

daviddm commented May 8, 2024

I made some bad git-stuff.
Remade the PR #258

@daviddm daviddm closed this May 8, 2024
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.

8 participants