thanos / beancount_ex / 383f46b4aa24cd98146b39e34aff7f1b4d137bde-PR-53

96%

main: 96%

Pull #53

github

The bench script was building a nested ledger, not a bug in DirectiveSort.

The for comprehension returned [[open, txn], [open, txn], ...] because |> List.flatten() only flattened each pair, not the outer list. DirectiveSort then hit a sublist where it expected a single directive.

Fix: use Enum.flat_map/2 so the ledger is a flat list of directives.

mix run bench/query_bench.exs completes successfully now.

345 of 367 new or added lines in 12 files covered. (94.01%)

1660 of 1729 relevant lines covered (96.01%)

433.43 hits per line