From 9b2f5a1e4b717cc8473d63554997760b99b4d7c9 Mon Sep 17 00:00:00 2001
From: Christian Koch <ckoch@cars.com>
Date: Thu, 4 Aug 2022 09:33:12 -0500
Subject: [PATCH] chore(cleanup): cleanup debug and readme

---
 README.md                         | 10 ++++++----
 test/ex_factor/cli_test.exs       |  3 +--
 test/ex_factor/evaluater_test.exs |  1 +
 3 files changed, 8 insertions(+), 6 deletions(-)

diff --git a/README.md b/README.md
index 07ff1d5..e06ea0c 100644
--- a/README.md
+++ b/README.md
@@ -28,6 +28,12 @@ Use at your peril, _for now._
 
 ## Roadmap TODO
 
+  - [] Option to only change the module name throughout the project
+  - [] find dead functions
+  - [] find module attrs and also move them?
+  - [] find types referenced in the moved specs
+  - [] find private functions references in refactored fn bodies.
+  - [] git stage all changes?
   - [] Add test for one file containing more than one `defmodule`
   - [] Add test for nested defmodules.
   - [] How does this work with macro code? Does that even make sense as a case to handle?
@@ -35,10 +41,6 @@ Use at your peril, _for now._
   - [] update test file refs by CLI option
   - [] Write tests to ensure we can find modules across umbrella apps.
   - [] Add configuration hooks?
-  - [] find dead functions
-  - [] find module attrs and also move them?
-  - [] find types referenced in the moved specs
-  - [] find private functions references in refactored fn bodies.
   - [] ElixirLS integration for VSCode?
   - [] Write the module code to rename usages of the refactored function
 
diff --git a/test/ex_factor/cli_test.exs b/test/ex_factor/cli_test.exs
index 7e6c4ca..52e32fc 100644
--- a/test/ex_factor/cli_test.exs
+++ b/test/ex_factor/cli_test.exs
@@ -88,11 +88,10 @@ defmodule ExFactor.CLITest do
     ]
 
     argv = OptionParser.to_argv(opts)
-    |> IO.inspect(label: "")
 
     {_cli_output, exit_status} = System.cmd("mix", ["ex_factor" | argv])
     assert exit_status == 0
-    file = File.read!(target_path) |> IO.inspect(label: "")
+    file = File.read!(target_path)
     assert file =~ "\n@spec pub1(term()) :: term()\ndef pub1(arg1) do\n  :pub1_ok\nend\nend"
   end
 end
diff --git a/test/ex_factor/evaluater_test.exs b/test/ex_factor/evaluater_test.exs
index 4b65f89..063f5d9 100644
--- a/test/ex_factor/evaluater_test.exs
+++ b/test/ex_factor/evaluater_test.exs
@@ -3,6 +3,7 @@ defmodule ExFactor.EvaluaterTest do
   alias ExFactor.Evaluater
 
   describe "modules_to_refactor/1" do
+    @tag :skip
     test "it should report callers of a module function" do
       assert ["test/support/support.ex" | _] =
                Evaluater.modules_to_refactor(ExFactor.Parser, :all_functions, 1)