refactor: ignore alias and aliases on some refactors

lib/refactorex/refactor/constant/extract_constant.ex

@@ -12,7 +12,7 @@ defmodule Refactorex.Refactor.Constant.ExtractConstant do
   @constant_name "extracted_constant"
 
   def can_refactor?(%{node: {id, _, _}}, _)
-      when id in ~w(@ &)a,
+      when id in ~w(@ & alias __aliases__)a,
       do: false
 
   def can_refactor?(%{node: node} = zipper, selection) do

lib/refactorex/refactor/function/extract_function.ex

@@ -1,6 +1,6 @@
 defmodule Refactorex.Refactor.Function.ExtractFunction do
   use Refactorex.Refactor,
-    title: "Extract private function",
+    title: "Extract function",
     kind: "refactor.extract",
     works_on: :selection
 
@@ -13,7 +13,9 @@ defmodule Refactorex.Refactor.Function.ExtractFunction do
 
   @function_name "extracted_function"
 
-  def can_refactor?(_, {id, _, _}) when id in ~w(@ & <-)a, do: :skip
+  def can_refactor?(%{node: {id, _, _}}, _)
+      when id in ~w(@ & <- alias __aliases__)a,
+      do: :skip
 
   def can_refactor?(%{node: node} = zipper, selection) do
     cond do

lib/refactorex/refactor/pipeline/introduce_io_inspect.ex

@@ -6,7 +6,9 @@ defmodule Refactorex.Refactor.Pipeline.IntroduceIOInspect do
 
   alias Refactorex.Refactor.Variable
 
-  def can_refactor?(_, {id, _, _}) when id in ~w(<- &)a, do: :skip
+  def can_refactor?(_, {id, _, _})
+      when id in ~w(<- & alias __aliases__)a,
+      do: :skip
 
   def can_refactor?(%{node: node} = zipper, selection) do
     cond do

lib/refactorex/refactor/variable/extract_variable.ex

@@ -12,6 +12,10 @@ defmodule Refactorex.Refactor.Variable.ExtractVariable do
 
   @variable_name "extracted_variable"
 
+  def can_refactor?(%{node: {id, _, _}}, _)
+      when id in ~w(alias __aliases__)a,
+      do: false
+
   def can_refactor?(%{node: node} = zipper, selection) do
     cond do
       not AST.equal?(node, selection) ->

test/refactor/constant/extract_constant_test.exs

@@ -215,4 +215,17 @@ defmodule Refactorex.Refactor.Constant.ExtractConstantTest do
       """
     )
   end
+
+  test "ignores alias" do
+    assert_not_refactored(
+      ExtractConstant,
+      """
+      defmodule Foo do
+      # v
+        alias Foo.Bar
+      #             ^
+      end
+      """
+    )
+  end
 end

test/refactor/function/extract_function_test.exs

@@ -260,4 +260,17 @@ defmodule Refactorex.Refactor.Function.ExtractFunctionTest do
       """
     )
   end
+
+  test "ignores alias" do
+    assert_not_refactored(
+      ExtractFunction,
+      """
+      defmodule Foo do
+      # v
+        alias Foo.Bar
+      #             ^
+      end
+      """
+    )
+  end
 end

test/refactor/pipeline/introduce_io_inspect_test.exs

@@ -58,4 +58,17 @@ defmodule Refactorex.Refactor.Pipeline.IntroduceIOInspectTest do
       """
     )
   end
+
+  test "ignores alias" do
+    assert_not_refactored(
+      IntroduceIOInspect,
+      """
+      defmodule Foo do
+      # v
+        alias Foo.Bar
+      #             ^
+      end
+      """
+    )
+  end
 end

test/refactor/variable/extract_variable_test.exs

@@ -298,4 +298,30 @@ defmodule Refactorex.Refactor.Variable.ExtractVariableTest do
       """
     )
   end
+
+  test "ignores alias" do
+    assert_not_refactored(
+      ExtractVariable,
+      """
+      defmodule Foo do
+      # v
+        alias Foo.Bar
+      #             ^
+      end
+      """
+    )
+
+    assert_not_refactored(
+      ExtractVariable,
+      """
+      defmodule Foo do
+        def foo() do
+      #   v
+          Foo.Bar
+      #         ^
+        end
+      end
+      """
+    )
+  end
 end