search

Joystream / storage-node-joystream

Colossus - Storage node for Joystream
GNU General Public License v3.0
4 stars 7 forks source link

Add tests to runtime api #77

Closed jfinkhaeuser closed 5 years ago

jfinkhaeuser commented 5 years ago

Implements #71

Not all functionality is tested, because the runtime state that's required is a bit complex to set up. But as a baseline of tests, it's probably a good start, to be fleshed out later. WDYT?

jfinkhaeuser commented 5 years ago

After this, it should be simple enough to move the code to #74

jfinkhaeuser commented 5 years ago

As discussed separately, you're right - the main point for me was to make development more independent of a running storage node rather than CI integration. For CI integration, we should either run a prepped node, or mock the base/polkadot API, effectively.

It seems like a lot of effort for what we're trying to get going right now, but we should open an issue for it in the new repo!