ocaml-batteries-team/batteries-included — Issue #185: A tool to check for inactive/malformed tests
Author: gasche
Created: 2011-12-09 05:55:45+00:00
State: open
Labels: Build, Feature request

The parsing of inline unit tests is rather fragile: the good syntax is `(**T ... **)`, and using `(** T ... **)` or `(**T *)` will result in silently ignoring the test. Inactive test is more dangerous than no test at all as it gives a false sense of security. We need a tool to check for such inactive tests.

In `testsuite/` we could also try to spot inactive tests. Identifiers of the form `test_foo` that are not added in a OUnit test value (that is, that are never right-hand-side of a `>::` operator) are suspicious.