-
Notifications
You must be signed in to change notification settings - Fork 177
Add/reorganize unit tests for CClaimTrieCache/CClaimTrie #104
Copy link
Copy link
Closed
Labels
type: improvementExisting (or partially existing) functionality needs to be changedExisting (or partially existing) functionality needs to be changed
Milestone
Description
We have decent coverage for integration testing the CClaimTrieCache together with CClaimTrie but we do not have any unit tests for testing them separately.
Especially for CClaimTrieCache, adding unit tests would help developers understand and debug the class. We'll have better confidence on the correctness of the code going forward.
Acceptance Criteria
Definition of Done
- Tested against acceptance criteria
- Tested against the assumptions of the user story
- The project builds without errors
- Unit tests are written and passing
- Tests on devices/browsers listed in the issue have passed
- QA performed & issues resolved
- Refactoring completed
- Any configuration or build changes documented
- Documentation updated
- Peer Code Review performed
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
type: improvementExisting (or partially existing) functionality needs to be changedExisting (or partially existing) functionality needs to be changed