search

sociomantic-tsunami / dlsproto

Distributed Log Store protocol definition, client, fake node, and tests
Boost Software License 1.0
3 stars 18 forks source link

Fix missing import for builds without unittest version #106

Closed bogdan-szabo-sociomantic closed 5 years ago

bogdan-szabo-sociomantic commented 5 years ago

Compiling turtle tests fail because the waitTotalRecords uses TestException that is imported only for unittest builds.

nemanja-boric-sociomantic commented 5 years ago

Should the same fix be applied to dmqproto?

bogdan-szabo-sociomantic commented 5 years ago

it looks like dmq proto imports the whole module: https://github.com/sociomantic-tsunami/dmqproto/blob/v14.x.x/src/turtle/env/Dmq.d#L22

it would be useful to have a build of the library without unit tests, to avoid these problems in the future.