Skip to content
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.

Sign up for GitHub

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

Jump to bottom

test: Coverage for repository #30

Open
2 tasks
mfw78 opened this issue Jun 7, 2022 · 0 comments
Open
2 tasks

test: Coverage for repository #30

mfw78 opened this issue Jun 7, 2022 · 0 comments
Labels
A-tests Area: tests C-database Components: database D-chore Difficulty: chore P-normal Priority: normal
Milestone
ETHBarcelona & ETHcc

Comments

@mfw78
Copy link
Contributor

mfw78 commented Jun 7, 2022

Problem
As we build on top of the repository layer that we wrote, it is critical that this layer performs as expected.

Solution
Unit tests to cover basic functionality, but particularly, we require:

  • Testing all getters to make sure that they return a consistent value (suggest type | null).
  • Testing of all indices to make sure that they are being managed correctly (add a stub, adds it to the indices., remove a stub, removes from the indices).
@mfw78 mfw78 added A-tests Area: tests D-chore Difficulty: chore P-normal Priority: normal C-database Components: database labels Jun 7, 2022
@mfw78 mfw78 added this to the ETHBarcelona & ETHcc milestone Jun 7, 2022
@mfw78 mfw78 changed the title Test coverage for repository test: Coverage for repository Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
A-tests Area: tests C-database Components: database D-chore Difficulty: chore P-normal Priority: normal
Projects
None yet
Milestone
ETHBarcelona & ETHcc
Development

No branches or pull requests
1 participant
@mfw78