Merge pull request #538 from skipkayhil/hm-fix-rm-first-module-method

Fix removing first method in class

lib/spoom/deadcode/remover.rb

@@ -312,7 +312,7 @@ def delete_node_and_comments_and_sigs(context)
           # Adjust the lines to remove to include following blank lines
           after = context.next_node
           if before.nil? && after && after.location.start_line > end_line + 1
-            end_line = after.location.end_line - 1
+            end_line = after.location.start_line - 1
           elsif after.nil? && context.parent_node.location.end_line > end_line + 1
             end_line = context.parent_node.location.end_line - 1
           end

test/spoom/deadcode/remover_test.rb

@@ -821,6 +821,25 @@ module Bar; end
         RB
       end
 
+      def test_deadcode_remover_removes_first_node_with_blank_lines
+        res = remove(<<~RB, "foo")
+          class Foo
+            def foo
+            end
+
+            def bar
+            end
+          end
+        RB
+
+        assert_equal(<<~RB, res)
+          class Foo
+            def bar
+            end
+          end
+        RB
+      end
+
       def test_deadcode_remover_removes_node_with_blank_lines
         res = remove(<<~RB, "bar")
           class Foo