summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorTuncer Ayaz <tuncer.ayaz@gmail.com>2014-06-14 18:12:18 +0200
committerTuncer Ayaz <tuncer.ayaz@gmail.com>2014-06-14 19:23:45 +0200
commit8fea315b0bfe120d8177c16bda2a6ee3e1343ac0 (patch)
treeb34215d3a42fa65af96a7d14537118aa84f4e686 /src
parent4a6dc223b7a53acf99ffebeab95daef6f57f4be1 (diff)
rebar_eunit: disambiguate internal naming
Partially revert naming changes introduced in 93689703c1: CoverageModules -> FilteredModules get_coverage_modules -> get_matching_modules Having the name "coverage" meaning "filtered/selected modules" can be confused with code coverage.
Diffstat (limited to 'src')
-rw-r--r--src/rebar_eunit.erl10
1 files changed, 5 insertions, 5 deletions
diff --git a/src/rebar_eunit.erl b/src/rebar_eunit.erl
index a6fa0de..faa6228 100644
--- a/src/rebar_eunit.erl
+++ b/src/rebar_eunit.erl
@@ -157,7 +157,7 @@ run_eunit(Config, CodePath, SrcErls) ->
%% Get matching tests and modules
AllModules = [rebar_utils:beam_to_mod(?EUNIT_DIR, N) || N <- AllBeamFiles],
- {Tests, CoverageModules} =
+ {Tests, FilteredModules} =
get_tests_and_modules(Config, ModuleBeamFiles, AllModules),
SrcModules = [rebar_utils:erl_to_mod(M) || M <- SrcErls],
@@ -168,7 +168,7 @@ run_eunit(Config, CodePath, SrcErls) ->
StatusBefore = status_before_eunit(),
EunitResult = perform_eunit(Config, Tests),
- rebar_cover_utils:perform_cover(Config, CoverageModules, SrcModules,
+ rebar_cover_utils:perform_cover(Config, FilteredModules, SrcModules,
eunit_dir()),
rebar_cover_utils:close(CoverLog),
@@ -221,9 +221,9 @@ get_tests_and_modules(Config, ModuleBeamFiles, AllModules) ->
{Tests, QualifiedTests} = get_qualified_and_unqualified_tests(Config),
Modules = get_test_modules(SelectedSuites, Tests,
QualifiedTests, ModuleBeamFiles),
- CoverageModules = get_coverage_modules(AllModules, Modules, QualifiedTests),
+ FilteredModules = get_matching_modules(AllModules, Modules, QualifiedTests),
MatchedTests = get_matching_tests(Modules, Tests, QualifiedTests),
- {MatchedTests, CoverageModules}.
+ {MatchedTests, FilteredModules}.
%%
%% == get suites specified via 'suites' option ==
@@ -331,7 +331,7 @@ get_test_modules(SelectedSuites, Tests, QualifiedTests, ModuleBeamFiles) ->
N <- ModuleBeamFiles]
end.
-get_coverage_modules(AllModules, Modules, QualifiedTests) ->
+get_matching_modules(AllModules, Modules, QualifiedTests) ->
ModuleFilterMapper =
fun({M, _}) ->
case lists:member(M, AllModules) of