Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

PDP Unit Tests #39

Open
5 tasks
hannahhoward opened this issue Feb 19, 2025 · 0 comments · May be fixed by #43
Open
5 tasks

PDP Unit Tests #39

hannahhoward opened this issue Feb 19, 2025 · 0 comments · May be fixed by #43
Assignees
@frrist

Comments

@hannahhoward
Copy link
Member

What

Currently most of the pkg/pdp module lacks unit tests.

Adding them should be a good way to get strong familiarty with how it works.

In order of importance:

  • pkg/pdp/aggregator/fns
  • pkg/pdp/aggregator (specifically steps.go)
  • pkg/pdp/piecefinder
  • pkg/pdp/pieceadder
  • pkg/pdp/curio (the curio http client)

Spend 2-3 days max, stop whereever you get to.
@hannahhoward hannahhoward moved this to Sprint Backlog in Storacha Project Planning Feb 19, 2025
@frrist frrist moved this from Sprint Backlog to In Progress in Storacha Project Planning Feb 20, 2025
@frrist frrist linked a pull request Feb 21, 2025 that will close this issue
Open
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@frrist frrist

Labels
None yet
Projects
Storacha Project Planning
Status: In Progress
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: add unit testing for pdp package storacha/storage
2 participants
@hannahhoward @frrist