fix: handling function lookup if call is followed by pipe

Author: gp-pereira

lib/refactorex/refactor/function.ex

@@ -26,13 +26,13 @@ defmodule Refactorex.Refactor.Function do
   def find_definitions(%{node: {:/, _, _}} = zipper),
     do: find_definitions(Z.down(zipper))
 
-  def find_definitions(%{node: {name, _, args}} = zipper) do
+  def find_definitions(%{node: {name, _, args} = node} = zipper) do
     num_args =
       case Z.up(zipper) do
         %{node: {:/, _, [_, {:__block__, _, [num_args]}]}} ->
           num_args
 
-        %{node: {:|>, _, _}} ->
+        %{node: {:|>, _, [_, ^node]}} ->
           length(args) + 1
 
         _ ->

test/refactor/function/rename_function_test.exs

@@ -103,6 +103,36 @@ defmodule Refactorex.Refactor.Function.RenameFunctionTest do
     )
   end
 
+  test "renames function call followed by pipe" do
+    assert_refactored(
+      RenameFunction,
+      """
+      defmodule Foo do
+        def foo() do
+        # v
+          extracted_function(zipper, aliases)
+        #                  ^
+          |> IO.inspect()
+        end
+
+        def extracted_function(a, b) do
+        end
+      end
+      """,
+      """
+      defmodule Foo do
+        def foo() do
+          #{placeholder()}(zipper, aliases)
+          |> IO.inspect()
+        end
+
+        def #{placeholder()}(a, b) do
+        end
+      end
+      """
+    )
+  end
+
   test "renames the selected public function with multiple definitions" do
     assert_refactored(
       RenameFunction,