Thanks for the review, Christian!
On 2021-02-08 13:42, Christian Ehrhardt wrote: > Recommended TODOs: > - upstream provides tests in test/*, could those be enabled at build > time?
As far as I can see they are already enabled via dh_auto_test. From the amd64 build log:
PASS: io-test PASS: trie-test PASS: base-test PASS: vector-test PASS: marisa-test ============================================================================ Testsuite summary for marisa 0.2.6 ============================================================================ # TOTAL: 5 # PASS: 5 # SKIP: 0 # XFAIL: 0 # FAIL: 0 # XPASS: 0 # ERROR: 0
Thanks for the review, Christian!
On 2021-02-08 13:42, Christian Ehrhardt wrote:
> Recommended TODOs:
> - upstream provides tests in test/*, could those be enabled at build
> time?
As far as I can see they are already enabled via dh_auto_test. From the amd64 build log:
PASS: io-test ======= ======= ======= ======= ======= ======= ======= ======= ======= ====== ======= ======= ======= ======= ======= ======= ======= ======= ======= ======
PASS: trie-test
PASS: base-test
PASS: vector-test
PASS: marisa-test
=======
Testsuite summary for marisa 0.2.6
=======
# TOTAL: 5
# PASS: 5
# SKIP: 0
# XFAIL: 0
# FAIL: 0
# XPASS: 0
# ERROR: 0