flow: Unit tests (more of them) needed.

[ygoldfeld]

Filed by @ygoldfeld pre-open-source:

The good news is Flow has been widely tested with tons of integration testing, elsewhere. It has proven itself solid in practice.

The bad news is it lacks systematic unit testing which reduces confidence and increases risk of regressions. As an open-source project this is a deficit. Flow is manageably-sized, and it is not a hugely difficult task to resolve this. We have a pretty large GoogleTest-based unit test suite in sister/dependent project, Flow-IPC.

Since net_flow is a bit of an outlier as essentially a demo/niche module, it could be excluded from this work at first (separate ticket to handle it later/separately - TODO).

[ygoldfeld]

Changing title from flow: Unit tests needed. to flow: Unit tests (more of them) needed. A PR is coming very soon that'll add the unit test framework (with one new feature being tested); but of course we should have tests covering existing stuff.

[ygoldfeld]

#111 did add tests for quite a few existing items including oft-used things Basic_blob, Blob_with_log_context, Linked_hash_map, Linked_hash_set, Log_context. Ticket remains open: go through the rest of the code.