From 8c23816f6a91eb0794b370dc47009c45e7288e36 Mon Sep 17 00:00:00 2001 From: markbrady Date: Fri, 12 Jun 2026 14:13:09 -0700 Subject: [PATCH] [IfChainToSwitch] test-only change: add explicit values for `EnableSafe` to tests, to replace implicit `false`s PiperOrigin-RevId: 931326552 --- .../bugpatterns/IfChainToSwitch.java | 6 +- .../bugpatterns/IfChainToSwitchTest.java | 206 +++++++++--------- 2 files changed, 111 insertions(+), 101 deletions(-) diff --git a/core/src/main/java/com/google/errorprone/bugpatterns/IfChainToSwitch.java b/core/src/main/java/com/google/errorprone/bugpatterns/IfChainToSwitch.java index dc1e5e392e7..b6469aa42c6 100644 --- a/core/src/main/java/com/google/errorprone/bugpatterns/IfChainToSwitch.java +++ b/core/src/main/java/com/google/errorprone/bugpatterns/IfChainToSwitch.java @@ -116,13 +116,15 @@ public final class IfChainToSwitch extends BugChecker implements IfTreeMatcher { private final boolean enableMain; private final boolean enableSafe; private final int maxChainLength; + private final int minChainLength; private final ConstantExpressions constantExpressions; @Inject IfChainToSwitch(ErrorProneFlags flags, ConstantExpressions constantExpressions) { - enableMain = flags.getBoolean("IfChainToSwitch:EnableMain").orElse(false); + enableMain = flags.getBoolean("IfChainToSwitch:EnableMain").orElse(true); enableSafe = flags.getBoolean("IfChainToSwitch:EnableSafe").orElse(false); maxChainLength = flags.getInteger("IfChainToSwitch:MaxChainLength").orElse(50); + minChainLength = flags.getInteger("IfChainToSwitch:MinChainLength").orElse(5); this.constantExpressions = constantExpressions; } @@ -184,7 +186,7 @@ private Description analyzeIfTree(IfTree ifTree, VisitorState state) { if (!ifChainAnalysisState.validity().equals(Validity.VALID) // Exclude short if-chains, since they may be more readable as-is - || ifChainAnalysisState.depth() < 3) { + || ifChainAnalysisState.depth() < minChainLength) { return NO_MATCH; } diff --git a/core/src/test/java/com/google/errorprone/bugpatterns/IfChainToSwitchTest.java b/core/src/test/java/com/google/errorprone/bugpatterns/IfChainToSwitchTest.java index 3e6a1ab4728..c2dca17ab4d 100644 --- a/core/src/test/java/com/google/errorprone/bugpatterns/IfChainToSwitchTest.java +++ b/core/src/test/java/com/google/errorprone/bugpatterns/IfChainToSwitchTest.java @@ -39,6 +39,12 @@ enum Suit { CLUB }; """; + + private static final String ENABLE_MAIN = "-XepOpt:IfChainToSwitch:EnableMain"; + private static final String DISABLE_SAFE = "-XepOpt:IfChainToSwitch:EnableSafe=false"; + private static final String ENABLE_SAFE = "-XepOpt:IfChainToSwitch:EnableSafe"; + private static final String ENABLE_SAFE_TRUE = "-XepOpt:IfChainToSwitch:EnableSafe=true"; + private static final String MIN_CHAIN_LENGTH_3 = "-XepOpt:IfChainToSwitch:MinChainLength=3"; private final CompilationTestHelper helper = CompilationTestHelper.newInstance(IfChainToSwitch.class, getClass()) .addSourceLines("Suit.java", SUIT); @@ -101,7 +107,7 @@ public void foo(Suit s) { } """)) .allowFormattingErrors() - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -135,7 +141,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -191,7 +197,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -251,7 +257,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -278,7 +284,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -304,7 +310,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -349,7 +355,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -400,7 +406,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -452,7 +458,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -483,7 +489,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -539,7 +545,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -593,7 +599,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -648,7 +654,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -695,7 +701,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -742,7 +748,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -792,7 +798,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -842,7 +848,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -886,7 +892,7 @@ public void foo(Suit suit) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -930,7 +936,7 @@ public void foo(Suit suit) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -980,7 +986,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1027,7 +1033,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1077,7 +1083,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1129,7 +1135,7 @@ public int foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1172,7 +1178,7 @@ boolean g(Proto proto, Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1214,7 +1220,7 @@ boolean g(Proto proto, Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1255,7 +1261,7 @@ boolean g(Proto proto, Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1283,7 +1289,7 @@ public void foo(Suit suit) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1328,7 +1334,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1356,7 +1362,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1388,7 +1394,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1423,7 +1429,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1474,7 +1480,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1502,7 +1508,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1532,7 +1538,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1566,7 +1572,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1632,7 +1638,7 @@ public void foo(Suit s) { } """) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1663,7 +1669,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1693,7 +1699,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1725,7 +1731,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1781,7 +1787,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1848,7 +1854,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -1877,7 +1883,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1907,7 +1913,7 @@ public void foo(Object o) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1933,7 +1939,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -1958,7 +1964,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -2075,7 +2081,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2124,7 +2130,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2173,7 +2179,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2263,7 +2269,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2352,7 +2358,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2413,7 +2419,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2471,7 +2477,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe=false") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2529,7 +2535,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2560,7 +2566,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe=false") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -2592,7 +2598,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe=false") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -2625,7 +2631,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe=false") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -2685,7 +2691,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2745,7 +2751,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2810,7 +2816,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2865,7 +2871,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2920,7 +2926,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -2943,7 +2949,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -2970,7 +2976,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3008,7 +3014,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3033,7 +3039,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe=true") + .setArgs(ENABLE_MAIN, ENABLE_SAFE_TRUE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3093,7 +3099,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3153,7 +3159,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3214,7 +3220,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3274,7 +3280,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3301,7 +3307,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3332,7 +3338,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3383,7 +3389,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3432,7 +3438,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3465,7 +3471,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3491,7 +3497,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3519,7 +3525,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3568,7 +3574,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3623,7 +3629,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3677,7 +3683,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3733,7 +3739,9 @@ public void foo(Suit s) { } """)) .setArgs( - "-XepOpt:IfChainToSwitch:EnableMain", + ENABLE_MAIN, + DISABLE_SAFE, + MIN_CHAIN_LENGTH_3, "--enable-preview", "--release", Integer.toString(Runtime.version().feature())) @@ -3773,7 +3781,7 @@ private void foo(Suit suit) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3815,7 +3823,7 @@ public void foo(Object obj) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3865,7 +3873,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3898,7 +3906,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -3939,7 +3947,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -3972,7 +3980,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -4013,7 +4021,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -4072,7 +4080,7 @@ public void foo(Die d) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -4114,7 +4122,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -4140,7 +4148,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -4185,7 +4193,7 @@ public void foo(Suit s) { } } """)) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -4228,7 +4236,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -4255,7 +4263,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -4281,7 +4289,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -4307,7 +4315,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -4333,7 +4341,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain", "-XepOpt:IfChainToSwitch:EnableSafe") + .setArgs(ENABLE_MAIN, ENABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -4359,7 +4367,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); } @@ -4401,7 +4409,7 @@ public void foo(Suit s) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .setFixChooser(IfChainToSwitchTest::assertOneFixAndChoose) .doTest(); } @@ -4429,7 +4437,7 @@ public void foo(Object o) { } } """) - .setArgs("-XepOpt:IfChainToSwitch:EnableMain") + .setArgs(ENABLE_MAIN, DISABLE_SAFE, MIN_CHAIN_LENGTH_3) .doTest(); }