From 8cfb4df3e055b4f1e323273faf9b5157994a583a Mon Sep 17 00:00:00 2001
From: Fred Hebert <mononcqc@ferd.ca>
Date: Sat, 7 Mar 2015 15:44:32 -0500
Subject: Fix namespace regressions

This adds some more convolution to command parsing, unfortunately, but
makes some cases clearer (`rebar3 as as as as` is no longer valid). It
unfortunately disallows using $REBAR_PROFILE along with `as` in a rebar
command.
---
 test/rebar_namespace_SUITE.erl | 24 ++++++++++++++++++++++--
 1 file changed, 22 insertions(+), 2 deletions(-)

(limited to 'test/rebar_namespace_SUITE.erl')

diff --git a/test/rebar_namespace_SUITE.erl b/test/rebar_namespace_SUITE.erl
index 40b7238..3a55573 100644
--- a/test/rebar_namespace_SUITE.erl
+++ b/test/rebar_namespace_SUITE.erl
@@ -5,7 +5,8 @@
 
 all() -> [implicit_compile, default_compile, do_compile,
           as_default_compile, as_do_compile,
-          notfound, do_notfound, default_notfound, ns_notfound].
+          notfound, do_notfound, default_notfound, ns_notfound, ns_found,
+          as_ns_invalid].
 
 init_per_testcase(Case, Config0) ->
     Config = rebar_test_utils:init_rebar_state(Config0),
@@ -77,6 +78,22 @@ ns_notfound(Config) ->
                             [fakecommand, ns])}
     ).
 
+ns_found(Config) ->
+    Command = ["ns", "fake_provider"],
+    rebar_test_utils:run_and_check(
+      add_fake_ns_provider(Config), [], Command,
+      {ok, []}
+    ).
+
+as_ns_invalid(Config) ->
+    %% The as namespace is not valid
+    Command = ["as", "profile", "as", "task"],
+    rebar_test_utils:run_and_check(
+      add_fake_ns_provider(Config), [], Command,
+      {error, "Namespace 'as' is forbidden"}
+    ).
+
+
 %%% Helpers %%%
 add_fake_ns_provider(Config) ->
     State = ?config(state, Config),
@@ -84,10 +101,13 @@ add_fake_ns_provider(Config) ->
       State,
       providers:create(
         [{name, fake_provider},
-         {module, fake_provider},
+         {module, ?MODULE},
          {namespace, ns},
          {deps, []},
          {opts, []}]
        )
      ),
     [{state, State1} | Config].
+
+%% callback for the test suite.
+do(State) -> {ok, State}.
-- 
cgit v1.1