IntersectMBO / IntersectMBO/cardano-coin-selection

Revise and simplify test suite.

Open
#34 0 comments 0 reactions 0 assignees View on GitHub
test coverage
Dominant language
Haskell
Stars
19
Forks
10
PR merge metrics
No merged PRs in 30d

Description

## Context

This issue collects together tasks relating to simplifying the test suite for `cardano-coin-selection`.

## Tasks

- [ ] **`TxIn` and `Address` types** (used for testing)

These can be simplified, as the tests only rely on the fact that generated values are unique and can be ordered. (The internal structure of these types is not relevant.)

In particular, the `Show` instance of `TxIn` is very noisy, making output of tests very difficult to read.

- [ ] **`Arbitrary` instances**

Many of these are duplicates of one another, or very similar. We should consider whether or not it would be worth merging similar instances.

Contributor guide

Open the contributing guide

Research direction

Locate the test definitions for the TxIn and Address types and the Arbitrary instances, then run the existing test suite to understand its current output. Simplify the test-only types, reduce noisy TxIn display output, and consolidate duplicate or similar Arbitrary instances while keeping the tests passing.

Written by the indexing model from the issue text.

Assessment

Tech stack
haskell
Domain
testing-qa
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.