Enable formatting in testbench-api module (#10852)

server/src/main/java/com/vaadin/ui/ComponentRootSetter.java

@@ -52,7 +52,8 @@ public class ComponentRootSetter implements Serializable {
     }
 
     /**
-     * Checks if the given custom component or composite may accept a root component.
+     * Checks if the given custom component or composite may accept a root
+     * component.
      * <p>
      * For internal use only.
      *
@@ -63,11 +64,11 @@ public class ComponentRootSetter implements Serializable {
      *
      */
     public static boolean canSetRoot(Component customComponent) {
-        if(customComponent instanceof  CustomComponent) {
+        if (customComponent instanceof CustomComponent) {
             return true;
         }
-        if(customComponent instanceof  Composite) {
-            return ((Composite)customComponent).getCompositionRoot() == null;
+        if (customComponent instanceof Composite) {
+            return ((Composite) customComponent).getCompositionRoot() == null;
         }
         return false;
     }

server/src/main/java/com/vaadin/ui/components/colorpicker/ColorPickerPreview.java

@@ -209,4 +209,4 @@ public class ColorPickerPreview extends CssLayout implements HasValue<Color> {
                         .concat("' is not in any recognized format");
     }
 
-}
+}

server/src/test/java/com/vaadin/tests/server/component/colorpicker/HexPatternParsingTest.java

@@ -36,7 +36,8 @@ public class HexPatternParsingTest {
     public static Collection<Object[]> hexdata() {
         Object[][] validValues = { { "#000000", 0, 0, 0, true },
                 { "#ffffff", 255, 255, 255, true },
-                { "#FF00ff", 255, 0, 255, true }, { "#aa90e3",170,144,227, true },
+                { "#FF00ff", 255, 0, 255, true },
+                { "#aa90e3", 170, 144, 227, true },
                 { "#016953", 1, 105, 83, true },
                 { "#bC64D0", 188, 100, 208, true },
                 { "#F100FF", 241, 0, 255, true },

server/src/test/java/com/vaadin/tests/server/component/tree/TreeDeclarativeTest.java

@@ -149,22 +149,19 @@ public class TreeDeclarativeTest
     public void testUpdateExisting() {
         Tree tree = new Tree();
 
-        String treeDesign =
-                "<vaadin-tree selection-mode=\"MULTI\">" +
-                        "<node item=\"A\">A</node>" +
-                        "<node item=\"B\">B</node>" +
-                        "<node item=\"AA\" parent=\"A\">AA</node>" +
-                        "</vaadin-tree>";
+        String treeDesign = "<vaadin-tree selection-mode=\"MULTI\">"
+                + "<node item=\"A\">A</node>" + "<node item=\"B\">B</node>"
+                + "<node item=\"AA\" parent=\"A\">AA</node>" + "</vaadin-tree>";
 
         Design.read(new ByteArrayInputStream(treeDesign.getBytes()), tree);
-        Object[] items = tree.getDataProvider().
-                fetchChildren(new HierarchicalQuery(null, null)).toArray();
-        assertArrayEquals(new Object[]{"A", "B"}, items);
-        Object[] itemsA = tree.getDataProvider().
-                fetchChildren(new HierarchicalQuery(null, "A")).toArray();
-        assertArrayEquals(new Object[]{"AA"}, itemsA);
-        long countB = tree.getDataProvider().
-                fetchChildren(new HierarchicalQuery(null, "B")).count();
+        Object[] items = tree.getDataProvider()
+                .fetchChildren(new HierarchicalQuery(null, null)).toArray();
+        assertArrayEquals(new Object[] { "A", "B" }, items);
+        Object[] itemsA = tree.getDataProvider()
+                .fetchChildren(new HierarchicalQuery(null, "A")).toArray();
+        assertArrayEquals(new Object[] { "AA" }, itemsA);
+        long countB = tree.getDataProvider()
+                .fetchChildren(new HierarchicalQuery(null, "B")).count();
         assertEquals(0L, countB);
         assertTrue(tree.getSelectionModel() instanceof SelectionModel.Multi);
     }

testbench-api/pom.xml

@@ -35,6 +35,10 @@
 
     <build>
         <plugins>
+            <plugin>
+                <groupId>net.revelc.code.formatter</groupId>
+                <artifactId>formatter-maven-plugin</artifactId>
+            </plugin>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-checkstyle-plugin</artifactId>

testbench-api/src/main/java/com/vaadin/testbench/elements/GridElement.java

@@ -462,7 +462,8 @@ public class GridElement extends AbstractComponentElement {
         Optional<WebElement> toggleButton = getDriver()
                 .findElement(By.className("v-grid-sidebar-content"))
                 .findElements(By.className("column-hiding-toggle")).stream()
-                .filter(element -> element.getText().equals(toggleCaption)).findAny();
+                .filter(element -> element.getText().equals(toggleCaption))
+                .findAny();
         if (toggleButton.isPresent()) {
             toggleButton.ifPresent(element -> element.click());
         } else {