Use :tmp_dir tag

Author: wojtekmach

test/ex_doc/formatter/html/erlang_test.exs

@@ -1,9 +1,9 @@
 defmodule ExDoc.Formatter.HTML.ErlangTest do
   use ExUnit.Case
   import TestHelper
-  setup :create_tmp_dir
 
   @moduletag :otp24
+  @moduletag :tmp_dir
 
   test "it works", c do
     erlc(c, :foo, ~S"""

test/ex_doc/language/erlang_test.exs

@@ -2,9 +2,8 @@ defmodule ExDoc.Language.ErlangTest do
   use ExUnit.Case
   import TestHelper
 
-  setup :create_tmp_dir
-
   @moduletag :otp24
+  @moduletag :tmp_dir
 
   describe "autolink_doc/2" do
     test "module", c do

test/ex_doc/retriever/elixir_test.exs

@@ -3,7 +3,7 @@ defmodule ExDoc.Retriever.ElixirTest do
   alias ExDoc.{Retriever, DocAST}
   import TestHelper
 
-  setup :create_tmp_dir
+  @moduletag :tmp_dir
 
   describe "docs_from_modules/2" do
     test "module", c do

test/ex_doc/retriever/erlang_test.exs

@@ -4,8 +4,7 @@ defmodule ExDoc.Retriever.ErlangTest do
   import TestHelper
 
   @moduletag :otp24
-
-  setup :create_tmp_dir
+  @moduletag :tmp_dir
 
   describe "docs_from_modules/2" do
     test "module", c do

test/ex_doc/retriever_test.exs

@@ -3,7 +3,7 @@ defmodule ExDoc.RetrieverTest do
   alias ExDoc.Retriever
   import TestHelper
 
-  setup :create_tmp_dir
+  @moduletag :tmp_dir
 
   describe "docs_from_modules/2: Generic" do
     test "module with no docs", c do

test/test_helper.exs

@@ -89,20 +89,4 @@ defmodule TestHelper do
       raise "not supported"
     end
   end
-
-  # TODO: replace with ExUnit @tag :tmp_dir feature when we require Elixir v1.11.
-  def create_tmp_dir(context) do
-    module = escape_path(inspect(context.module))
-    name = escape_path(to_string(context.test))
-    dir = Path.join(["tmp", module, name])
-    File.rm_rf!(dir)
-    File.mkdir_p!(dir)
-    Map.put(context, :tmp_dir, dir)
-  end
-
-  @escape Enum.map(' [~#%&*{}\\:<>?/+|"]', &<<&1::utf8>>)
-
-  defp escape_path(path) do
-    String.replace(path, @escape, "-")
-  end
 end