add test to verify only project_plugins override providers

test/mock_git_resource.erl

@@ -1,6 +1,6 @@
 %%% Mock a git resource and create an app magically for each URL submitted.
 -module(mock_git_resource).
--export([mock/0, mock/1, unmock/0]).
+-export([mock/0, mock/1, mock/2, unmock/0]).
 -define(MOD, rebar_git_resource).
 
 %%%%%%%%%%%%%%%%%
@@ -24,11 +24,14 @@ mock() -> mock([]).
          | {pkg, App, term()},
     Vsn :: string().
 mock(Opts) ->
+    mock(Opts, create_app).
+
+mock(Opts, CreateType) ->
     meck:new(?MOD, [no_link]),
     mock_lock(Opts),
     mock_update(Opts),
     mock_vsn(Opts),
-    mock_download(Opts),
+    mock_download(Opts, CreateType),
     ok.
 
 unmock() ->
@@ -98,7 +101,7 @@ mock_vsn(Opts) ->
 %%   `{deps, [{"app1", [{app2, ".*", {git, ...}}]}]}' -- basically
 %%   the `deps' option takes a key/value list of terms to output directly
 %%   into a `rebar.config' file to describe dependencies.
-mock_download(Opts) ->
+mock_download(Opts, CreateType) ->
     Deps = proplists:get_value(deps, Opts, []),
     Config = proplists:get_value(config, Opts, []),
     Default = proplists:get_value(default_vsn, Opts, "0.0.0"),
@@ -110,7 +113,7 @@ mock_download(Opts) ->
             {git, Url, {_, Vsn}} = normalize_git(Git, Overrides, Default),
             App = app(Url),
             AppDeps = proplists:get_value({App,Vsn}, Deps, []),
-            rebar_test_utils:create_app(
+            rebar_test_utils:CreateType(
                 Dir, App, Vsn,
                 [kernel, stdlib] ++ [element(1,D) || D  <- AppDeps]
             ),

test/rebar_plugins_SUITE.erl

@@ -12,7 +12,8 @@
          list/1,
          upgrade/1,
          sub_app_plugins/1,
-         sub_app_plugin_overrides/1]).
+         sub_app_plugin_overrides/1,
+         project_plugins/1]).
 
 -include_lib("common_test/include/ct.hrl").
 -include_lib("eunit/include/eunit.hrl").
@@ -34,7 +35,7 @@ end_per_testcase(_, _Config) ->
     catch meck:unload().
 
 all() ->
-    [compile_plugins, compile_global_plugins, complex_plugins, list, upgrade, sub_app_plugins, sub_app_plugin_overrides].
+    [compile_plugins, compile_global_plugins, complex_plugins, list, upgrade, sub_app_plugins, sub_app_plugin_overrides, project_plugins].
 
 %% Tests that compiling a project installs and compiles the plugins of deps
 compile_plugins(Config) ->
@@ -281,3 +282,50 @@ sub_app_plugin_overrides(Config) ->
       Config, RConf, ["compile"],
       {ok, [{app, Name}, {dep, Dep2Name, Vsn}, {plugin, DepName, Vsn2}, {plugin, PluginName}]}
      ).
+
+%% Check that project plugins are first in providers even if they override defaults but that
+%% normal plugins do not
+project_plugins(Config) ->
+    AppDir = ?config(apps, Config),
+
+    Name = rebar_test_utils:create_random_name("app1_"),
+    Vsn = rebar_test_utils:create_random_vsn(),
+    rebar_test_utils:create_app(AppDir, Name, Vsn, [kernel, stdlib]),
+
+    DepName = rebar_test_utils:create_random_name("dep1_"),
+    PluginName = "compile",
+    PluginName2 = "release",
+
+    Plugins = rebar_test_utils:expand_deps(git, [{PluginName, Vsn, []}, {PluginName2, Vsn, []}]),
+    {SrcDeps, _} = rebar_test_utils:flat_deps(Plugins),
+    mock_git_resource:mock([{deps, SrcDeps}], create_plugin),
+
+    mock_pkg_resource:mock([{pkgdeps, [{{list_to_binary(DepName), list_to_binary(Vsn)}, []}]},
+                            {config, [{plugins, [
+                                                {list_to_atom(PluginName),
+                                                 {git, "http://site.com/user/"++PluginName++".git",
+                                                 {tag, Vsn}}}]}]}]),
+
+    RConfFile =
+        rebar_test_utils:create_config(AppDir,
+                                      [{deps, [
+                                              list_to_atom(DepName)
+                                              ]},
+                                      {project_plugins, [
+                                                        {list_to_atom(PluginName2),
+                                                         {git, "http://site.com/user/"++PluginName2++".git",
+                                                          {tag, Vsn}}}]}]),
+    {ok, RConf} = file:consult(RConfFile),
+
+    %% Build with deps.
+    {ok, State} = rebar_test_utils:run_and_check(
+                    Config, RConf, ["compile"],
+                    {ok, [{app, Name}, {plugin, PluginName}, {plugin, PluginName2}, {dep, DepName}]}
+                   ),
+
+    %% Should have 2 release providers but only 1 compile provider
+    Release = [P || P <- rebar_state:providers(State), providers:impl(P) =:= release, providers:namespace(P) =:= default],
+    Compile = [P || P <- rebar_state:providers(State), providers:impl(P) =:= compile, providers:namespace(P) =:= default],
+
+    ?assertEqual(length(Release), 2),
+    ?assertEqual(length(Compile), 1).

test/rebar_test_utils.erl

@@ -3,7 +3,7 @@
 -include_lib("eunit/include/eunit.hrl").
 -export([init_rebar_state/1, init_rebar_state/2, run_and_check/4, check_results/3]).
 -export([expand_deps/2, flat_deps/1, top_level_deps/1]).
--export([create_app/4, create_eunit_app/4, create_empty_app/4,
+-export([create_app/4, create_plugin/4, create_eunit_app/4, create_empty_app/4,
          create_config/2, create_config/3, package_app/3]).
 -export([create_random_name/1, create_random_vsn/0, write_src_file/2]).
 
@@ -82,6 +82,16 @@ create_app(AppDir, Name, Vsn, Deps) ->
     write_app_src_file(AppDir, Name, Vsn, Deps),
     rebar_app_info:new(Name, Vsn, AppDir, Deps).
 
+%% @doc Creates a dummy plugin including:
+%% - src/<file>.erl
+%% - src/<file>.app.src
+%% And returns a `rebar_app_info' object.
+create_plugin(AppDir, Name, Vsn, Deps) ->
+    write_plugin_file(AppDir, Name ++ ".erl"),
+    write_src_file(AppDir, "not_a_real_src_" ++ Name ++ ".erl"),
+    write_app_src_file(AppDir, Name, Vsn, Deps),
+    rebar_app_info:new(Name, Vsn, AppDir, Deps).
+
 %% @doc Creates a dummy application including:
 %% - src/<file>.erl
 %% - src/<file>.app.src
