Use Eclipse formatter (behind -Pcom.palantir.baseline-format.eclipse feature flag)

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

@@ -0,0 +1,5 @@
+type: improvement
+improvement:
+  description: Use the Eclipse formatter in Spotless
+  links:
+  - https://github.com/palantir/gradle-baseline/pull/707

gradle-baseline-java/src/main/groovy/com/palantir/baseline/plugins/BaselineFormat.java

@@ -17,6 +17,7 @@
 package com.palantir.baseline.plugins;
 
 import com.diffplug.gradle.spotless.SpotlessExtension;
+import java.nio.file.Paths;
 import org.gradle.api.Project;
 import org.gradle.api.Task;
 import org.gradle.api.file.ConfigurableFileCollection;
@@ -46,14 +47,12 @@ public void apply(Project project) {
                 // use empty string to specify one group for all non-static imports
                 java.importOrder("");
                 java.trimTrailingWhitespace();
-                java.indentWithSpaces(4);
-                java.endWithNewline();
+
+                java.eclipse().configFile(
+                        project.file(Paths.get(getConfigDir(), "spotless/eclipse.xml").toString()));
 
                 // No empty lines at start of blocks
                 java.replaceRegex("Block starts with blank lines", "\\) \\{\n+", ") {\n");
-
-                // No dangling parentheses - closing paren must be on the same line as the expression
-                java.replaceRegex("Dangling closing parenthesis", "(\n\\s+)+\\)", ")");
             });
 
             // necessary because SpotlessPlugin creates tasks in an afterEvaluate block
@@ -65,4 +64,5 @@ public void apply(Project project) {
             });
         });
     }
+
 }

gradle-baseline-java/src/test/groovy/com/palantir/baseline/BaselineConfigIntegrationTest.groovy

@@ -22,7 +22,7 @@ import org.gradle.testkit.runner.TaskOutcome
  * This test relies on running ./gradlew :gradle-baseline-java-config:publishToMavenLocal.
  */
 class BaselineConfigIntegrationTest extends AbstractPluginTest {
-    def projectVersion = "git describe --tags --first-parent --dirty=.dirty".execute().text.trim()
+    def projectVersion = "git describe --tags --first-parent --dirty=.dirty --abbrev=7".execute().text.trim()
     def standardBuildFile = """
         plugins {
             id 'com.palantir.baseline-config'
@@ -46,7 +46,9 @@ class BaselineConfigIntegrationTest extends AbstractPluginTest {
 
         then:
         with('--stacktrace', '--info', 'baselineUpdateConfig').build()
-        directory('.baseline').list().toList().toSet() == ['checkstyle', 'copyright', 'eclipse', 'idea'].toSet()
+        directory('.baseline').list().toList().toSet() == [
+                'checkstyle', 'copyright', 'eclipse', 'idea', 'spotless'
+        ].toSet()
         directory('project').list().toList().isEmpty()
     }
 
@@ -65,7 +67,9 @@ class BaselineConfigIntegrationTest extends AbstractPluginTest {
 
         then:
         with('--stacktrace', '--info', 'baselineUpdateConfig').build()
-        directory('.baseline').list().toList().toSet() == ['checkstyle', 'copyright', 'eclipse', 'idea'].toSet()
+        directory('.baseline').list().toList().toSet() == [
+                'checkstyle', 'copyright', 'eclipse', 'idea', 'spotless'
+        ].toSet()
         directory('project').list().toList().toSet() == ['scalastyle_config.xml'].toSet()
     }
 

gradle-baseline-java/src/test/groovy/com/palantir/baseline/BaselineFormatIntegrationTest.groovy

@@ -16,11 +16,18 @@
 
 package com.palantir.baseline
 
+import org.apache.commons.io.FileUtils
 import org.gradle.testkit.runner.BuildResult
 import org.gradle.testkit.runner.TaskOutcome
 
 class BaselineFormatIntegrationTest extends AbstractPluginTest {
 
+    def setup() {
+        FileUtils.copyDirectory(
+                new File("../gradle-baseline-java-config/resources"),
+                new File(projectDir, ".baseline"))
+    }
+
     def standardBuildFile = '''
         plugins {
             id 'java'
@@ -36,14 +43,17 @@ class BaselineFormatIntegrationTest extends AbstractPluginTest {
 
     def validJavaFile = '''
     package test;
-    public class Test { void test() {
-        int x = 1;
-        System.out.println(
-            "Hello");
-        Optional.of("hello").orElseGet(() -> {
-            return "Hello World";
-        });
-    } }
+
+    public class Test {
+        void test() {
+            int x = 1;
+            System.out.println(
+                    "Hello");
+            Optional.of("hello").orElseGet(() -> {
+                return "Hello World";
+            });
+        }
+    }
     '''.stripIndent()
 
     def invalidJavaFile = '''

gradle-baseline-java/src/test/groovy/com/palantir/baseline/BaselineFormatTest.groovy

@@ -28,6 +28,11 @@ class BaselineFormatTest extends Specification {
 
     def setup() {
         project = ProjectBuilder.builder().build()
+        project.buildscript {
+            repositories {
+                mavenCentral()
+            }
+        }
         project.plugins.apply 'java'
         project.plugins.apply BaselineFormat
         project.evaluate()