TMP: ignore thor commands

Signed-off-by: Alexandre Terrasa <[email protected]>

lib/spoom/deadcode/plugins/thor.rb

@@ -11,9 +11,6 @@ class Thor < Base
 
         sig { override.params(symbol_def: Model::Method, definition: Definition).void }
         def on_define_method(symbol_def, definition)
-          # TODO?
-          # return if indexer.nesting_block # method defined in `no_commands do ... end`, we don't want to ignore it
-
           owner = symbol_def.owner
           return unless owner.is_a?(Model::Class)
 

test/spoom/deadcode/plugins/thor_test.rb

@@ -13,10 +13,10 @@ class ThorTest < TestWithProject
         def test_ignore_thor_commands
           @project.write!("foo.rb", <<~RB)
             class Foo < Thor
-              def ignored; end
+              def ignored1; end
 
               no_commands do
-                def dead; end
+                def ignored2; end
               end
             end
 
@@ -26,8 +26,8 @@ class Bar; end
           index = index_with_plugins
           refute_ignored(index, "Foo")
           refute_ignored(index, "Bar")
-          assert_ignored(index, "ignored")
-          refute_ignored(index, "dead")
+          assert_ignored(index, "ignored1")
+          assert_ignored(index, "ignored2")
         end
 
         def test_ignore_thor_methods