-
Notifications
You must be signed in to change notification settings - Fork 315
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
test(trustchain): getMembers should be Unauthorized if no longer member #7236
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
4 Skipped Deployments
|
// first member initializes itself | ||
const name1 = "Member 1"; | ||
const sdk1 = getSdk(!!getEnv("MOCK"), { applicationId, name: name1 }); | ||
const member1creds = await sdk1.initMemberCredentials(); | ||
// second member initializes itself | ||
const name2 = "Member 2"; | ||
const sdk2 = getSdk(!!getEnv("MOCK"), { applicationId, name: name2 }); | ||
const member2creds = await sdk2.initMemberCredentials(); | ||
const member2 = { name: name2, id: member2creds.pubkey, permissions: 0xffffffff }; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As we use this on every test, we could make an helper to create that no?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes probably.
like #7262 has done it for the sdkForName one π
β Checklist
npx changeset
was attached.π Description
β Context
π§ Checklist for the PR Reviewers