From 4909a65d8cecde7fd5c2d10956760e0c719a6e11 Mon Sep 17 00:00:00 2001
From: Avgustin Marinov <Avgustin.Marinov@bosch.com>
Date: Wed, 22 Jan 2025 15:20:57 +0200
Subject: [PATCH] Improve autowiring for tests (Sonar) (#2217)

Signed-off-by: Avgustin Marinov <Avgustin.Marinov@bosch.com>
---
 .../BusProtoStuffMessageConverterTest.java    |   5 +-
 .../AbstractDsAssignmentStrategy.java         |   7 +-
 .../event/remote/AbstractRemoteEventTest.java |   1 +
 .../jpa/AbstractJpaIntegrationTest.java       |  19 +-
 .../autoassign/AutoAssignCheckerIntTest.java  |   1 +
 .../autocleanup/AutoActionCleanupTest.java    |   1 +
 .../autocleanup/AutoCleanupSchedulerTest.java |   1 +
 .../jpa/event/RepositoryEntityEventTest.java  |   1 +
 .../ConfirmationManagementTest.java           |   1 -
 .../management/DeploymentManagementTest.java  |   4 +-
 .../rsql/RSQLParserValidationOracleTest.java  |   2 +
 .../repository/test/TestConfiguration.java    |  35 ++-
 .../test/util/AbstractIntegrationTest.java    |   3 +
 .../repository/test/util/TestdataFactory.java | 266 ++++++++----------
 14 files changed, 169 insertions(+), 178 deletions(-)

diff --git a/hawkbit-repository/hawkbit-repository-core/src/test/java/org/eclipse/hawkbit/event/BusProtoStuffMessageConverterTest.java b/hawkbit-repository/hawkbit-repository-core/src/test/java/org/eclipse/hawkbit/event/BusProtoStuffMessageConverterTest.java
index b0cba6c255..73573f99a0 100644
--- a/hawkbit-repository/hawkbit-repository-core/src/test/java/org/eclipse/hawkbit/event/BusProtoStuffMessageConverterTest.java
+++ b/hawkbit-repository/hawkbit-repository-core/src/test/java/org/eclipse/hawkbit/event/BusProtoStuffMessageConverterTest.java
@@ -57,8 +57,9 @@ void successfullySerializeAndDeserializeEvent() {
         // deserialize
         when(messageMock.getPayload()).thenReturn(serializedEvent);
         final Object deserializedEvent = underTest.convertFromInternal(messageMock, RemoteApplicationEvent.class, null);
-        assertThat(deserializedEvent).isInstanceOf(TargetCreatedEvent.class);
-        assertThat(deserializedEvent).isEqualTo(targetCreatedEvent);
+        assertThat(deserializedEvent)
+                .isInstanceOf(TargetCreatedEvent.class)
+                .isEqualTo(targetCreatedEvent);
     }
 
     @Test
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/main/java/org/eclipse/hawkbit/repository/jpa/management/AbstractDsAssignmentStrategy.java b/hawkbit-repository/hawkbit-repository-jpa/src/main/java/org/eclipse/hawkbit/repository/jpa/management/AbstractDsAssignmentStrategy.java
index 9099bb8f3a..ed51640262 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/main/java/org/eclipse/hawkbit/repository/jpa/management/AbstractDsAssignmentStrategy.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/main/java/org/eclipse/hawkbit/repository/jpa/management/AbstractDsAssignmentStrategy.java
@@ -26,6 +26,7 @@
 import org.eclipse.hawkbit.repository.event.remote.entity.TargetUpdatedEvent;
 import org.eclipse.hawkbit.repository.jpa.configuration.Constants;
 import org.eclipse.hawkbit.repository.jpa.executor.AfterTransactionCommitExecutor;
+import org.eclipse.hawkbit.repository.jpa.model.AbstractJpaBaseEntity_;
 import org.eclipse.hawkbit.repository.jpa.model.JpaAction;
 import org.eclipse.hawkbit.repository.jpa.model.JpaActionStatus;
 import org.eclipse.hawkbit.repository.jpa.model.JpaAction_;
@@ -60,12 +61,14 @@ public abstract class AbstractDsAssignmentStrategy {
     protected final AfterTransactionCommitExecutor afterCommit;
     protected final EventPublisherHolder eventPublisherHolder;
     protected final ActionRepository actionRepository;
+
     private final ActionStatusRepository actionStatusRepository;
     private final QuotaManagement quotaManagement;
     private final BooleanSupplier multiAssignmentsConfig;
     private final BooleanSupplier confirmationFlowConfig;
     private final RepositoryProperties repositoryProperties;
 
+    @SuppressWarnings("java:S107")
     AbstractDsAssignmentStrategy(
             final TargetRepository targetRepository,
             final AfterTransactionCommitExecutor afterCommit, final EventPublisherHolder eventPublisherHolder,
@@ -149,7 +152,7 @@ protected List<Long> overrideObsoleteUpdateActions(final Collection<Long> target
                     cb.equal(root.get(JpaAction_.active), true),
                     cb.equal(root.get(JpaAction_.distributionSet).get(JpaDistributionSet_.requiredMigrationStep), false),
                     cb.notEqual(root.get(JpaAction_.status), Action.Status.CANCELING),
-                    root.get(JpaAction_.target).get(JpaTarget_.id).in(targetsIds)
+                    root.get(JpaAction_.target).get(AbstractJpaBaseEntity_.id).in(targetsIds)
             );
         });
 
@@ -187,7 +190,7 @@ protected List<Long> closeObsoleteUpdateActions(final Collection<Long> targetsId
             return cb.and(
                     cb.equal(root.get(JpaAction_.active), true),
                     cb.equal(root.get(JpaAction_.distributionSet).get(JpaDistributionSet_.requiredMigrationStep), false),
-                    root.get(JpaAction_.target).get(JpaTarget_.id).in(targetsIds)
+                    root.get(JpaAction_.target).get(AbstractJpaBaseEntity_.id).in(targetsIds)
             );
         });
 
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/event/remote/AbstractRemoteEventTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/event/remote/AbstractRemoteEventTest.java
index 84c652c868..d8d5f14bf0 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/event/remote/AbstractRemoteEventTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/event/remote/AbstractRemoteEventTest.java
@@ -37,6 +37,7 @@
 /**
  * Test the remote entity events.
  */
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
  public abstract class AbstractRemoteEventTest extends AbstractJpaIntegrationTest {
 
     @Autowired
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/AbstractJpaIntegrationTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/AbstractJpaIntegrationTest.java
index 072e63af4a..ddaa6e6d59 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/AbstractJpaIntegrationTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/AbstractJpaIntegrationTest.java
@@ -78,6 +78,7 @@
 @ContextConfiguration(classes = { RepositoryApplicationConfiguration.class, TestConfiguration.class })
 @Import(TestChannelBinderConfiguration.class)
 @TestPropertySource(locations = "classpath:/jpa-test.properties")
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
 public abstract class AbstractJpaIntegrationTest extends AbstractIntegrationTest {
 
     protected static final String INVALID_TEXT_HTML = "</noscript><br><script>";
@@ -91,58 +92,40 @@ public abstract class AbstractJpaIntegrationTest extends AbstractIntegrationTest
 
     @Autowired
     protected TargetRepository targetRepository;
-
     @Autowired
     protected ActionRepository actionRepository;
-
     @Autowired
     protected DistributionSetRepository distributionSetRepository;
-
     @Autowired
     protected SoftwareModuleRepository softwareModuleRepository;
-
     @Autowired
     protected TenantMetaDataRepository tenantMetaDataRepository;
-
     @Autowired
     protected DistributionSetTypeRepository distributionSetTypeRepository;
-
     @Autowired
     protected SoftwareModuleTypeRepository softwareModuleTypeRepository;
-
     @Autowired
     protected TargetTagRepository targetTagRepository;
-
     @Autowired
     protected TargetTypeRepository targetTypeRepository;
-
     @Autowired
     protected DistributionSetTagRepository distributionSetTagRepository;
-
     @Autowired
     protected SoftwareModuleMetadataRepository softwareModuleMetadataRepository;
-
     @Autowired
     protected ActionStatusRepository actionStatusRepository;
-
     @Autowired
     protected LocalArtifactRepository artifactRepository;
-
     @Autowired
     protected RolloutGroupRepository rolloutGroupRepository;
-
     @Autowired
     protected RolloutTargetGroupRepository rolloutTargetGroupRepository;
-
     @Autowired
     protected RolloutRepository rolloutRepository;
-
     @Autowired
     protected TenantConfigurationProperties tenantConfigurationProperties;
-
     @Autowired
     protected RolloutTestApprovalStrategy approvalStrategy;
-
     @Autowired
     private JpaProperties jpaProperties;
 
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autoassign/AutoAssignCheckerIntTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autoassign/AutoAssignCheckerIntTest.java
index fb7e941670..4596e15711 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autoassign/AutoAssignCheckerIntTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autoassign/AutoAssignCheckerIntTest.java
@@ -52,6 +52,7 @@
  */
 @Feature("Component Tests - Repository")
 @Story("Auto assign checker")
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
 class AutoAssignCheckerIntTest extends AbstractJpaIntegrationTest {
 
     private static final String SPACE_AND_DESCRIPTION = " description";
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoActionCleanupTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoActionCleanupTest.java
index 8f5148a95e..f05b9004a7 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoActionCleanupTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoActionCleanupTest.java
@@ -33,6 +33,7 @@
  */
 @Feature("Component Tests - Repository")
 @Story("Action cleanup handler")
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
 class AutoActionCleanupTest extends AbstractJpaIntegrationTest {
 
     @Autowired
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoCleanupSchedulerTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoCleanupSchedulerTest.java
index d9f59f2b14..1f3e3ddeff 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoCleanupSchedulerTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/autocleanup/AutoCleanupSchedulerTest.java
@@ -28,6 +28,7 @@
  */
 @Feature("Component Tests - Repository")
 @Story("Auto cleanup scheduler")
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
 class AutoCleanupSchedulerTest extends AbstractJpaIntegrationTest {
 
     private final AtomicInteger counter = new AtomicInteger();
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/event/RepositoryEntityEventTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/event/RepositoryEntityEventTest.java
index c0d84637da..6da8078574 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/event/RepositoryEntityEventTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/event/RepositoryEntityEventTest.java
@@ -50,6 +50,7 @@
 @Feature("Component Tests - Repository")
 @Story("Entity Events")
 @SpringBootTest(classes = { RepositoryTestConfiguration.class }, webEnvironment = SpringBootTest.WebEnvironment.NONE)
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
 public class RepositoryEntityEventTest extends AbstractJpaIntegrationTest {
 
     @Autowired
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/ConfirmationManagementTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/ConfirmationManagementTest.java
index bbcbd90e33..2a04956118 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/ConfirmationManagementTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/ConfirmationManagementTest.java
@@ -16,7 +16,6 @@
 import java.util.Arrays;
 import java.util.List;
 import java.util.Objects;
-import java.util.stream.Collectors;
 import java.util.stream.Stream;
 
 import io.qameta.allure.Description;
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/DeploymentManagementTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/DeploymentManagementTest.java
index a1aae6e834..7cebd9e070 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/DeploymentManagementTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/management/DeploymentManagementTest.java
@@ -1283,7 +1283,7 @@ void deleteDistributionSet() {
                         .asList(DistributionSetSpecification.isDeleted(true), DistributionSetSpecification.isCompleted(true))),
                 PAGE).getContent()).as("wrong size of founded ds").hasSize(noOfDistributionSets);
 
-        for (final DistributionSet ds : deploymentResult.getDistributionSets()) {
+        for (final DistributionSet ignored : deploymentResult.getDistributionSets()) {
             testdataFactory.sendUpdateActionStatusToTargets(deploymentResult.getDeployedTargets(), Status.FINISHED,
                     Collections.singletonList("blabla alles gut"));
         }
@@ -1316,7 +1316,7 @@ void deletesTargetsAndVerifyCascadeDeletes() {
         final DeploymentResult deploymentResult = prepareComplexRepo(undeployedTargetPrefix, noOfUndeployedTargets,
                 deployedTargetPrefix, noOfDeployedTargets, noOfDistributionSets, "myTestDS");
 
-        for (final DistributionSet ds : deploymentResult.getDistributionSets()) {
+        for (final DistributionSet ignored : deploymentResult.getDistributionSets()) {
             testdataFactory.sendUpdateActionStatusToTargets(deploymentResult.getDeployedTargets(), Status.FINISHED,
                     Collections.singletonList("blabla alles gut"));
         }
diff --git a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/rsql/RSQLParserValidationOracleTest.java b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/rsql/RSQLParserValidationOracleTest.java
index 33bbb52b8a..48fa846ff8 100644
--- a/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/rsql/RSQLParserValidationOracleTest.java
+++ b/hawkbit-repository/hawkbit-repository-jpa/src/test/java/org/eclipse/hawkbit/repository/jpa/rsql/RSQLParserValidationOracleTest.java
@@ -27,6 +27,7 @@
 
 @Feature("Component Tests - Repository")
 @Story("RSQL filter suggestion")
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
 public class RSQLParserValidationOracleTest extends AbstractJpaIntegrationTest {
 
     private static final String[] OP_SUGGESTIONS = new String[] { "==", "!=", "=ge=", "=le=", "=gt=", "=lt=", "=in=",
@@ -35,6 +36,7 @@ public class RSQLParserValidationOracleTest extends AbstractJpaIntegrationTest {
             .map(field -> field.name().toLowerCase()).toArray(String[]::new);
     private static final String[] AND_OR_SUGGESTIONS = new String[] { "and", "or" };
     private static final String[] NAME_VERSION_SUGGESTIONS = new String[] { "name", "version" };
+
     @Autowired
     private RsqlValidationOracle rsqlValidationOracle;
 
diff --git a/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/TestConfiguration.java b/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/TestConfiguration.java
index 0a1230d21b..9770714208 100644
--- a/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/TestConfiguration.java
+++ b/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/TestConfiguration.java
@@ -25,8 +25,25 @@
 import org.eclipse.hawkbit.cache.TenantAwareCacheManager;
 import org.eclipse.hawkbit.event.BusProtoStuffMessageConverter;
 import org.eclipse.hawkbit.im.authentication.SpRole;
+import org.eclipse.hawkbit.repository.ArtifactManagement;
+import org.eclipse.hawkbit.repository.ControllerManagement;
+import org.eclipse.hawkbit.repository.DeploymentManagement;
+import org.eclipse.hawkbit.repository.DistributionSetInvalidationManagement;
+import org.eclipse.hawkbit.repository.DistributionSetManagement;
+import org.eclipse.hawkbit.repository.DistributionSetTagManagement;
+import org.eclipse.hawkbit.repository.DistributionSetTypeManagement;
+import org.eclipse.hawkbit.repository.EntityFactory;
+import org.eclipse.hawkbit.repository.QuotaManagement;
 import org.eclipse.hawkbit.repository.RolloutApprovalStrategy;
+import org.eclipse.hawkbit.repository.RolloutHandler;
+import org.eclipse.hawkbit.repository.RolloutManagement;
 import org.eclipse.hawkbit.repository.RolloutStatusCache;
+import org.eclipse.hawkbit.repository.SoftwareModuleManagement;
+import org.eclipse.hawkbit.repository.SoftwareModuleTypeManagement;
+import org.eclipse.hawkbit.repository.TargetFilterQueryManagement;
+import org.eclipse.hawkbit.repository.TargetManagement;
+import org.eclipse.hawkbit.repository.TargetTagManagement;
+import org.eclipse.hawkbit.repository.TargetTypeManagement;
 import org.eclipse.hawkbit.repository.event.ApplicationEventFilter;
 import org.eclipse.hawkbit.repository.model.helper.EventPublisherHolder;
 import org.eclipse.hawkbit.repository.rsql.VirtualPropertyReplacer;
@@ -141,8 +158,22 @@ SystemManagementHolder systemManagementHolder() {
     }
 
     @Bean
-    TestdataFactory testdataFactory() {
-        return new TestdataFactory();
+    TestdataFactory testdataFactory(
+            final ControllerManagement controllerManagement, final ArtifactManagement artifactManagement,
+            final SoftwareModuleManagement softwareModuleManagement, final SoftwareModuleTypeManagement softwareModuleTypeManagement,
+            final DistributionSetManagement distributionSetManagement,
+            final DistributionSetInvalidationManagement distributionSetInvalidationManagement,
+            final DistributionSetTypeManagement distributionSetTypeManagement,
+            final TargetManagement targetManagement, final TargetFilterQueryManagement targetFilterQueryManagement,
+            final TargetTypeManagement targetTypeManagement, final TargetTagManagement targetTagManagement,
+            final DeploymentManagement deploymentManagement, final DistributionSetTagManagement distributionSetTagManagement,
+            final RolloutManagement rolloutManagement, final RolloutHandler rolloutHandler,
+            final QuotaManagement quotaManagement,
+            final EntityFactory entityFactory) {
+        return new TestdataFactory(controllerManagement, artifactManagement, softwareModuleManagement, softwareModuleTypeManagement,
+                distributionSetManagement, distributionSetInvalidationManagement, distributionSetTypeManagement, targetManagement,
+                targetFilterQueryManagement, targetTypeManagement, targetTagManagement, deploymentManagement,
+                distributionSetTagManagement, rolloutManagement, rolloutHandler, quotaManagement, entityFactory);
     }
 
     @Bean
diff --git a/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/AbstractIntegrationTest.java b/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/AbstractIntegrationTest.java
index 80d18f73c4..bdb1049af1 100644
--- a/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/AbstractIntegrationTest.java
+++ b/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/AbstractIntegrationTest.java
@@ -107,6 +107,7 @@
         listeners = { EventVerifier.class, CleanupTestExecutionListener.class },
         mergeMode = MergeMode.MERGE_WITH_DEFAULTS)
 @TestPropertySource(properties = "spring.main.allow-bean-definition-overriding=true")
+@SuppressWarnings("java:S6813") // constructor injects are not possible for test classes
 public abstract class AbstractIntegrationTest {
 
     protected static final Pageable PAGE = PageRequest.of(0, 500, Sort.by(Direction.ASC, "id"));
@@ -120,6 +121,7 @@ public abstract class AbstractIntegrationTest {
      * {@link SystemManagement#getTenantMetadata()};
      */
     protected static final int DEFAULT_DS_TYPES = RepositoryConstants.DEFAULT_DS_TYPES_IN_TENANT + 1;
+
     @Autowired
     protected EntityFactory entityFactory;
     @Autowired
@@ -478,6 +480,7 @@ protected void waitNextMillis() {
         }
     }
 
+    @SuppressWarnings("java:S4042")
     private static File createTempDir() {
         try {
             final File file = Files.createTempDirectory(System.currentTimeMillis() + "_").toFile();
diff --git a/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/TestdataFactory.java b/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/TestdataFactory.java
index e456b360f3..75c94bb55c 100644
--- a/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/TestdataFactory.java
+++ b/hawkbit-repository/hawkbit-repository-test/src/main/java/org/eclipse/hawkbit/repository/test/util/TestdataFactory.java
@@ -78,7 +78,6 @@
 import org.eclipse.hawkbit.repository.model.TargetTag;
 import org.eclipse.hawkbit.repository.model.TargetType;
 import org.eclipse.hawkbit.repository.model.TargetUpdateStatus;
-import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.data.domain.PageRequest;
 import org.springframework.data.domain.Pageable;
 import org.springframework.util.ObjectUtils;
@@ -139,56 +138,54 @@ public class TestdataFactory {
 
     private static final String SPACE_AND_DESCRIPTION = " description";
 
-    @Autowired
-    private ControllerManagement controllerManagament;
-
-    @Autowired
-    private SoftwareModuleManagement softwareModuleManagement;
-
-    @Autowired
-    private SoftwareModuleTypeManagement softwareModuleTypeManagement;
-
-    @Autowired
-    private DistributionSetManagement distributionSetManagement;
-
-    @Autowired
-    private DistributionSetInvalidationManagement distributionSetInvalidationManagement;
-
-    @Autowired
-    private DistributionSetTypeManagement distributionSetTypeManagement;
-
-    @Autowired
-    private TargetManagement targetManagement;
-
-    @Autowired
-    private TargetFilterQueryManagement targetFilterQueryManagement;
-
-    @Autowired
-    private TargetTypeManagement targetTypeManagement;
-
-    @Autowired
-    private DeploymentManagement deploymentManagement;
-
-    @Autowired
-    private TargetTagManagement targetTagManagement;
-
-    @Autowired
-    private DistributionSetTagManagement distributionSetTagManagement;
-
-    @Autowired
-    private EntityFactory entityFactory;
-
-    @Autowired
-    private ArtifactManagement artifactManagement;
-
-    @Autowired
-    private RolloutManagement rolloutManagement;
-
-    @Autowired
-    private RolloutHandler rolloutHandler;
-
-    @Autowired
-    private QuotaManagement quotaManagement;
+    private final ControllerManagement controllerManagement;
+    private final ArtifactManagement artifactManagement;
+    private final SoftwareModuleManagement softwareModuleManagement;
+    private final SoftwareModuleTypeManagement softwareModuleTypeManagement;
+    private final DistributionSetManagement distributionSetManagement;
+    private final DistributionSetInvalidationManagement distributionSetInvalidationManagement;
+    private final DistributionSetTypeManagement distributionSetTypeManagement;
+    private final TargetManagement targetManagement;
+    private final TargetFilterQueryManagement targetFilterQueryManagement;
+    private final TargetTypeManagement targetTypeManagement;
+    private final TargetTagManagement targetTagManagement;
+    private final DeploymentManagement deploymentManagement;
+    private final DistributionSetTagManagement distributionSetTagManagement;
+    private final RolloutManagement rolloutManagement;
+    private final RolloutHandler rolloutHandler;
+    private final QuotaManagement quotaManagement;
+    private final EntityFactory entityFactory;
+
+    public TestdataFactory(
+            final ControllerManagement controllerManagement, final ArtifactManagement artifactManagement,
+            final SoftwareModuleManagement softwareModuleManagement, final SoftwareModuleTypeManagement softwareModuleTypeManagement,
+            final DistributionSetManagement distributionSetManagement,
+            final DistributionSetInvalidationManagement distributionSetInvalidationManagement,
+            final DistributionSetTypeManagement distributionSetTypeManagement,
+            final TargetManagement targetManagement, final TargetFilterQueryManagement targetFilterQueryManagement,
+            final TargetTypeManagement targetTypeManagement, final TargetTagManagement targetTagManagement,
+            final DeploymentManagement deploymentManagement, final DistributionSetTagManagement distributionSetTagManagement,
+            final RolloutManagement rolloutManagement, final RolloutHandler rolloutHandler,
+            final QuotaManagement quotaManagement,
+            final EntityFactory entityFactory) {
+        this.controllerManagement = controllerManagement;
+        this.softwareModuleManagement = softwareModuleManagement;
+        this.softwareModuleTypeManagement = softwareModuleTypeManagement;
+        this.distributionSetManagement = distributionSetManagement;
+        this.distributionSetInvalidationManagement = distributionSetInvalidationManagement;
+        this.distributionSetTypeManagement = distributionSetTypeManagement;
+        this.targetManagement = targetManagement;
+        this.targetFilterQueryManagement = targetFilterQueryManagement;
+        this.targetTypeManagement = targetTypeManagement;
+        this.targetTagManagement = targetTagManagement;
+        this.deploymentManagement = deploymentManagement;
+        this.distributionSetTagManagement = distributionSetTagManagement;
+        this.entityFactory = entityFactory;
+        this.artifactManagement = artifactManagement;
+        this.rolloutManagement = rolloutManagement;
+        this.rolloutHandler = rolloutHandler;
+        this.quotaManagement = quotaManagement;
+    }
 
     public static String randomString(final int len) {
         return RANDOM_STRING_UTILS.next(len, true, false);
@@ -200,12 +197,10 @@ public static byte[] randomBytes(final int len) {
 
     public Action performAssignment(final DistributionSet distributionSet) {
         final Target target = createTarget(randomString(5));
-        final DeploymentRequest deploymentRequest = new DeploymentRequest(target.getControllerId(),
-                distributionSet.getId(), ActionType.FORCED, 0, null, null, null, null, false);
+        final DeploymentRequest deploymentRequest = new DeploymentRequest(
+                target.getControllerId(), distributionSet.getId(), ActionType.FORCED, 0, null, null, null, null, false);
         deploymentManagement.assignDistributionSets(Collections.singletonList(deploymentRequest));
-
-        return deploymentManagement.findActionsByTarget(target.getControllerId(), Pageable.unpaged()).getContent()
-                .get(0);
+        return deploymentManagement.findActionsByTarget(target.getControllerId(), Pageable.unpaged()).getContent().get(0);
     }
 
     /**
@@ -214,8 +209,7 @@ public Action performAssignment(final DistributionSet distributionSet) {
      * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} and
      * {@link DistributionSet#isRequiredMigrationStep()} <code>false</code>.
      *
-     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name,
-     *         vendor and description.
+     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name, vendor and description.
      * @return {@link DistributionSet} entity.
      */
     public DistributionSet createDistributionSet(final String prefix) {
@@ -254,8 +248,7 @@ public DistributionSet createDistributionSet(final Collection<SoftwareModule> mo
      * {@link DistributionSet#isRequiredMigrationStep()} <code>false</code>.
      *
      * @param modules of {@link DistributionSet#getModules()}
-     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name,
-     *         vendor and description.
+     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name, vendor and description.
      * @return {@link DistributionSet} entity.
      */
     public DistributionSet createDistributionSet(final Collection<SoftwareModule> modules, final String prefix) {
@@ -267,8 +260,7 @@ public DistributionSet createDistributionSet(final Collection<SoftwareModule> mo
      * {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
      * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION}.
      *
-     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name,
-     *         vendor and description.
+     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name, vendor and description.
      * @param isRequiredMigrationStep for {@link DistributionSet#isRequiredMigrationStep()}
      * @return {@link DistributionSet} entity.
      */
@@ -282,8 +274,7 @@ public DistributionSet createDistributionSet(final String prefix, final boolean
      * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} and
      * {@link DistributionSet#isRequiredMigrationStep()} <code>false</code>.
      *
-     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name,
-     *         vendor and description.
+     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name, vendor and description.
      * @param tags DistributionSet tags
      * @return {@link DistributionSet} entity.
      */
@@ -292,14 +283,11 @@ public DistributionSet createDistributionSet(final String prefix, final Collecti
     }
 
     /**
-     * Creates {@link DistributionSet} in repository including three
-     * {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
+     * Creates {@link DistributionSet} in repository including three {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
      * {@link #SM_TYPE_APP}.
      *
-     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name,
-     *         vendor and description.
-     * @param version {@link DistributionSet#getVersion()} and
-     *         {@link SoftwareModule#getVersion()} extended by a random number.
+     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name, vendor and description.
+     * @param version {@link DistributionSet#getVersion()} and {@link SoftwareModule#getVersion()} extended by a random number.
      * @param isRequiredMigrationStep for {@link DistributionSet#isRequiredMigrationStep()}
      * @return {@link DistributionSet} entity.
      */
@@ -352,9 +340,8 @@ public void addSoftwareModuleMetadata(final DistributionSet set) {
      * @param modules for {@link DistributionSet#getModules()}
      * @return {@link DistributionSet} entity.
      */
-    public DistributionSet createDistributionSet(final String prefix, final String version,
-            final boolean isRequiredMigrationStep, final Collection<SoftwareModule> modules) {
-
+    public DistributionSet createDistributionSet(
+            final String prefix, final String version, final boolean isRequiredMigrationStep, final Collection<SoftwareModule> modules) {
         return distributionSetManagement.create(
                 entityFactory.distributionSet().create().name(prefix != null && !prefix.isEmpty() ? prefix : "DS")
                         .version(version).description(randomDescriptionShort()).type(findOrCreateDefaultTestDsType())
@@ -363,15 +350,11 @@ public DistributionSet createDistributionSet(final String prefix, final String v
     }
 
     /**
-     * Creates {@link DistributionSet} in repository including three
-     * {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
+     * Creates {@link DistributionSet} in repository including three {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
      * {@link #SM_TYPE_APP}.
      *
-     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name,
-     *         vendor and description.
-     * @param version {@link DistributionSet#getVersion()} and
-     *         {@link SoftwareModule#getVersion()} extended by a random
-     *         number.updat
+     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name, vendor and description.
+     * @param version {@link DistributionSet#getVersion()} and {@link SoftwareModule#getVersion()} extended by a random number.updat
      * @param tags DistributionSet tags
      * @return {@link DistributionSet} entity.
      */
@@ -382,17 +365,14 @@ public DistributionSet createDistributionSet(final String prefix, final String v
     }
 
     /**
-     * Creates {@link DistributionSet}s in repository including three
-     * {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
-     * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} followed by an iterative
-     * number and {@link DistributionSet#isRequiredMigrationStep()}
+     * Creates {@link DistributionSet}s in repository including three {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
+     * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} followed by an iterative number and {@link DistributionSet#isRequiredMigrationStep()}
      * <code>false</code>.
      *
      * @param number of {@link DistributionSet}s to create
      * @return {@link List} of {@link DistributionSet} entities
      */
     public List<DistributionSet> createDistributionSets(final int number) {
-
         return createDistributionSets("", number);
     }
 
@@ -403,57 +383,53 @@ public List<DistributionSet> createDistributionSets(final int number) {
      * @return {@link List} of {@link DistributionSet} entities
      */
     public List<DistributionSet> createDistributionSetsWithoutModules(final int number) {
-
         final List<DistributionSet> sets = new ArrayList<>(number);
         for (int i = 0; i < number; i++) {
             sets.add(distributionSetManagement
-                    .create(entityFactory.distributionSet().create().name("DS" + i).version(DEFAULT_VERSION + "." + i)
-                            .description(randomDescriptionShort()).type(findOrCreateDefaultTestDsType())));
+                    .create(entityFactory.distributionSet().create()
+                            .name("DS" + i)
+                            .version(DEFAULT_VERSION + "." + i)
+                            .description(randomDescriptionShort())
+                            .type(findOrCreateDefaultTestDsType())));
         }
-
         return sets;
     }
 
     /**
-     * Creates {@link DistributionSet}s in repository including three
-     * {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
-     * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} followed by an iterative
-     * number and {@link DistributionSet#isRequiredMigrationStep()}
+     * Creates {@link DistributionSet}s in repository including three {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
+     * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} followed by an iterative number and {@link DistributionSet#isRequiredMigrationStep()}
      * <code>false</code>.
      *
-     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name,
-     *         vendor and description.
+     * @param prefix for {@link SoftwareModule}s and {@link DistributionSet}s name, vendor and description.
      * @param number of {@link DistributionSet}s to create
      * @return {@link List} of {@link DistributionSet} entities
      */
     public List<DistributionSet> createDistributionSets(final String prefix, final int number) {
-
         final List<DistributionSet> sets = new ArrayList<>();
         for (int i = 0; i < number; i++) {
             sets.add(createDistributionSet(prefix, DEFAULT_VERSION + "." + i, false));
         }
-
         return sets;
     }
 
     /**
-     * Creates {@link DistributionSet}s in repository with
-     * {@link #DEFAULT_DESCRIPTION} and
-     * {@link DistributionSet#isRequiredMigrationStep()} <code>false</code>.
+     * Creates {@link DistributionSet}s in repository with {@link #DEFAULT_DESCRIPTION} and {@link DistributionSet#isRequiredMigrationStep()}
+     * <code>false</code>.
      *
      * @param name {@link DistributionSet#getName()}
      * @param version {@link DistributionSet#getVersion()}
      * @return {@link DistributionSet} entity
      */
     public DistributionSet createDistributionSetWithNoSoftwareModules(final String name, final String version) {
-
-        return distributionSetManagement.create(entityFactory.distributionSet().create().name(name).version(version)
-                .description(DEFAULT_DESCRIPTION).type(findOrCreateDefaultTestDsType()));
+        return distributionSetManagement.create(entityFactory.distributionSet().create()
+                .name(name)
+                .version(version)
+                .description(DEFAULT_DESCRIPTION)
+                .type(findOrCreateDefaultTestDsType()));
     }
 
     /**
-     * Creates {@link Artifact}s for given {@link SoftwareModule} with a small text
-     * payload.
+     * Creates {@link Artifact}s for given {@link SoftwareModule} with a small text payload.
      *
      * @param moduleId the {@link Artifact}s belong to.
      * @return {@link Artifact} entity.
@@ -464,45 +440,39 @@ public List<Artifact> createArtifacts(final Long moduleId) {
             final String artifactData = "some test data" + i;
             artifacts.add(createArtifact(artifactData, moduleId, "filename" + i));
         }
-
         return artifacts;
     }
 
     /**
-     * Create an {@link Artifact} for given {@link SoftwareModule} with a small text
-     * payload.
+     * Create an {@link Artifact} for given {@link SoftwareModule} with a small text payload.
      *
-     * @param artifactData the {@link Artifact} Inputstream
+     * @param artifactData the {@link Artifact} InputStream
      * @param moduleId the {@link Artifact} belongs to
      * @param filename that was provided during upload.
      * @return {@link Artifact} entity.
      */
     public Artifact createArtifact(final String artifactData, final Long moduleId, final String filename) {
         final InputStream stubInputStream = IOUtils.toInputStream(artifactData, StandardCharsets.UTF_8);
-        return artifactManagement
-                .create(new ArtifactUpload(stubInputStream, moduleId, filename, false, artifactData.length()));
+        return artifactManagement.create(new ArtifactUpload(stubInputStream, moduleId, filename, false, artifactData.length()));
     }
 
     /**
-     * Create an {@link Artifact} for given {@link SoftwareModule} with a small text
-     * payload.
+     * Create an {@link Artifact} for given {@link SoftwareModule} with a small text payload.
      *
-     * @param artifactData the {@link Artifact} Inputstream
+     * @param artifactData the {@link Artifact} InputStream
      * @param moduleId the {@link Artifact} belongs to
      * @param filename that was provided during upload.
      * @param fileSize the file size
      * @return {@link Artifact} entity.
      */
-    public Artifact createArtifact(final byte[] artifactData, final Long moduleId, final String filename,
-            final int fileSize) {
+    public Artifact createArtifact(final byte[] artifactData, final Long moduleId, final String filename, final int fileSize) {
         return artifactManagement.create(
                 new ArtifactUpload(new ByteArrayInputStream(artifactData), moduleId, filename, false, fileSize));
     }
 
     /**
-     * Creates {@link SoftwareModule} with {@link #DEFAULT_VENDOR} and
-     * {@link #DEFAULT_VERSION} and random generated {@link Target#getDescription()}
-     * in the repository.
+     * Creates {@link SoftwareModule} with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION} and random generated
+     * {@link Target#getDescription()} in the repository.
      *
      * @param typeKey of the {@link SoftwareModuleType}
      * @return persisted {@link SoftwareModule}.
@@ -512,9 +482,8 @@ public SoftwareModule createSoftwareModule(final String typeKey) {
     }
 
     /**
-     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_APP_KEY}
-     * with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION} and random
-     * generated {@link Target#getDescription()} in the repository.
+     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_APP_KEY} with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION}
+     * and random generated {@link Target#getDescription()} in the repository.
      *
      * @return persisted {@link SoftwareModule}.
      */
@@ -523,9 +492,8 @@ public SoftwareModule createSoftwareModuleApp() {
     }
 
     /**
-     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_APP_KEY}
-     * with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION} and random
-     * generated {@link Target#getDescription()} in the repository.
+     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_APP_KEY} with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION}
+     * and random generated {@link Target#getDescription()} in the repository.
      *
      * @param prefix added to name and version
      * @return persisted {@link SoftwareModule}.
@@ -535,9 +503,8 @@ public SoftwareModule createSoftwareModuleApp(final String prefix) {
     }
 
     /**
-     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_OS_KEY}
-     * with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION} and random
-     * generated {@link Target#getDescription()} in the repository.
+     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_OS_KEY} with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION}
+     * and random generated {@link Target#getDescription()} in the repository.
      *
      * @return persisted {@link SoftwareModule}.
      */
@@ -546,9 +513,8 @@ public SoftwareModule createSoftwareModuleOs() {
     }
 
     /**
-     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_OS_KEY}
-     * with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION} and random
-     * generated {@link Target#getDescription()} in the repository.
+     * Creates {@link SoftwareModule} of type {@value Constants#SMT_DEFAULT_OS_KEY} with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION}
+     * and random generated {@link Target#getDescription()} in the repository.
      *
      * @param prefix added to name and version
      * @return persisted {@link SoftwareModule}.
@@ -558,9 +524,8 @@ public SoftwareModule createSoftwareModuleOs(final String prefix) {
     }
 
     /**
-     * Creates {@link SoftwareModule} with {@link #DEFAULT_VENDOR} and
-     * {@link #DEFAULT_VERSION} and random generated {@link Target#getDescription()}
-     * in the repository.
+     * Creates {@link SoftwareModule} with {@link #DEFAULT_VENDOR} and {@link #DEFAULT_VERSION} and random generated
+     * {@link Target#getDescription()} in the repository.
      *
      * @param typeKey of the {@link SoftwareModuleType}
      * @param prefix added to name and version
@@ -568,8 +533,12 @@ public SoftwareModule createSoftwareModuleOs(final String prefix) {
      */
     public SoftwareModule createSoftwareModule(final String typeKey, final String prefix, final boolean encrypted) {
         return softwareModuleManagement.create(entityFactory.softwareModule().create()
-                .type(findOrCreateSoftwareModuleType(typeKey)).name(prefix + typeKey).version(prefix + DEFAULT_VERSION)
-                .description(randomDescriptionShort()).vendor(DEFAULT_VENDOR).encrypted(encrypted));
+                .type(findOrCreateSoftwareModuleType(typeKey))
+                .name(prefix + typeKey)
+                .version(prefix + DEFAULT_VERSION)
+                .description(randomDescriptionShort())
+                .vendor(DEFAULT_VENDOR)
+                .encrypted(encrypted));
     }
 
     /**
@@ -593,15 +562,14 @@ public Target createTarget(final String controllerId) {
      * @return persisted {@link Target}
      */
     public Target createTarget(final String controllerId, final String targetName) {
-        final Target target = targetManagement
-                .create(entityFactory.target().create().controllerId(controllerId).name(targetName));
+        final Target target = targetManagement.create(entityFactory.target().create().controllerId(controllerId).name(targetName));
         assertTargetProperlyCreated(target);
         return target;
     }
 
     public Target createTarget(final String controllerId, final String targetName, final String address) {
-        final Target target = targetManagement
-                .create(entityFactory.target().create().controllerId(controllerId).name(targetName).address(address));
+        final Target target = targetManagement.create(
+                entityFactory.target().create().controllerId(controllerId).name(targetName).address(address));
         assertTargetProperlyCreated(target);
         return target;
     }
@@ -620,15 +588,12 @@ public Target createTarget(final String controllerId, final String targetName, f
     }
 
     /**
-     * Creates {@link DistributionSet}s in repository including three
-     * {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
-     * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} followed by an iterative
-     * number and {@link DistributionSet#isRequiredMigrationStep()}
+     * Creates {@link DistributionSet}s in repository including three {@link SoftwareModule}s of types {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT} ,
+     * {@link #SM_TYPE_APP} with {@link #DEFAULT_VERSION} followed by an iterative number and {@link DistributionSet#isRequiredMigrationStep()}
      * <code>false</code>.
      *
-     * In addition it updates the created {@link DistributionSet}s and
-     * {@link SoftwareModule}s to ensure that {@link BaseEntity#getLastModifiedAt()}
-     * and {@link BaseEntity#getLastModifiedBy()} is filled.
+     * In addition, it updates the created {@link DistributionSet}s and {@link SoftwareModule}s to ensure that
+     * {@link BaseEntity#getLastModifiedAt()} and {@link BaseEntity#getLastModifiedBy()} is filled.
      *
      * @return persisted {@link DistributionSet}.
      */
@@ -645,9 +610,8 @@ public DistributionSet createUpdatedDistributionSet() {
     }
 
     /**
-     * @return {@link DistributionSetType} with key {@link #DS_TYPE_DEFAULT} and
-     *         {@link SoftwareModuleType}s {@link #SM_TYPE_OS}, {@link #SM_TYPE_RT}
-     *         , {@link #SM_TYPE_APP}.
+     * @return {@link DistributionSetType} with key {@link #DS_TYPE_DEFAULT} and {@link SoftwareModuleType}s {@link #SM_TYPE_OS},
+     *         {@link #SM_TYPE_RT} , {@link #SM_TYPE_APP}.
      */
     public DistributionSetType findOrCreateDefaultTestDsType() {
         final List<SoftwareModuleType> mand = new ArrayList<>();
@@ -1316,7 +1280,7 @@ private List<Target> createTargets(final Collection<TargetCreate> targetCreates)
     private Action sendUpdateActionStatusToTarget(final Status status, final Action updActA,
             final Collection<String> msgs) {
 
-        return controllerManagament.addUpdateActionStatus(
+        return controllerManagement.addUpdateActionStatus(
                 entityFactory.actionStatus().create(updActA.getId()).status(status).messages(msgs));
     }