WIP add top-level refactor fn and test

ckochx · Sep 17, 2021 · aa940c3 · aa940c3
1 parent a0a352b
commit aa940c3
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/lib/ex_factor/extractor.ex b/lib/ex_factor/extractor.ex
@@ -56,6 +56,7 @@ defmodule ExFactor.Extractor do
     source_function = Keyword.get(opts, :source_function)
     arity = Keyword.get(opts, :arity)
     source_path = Keyword.get(opts, :source_path, path(source_module))
+    |> IO.inspect(label: "REMOVE source_path")
 
     Remover.remove(source_path, source_function, arity)
   end

diff --git a/test/ex_factor_test.exs b/test/ex_factor_test.exs
@@ -2,7 +2,7 @@ defmodule ExFactorTest do
   use ExUnit.Case
 
   describe "refactor/1" do
-    test "it should refactors the functions into a new module, specified in opts" do
+    test "it refactors the functions into a new module, specified in opts" do
       File.rm("lib/ex_factor/source_module.ex")
       File.rm("lib/ex_factor/target_module.ex")
 
@@ -49,9 +49,15 @@ defmodule ExFactorTest do
       assert file =~ "def(refactor1(arg1)) do"
       assert file =~ "def(refactor1([])) do"
       assert file =~ " @doc \"some docs\""
+      assert file =~ "def pub_exists(arg_exists) do"
 
-      File.rm("lib/ex_factor/source_module.ex")
-      File.rm("lib/ex_factor/target_module.ex")
+      file = File.read!("lib/ex_factor/source_module.ex")
+      |> IO.inspect(label: "")
+      refute file =~ "def refactor1(arg1) do"
+      refute file =~ "def refactor1([]) do"
+
+      # File.rm("lib/ex_factor/source_module.ex")
+      # File.rm("lib/ex_factor/target_module.ex")
     end
   end
 end