Add a test for the broken behavior due to missing grouping parenthesis.

PiperOrigin-RevId: 684584755
google · Oct 10, 2024 · 1dbcb6e · 1dbcb6e
1 parent 5145dae
commit 1dbcb6e
Showing 1 changed file with 40 additions and 0 deletions.
diff --git a/core/src/test/java/com/google/errorprone/bugpatterns/inlineme/InlinerTest.java b/core/src/test/java/com/google/errorprone/bugpatterns/inlineme/InlinerTest.java
@@ -1471,6 +1471,46 @@ public void doTest() {
         .doTest();
   }
 
+  @Test
+  public void math_b308614050() {
+    refactoringTestHelper
+        .addInputLines(
+            "Client.java",
+            """
+            package com.google.foo;
+            import com.google.errorprone.annotations.InlineMe;
+            public final class Client {
+              @InlineMe(replacement = "x * 2")
+              public static int timesTwo(int x) {
+                return x * 2;
+              }
+            }
+            """)
+        .expectUnchanged()
+        .addInputLines(
+            "Caller.java",
+            """
+            import com.google.foo.Client;
+            public final class Caller {
+              public void doTest() {
+                long four = Client.timesTwo(1 + 1);
+              }
+            }
+            """)
+        // This is a bug since it now evaluates to 3, not 4!
+        .addOutputLines(
+            "Caller.java",
+            """
+            import com.google.foo.Client;
+            public final class Caller {
+              public void doTest() {
+                long four = 1 + 1 * 2;
+              }
+            }
+            """)
+        .doTest();
+  }
+
   private BugCheckerRefactoringTestHelper bugCheckerWithPrefixFlag(String prefix) {
     return BugCheckerRefactoringTestHelper.newInstance(Inliner.class, getClass())
         .setArgs("-XepOpt:" + PREFIX_FLAG + "=" + prefix);