Closes Taskana#2459: Fix Code Smells

.github/workflows/continuous-integration.yml

@@ -44,6 +44,8 @@ jobs:
         with:
           distribution: adopt
           java-version: ${{ env.JAVA_VERSION }}
+      - name: Print Java version
+        run: java -version
       - name: Cache maven dependencies
         id: cache
         uses: actions/cache@v3
@@ -53,6 +55,8 @@ jobs:
           restore-keys: ${{ runner.os }}-${{ env.CACHE_MAVEN_NAME }}
       - name: Change versions to match tag
         run: ci/change_version.sh -m .
+      - name: Print Compilation Command
+        run: mvn clean compile -X
       - name: Compile & build
         run: ./mvnw -B install -DskipTests -Dasciidoctor.skip -Djacoco.skip
       - name: Populate cache

common/taskana-common-data/src/main/java/pro/taskana/sampledata/SampleDataGenerator.java

@@ -13,7 +13,6 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.function.Consumer;
-import java.util.stream.Collectors;
 import java.util.stream.Stream;
 import javax.sql.DataSource;
 import org.apache.ibatis.jdbc.RuntimeSqlException;
@@ -78,9 +77,7 @@ public void dropDb() {
   private List<String> parseScripts(Stream<String> scripts) {
     try (Connection connection = dataSource.getConnection()) {
       DB db = DB.getDB(connection);
-      return scripts
-          .map(script -> SqlReplacer.getScriptAsSql(db, now, script))
-          .collect(Collectors.toList());
+      return scripts.map(script -> SqlReplacer.getScriptAsSql(db, now, script)).toList();
     } catch (SQLException e) {
       throw new RuntimeSqlException("Connection to database failed.", e);
     }

common/taskana-common-security/src/main/java/pro/taskana/common/internal/security/CurrentUserContextImpl.java

@@ -10,7 +10,6 @@
 import java.util.List;
 import java.util.Objects;
 import java.util.Set;
-import java.util.stream.Collectors;
 import javax.security.auth.Subject;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -61,7 +60,7 @@ public List<String> getGroupIds() {
           .map(Principal::getName)
           .filter(Objects::nonNull)
           .map(this::convertAccessId)
-          .collect(Collectors.toList());
+          .toList();
     }
     LOGGER.trace("No groupIds found in subject!");
     return Collections.emptyList();

common/taskana-common-test/src/main/java/pro/taskana/common/test/security/JaasExtension.java

@@ -107,17 +107,17 @@ public <T> T interceptTestFactoryMethod(
 
       Iterable<DynamicNode> newChildrenForDynamicContainer;
       // TestFactory must have one of the following return types. See link above for further details
-      if (factoryResult instanceof DynamicNode) {
-        newChildrenForDynamicContainer = Collections.singleton((DynamicNode) factoryResult);
+      if (factoryResult instanceof DynamicNode dynamicNode) {
+        newChildrenForDynamicContainer = Collections.singleton(dynamicNode);
       } else if (factoryResult instanceof Stream) {
         Stream<DynamicNode> nodes = (Stream<DynamicNode>) factoryResult;
-        newChildrenForDynamicContainer = nodes.collect(Collectors.toList());
+        newChildrenForDynamicContainer = nodes.toList();
       } else if (factoryResult instanceof Iterable) {
         newChildrenForDynamicContainer = (Iterable<DynamicNode>) factoryResult;
       } else if (factoryResult instanceof Iterator) {
         newChildrenForDynamicContainer = () -> (Iterator<DynamicNode>) factoryResult;
-      } else if (factoryResult instanceof DynamicNode[]) {
-        newChildrenForDynamicContainer = Arrays.asList((DynamicNode[]) factoryResult);
+      } else if (factoryResult instanceof DynamicNode[] dynamicNodes) {
+        newChildrenForDynamicContainer = Arrays.asList(dynamicNodes);
       } else {
         throw new SystemException(
             String.format(
@@ -142,8 +142,11 @@ public <T> T interceptTestFactoryMethod(
       Store store = getMethodLevelStore(extensionContext);
       return (T)
           Stream.of(annotation.value())
-              .peek(a -> store.put(ACCESS_IDS_STORE_KEY, a))
-              .map(wrapTestsInDynamicContainer);
+              .map(
+                  a -> {
+                    store.put(ACCESS_IDS_STORE_KEY, a);
+                    return wrapTestsInDynamicContainer.apply(a);
+                  });
     }
 
     return extractAccessIdAndPerformInvocation(invocation, invocationContext.getExecutable());
@@ -207,8 +210,11 @@ public Stream<TestTemplateInvocationContext> provideTestTemplateInvocationContex
         AnnotationSupport.findRepeatableAnnotations(context.getElement(), WithAccessId.class);
     Store store = getMethodLevelStore(context);
     return accessIds.stream()
-        .peek(a -> store.put(ACCESS_IDS_STORE_KEY, a))
-        .map(JaasExtensionInvocationContext::new);
+        .map(
+            a -> {
+              store.put(ACCESS_IDS_STORE_KEY, a);
+              return new JaasExtensionInvocationContext(a);
+            });
   }
 
   // endregion
@@ -217,8 +223,7 @@ private static void persistDynamicContainerChildren(
       Iterable<DynamicNode> nodes, Map<String, List<DynamicNode>> childrenMap) {
     nodes.forEach(
         node -> {
-          if (node instanceof DynamicContainer) {
-            DynamicContainer container = (DynamicContainer) node;
+          if (node instanceof DynamicContainer container) {
             List<DynamicNode> children = container.getChildren().collect(Collectors.toList());
             childrenMap.put(container.hashCode() + container.getDisplayName(), children);
             persistDynamicContainerChildren(children, childrenMap);
@@ -228,8 +233,7 @@ private static void persistDynamicContainerChildren(
 
   private static DynamicNode duplicateDynamicNode(
       DynamicNode node, Map<String, List<DynamicNode>> lookupMap) {
-    if (node instanceof DynamicContainer) {
-      DynamicContainer container = (DynamicContainer) node;
+    if (node instanceof DynamicContainer container) {
       Stream<DynamicNode> children =
           lookupMap.get(node.hashCode() + node.getDisplayName()).stream()
               .map(x -> duplicateDynamicNode(x, lookupMap));
@@ -259,7 +263,7 @@ private static List<Principal> getPrincipals(WithAccessId withAccessId) {
       return Stream.concat(
               Stream.of(withAccessId.user()).map(UserPrincipal::new),
               Arrays.stream(withAccessId.groups()).map(GroupPrincipal::new))
-          .collect(Collectors.toList());
+          .toList();
     }
     return Collections.emptyList();
   }

common/taskana-common/src/main/java/pro/taskana/common/internal/configuration/DbSchemaCreator.java

@@ -64,9 +64,9 @@ public boolean run() throws SQLException {
     }
     if (LOGGER.isDebugEnabled()) {
       LOGGER.debug(outWriter.toString());
-    }
-    if (!errorWriter.toString().trim().isEmpty()) {
-      LOGGER.error(errorWriter.toString());
+      if (!errorWriter.toString().trim().isEmpty()) {
+        LOGGER.error(errorWriter.toString());
+      }
     }
     return false;
   }

common/taskana-common/src/main/java/pro/taskana/common/internal/configuration/parser/SimpleParser.java

@@ -8,7 +8,6 @@
 import java.util.Optional;
 import java.util.function.Function;
 import java.util.regex.Pattern;
-import java.util.stream.Collectors;
 import pro.taskana.common.api.exceptions.SystemException;
 import pro.taskana.common.internal.util.ReflectionUtil;
 
@@ -43,6 +42,6 @@ protected static List<String> splitStringAndTrimElements(String str, String sepa
     return Arrays.stream(str.split(Pattern.quote(separator)))
         .filter(not(String::isEmpty))
         .map(String::trim)
-        .collect(Collectors.toList());
+        .toList();
   }
 }

mvnw

@@ -306,3 +306,4 @@ exec "$JAVACMD" \
   -classpath "$MAVEN_PROJECTBASEDIR/.mvn/wrapper/maven-wrapper.jar" \
   "-Dmaven.multiModuleProjectDirectory=${MAVEN_PROJECTBASEDIR}" \
   ${WRAPPER_LAUNCHER} $MAVEN_CONFIG "$@"
+echo "JAVA_HOME is set to: $JAVA_HOME"