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.

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

Testing refactors #21

Merged
merged 3 commits into from
Dec 19, 2019

Conversation

shannonwells
Copy link
Contributor

@shannonwells shannonwells commented Dec 19, 2019

Problem

There is some good but complicated libp2p mocknet setup in datatransfer/graphsync, which is almost entirely usable and needed for retrieval market testing.

Solution

extract mocknet setup from datatransfer testing, fold whitebox testin…g into graphsync_impl_test

Also:

  • eliminate a whitebox test with an additional Subscribers function in graphsync impl and use of a test interface.
  • a few small changes to please the linters

Copy link
Contributor

@ingar ingar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@shannonwells shannonwells merged commit 036ced7 into feat/retrieval-extraction Dec 19, 2019
@shannonwells shannonwells deleted the techdebt/testing-refactors branch December 19, 2019 21:32
ingar pushed a commit that referenced this pull request Dec 19, 2019
* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test
ingar pushed a commit that referenced this pull request Dec 19, 2019
* feat(retrievalmarket): extract from lotus

Extract retrievalmarket from lotus, migrate to shared types, use cbor-gen-for, etc

* Testing refactors (#21)

* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test

Co-authored-by: Shannon Wells <shannonwells@users.noreply.github.com>
shannonwells added a commit that referenced this pull request Dec 20, 2019
* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test
shannonwells added a commit that referenced this pull request Dec 20, 2019
* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test
shannonwells added a commit that referenced this pull request Jan 2, 2020
* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test
shannonwells added a commit that referenced this pull request Jan 2, 2020
* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test
shannonwells added a commit that referenced this pull request Jan 8, 2020
* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test
shannonwells added a commit that referenced this pull request Jan 8, 2020
* extract mocknet setup from datatransfer testing, fold whitebox testing into graphsync_impl_test
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants