-
Notifications
You must be signed in to change notification settings - Fork 41
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 virtual cards backend #271
Comments
@EresDev, You are not allowed to add Time: <1 Week |
! No permission to set labels |
/start |
! Please set your wallet address with the /wallet command first and try again. |
/wallet 0x229674f3db146E0AF0182b660F5c9232538E7e7b |
+ Successfully registered wallet address |
+ Successfully registered wallet address |
/start |
Tips:
|
Tips:
|
# No linked pull requests to close |
/start |
Tips:
|
/start |
Tip
|
|
View | Contribution | Count | Reward |
---|---|---|---|
Issue | Task | 1 | 800 |
Issue | Specification | 1 | 36.42 |
Review | Comment | 1 | 0 |
Conversation Incentives
Comment | Formatting | Relevance | Reward |
---|---|---|---|
The tests with this repository are e2e tests. That's how I start… | 12.14content: content: p: score: 0 elementCount: 4 a: score: 5 elementCount: 1 ul: score: 1 elementCount: 1 li: score: 0.5 elementCount: 2 result: 7 regex: wordCount: 103 wordValue: 0.1 result: 5.14 | 1 | 36.42 |
Resolves #271With unit tests, it also includes some renaming a… | 4.5content: content: p: score: 0 elementCount: 5 h3: score: 1 elementCount: 2 ul: score: 1 elementCount: 1 li: score: 0.5 elementCount: 3 result: 4.5 regex: wordCount: 65 wordValue: 0 result: 0 | 0.9 | 0 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The tests with this repository are e2e tests. That's how I started the virtual cards feature and used e2e tests.
As the feature is getting refined, I think it would be good idea to bring in unit tests for the backend functions. Cloudflare recommends vitest because backend functions are Cloudflare pages function. Along with the general advantages of unit tests over e2e tests, I see following reasons to have them.
The text was updated successfully, but these errors were encountered: