diff options
author | Arjan Scherpenisse <arjan@scherpenisse.net> | 2012-08-01 20:47:47 +0200 |
---|---|---|
committer | Dave Smith <dizzyd@dizzyd.com> | 2012-10-31 20:37:42 -0600 |
commit | cc67814b657380507c3aee1222f9f2c8b8b061b4 (patch) | |
tree | 0cb91b92f7f731ada3a4549d7a0b341a854a5534 /inttest/retest | |
parent | 380506c38031722e928bacf1c5f143e5655ab349 (diff) |
ct: skip instead of halt on missing/unknown suite
This allows `rebar ct suites=abc` to consider all suites when
you have a rebar setup with multiple sub_dirs.
Previously, rebar halted after it could not find the suite in the
first dir. But the suite might be present in another dir (when
sub_dirs contains multiple dirs).
This commit makes it so instead of halting, it prints a warning and
continues with looking for the suite in the other `sub_dir`s.
Note -- This uses try/catch to cause the test to be skipped because
otherwise I needed to adjust the return values of 4 functions, the
code path is pretty deeply nested here. Otherwise the whole call chain
needed to be adjusted for this return value:
`run_test -> make_cmd -> get_suites -> find_suite_path`
IMHO, I think for exceptional cases like this it is fine to use throw;
specially since only the {skip} is catched and nothing else.
Diffstat (limited to 'inttest/retest')
0 files changed, 0 insertions, 0 deletions