@@ -365,6 +375,11 @@ check_results(AppDir, Expected, ProfileRun) ->
                 ?assert(filelib:is_dir(Dirname))
         end, Expected).
 
+write_plugin_file(Dir, Name) ->
+    Erl = filename:join([Dir, "src", Name]),
+    ok = filelib:ensure_dir(Erl),
+    ok = ec_file:write(Erl, plugin_src_file(Name)).
+
 write_src_file(Dir, Name) ->
     Erl = filename:join([Dir, "src", Name]),
     ok = filelib:ensure_dir(Erl),
@@ -395,6 +410,18 @@ erl_src_file(Name) ->
                   "-export([main/0]).\n"
                   "main() -> ok.\n", [filename:basename(Name, ".erl")]).
 
+plugin_src_file(Name) ->
+    io_lib:format("-module('~s').\n"
+                 "-export([init/1]).\n"
+                 "init(State) -> \n"
+                 "Provider = providers:create([\n"
+                 "{name, '~s'},\n"
+                 "{module, '~s'}\n"
+                 "]),\n"
+                 "{ok, rebar_state:add_provider(State, Provider)}.\n", [filename:basename(Name, ".erl"),
+                                                                        filename:basename(Name, ".erl"),
+                                                                        filename:basename(Name, ".erl")]).
+
 erl_eunitized_src_file(Name) ->
     io_lib:format("-module('~s').\n"
                   "-export([main/0]).\n"