IntersectMBO / IntersectMBO/ouroboros-network

Refactor diffusion testnet simulation

Open
#4,085 4 comments 0 reactions 0 assignees View on GitHub
technical debt testing
Dominant language
Haskell
Stars
296
Forks
104
Avg merge
2d 10h
Merged PRs (30d)
4

Description

The current module structure and some abstractions makes things unnecessarily
complicated. I think we should:

* merge parts of `Test.Ouroboros.Network.Diffusion.Node` and `Test.Ouroboros.Network.Testnet.Simulation.Node` which are used to run a single diffusion, exposing only the simpler interface (from the latter module);
* split Test.Ouroboros.Network.Testnet.Simulation.Node into two parts:

- `Test.Ouroboros.Network.Diffusion.Testnet` which runs a network
- `Test.Ouroboros.Network.Diffusion.Types` which includes `NodeArgs`, `SimArgs` and their arbitrary generators;

* use `Test.Ouroboros.Network.Diffusion` as the module which exposes all the
tests.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.