Merge pull request #760 from tsloughter/app_info_state

move state into app_info

3 files changed, 28 insertions, 29 deletions

diff --git a/test/rebar_dir_SUITE.erl b/test/rebar_dir_SUITE.erl
index a3c5052..6fbc081 100644
--- a/test/rebar_dir_SUITE.erl
+++ b/test/rebar_dir_SUITE.erl
@@ -28,39 +28,39 @@ end_per_testcase(_, _Config) -> ok.
 
 default_src_dirs(Config) ->
     {ok, State} = rebar_test_utils:run_and_check(Config, [], ["compile"], return),
-    
-    [] = rebar_dir:src_dirs(State),
-    ["src"] = rebar_dir:src_dirs(State, ["src"]).
+
+    [] = rebar_dir:src_dirs(rebar_state:opts(State)),
+    ["src"] = rebar_dir:src_dirs(rebar_state:opts(State), ["src"]).
 
 default_extra_src_dirs(Config) ->
     {ok, State} = rebar_test_utils:run_and_check(Config, [], ["compile"], return),
-  
-    [] = rebar_dir:extra_src_dirs(State),
-    ["src"] = rebar_dir:extra_src_dirs(State, ["src"]).
+
+    [] = rebar_dir:extra_src_dirs(rebar_state:opts(State)),
+    ["src"] = rebar_dir:extra_src_dirs(rebar_state:opts(State), ["src"]).
 
 default_all_src_dirs(Config) ->
     {ok, State} = rebar_test_utils:run_and_check(Config, [], ["compile"], return),
-  
-    [] = rebar_dir:all_src_dirs(State),
-    ["src", "test"] = rebar_dir:all_src_dirs(State, ["src"], ["test"]).
+
+    [] = rebar_dir:all_src_dirs(rebar_state:opts(State)),
+    ["src", "test"] = rebar_dir:all_src_dirs(rebar_state:opts(State), ["src"], ["test"]).
 
 src_dirs(Config) ->
     RebarConfig = [{erl_opts, [{src_dirs, ["foo", "bar", "baz"]}]}],
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
-    
-    ["foo", "bar", "baz"] = rebar_dir:src_dirs(State).
+
+    ["foo", "bar", "baz"] = rebar_dir:src_dirs(rebar_state:opts(State)).
 
 extra_src_dirs(Config) ->
     RebarConfig = [{erl_opts, [{extra_src_dirs, ["foo", "bar", "baz"]}]}],
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
-    
-    ["foo", "bar", "baz"] = rebar_dir:extra_src_dirs(State).
+
+    ["foo", "bar", "baz"] = rebar_dir:extra_src_dirs(rebar_state:opts(State)).
 
 all_src_dirs(Config) ->
     RebarConfig = [{erl_opts, [{src_dirs, ["foo", "bar"]}, {extra_src_dirs, ["baz", "qux"]}]}],
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
-    
-    ["foo", "bar", "baz", "qux"] = rebar_dir:all_src_dirs(State).
+
+    ["foo", "bar", "baz", "qux"] = rebar_dir:all_src_dirs(rebar_state:opts(State)).
 
 profile_src_dirs(Config) ->
     RebarConfig = [
@@ -70,9 +70,9 @@ profile_src_dirs(Config) ->
         ]}
     ],
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["as", "more", "compile"], return),
-    
+
     R = lists:sort(["foo", "bar", "baz", "qux"]),
-    R = lists:sort(rebar_dir:src_dirs(State)).
+    R = lists:sort(rebar_dir:src_dirs(rebar_state:opts(State))).
 
 profile_extra_src_dirs(Config) ->
     RebarConfig = [
@@ -82,9 +82,9 @@ profile_extra_src_dirs(Config) ->
         ]}
     ],
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["as", "more", "compile"], return),
-    
+
     R = lists:sort(["foo", "bar", "baz", "qux"]),
-    R = lists:sort(rebar_dir:extra_src_dirs(State)).
+    R = lists:sort(rebar_dir:extra_src_dirs(rebar_state:opts(State))).
 
 profile_all_src_dirs(Config) ->
     RebarConfig = [
@@ -94,6 +94,6 @@ profile_all_src_dirs(Config) ->
         ]}
     ],
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["as", "more", "compile"], return),
-    
+
     R = lists:sort(["foo", "bar", "baz", "qux"]),
