-
Notifications
You must be signed in to change notification settings - Fork 20
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
Data providers tests for Achainable, Nodereal, Discord, Litentry, and Twitter #2434
Conversation
6229603
to
eb1dd48
Compare
496fc41
to
85e12c9
Compare
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.
Looks good to me. ❤️
"mockWeb3Network": "litentry,polkadot", | ||
"expectedCredentialValue": false | ||
} | ||
] |
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.
is there any option that we have a successful result and unsuccessful? Like early bird user and non EB user
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.
Of course, It depends on the configuration of mockDid; that's the purpose of this PR. You can try it locally.
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.
You can use this address if you want to have a credential with true
value
Context
this pr:
resolves p-367
resolves p-368
resolves p-369
resolves p-370
Add all vcs request json(If anything is missing, please let me know, and will add it.)
Note:
Since mockDid is a real address, but uncontrollable, it can lead to uncertainty about whether expectedCredentialValue is
true
orfalse
. If we want to test, I believe we need to find the address yourself, update it, and then run GHA to check the results.