Merge pull request #95 from talentdeficit/stable_test_dirs

when compiling `common_test` or `eunit` use the same tmp dir

1 files changed, 17 insertions, 5 deletions

diff --git a/src/rebar_prv_common_test.erl b/src/rebar_prv_common_test.erl
index 7a52c92..f17134a 100644
--- a/src/rebar_prv_common_test.erl
+++ b/src/rebar_prv_common_test.erl
@@ -178,13 +178,25 @@ split_ct_dirs(State, RawOpts) ->
             proplists:get_value(dir, CTOpts, []);
         Dirs -> split_string(Dirs)
     end,
-    OutDir = case proplists:get_value(outdir, RawOpts) of
-        undefined -> filename:join([rebar_state:dir(State),
-                                    ec_file:insecure_mkdtemp()]);
-        Out -> Out
-    end,
+    OutDir = proplists:get_value(outdir, RawOpts, default_test_dir(State)),
     {InDirs, OutDir}.
 
+default_test_dir(State) ->
+    Tmp = case erlang:system_info(system_architecture) of
+        "win32" ->
+            "./tmp";
+        _SysArch ->
+            "/tmp"
+    end,
+    Root = filename:join([rebar_state:dir(State), Tmp]),
+    Project = filename:basename(rebar_state:dir(State)),
+    OutDir = filename:join([Root, Project ++ "_rebar3_ct"]),
+    %% delete the directory if it exists so tests run with clean state
+    _ = ec_file:remove(OutDir, [recursive]),
+    %% recreate the directory
+    ok = filelib:ensure_dir(filename:join([OutDir, "dummy.beam"])),
+    OutDir.
+
 transform_opts(Opts) ->
     transform_opts(Opts, []).