Formatting

Author: aurelien-coet-sonarsource

java-checks-common/src/test/java/org/sonar/java/checks/helpers/JParserTestUtils.java

@@ -17,13 +17,11 @@
 package org.sonar.java.checks.helpers;
 
 import com.google.common.io.Files;
-
 import java.io.File;
 import java.nio.charset.StandardCharsets;
 import java.util.Arrays;
 import java.util.List;
 import java.util.stream.Collectors;
-
 import org.sonar.java.model.FrontendParser;
 import org.sonar.java.model.JParserConfig;
 import org.sonar.java.test.TestResourceScope;

java-checks-common/src/test/java/org/sonar/java/checks/helpers/ReassignmentFinderTest.java

@@ -18,7 +18,6 @@
 
 import java.lang.reflect.Constructor;
 import java.util.List;
-
 import org.junit.jupiter.api.Test;
 import org.sonar.java.test.TestResourceScope;
 import org.sonar.plugins.java.api.semantic.Symbol;

java-checks-test-sources/test-classpath-reader/src/main/java/org/sonar/java/test/TestResourceScope.java

@@ -30,7 +30,9 @@
 import org.junit.jupiter.api.parallel.Execution;
 import org.junit.jupiter.api.parallel.ExecutionMode;
 
-/** Retains lazy fixtures through instance initialization, nested tests and all assertions in a test class. */
+/**
+ * Retains lazy fixtures through instance initialization, nested tests and all assertions in a test class.
+ */
 public final class TestResourceScope implements BeforeAllCallback, AfterAllCallback {
 
   private static final ThreadLocal<Deque<Scope>> SCOPES = ThreadLocal.withInitial(ArrayDeque::new);
@@ -85,7 +87,9 @@ public static final class Scope implements AutoCloseable {
     private Scope() {
     }
 
-    /** Attempts every fixture even after errors, then rethrows the first failure with later failures suppressed. */
+    /**
+     * Attempts every fixture even after errors, then rethrows the first failure with later failures suppressed.
+     */
     @Override
     public void close() throws Exception {
       if (closed) {

java-checks-test-sources/test-classpath-reader/src/test/java/org/sonar/java/test/TestResourceScopeTest.java

@@ -46,8 +46,14 @@ void scope_closes_every_resource_in_reverse_order_even_when_cleanup_fails() thro
     IllegalStateException first = new IllegalStateException("first");
     Exception second = new Exception("second");
     TestResourceScope.retain(() -> calls.add("last"));
-    TestResourceScope.retain(() -> { calls.add("second"); throw second; });
-    TestResourceScope.retain(() -> { calls.add("first"); throw first; });
+    TestResourceScope.retain(() -> {
+      calls.add("second");
+      throw second;
+    });
+    TestResourceScope.retain(() -> {
+      calls.add("first");
+      throw first;
+    });
     assertThatThrownBy(scope::close).isSameAs(first).hasSuppressedException(second);
     scope.close();
     assertThat(calls).containsExactly("first", "second", "last");
@@ -59,9 +65,13 @@ void scope_cleanup_attempts_every_resource_and_suppresses_errors() throws Except
     List<String> calls = new ArrayList<>();
     AssertionError failure = new AssertionError("cleanup");
     TestResourceScope.retain(() -> calls.add("last"));
-    TestResourceScope.retain(() -> { throw failure; });
+    TestResourceScope.retain(() -> {
+      throw failure;
+    });
     Exception first = new Exception("first");
-    TestResourceScope.retain(() -> { throw first; });
+    TestResourceScope.retain(() -> {
+      throw first;
+    });
 
     assertThatThrownBy(scope::close).isSameAs(first).hasSuppressedException(failure);
     scope.close();
@@ -79,9 +89,15 @@ void scope_cleanup_rethrows_first_error_after_attempting_every_resource() throws
     AssertionError first = new AssertionError("first");
     Exception second = new Exception("second");
     TestResourceScope.retain(() -> calls.add("last"));
-    TestResourceScope.retain(() -> { throw second; });
-    TestResourceScope.retain(() -> { throw first; });
-    TestResourceScope.retain(() -> { throw first; });
+    TestResourceScope.retain(() -> {
+      throw second;
+    });
+    TestResourceScope.retain(() -> {
+      throw first;
+    });
+    TestResourceScope.retain(() -> {
+      throw first;
+    });
 
     assertThatThrownBy(scope::close).isSameAs(first).hasSuppressedException(second);
     scope.close();
@@ -99,7 +115,9 @@ void missing_scope_fails_loudly_and_closes_the_unretained_fixture() {
   @Test
   void missing_scope_preserves_cleanup_exceptions_as_suppressed() {
     Exception failure = new Exception("cleanup failed");
-    assertThatThrownBy(() -> TestResourceScope.retain(() -> { throw failure; }))
+    assertThatThrownBy(() -> TestResourceScope.retain(() -> {
+      throw failure;
+    }))
       .isInstanceOf(IllegalStateException.class)
       .hasMessageContaining("@TestResourceScope.Scoped")
       .hasSuppressedException(failure);
@@ -108,7 +126,9 @@ void missing_scope_preserves_cleanup_exceptions_as_suppressed() {
   @Test
   void missing_scope_propagates_cleanup_errors() {
     AssertionError failure = new AssertionError("cleanup failed");
-    assertThatThrownBy(() -> TestResourceScope.retain(() -> { throw failure; }))
+    assertThatThrownBy(() -> TestResourceScope.retain(() -> {
+      throw failure;
+    }))
       .isSameAs(failure);
   }
 }