Add :require_ast

Author: josevalim

lib/mix/test/mix/tasks/source_test.exs

@@ -19,6 +19,7 @@ defmodule Mix.Tasks.SourceTest do
     end)
   end
 
+  @tag :require_ast
   test "source MODULE.FUN", context do
     in_tmp(context.test, fn ->
       Mix.Tasks.Source.run(["Enum.map"])
@@ -27,6 +28,7 @@ defmodule Mix.Tasks.SourceTest do
     end)
   end
 
+  @tag :require_ast
   test "source MODULE.FUN/ARITY", context do
     in_tmp(context.test, fn ->
       Mix.Tasks.Source.run(["Enum.map/2"])
@@ -80,23 +82,6 @@ defmodule Mix.Tasks.SourceTest do
       else: System.delete_env("ELIXIR_EDITOR")
   end
 
-  test "source -o opens with shortcut", context do
-    System.put_env("ELIXIR_EDITOR", "echo __LINE__:__FILE__")
-
-    in_tmp(context.test, fn ->
-      output =
-        capture_io(fn ->
-          Mix.Tasks.Source.run(["-o", "Enum.map/2"])
-        end)
-
-      assert output =~ ~r"\d+:.*lib/elixir/lib/enum\.ex"
-    end)
-  after
-    if @editor,
-      do: System.put_env("ELIXIR_EDITOR", @editor),
-      else: System.delete_env("ELIXIR_EDITOR")
-  end
-
   test "source --open without editor" do
     System.delete_env("ELIXIR_EDITOR")
     System.delete_env("EDITOR")

lib/mix/test/test_helper.exs

@@ -36,7 +36,7 @@ git_exclude =
 {line_exclude, line_include} =
   if line = System.get_env("LINE"), do: {[:test], [line: line]}, else: {[], []}
 
-cover_exclude =
+deterministic_exclude =
   if :deterministic in :compile.env_compiler_options() do
     [:cover, :requires_source]
   else
@@ -52,7 +52,13 @@ re_import_exclude =
   end
 
 Code.require_file("../../elixir/scripts/cover_record.exs", __DIR__)
-CoverageRecorder.maybe_record("mix")
+
+cover_exclude =
+  if CoverageRecorder.maybe_record("mix") do
+    [:require_ast]
+  else
+    []
+  end
 
 maybe_seed_opt = if seed = System.get_env("SEED"), do: [seed: String.to_integer(seed)], else: []
 
@@ -61,6 +67,7 @@ ex_unit_opts =
     trace: !!System.get_env("TRACE"),
     exclude:
       epmd_exclude ++
+        deterministic_exclude ++
         os_exclude ++ git_exclude ++ line_exclude ++ cover_exclude ++ re_import_exclude,
     include: line_include,
     assert_receive_timeout: String.to_integer(System.get_env("ELIXIR_ASSERT_TIMEOUT", "300"))