Do not count pin operator for ABC size

Refs #1155
rrrene · Oct 20, 2024 · 3d9dae8 · 3d9dae8
1 parent 599a7d8
commit 3d9dae8
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/lib/credo/check/refactor/abc_size.ex b/lib/credo/check/refactor/abc_size.ex
@@ -27,7 +27,7 @@ defmodule Credo.Check.Refactor.ABCSize do
   @def_ops [:def, :defp, :defmacro]
   @branch_ops [:.]
   @condition_ops [:if, :unless, :for, :try, :case, :cond, :and, :or, :&&, :||]
-  @non_calls [:==, :fn, :__aliases__, :__block__, :if, :or, :|>, :%{}]
+  @non_calls [:==, :fn, :__aliases__, :__block__, :if, :or, :|>, :%{}, :^]
 
   @doc false
   @impl true

diff --git a/test/credo/check/refactor/abc_size_test.exs b/test/credo/check/refactor/abc_size_test.exs
@@ -336,6 +336,20 @@ defmodule Credo.Check.Refactor.ABCSizeTest do
     assert rounded_abc_size(source) == 3
   end
 
+  test "it should NOT count pin operators (^) for abc size" do
+    source = """
+      def test(param, foo) do
+        case param do
+          ^foo -> foo.bar
+          "bar" -> bar.baz
+          "baz" -> bux.bus
+        end
+      end
+    """
+
+    assert rounded_abc_size(source) == 6
+  end
+
   test "it should NOT count functions given to ignore for abc size" do
     source = """
     def fun() do