diff options
| author | Fred Hebert <mononcqc@ferd.ca> | 2016-11-20 21:40:54 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2016-11-20 21:40:54 -0500 |
| commit | 8c4da25be01969d2392ed8229cccfda42316c306 (patch) | |
| tree | c10cd279eacf1a2af637d42f8b2a876da991c7a3 /src/rebar_prv_common_test.erl | |
| parent | ca0995b73b104964ab20ceed440aa323976cbc09 (diff) | |
| parent | e85cf555e24b2a4b9b8681b9028c87826a2c0ea6 (diff) | |
Merge pull request #1381 from ferd/fix-dialyzer-warnings
Fix rebar3 dialyzer warnings
Diffstat (limited to 'src/rebar_prv_common_test.erl')
| -rw-r--r-- | src/rebar_prv_common_test.erl | 15 |
1 files changed, 4 insertions, 11 deletions
diff --git a/src/rebar_prv_common_test.erl b/src/rebar_prv_common_test.erl index b91c90f..be31e8c 100644 --- a/src/rebar_prv_common_test.erl +++ b/src/rebar_prv_common_test.erl @@ -212,7 +212,6 @@ add_hooks(Opts, State) -> lists:keyreplace(ct_hooks, 1, Opts, {ct_hooks, NewHooks}) end. -select_tests(_, _, {error, _} = Error, _) -> Error; select_tests(_, _, _, {error, _} = Error) -> Error; select_tests(State, ProjectApps, CmdOpts, CfgOpts) -> %% set application env if sys_config argument is provided @@ -293,14 +292,9 @@ compile(State, {ok, _} = Tests) -> compile(_State, Error) -> Error. do_compile(State) -> - case rebar_prv_compile:do(State) of - %% successfully compiled apps - {ok, S} -> - ok = maybe_cover_compile(S), - {ok, S}; - %% this should look like a compiler error, not an eunit error - Error -> Error - end. + {ok, S} = rebar_prv_compile:do(State), + ok = maybe_cover_compile(S), + {ok, S}. inject_ct_state(State, {ok, Tests}) -> Apps = rebar_state:project_apps(State), @@ -308,8 +302,7 @@ inject_ct_state(State, {ok, Tests}) -> {ok, {NewState, ModdedApps}} -> test_dirs(NewState, ModdedApps, Tests); {error, _} = Error -> Error - end; -inject_ct_state(_State, Error) -> Error. + end. inject_ct_state(State, Tests, [App|Rest], Acc) -> case inject(rebar_app_info:opts(App), State, Tests) of |