ipfs-shipyard / net-ipfs-http-client

InterPlanetary File System client for .Net (C#, VB, F# ...)
MIT License
35 stars 13 forks source link

Update CI to load current Kubo and run loopback tests #22

Open erikmav opened 1 year ago

erikmav commented 1 year ago

The current TestFixture.Ipfs implementation for running unit tests currently just creates an IpfsClient instance with the assumption of an IPFS local server to connect to. Update the test code to:

Note this item will probably illuminate a lot of missing or out of date functionality. File incompatibilities as new issues.

erikmav commented 1 year ago

@Arlodotexe possible to add me as a contributor on this repo to be able to run CI? Also on IpfsCore as both repos need simultaneous updating.

Arlodotexe commented 1 year ago

This part of Richard's tests always bothered me a bit. I noticed this the first time I ported his code over. It wasn't a simple fix, so I moved on and worked on developing tooling to make this easier.

I'm actually using that tooling in my own apps and unit tests, dogfooding it. Specifically, we'll want to make use of KuboDownloader and KuboBootstrapper if there's no HTTP API already running.

Note If our test project takes this new dependency, https://github.com/Arlodotexe/OwlCore.Kubo/issues/3 gets a priority bump, but isn't required to close this ticket.

Arlodotexe commented 2 months ago

To give an update on this.

We're still doing various API cleanup to get us aligned with the most recent version of Kubo. Until we've finished fixing all remaining discrepancies, our unit tests won't fully pass.

When we've made more progress, we'll get Kubo bootstrapped and running as part of the tests, and run it in the CI.