fission-codes / fission-cli

Rewrite it in Rust 🦀

Apache License 2.0

1 stars 1 forks source link

Add wiremock for testing #45

Closed BoisterousCoder closed 1 year ago

BoisterousCoder commented 1 year ago

Summary

Problem

Currently the IPFS trait tests are run against an actual IPFS instance.

Impact

This makes testing harder, for the following reasons

There is no way to cases where IPFS fails in an easy an consistent way
There is no way to run the tests automatically on GitHub without somehow installing IPFS
There might be hardware specific errors that might be hard to diagnose
There is no way to access IPFS without having IPFS setup in a way that the CLI expects

Solution

Rewrite the tests so they use wiremock to send back data like it was the instance

BoisterousCoder commented 1 year ago

Woops seems I added a duplicate but accident. #29