-    R = lists:sort(rebar_dir:all_src_dirs(State)).
+    R = lists:sort(rebar_dir:all_src_dirs(rebar_state:opts(State))).
diff --git a/test/rebar_profiles_SUITE.erl b/test/rebar_profiles_SUITE.erl
index 41bb535..d4c10c5 100644
--- a/test/rebar_profiles_SUITE.erl
+++ b/test/rebar_profiles_SUITE.erl
@@ -428,8 +428,7 @@ test_profile_applied_to_apps(Config) ->
 
     Apps = rebar_state:project_apps(State),
     lists:foreach(fun(App) ->
-        AppState = rebar_app_info:state(App),
-        Opts = rebar_state:opts(AppState),
+        Opts = rebar_app_info:opts(App),
         ErlOpts = dict:fetch(erl_opts, Opts),
         true = lists:member({d, 'TEST'}, ErlOpts)
     end, Apps).
diff --git a/test/rebar_src_dirs_SUITE.erl b/test/rebar_src_dirs_SUITE.erl
index e322190..e29dcf0 100644
--- a/test/rebar_src_dirs_SUITE.erl
+++ b/test/rebar_src_dirs_SUITE.erl
@@ -49,7 +49,7 @@ src_dirs_at_root(Config) ->
 
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
 
-    ["foo", "bar", "baz"] = rebar_dir:src_dirs(State, []).
+    ["foo", "bar", "baz"] = rebar_dir:src_dirs(rebar_state:opts(State), []).
 
 extra_src_dirs_at_root(Config) ->
     AppDir = ?config(apps, Config),
@@ -62,7 +62,7 @@ extra_src_dirs_at_root(Config) ->
 
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
 
-    ["foo", "bar", "baz"] = rebar_dir:extra_src_dirs(State, []).
+    ["foo", "bar", "baz"] = rebar_dir:extra_src_dirs(rebar_state:opts(State), []).
 
 src_dirs_in_erl_opts(Config) ->
     AppDir = ?config(apps, Config),
@@ -75,7 +75,7 @@ src_dirs_in_erl_opts(Config) ->
 
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
 
-    ["foo", "bar", "baz"] = rebar_dir:src_dirs(State, []).
+    ["foo", "bar", "baz"] = rebar_dir:src_dirs(rebar_state:opts(State), []).
 
 extra_src_dirs_in_erl_opts(Config) ->
     AppDir = ?config(apps, Config),
@@ -88,7 +88,7 @@ extra_src_dirs_in_erl_opts(Config) ->
 
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
 
-    ["foo", "bar", "baz"] = rebar_dir:extra_src_dirs(State, []).
+    ["foo", "bar", "baz"] = rebar_dir:extra_src_dirs(rebar_state:opts(State), []).
 
 src_dirs_at_root_and_in_erl_opts(Config) ->
     AppDir = ?config(apps, Config),
@@ -101,7 +101,7 @@ src_dirs_at_root_and_in_erl_opts(Config) ->
 
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
 
-    ["baz", "qux", "foo", "bar"] = rebar_dir:src_dirs(State, []).
+    ["baz", "qux", "foo", "bar"] = rebar_dir:src_dirs(rebar_state:opts(State), []).
 
 extra_src_dirs_at_root_and_in_erl_opts(Config) ->
     AppDir = ?config(apps, Config),
@@ -114,7 +114,7 @@ extra_src_dirs_at_root_and_in_erl_opts(Config) ->
 
     {ok, State} = rebar_test_utils:run_and_check(Config, RebarConfig, ["compile"], return),
 
-    ["baz", "qux", "foo", "bar"] = rebar_dir:extra_src_dirs(State, []).
+    ["baz", "qux", "foo", "bar"] = rebar_dir:extra_src_dirs(rebar_state:opts(State), []).
 
 build_basic_app(Config) ->
     AppDir = ?config(apps, Config),