17d49d87

By: Tom Sydney Kerckhove <syd@cs-syd.eu>

Inline test-only helpers per the testing style rule

The testing style guide says: "Avoid helper functions in test modules,
especially if they are only used in one module and not exported.
Instead, prefer to duplicate that code in tests."

Inline the per-module helpers (aTestId, aRecord, chunksToString,
colouredChunksToString, mkRec, scriptedAction, countingOnRetry,
runScripted, asResult, ScriptedOutcome, the example* fixtures, the
is* predicates, skippedCauseId, recordsOf) into the individual 'it'
blocks that consume them.  Local 'let' bindings within a single test
are kept where they avoid duplicating non-trivial setup within that
test.

Timeline