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

fix: clean up and fix retrieval framework & tests #1848

Merged
merged 2 commits into from
Dec 19, 2023

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Dec 19, 2023

  • Use new go-trustless-utils primitives and traversal tools for requesting and verifying test retrievals
  • Simplify Framework#Retrieve and remove test logic that doesn't make sense for these kinds of retrievals
  • Wait for full startup of data transfer when making a retrieval client

Some lint errors in here that can be taken care of separately.

* Use new go-trustless-utils primitives and traversal tools
* Simplify Framework#Retrieve
@rvagg rvagg requested a review from LexLuthr December 19, 2023 09:29
@LexLuthr LexLuthr merged commit 7907a9f into remove-legacy-markets Dec 19, 2023
17 of 18 checks passed
@LexLuthr LexLuthr deleted the rvagg/remove-legacy-markets branch December 19, 2023 09:39
LexLuthr added a commit that referenced this pull request Feb 19, 2024
* remove gfm, pull datatransfer

* make gen, docsgen

* fix circular depndencies

* go mod tidy

* fix lint errors

* convert RetrievalAskGetter to interface

* fix interface pointer

* fix dependency injection, move ask db

* fix itests

* clean up mocks

* chore: lots more dt & gs debugging (#1776)

* fix lint errs, add back ProviderDataTransfer

* regenrate cbor

* cleanup go-data-transfer

* fix graphsync retrievals

* use new client for GS tests

* fix selector type

* restore selector type

* fix: clean up and fix retrieval framework & tests (#1848)

* fix: clean up and fix retrieval framework & tests

* Use new go-trustless-utils primitives and traversal tools
* Simplify Framework#Retrieve

* fix: wait for full startup of data transfer when making client

* fix lint errors

* fix go mod

* cleanup itest fixtures, fix gql

* go mod tidy

* fix directDealProv

* handle storage ask

* cleanup config

* remove graphsync unit tests

* add retrieval config headers

* update retrieval configs

* fix storage ask db command

* fix BoostLegacyDealByProposalCid api

* make cbor-gen

* fix missing api example

* migrate config, dump dagstore config

---------

Co-authored-by: Rod Vagg <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants