-
Notifications
You must be signed in to change notification settings - Fork 278
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: refactor jest test negative test cases - phase 1 #3483
Labels
enhancement
New feature or request
Comments
ashnashahgrover
added a commit
to ashnashahgrover/cacti
that referenced
this issue
Aug 18, 2024
--------------- 1. Refactored all remaining negative test case exception assertions under cacti/packages Removed try-catch blocks, replaced with declarations through jest-extended's own API. Fixes hyperledger#3483 Signed-off-by: ashnashahgrover <ashnashahgrover777@gmail.com>
ashnashahgrover
added a commit
to ashnashahgrover/cacti
that referenced
this issue
Aug 18, 2024
Primary Changes --------------- 1. Refactored all remaining negative test case exception assertions under cacti/packages Removed try-catch blocks, replaced with declarations through jest-extended's own API. Fixes hyperledger#3483 Signed-off-by: ashnashahgrover <ashnashahgrover777@gmail.com>
ashnashahgrover
added a commit
to ashnashahgrover/cacti
that referenced
this issue
Aug 26, 2024
Primary Changes --------------- 1. Refactored all remaining negative test case exception assertions under cacti/packages Removed try-catch blocks, replaced with declarations through jest-extended's own API. Fixes hyperledger#3483 Signed-off-by: ashnashahgrover <ashnashahgrover777@gmail.com>
This issue is now dependent on this issue |
ashnashahgrover
added a commit
to ashnashahgrover/cacti
that referenced
this issue
Aug 26, 2024
Primary Changes --------------- 1. Refactored all remaining negative test case exception assertions under cacti/packages Removed try-catch blocks, replaced with declarations through jest-extended's own API. Fixes hyperledger#3483 Signed-off-by: ashnashahgrover <ashnashahgrover777@gmail.com>
ashnashahgrover
added a commit
to ashnashahgrover/cacti
that referenced
this issue
Sep 3, 2024
Primary Changes --------------- 1. Refactored all remaining negative test case exception assertions under cacti/packages Removed try-catch blocks, replaced with declarations through jest-extended's own API. Fixes hyperledger#3483 Signed-off-by: ashnashahgrover <ashnashahgrover777@gmail.com>
5 tasks
ashnashahgrover
added a commit
to ashnashahgrover/cacti
that referenced
this issue
Sep 21, 2024
Primary Changes --------------- 1. Refactored all remaining negative test case exception assertions under cacti/packages Removed try-catch blocks, replaced with declarations through jest-extended's own API. Fixes hyperledger#3483 Signed-off-by: ashnashahgrover <ashnashahgrover777@gmail.com>
ashnashahgrover
added a commit
to ashnashahgrover/cacti
that referenced
this issue
Sep 21, 2024
Primary Changes --------------- 1. Refactored all remaining negative test case exception assertions under cacti/packages Removed try-catch blocks, replaced with declarations through jest-extended's own API. Fixes hyperledger#3483 Signed-off-by: ashnashahgrover <ashnashahgrover777@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Refactor all the remaining negative test case exception assertions for all tests in cacti/packages so that they do not use try-catch blocks but instead declare the requirements through Jest's (and jest-extended's) own API.
The parent issue with a much more detailed explanation of what's needed and code examples on how to do it:
Parent Issue: #3455
Acceptance Criteria
Test passes the same as before
Negative test case assertions have been migrated the way it is explained in the parent issue.
The text was updated successfully, but these errors were encountered: