Fix: Properly format switch expressions

changelog/@unreleased/pr-1069.v2.yml

@@ -0,0 +1,5 @@
+type: fix
+fix:
+  description: Properly format switch expressions
+  links:
+  - https://github.com/palantir/palantir-java-format/pull/1069

gradle-palantir-java-format/src/test/groovy/com/palantir/javaformat/gradle/FormatDiffTest.java

@@ -63,6 +63,7 @@ void reformat_a_subpath_of_a_git_directory_for_only_changed_lines() throws IOExc
         runCommandInRepo("git", "init");
         runCommandInRepo("git", "config", "user.name", "Test User");
         runCommandInRepo("git", "config", "user.email", "[email protected]");
+        runCommandInRepo("git", "config", "commit.gpgsign", "false");
         runCommandInRepo("git", "commit", "--allow-empty", "-m", "Init");
 
         Path subdir = repo.resolve("subdir");

palantir-java-format/src/main/java/com/palantir/javaformat/java/java14/Java14InputAstVisitor.java

@@ -303,6 +303,7 @@ public Void visitCase(CaseTree node, Void unused) {
                     scan(node.getBody(), null);
                 }
                 builder.guessToken(";");
+                builder.forcedBreak(minusTwo);
                 break;
             default:
                 throw new IllegalArgumentException(node.getCaseKind().name());

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata/ExpressionSwitch.input

@@ -41,4 +41,21 @@ class ExpressionSwitch {
       default -> false;
     };
   }
+
+  public void test1(int y) {
+      int x =
+              switch (y) {
+                  case 1 -> 1;
+                  case 2 -> throw new IllegalArgumentException();
+                  default -> throw new IllegalStateException();
+              };
+  }
+
+  public void test2(int y) {
+      int x;
+      x = switch (y) {
+          case 1 -> 1;
+          case 2 -> throw new IllegalArgumentException();
+          default -> throw new IllegalStateException();};
+  }
 }

palantir-java-format/src/test/resources/com/palantir/javaformat/java/testdata/ExpressionSwitch.output

@@ -47,4 +47,22 @@ class ExpressionSwitch {
             default -> false;
         };
     }
+
+    public void test1(int y) {
+        int x =
+                switch (y) {
+                    case 1 -> 1;
+                    case 2 -> throw new IllegalArgumentException();
+                    default -> throw new IllegalStateException();
+                };
+    }
+
+    public void test2(int y) {
+        int x;
+        x = switch (y) {
+            case 1 -> 1;
+            case 2 -> throw new IllegalArgumentException();
+            default -> throw new IllegalStateException();
+        };
+    }
 }