Migrate sdc-workflow-designer to Junit5 25/139325/3
authorFiete Ostkamp <Fiete.Ostkamp@telekom.de>
Tue, 5 Nov 2024 07:29:42 +0000 (08:29 +0100)
committerFiete Ostkamp <Fiete.Ostkamp@telekom.de>
Tue, 5 Nov 2024 08:09:30 +0000 (09:09 +0100)
Issue-ID: SDC-4694
Change-Id: Iaae2e171e82cbfaeaa7c66aa441b0d5543b08f99
Signed-off-by: Fiete Ostkamp <Fiete.Ostkamp@telekom.de>
26 files changed:
docs/requirements-docs.txt
docs/tox.ini
pom.xml
sdc-workflow-designer-be/pom.xml
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/ArtifactAssociationHandlerTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/ExceptionsHandlerTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/WorkflowControllerTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/WorkflowVersionControllerTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/swagger/UserIdReaderTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/types/PagingTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/types/SortingTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/types/VersionStatesFormatterTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/validation/ActivitySpecParameterNameValidatorTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/validation/ArchivingStatusValidatorTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/api/validation/NoDuplicatesValidatorTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/persistence/impl/ActivitySpecRepositoryImplTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/persistence/impl/ArtifactRepositoryTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/persistence/impl/ParameterRepositoryTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/server/resolvers/UserIdResolverTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/services/UniqueValueServiceTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/services/impl/ActivitySpecManagerImplTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/services/impl/WorkflowManagerImplTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/services/impl/WorkflowVersionManagerImplTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/services/impl/mappers/VersionMapperTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/services/impl/mappers/VersionStateMapperTest.java
sdc-workflow-designer-be/src/test/java/org/onap/sdc/workflow/services/impl/mappers/WorkflowMapperTest.java

index 71df2ab..981874f 100644 (file)
@@ -1,3 +1,4 @@
+six
 sphinx>=4.2.0  # BSD
 sphinx-rtd-theme>=1.0.0  # MIT
 sphinxcontrib-blockdiag  # BSD
index b9d7eb4..46075fa 100644 (file)
@@ -7,7 +7,7 @@ skipsdist = true
 basepython = python3.8
 deps =
     -r{toxinidir}/requirements-docs.txt
-    -chttps://raw.githubusercontent.com/openstack/requirements/stable/yoga/upper-constraints.txt
+    -chttps://releases.openstack.org/constraints/upper/yoga
     -chttps://git.onap.org/doc/plain/etc/upper-constraints.onap.txt?h=master
 commands =
     sphinx-build -W -q -b html -n -d {envtmpdir}/doctrees {toxinidir} {toxinidir}/_build/html
@@ -16,7 +16,7 @@ commands =
 basepython = python3.8
 deps =
     -r{toxinidir}/requirements-docs.txt
-    -chttps://raw.githubusercontent.com/openstack/requirements/stable/yoga/upper-constraints.txt
+    -chttps://releases.openstack.org/constraints/upper/yoga
     -chttps://git.onap.org/doc/plain/etc/upper-constraints.onap.txt?h=master
 commands =
     sphinx-build -W -q -b linkcheck -d {envtmpdir}/doctrees {toxinidir} {toxinidir}/_build/linkcheck
@@ -25,7 +25,7 @@ commands =
 basepython = python3.8
 deps =
     -r{toxinidir}/requirements-docs.txt
-    -chttps://raw.githubusercontent.com/openstack/requirements/stable/yoga/upper-constraints.txt
+    -chttps://releases.openstack.org/constraints/upper/yoga
     -chttps://git.onap.org/doc/plain/etc/upper-constraints.onap.txt?h=master
 commands =
     sphinx-build -W -q -b spelling -d {envtmpdir}/doctrees {toxinidir} {toxinidir}/_build/spellcheck
diff --git a/pom.xml b/pom.xml
index 2b20628..1eaa9c9 100644 (file)
--- a/pom.xml
+++ b/pom.xml
@@ -80,7 +80,7 @@
                 <plugin>
                     <groupId>org.apache.maven.plugins</groupId>
                     <artifactId>maven-surefire-plugin</artifactId>
-                    <version>2.22.2</version>
+                    <version>3.1.2</version>
                 </plugin>
                 <plugin>
                     <groupId>org.jacoco</groupId>
index b55553f..f5dce3d 100644 (file)
                 </exclusion>
             </exclusions>
         </dependency>
-        <dependency>
-            <groupId>org.springframework</groupId>
-            <artifactId>spring-test</artifactId>
-            <version>${org.springframework.version}</version>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.springframework</groupId>
             <artifactId>spring-jcl</artifactId>
                     <groupId>org.springframework</groupId>
                     <artifactId>spring-core</artifactId>
                 </exclusion>
-                <exclusion>
-                    <groupId>org.springframework</groupId>
-                    <artifactId>spring-test</artifactId>
-                </exclusion>
+        <exclusion>
+          <groupId>org.junit.vintage</groupId>
+          <artifactId>junit-vintage-engine</artifactId>
+        </exclusion>
             </exclusions>
         </dependency>
         <dependency>
                 </exclusion>
             </exclusions>
         </dependency>
-
-        <dependency>
-            <groupId>org.junit.vintage</groupId>
-            <artifactId>junit-vintage-engine</artifactId>
-            <version>${junitJupiter.version}</version>
-            <scope>test</scope>
-        </dependency>
         <dependency>
             <groupId>org.junit.platform</groupId>
             <artifactId>junit-platform-commons</artifactId>
             <plugin>
                 <groupId>org.apache.maven.plugins</groupId>
                 <artifactId>maven-surefire-plugin</artifactId>
-                <version>2.22.2</version>
-                <dependencies>
-                    <dependency>
-                        <groupId>org.apache.maven.surefire</groupId>
-                        <artifactId>surefire-junit-platform</artifactId>
-                        <version>2.22.2</version>
-                    </dependency>
-                </dependencies>
+                <version>3.1.2</version>
             </plugin>
         </plugins>
     </build>
index b4d1180..131e14b 100644 (file)
@@ -16,7 +16,7 @@
 
 package org.onap.sdc.workflow.api;
 
-import static junit.framework.TestCase.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.when;
@@ -24,9 +24,8 @@ import static org.mockito.Mockito.when;
 import java.io.IOException;
 import java.io.InputStream;
 import org.apache.commons.io.IOUtils;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 import org.mockito.Mockito;
 import org.onap.sdc.workflow.api.types.dto.ArtifactDeliveriesRequestDto;
 import org.onap.sdc.workflow.persistence.types.ArtifactEntity;
@@ -40,14 +39,12 @@ import org.springframework.http.HttpMethod;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
 import org.springframework.stereotype.Component;
-import org.springframework.test.context.ContextConfiguration;
 import org.springframework.test.context.TestPropertySource;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringJUnitConfig;
 import org.springframework.web.client.RestTemplate;
 
 
-@RunWith(SpringJUnit4ClassRunner.class)
-@ContextConfiguration(
+@SpringJUnitConfig(
         classes = {ArtifactAssociationHandlerTest.RestBuilderMockProvider.class, ArtifactAssociationService.class})
 @TestPropertySource(locations = "classpath:application-test.properties")
 @Component("ArtifactAssociationHandlerTest")
@@ -92,7 +89,7 @@ public class ArtifactAssociationHandlerTest {
     @Autowired
     private ArtifactAssociationService associationService;
 
-    @Before
+    @BeforeEach
     public void setUp() throws IOException {
         inputStreamMock = IOUtils.toInputStream("some test data for my input stream", "UTF-8");
         artifactMock = new ArtifactEntity(FILE_NAME, inputStreamMock);
index ee92400..71fe684 100644 (file)
 
 package org.onap.sdc.workflow.api;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertNotNull;
 import static org.springframework.http.HttpStatus.INTERNAL_SERVER_ERROR;
 import static org.springframework.http.HttpStatus.NOT_FOUND;
 import static org.springframework.http.HttpStatus.UNPROCESSABLE_ENTITY;
 
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
 import org.onap.sdc.workflow.api.types.ErrorResponse;
 import org.onap.sdc.workflow.api.types.UnexpectedErrorResponse;
@@ -33,9 +33,9 @@ import org.onap.sdc.workflow.services.exceptions.VersionModificationException;
 import org.onap.sdc.workflow.services.exceptions.VersionStateModificationMissingArtifactException;
 import org.onap.sdc.workflow.services.types.WorkflowVersionState;
 import org.springframework.http.ResponseEntity;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class ExceptionsHandlerTest {
 
     @InjectMocks
index 5e0f61d..2368d68 100644 (file)
@@ -17,7 +17,7 @@ package org.onap.sdc.workflow.api;
 
 import static org.hamcrest.Matchers.is;
 import static org.hamcrest.collection.IsCollectionWithSize.hasSize;
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.verify;
@@ -39,14 +39,14 @@ import com.amdocs.zusammen.datatypes.item.Item;
 import java.util.Arrays;
 import java.util.Collections;
 import java.util.List;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.ArgumentCaptor;
 import org.mockito.Captor;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
 import org.onap.sdc.workflow.RestPath;
 import org.onap.sdc.workflow.server.resolvers.UserIdResolver;
 import org.onap.sdc.workflow.services.WorkflowManager;
@@ -62,7 +62,7 @@ import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.ResultActions;
 import org.springframework.test.web.servlet.setup.MockMvcBuilders;
 
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
 public class WorkflowControllerTest {
 
     private static final String USER_ID = "userId";
@@ -78,7 +78,7 @@ public class WorkflowControllerTest {
     @InjectMocks
     private WorkflowController workflowController;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         mockMvc = MockMvcBuilders.standaloneSetup(workflowController)
                                  .setCustomArgumentResolvers(new PageableHandlerMethodArgumentResolver())
index a8813c0..732ae40 100644 (file)
@@ -18,7 +18,7 @@ package org.onap.sdc.workflow.api;
 
 import static org.hamcrest.Matchers.hasSize;
 import static org.hamcrest.Matchers.is;
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.verify;
 import static org.springframework.http.MediaType.APPLICATION_JSON;
@@ -31,12 +31,12 @@ import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.
 
 import java.util.Arrays;
 import java.util.Collection;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
 import org.onap.sdc.workflow.RestPath;
 import org.onap.sdc.workflow.api.mappers.WorkflowVersionDtoMapper;
 import org.onap.sdc.workflow.api.types.Parameter;
@@ -51,7 +51,7 @@ import org.springframework.mock.web.MockHttpServletResponse;
 import org.springframework.test.web.servlet.MockMvc;
 import org.springframework.test.web.servlet.setup.MockMvcBuilders;
 
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
 public class WorkflowVersionControllerTest {
 
     private static final String USER_ID = "cs0008";
@@ -68,7 +68,7 @@ public class WorkflowVersionControllerTest {
     @InjectMocks
     private WorkflowVersionController workflowVersionController;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         mockMvc = MockMvcBuilders.standaloneSetup(workflowVersionController).build();
     }
index b82cb43..73cfe40 100644 (file)
@@ -24,18 +24,18 @@ import static org.mockito.Mockito.when;
 
 import com.fasterxml.classmate.TypeResolver;
 import java.util.Optional;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import java.lang.annotation.Annotation;
-import org.junit.Test;
-import org.junit.runner.RunWith;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.onap.sdc.workflow.services.annotations.UserId;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 import springfox.documentation.builders.ParameterBuilder;
 import springfox.documentation.service.ResolvedMethodParameter;
 import springfox.documentation.spi.service.contexts.ParameterContext;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class UserIdReaderTest {
 
     private static final UserId USER_ID_ANNOTATION = new UserId() {
index a0392eb..9c29c8e 100644 (file)
 
 package org.onap.sdc.workflow.api.types;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNull;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.onap.sdc.workflow.services.types.PagingConstants.MAX_LIMIT;
 
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class PagingTest {
 
     @InjectMocks
index 9cd95a5..47570ac 100644 (file)
 
 package org.onap.sdc.workflow.api.types;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import java.util.Arrays;
 import java.util.Collections;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
 import org.onap.sdc.workflow.services.types.Sort;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class SortingTest {
 
     @InjectMocks
index 294577f..bd11fb4 100644 (file)
 
 package org.onap.sdc.workflow.api.types;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertNull;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.onap.sdc.workflow.services.types.WorkflowVersionState.CERTIFIED;
 import static org.onap.sdc.workflow.services.types.WorkflowVersionState.DRAFT;
 
 import java.util.Collections;
 import java.util.stream.Collectors;
 import java.util.stream.Stream;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class VersionStatesFormatterTest {
 
     @InjectMocks
index 75fb5a2..0e3a37f 100644 (file)
 
 package org.onap.sdc.workflow.api.validation;
 
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
 import javax.validation.ConstraintValidatorContext;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
 import org.onap.sdc.workflow.persistence.types.ActivitySpecParameter;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class ActivitySpecParameterNameValidatorTest {
 
     class AnnotationWrapper {
@@ -53,7 +53,7 @@ public class ActivitySpecParameterNameValidatorTest {
     private ActivitySpecParameterNameValidator validator;
 
 
-    @Before
+    @BeforeEach
     public void setup() throws NoSuchFieldException {
         MockitoAnnotations.initMocks(this);
         when(context.buildConstraintViolationWithTemplate(anyString())).thenReturn(constraintViolationBuilder);
index d5e5f20..6d45164 100644 (file)
 
 package org.onap.sdc.workflow.api.validation;
 
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.anyString;
-import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
 
 import javax.validation.ConstraintValidatorContext;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class ArchivingStatusValidatorTest {
 
     class AnnotationWrapper {
@@ -48,7 +47,7 @@ public class ArchivingStatusValidatorTest {
 
     private ArchivingStatusValidator validator;
 
-    @Before
+    @BeforeEach
     public void setup() throws NoSuchFieldException {
         MockitoAnnotations.initMocks(this);
         when(context.buildConstraintViolationWithTemplate(anyString())).thenReturn(constraintViolationBuilder);
index f8f6b74..e000455 100644 (file)
@@ -16,8 +16,8 @@
 
 package org.onap.sdc.workflow.api.validation;
 
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.when;
 
@@ -26,8 +26,8 @@ import java.util.Arrays;
 import java.util.Collection;
 import java.util.Collections;
 import javax.validation.ConstraintValidatorContext;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
 import org.onap.sdc.workflow.api.types.Parameter;
@@ -50,7 +50,7 @@ public class NoDuplicatesValidatorTest {
     private ConstraintValidatorContext.ConstraintViolationBuilder.NodeBuilderCustomizableContext
             nodeBuilderCustomizableContext;
 
-    @Before
+    @BeforeEach
     public void init() throws NoSuchFieldException {
         MockitoAnnotations.initMocks(this);
         when(context.buildConstraintViolationWithTemplate(anyString())).thenReturn(constraintViolationBuilder);
index e69d498..c01c205 100644 (file)
@@ -18,6 +18,7 @@ package org.onap.sdc.workflow.persistence.impl;
 
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.Mockito.doAnswer;
+import static org.mockito.Mockito.lenient;
 import static org.onap.sdc.common.zusammen.services.ZusammenElementUtil.ELEMENT_TYPE_PROPERTY;
 
 import com.amdocs.zusammen.adaptor.inbound.api.types.item.Element;
@@ -35,13 +36,13 @@ import java.util.List;
 import java.util.Map;
 import java.util.Optional;
 import java.util.UUID;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
 import org.onap.sdc.common.versioning.persistence.zusammen.ZusammenSessionContextCreator;
 import org.onap.sdc.common.zusammen.services.ZusammenAdaptor;
 import org.onap.sdc.workflow.persistence.impl.ActivitySpecRepositoryImpl.InfoPropertyName;
@@ -51,7 +52,7 @@ import org.onap.sdc.workflow.persistence.types.ActivitySpecEntity;
 import org.onap.sdc.workflow.persistence.types.ActivitySpecParameter;
 import org.onap.sdc.workflow.services.utilities.JsonUtil;
 
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
 public class ActivitySpecRepositoryImplTest {
 
     private static final String versionId = "1234";
@@ -68,7 +69,7 @@ public class ActivitySpecRepositoryImplTest {
     private final Map<String, Element> elementMap = new HashMap<>();
     private String elementId;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         daoImpl = new ActivitySpecRepositoryImpl(zusammenAdaptor, contextCreator);
         entity = new ActivitySpecEntity();
@@ -100,14 +101,14 @@ public class ActivitySpecRepositoryImplTest {
         Optional<ElementInfo> testElementInfo = zusammenAdaptor
                                                         .getElementInfoByName(contextCreator.create(), elementContext,
                                                                 Id.ZERO, ActivitySpecElementType.ACTIVITYSPEC.name());
-        Assert.assertTrue(testElementInfo.isPresent());
-        Assert.assertEquals(testElementInfo.get().getInfo().getName(), ActivitySpecElementType.ACTIVITYSPEC.name());
-        Assert.assertEquals(testElementInfo.get().getInfo()
+        Assertions.assertTrue(testElementInfo.isPresent());
+        Assertions.assertEquals(testElementInfo.get().getInfo().getName(), ActivitySpecElementType.ACTIVITYSPEC.name());
+        Assertions.assertEquals(testElementInfo.get().getInfo()
                                     .getProperty(ActivitySpecRepositoryImpl.InfoPropertyName.DESCRIPTION.getValue()),
                 entity.getDescription());
-        Assert.assertEquals(testElementInfo.get().getInfo().getProperty(InfoPropertyName.CATEGORY.getValue()),
+        Assertions.assertEquals(testElementInfo.get().getInfo().getProperty(InfoPropertyName.CATEGORY.getValue()),
                 entity.getCategoryList());
-        Assert.assertEquals(testElementInfo.get().getInfo()
+        Assertions.assertEquals(testElementInfo.get().getInfo()
                                     .getProperty(ActivitySpecRepositoryImpl.InfoPropertyName.NAME.getValue()),
                 entity.getName());
 
@@ -115,15 +116,15 @@ public class ActivitySpecRepositoryImplTest {
                 zusammenAdaptor.getElement(contextCreator.create(), elementContext, new Id(elementId));
         final InputStream data = testElement.get().getData();
         final ActivitySpecData activitySpecData = JsonUtil.json2Object(data, ActivitySpecData.class);
-        Assert.assertEquals(activitySpecData.getInputs().get(0).getName(), entity.getInputs().get(0).getName());
+        Assertions.assertEquals(activitySpecData.getInputs().get(0).getName(), entity.getInputs().get(0).getName());
     }
 
     @Test
     public void testGet() {
         final ActivitySpecEntity retrieved = daoImpl.get(entity);
-        Assert.assertEquals(retrieved.getName(), entity.getName());
-        Assert.assertEquals(retrieved.getDescription(), entity.getDescription());
-        Assert.assertEquals(retrieved.getCategoryList(), entity.getCategoryList());
+        Assertions.assertEquals(retrieved.getName(), entity.getName());
+        Assertions.assertEquals(retrieved.getDescription(), entity.getDescription());
+        Assertions.assertEquals(retrieved.getCategoryList(), entity.getCategoryList());
     }
 
     @Test
@@ -131,20 +132,20 @@ public class ActivitySpecRepositoryImplTest {
         entity.setDescription("Update AS version1");
         daoImpl.update(entity);
         final ActivitySpecEntity retrieved = daoImpl.get(entity);
-        Assert.assertEquals(retrieved.getName(), entity.getName());
-        Assert.assertEquals(retrieved.getDescription(), entity.getDescription());
-        Assert.assertEquals(retrieved.getCategoryList(), entity.getCategoryList());
+        Assertions.assertEquals(retrieved.getName(), entity.getName());
+        Assertions.assertEquals(retrieved.getDescription(), entity.getDescription());
+        Assertions.assertEquals(retrieved.getCategoryList(), entity.getCategoryList());
     }
 
 
     private void mockZusammenAdapter() {
 
-        doAnswer(invocationOnMock -> {
+        lenient().doAnswer(invocationOnMock -> {
             Id elementId = invocationOnMock.getArgument(2);
             return Optional.of(elementMap.get(elementId.getValue()));
         }).when(zusammenAdaptor).getElement(any(), any(), any());
 
-        doAnswer(invocationOnMock -> {
+        lenient().doAnswer(invocationOnMock -> {
             ZusammenElement element = new ZusammenElement();
             Info info = new Info();
             element.setElementId(Id.ZERO);
@@ -155,7 +156,7 @@ public class ActivitySpecRepositoryImplTest {
             return Optional.of(element);
         }).when(zusammenAdaptor).getElementByName(any(), any(), any(), any());
 
-        doAnswer(invocationOnMock -> {
+        lenient().doAnswer(invocationOnMock -> {
             String elementName = invocationOnMock.getArgument(3);
             return elementMap.values().stream()
                            .filter(element -> elementName.equals(element.getInfo().getProperty(ELEMENT_TYPE_PROPERTY)))
@@ -167,7 +168,7 @@ public class ActivitySpecRepositoryImplTest {
                            }).findAny();
         }).when(zusammenAdaptor).getElementInfoByName(any(), any(), any(), any());
 
-        doAnswer(invocationOnMock -> {
+        lenient().doAnswer(invocationOnMock -> {
             ZusammenElement element = invocationOnMock.getArgument(2);
             if (element.getAction().equals(Action.CREATE) || element.getAction().equals(Action.UPDATE)) {
                 element.setElementId(new Id(UUID.randomUUID().toString()));
index 96f74d5..374ad14 100644 (file)
@@ -16,9 +16,9 @@
 
 package org.onap.sdc.workflow.persistence.impl;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.ArgumentMatchers.isNull;
@@ -38,18 +38,18 @@ import java.io.IOException;
 import java.io.InputStream;
 import java.util.Optional;
 import org.apache.commons.io.IOUtils;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
 import org.onap.sdc.common.versioning.persistence.zusammen.ZusammenSessionContextCreator;
 import org.onap.sdc.common.zusammen.services.ZusammenAdaptor;
 import org.onap.sdc.workflow.persistence.impl.types.WorkflowElementType;
 import org.onap.sdc.workflow.persistence.types.ArtifactEntity;
 
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
 public class ArtifactRepositoryTest {
 
     private static final String FILE_NAME_PROPERTY = "fileName";
@@ -66,7 +66,7 @@ public class ArtifactRepositoryTest {
     @InjectMocks
     private ArtifactRepositoryImpl artifactRepository;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         doReturn(SESSION_CONTEXT).when(contextCreatorMock).create();
     }
index 4112ab4..c2d897c 100644 (file)
 
 package org.onap.sdc.workflow.persistence.impl;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.isNull;
-import static org.mockito.Matchers.any;
-import static org.mockito.Matchers.eq;
+import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.doReturn;
 import static org.mockito.Mockito.verify;
 
@@ -33,14 +34,14 @@ import com.amdocs.zusammen.datatypes.item.Info;
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Optional;
-import org.junit.Before;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mapstruct.ap.internal.util.Collections;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.Spy;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
 import org.onap.sdc.common.versioning.persistence.zusammen.ZusammenSessionContextCreator;
 import org.onap.sdc.common.zusammen.services.ZusammenAdaptor;
 import org.onap.sdc.workflow.persistence.impl.types.ParameterPropertyName;
@@ -49,7 +50,7 @@ import org.onap.sdc.workflow.persistence.types.ParameterEntity;
 import org.onap.sdc.workflow.persistence.types.ParameterRole;
 import org.onap.sdc.workflow.persistence.types.ParameterType;
 
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
 public class ParameterRepositoryTest {
 
     private static final String ITEM1_ID = "item_id_1";
@@ -67,7 +68,7 @@ public class ParameterRepositoryTest {
     @InjectMocks
     private ParameterRepositoryImpl parameterRepository;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         doReturn(SESSION_CONTEXT).when(contextCreatorMock).create();
     }
@@ -181,12 +182,14 @@ public class ParameterRepositoryTest {
                         eq("Delete all INPUT"));
     }
 
-    @Test(expected = IllegalStateException.class)
+    @Test
     public void shouldFailIfParentElementDoesNotExist() {
-        doReturn(Optional.empty()).when(zusammenAdaptorMock)
-                                 .getElementInfoByName(eq(SESSION_CONTEXT), any(ElementContext.class), isNull(),
-                                         eq(WorkflowElementType.INPUTS.name()));
-        parameterRepository.deleteAll(ITEM1_ID, VERSION1_ID, ParameterRole.INPUT);
+        assertThrows(IllegalStateException.class, () -> {
+            doReturn(Optional.empty()).when(zusammenAdaptorMock)
+                    .getElementInfoByName(eq(SESSION_CONTEXT), any(ElementContext.class), isNull(),
+                            eq(WorkflowElementType.INPUTS.name()));
+            parameterRepository.deleteAll(ITEM1_ID, VERSION1_ID, ParameterRole.INPUT);
+        });
     }
 
     @Test
index 464af46..5fcb11e 100644 (file)
 
 package org.onap.sdc.workflow.server.resolvers;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.when;
 import static org.onap.sdc.workflow.api.RestParams.USER_ID_HEADER;
 
 import javax.servlet.http.HttpServletRequest;
-import org.junit.Test;
+import org.junit.jupiter.api.Test;
 import org.onap.sdc.workflow.services.annotations.UserId;
 import org.springframework.core.MethodParameter;
 import org.springframework.web.bind.ServletRequestBindingException;
@@ -53,26 +54,32 @@ public class UserIdResolverTest {
         assertEquals(headerValue, resolved);
     }
 
-    @Test(expected = IllegalStateException.class)
+    @Test
     public void illegalTypeErrorThrownWhenAnnotatedParameterIsNotOfTypeString() {
-        MethodParameter methodParameterMock = mock(MethodParameter.class);
-        when(methodParameterMock.hasParameterAnnotation(UserId.class)).thenReturn(true);
-        //noinspection unchecked
-        when(methodParameterMock.getParameterType()).thenReturn((Class)String[].class);
-        new UserIdResolver().supportsParameter(methodParameterMock);
+        assertThrows(IllegalStateException.class, () -> {
+            MethodParameter methodParameterMock = mock(MethodParameter.class);
+            when(methodParameterMock.hasParameterAnnotation(UserId.class)).thenReturn(true);
+            //noinspection unchecked
+            when(methodParameterMock.getParameterType()).thenReturn((Class) String[].class);
+            new UserIdResolver().supportsParameter(methodParameterMock);
+        });
     }
 
-    @Test(expected = ServletRequestBindingException.class)
-    public void missingHeaderErrorThrownWhenUserIdHeaderNotPopulated() throws ServletRequestBindingException {
-        NativeWebRequest webRequestMock = mock(NativeWebRequest.class);
-        when(webRequestMock.getNativeRequest(HttpServletRequest.class)).thenReturn(mock(HttpServletRequest.class));
-        new UserIdResolver().resolveArgument(null, null, webRequestMock, null);
+    @Test
+    public void missingHeaderErrorThrownWhenUserIdHeaderNotPopulated() {
+        assertThrows(ServletRequestBindingException.class, () -> {
+            NativeWebRequest webRequestMock = mock(NativeWebRequest.class);
+            when(webRequestMock.getNativeRequest(HttpServletRequest.class)).thenReturn(mock(HttpServletRequest.class));
+            new UserIdResolver().resolveArgument(null, null, webRequestMock, null);
+        });
     }
 
-    @Test(expected = NullPointerException.class)
-    public void exceptionThrownWhenRequestTypeIsNotHttpRequest() throws ServletRequestBindingException {
-        NativeWebRequest webRequestMock = mock(NativeWebRequest.class);
-        new UserIdResolver().resolveArgument(null, null, webRequestMock, null);
+    @Test
+    public void exceptionThrownWhenRequestTypeIsNotHttpRequest() {
+        assertThrows(NullPointerException.class, () -> {
+            NativeWebRequest webRequestMock = mock(NativeWebRequest.class);
+            new UserIdResolver().resolveArgument(null, null, webRequestMock, null);
+        });
     }
 
     @Test
index da2d8fa..b099669 100644 (file)
@@ -16,8 +16,9 @@
 
 package org.onap.sdc.workflow.services;
 
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertFalse;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.Mockito.doReturn;
@@ -26,8 +27,8 @@ import static org.mockito.Mockito.times;
 import static org.mockito.Mockito.verify;
 
 import java.util.Optional;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
@@ -48,7 +49,7 @@ public class UniqueValueServiceTest {
     @InjectMocks
     private UniqueValueService uniqueValueService;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         MockitoAnnotations.initMocks(this);
     }
@@ -66,10 +67,12 @@ public class UniqueValueServiceTest {
         verify(uniqueValueRepositoryMock, never()).findById(any(UniqueValueEntity.class));
     }
 
-    @Test(expected = UniqueValueViolationException.class)
+    @Test
     public void shouldThrowExceptionIfValueIsNotUnique() {
-        doReturn(Optional.of("xxx")).when(uniqueValueRepositoryMock).findById(any());
-        uniqueValueService.createUniqueValue(TYPE, DUMMY_COMBINATION);
+        assertThrows(UniqueValueViolationException.class, () -> {
+            doReturn(Optional.of("xxx")).when(uniqueValueRepositoryMock).findById(any());
+            uniqueValueService.createUniqueValue(TYPE, DUMMY_COMBINATION);
+        });
     }
 
     @Test
index 3166e06..55326e8 100644 (file)
@@ -16,6 +16,7 @@
 
 package org.onap.sdc.workflow.services.impl;
 
+import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.ArgumentMatchers.eq;
@@ -30,10 +31,10 @@ import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
-import org.junit.After;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
+import org.junit.jupiter.api.AfterEach;
+import org.junit.jupiter.api.Assertions;
+import org.junit.jupiter.api.BeforeEach;
+import org.junit.jupiter.api.Test;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.Mockito;
@@ -78,12 +79,12 @@ public class ActivitySpecManagerImplTest {
     @Mock
     private ActivitySpecMapper activitySpecMapperMock;
 
-    @Before
+    @BeforeEach
     public void setUp() {
         MockitoAnnotations.initMocks(this);
     }
 
-    @After
+    @AfterEach
     public void tearDown() {
         activitySpecManager = null;
     }
@@ -128,19 +129,19 @@ public class ActivitySpecManagerImplTest {
 
         ActivitySpecEntity activitySpec = activitySpecManager.createActivitySpec(activitySpecToCreate);
 
-        Assert.assertNotNull(activitySpec);
+        Assertions.assertNotNull(activitySpec);
         activitySpec.setId(itemId);
         activitySpec.setStatus(VersionStatus.Draft.name());
         assertActivitySpecEquals(activitySpec, activitySpecToCreate);
     }
 
     private void assertActivitySpecEquals(ActivitySpecEntity actual, ActivitySpecEntity expected) {
-        Assert.assertEquals(actual.getId(), expected.getId());
-        Assert.assertEquals(actual.getName(), expected.getName());
-        Assert.assertEquals(actual.getDescription(), expected.getDescription());
-        Assert.assertEquals(actual.getCategoryList(), expected.getCategoryList());
-        Assert.assertEquals(actual.getInputs(), expected.getInputs());
-        Assert.assertEquals(actual.getOutputs(), expected.getOutputs());
+        Assertions.assertEquals(actual.getId(), expected.getId());
+        Assertions.assertEquals(actual.getName(), expected.getName());
+        Assertions.assertEquals(actual.getDescription(), expected.getDescription());
+        Assertions.assertEquals(actual.getCategoryList(), expected.getCategoryList());
+        Assertions.assertEquals(actual.getInputs(), expected.getInputs());
+        Assertions.assertEquals(actual.getOutputs(), expected.getOutputs());
     }
 
     @Test
@@ -150,20 +151,20 @@ public class ActivitySpecManagerImplTest {
         doReturn(new ActivitySpecEntity(ID, null)).when(activitySpecMapperMock).itemToActivitySpec(item);
 
         final Collection<ActivitySpecEntity> activitySpecs = activitySpecManager.list("Certified");
-        Assert.assertEquals(1, activitySpecs.size());
-        Assert.assertEquals(ID, activitySpecs.iterator().next().getId());
+        Assertions.assertEquals(1, activitySpecs.size());
+        Assertions.assertEquals(ID, activitySpecs.iterator().next().getId());
     }
 
     @Test
     public void testListInvalidFilter() {
         final Collection<ActivitySpecEntity> activitySpecs = activitySpecManager.list("invalid_status");
-        Assert.assertEquals(0, activitySpecs.size());
+        Assertions.assertEquals(0, activitySpecs.size());
     }
 
     @Test
     public void testListNoFilter() {
         final Collection<ActivitySpecEntity> activitySpecs = activitySpecManager.list(null);
-        Assert.assertEquals(0, activitySpecs.size());
+        Assertions.assertEquals(0, activitySpecs.size());
     }
 
     @Test
@@ -177,12 +178,12 @@ public class ActivitySpecManagerImplTest {
         doReturn(createRetrievedVersion(version01,VersionStatus.Draft)).when(versionManagerMock).get(any(), any());
         ActivitySpecEntity retrieved = activitySpecManager.get(input);
         assertActivitySpecEquals(retrieved, input);
-        Assert.assertEquals(retrieved.getStatus(), VersionStatus.Draft.name());
+        Assertions.assertEquals(retrieved.getStatus(), VersionStatus.Draft.name());
 
 
         retrieved = activitySpecManager.get(input);
         assertActivitySpecEquals(retrieved, input);
-        Assert.assertEquals(retrieved.getStatus(), VersionStatus.Draft.name());
+        Assertions.assertEquals(retrieved.getStatus(), VersionStatus.Draft.name());
     }
 
     private void mockListVersions() {
@@ -199,9 +200,9 @@ public class ActivitySpecManagerImplTest {
         Mockito.doThrow(new RuntimeException(TEST_ERROR_MSG)).when(activitySpecRepositoryMock).get(any());
         try {
             activitySpecManager.get(input);
-            Assert.fail();
+            Assertions.fail();
         } catch (EntityNotFoundException exception) {
-            Assert.assertEquals(ACTIVITY_SPEC_NOT_FOUND, exception.getMessage());
+            Assertions.assertEquals(ACTIVITY_SPEC_NOT_FOUND, exception.getMessage());
         }
     }
 
@@ -213,23 +214,27 @@ public class ActivitySpecManagerImplTest {
         Mockito.doThrow(new RuntimeException(TEST_ERROR_MSG)).when(versionManagerMock).list(any());
         try {
             activitySpecManager.get(input);
-            Assert.fail();
+            Assertions.fail();
         } catch (EntityNotFoundException exception) {
-            Assert.assertEquals(ACTIVITY_SPEC_NOT_FOUND, exception.getMessage());
+            Assertions.assertEquals(ACTIVITY_SPEC_NOT_FOUND, exception.getMessage());
         }
     }
 
-    @Test(expected = VersionStatusModificationException.class)
+    @Test
     public void testInvalidDeprecate() {
-        InternalVersion version = createRetrievedVersion(version01, VersionStatus.Draft);
-        doReturn(version).when(versionManagerMock).get(ID, version01);
-        activitySpecManager.actOnAction(new ActivitySpecEntity(ID, version01), ActivitySpecAction.DEPRECATE);
+        assertThrows(VersionStatusModificationException.class, () -> {
+            InternalVersion version = createRetrievedVersion(version01, VersionStatus.Draft);
+            doReturn(version).when(versionManagerMock).get(ID, version01);
+            activitySpecManager.actOnAction(new ActivitySpecEntity(ID, version01), ActivitySpecAction.DEPRECATE);
+        });
     }
 
-    @Test(expected = VersionStatusModificationException.class)
+    @Test
     public void testInvalidDelete() {
-        mockCertifiedVersion();
-        activitySpecManager.actOnAction(new ActivitySpecEntity(ID, version01), ActivitySpecAction.DELETE);
+        assertThrows(VersionStatusModificationException.class, () -> {
+            mockCertifiedVersion();
+            activitySpecManager.actOnAction(new ActivitySpecEntity(ID, version01), ActivitySpecAction.DELETE);
+        });
     }
 
     private void mockCertifiedVersion() {
@@ -237,10 +242,12 @@ public class ActivitySpecManagerImplTest {
         doReturn(version).when(versionManagerMock).get(ID, version01);
     }
 
-    @Test(expected = VersionStatusModificationException.class)
+    @Test
     public void testInvalidCertify() {
-        mockCertifiedVersion();
-        activitySpecManager.actOnAction(new ActivitySpecEntity(ID, version01), ActivitySpecAction.CERTIFY);
+        assertThrows(VersionStatusModificationException.class, () -> {
+            mockCertifiedVersion();
+            activitySpecManager.actOnAction(new ActivitySpecEntity(ID, version01), ActivitySpecAction.CERTIFY);
+        });
     }
 
     @Test
@@ -259,9 +266,9 @@ public class ActivitySpecManagerImplTest {
         mockListVersions();
         try {
             activitySpecManager.actOnAction(new ActivitySpecEntity(ID, version01), ActivitySpecAction.CERTIFY);
-            Assert.fail();
+            Assertions.fail();
         } catch (EntityNotFoundException exception) {
-            Assert.assertEquals(ACTIVITY_SPEC_NOT_FOUND, exception.getMessage());
+            Assertions.assertEquals(ACTIVITY_SPEC_NOT_FOUND, exception.getMessage());
         }
     }
 
index 1db2409..218f443 100644 (file)
@@ -16,8 +16,9 @@
 
 package org.onap.sdc.workflow.services.impl;
 
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.eq;
 import static org.mockito.Mockito.doReturn;
@@ -38,8 +39,8 @@ import java.util.List;
 import java.util.Map;
 import java.util.function.Function;
 import java.util.stream.Collectors;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.onap.sdc.common.versioning.services.ItemManager;
@@ -58,9 +59,9 @@ import org.onap.sdc.workflow.services.types.Sort;
 import org.onap.sdc.workflow.services.types.SortingRequest;
 import org.onap.sdc.workflow.services.types.Workflow;
 import org.onap.sdc.workflow.services.types.WorkflowVersionState;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringExtension;
 
-@RunWith(SpringJUnit4ClassRunner.class)
+@ExtendWith(SpringExtension.class)
 public class WorkflowManagerImplTest {
 
 
@@ -131,13 +132,15 @@ public class WorkflowManagerImplTest {
         assertPaging(workflows.getPaging(), requestSpec.getPaging().getOffset(), requestSpec.getPaging().getLimit(), 2);
     }
 
-    @Test(expected = EntityNotFoundException.class)
+    @Test
     public void shouldThrowExceptionWhenWorkflowDontExist() {
-        Workflow nonExistingWorkflow = new Workflow();
-        nonExistingWorkflow.setId(ITEM1_ID);
-        doReturn(null).when(itemManagerMock).get(ITEM1_ID);
-        workflowManager.get(nonExistingWorkflow);
-        verify(workflowMapperMock, times(3)).fromItem(any(Item.class));
+        assertThrows(EntityNotFoundException.class, () -> {
+            Workflow nonExistingWorkflow = new Workflow();
+            nonExistingWorkflow.setId(ITEM1_ID);
+            doReturn(null).when(itemManagerMock).get(ITEM1_ID);
+            workflowManager.get(nonExistingWorkflow);
+            verify(workflowMapperMock, times(3)).fromItem(any(Item.class));
+        });
     }
 
     @Test
@@ -170,10 +173,12 @@ public class WorkflowManagerImplTest {
 
     }
 
-    @Test(expected = EntityNotFoundException.class)
+    @Test
     public void shouldThrowExceptionWhenWorkflowToUpdateNotFound() {
-        doReturn(null).when(itemManagerMock).get(ITEM1_ID);
-        workflowManager.update(createWorkflow(1, true, ArchivingStatus.ACTIVE));
+        assertThrows(EntityNotFoundException.class, () -> {
+            doReturn(null).when(itemManagerMock).get(ITEM1_ID);
+            workflowManager.update(createWorkflow(1, true, ArchivingStatus.ACTIVE));
+        });
     }
 
     @Test
index 6648f37..35ab8ba 100644 (file)
@@ -16,7 +16,9 @@
 
 package org.onap.sdc.workflow.services.impl;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertThrows;
+import static org.junit.jupiter.api.Assertions.assertTrue;
 import static org.mockito.ArgumentMatchers.any;
 import static org.mockito.ArgumentMatchers.anyString;
 import static org.mockito.ArgumentMatchers.eq;
@@ -39,15 +41,13 @@ import java.util.Collections;
 import java.util.List;
 import java.util.Optional;
 import org.apache.commons.io.IOUtils;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.rules.ExpectedException;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
+import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.ArgumentCaptor;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
 import org.mockito.Spy;
-import org.mockito.junit.MockitoJUnitRunner;
+import org.mockito.junit.jupiter.MockitoExtension;
 import org.onap.sdc.common.versioning.persistence.types.InternalItem;
 import org.onap.sdc.common.versioning.persistence.types.InternalVersion;
 import org.onap.sdc.common.versioning.services.ItemManager;
@@ -74,7 +74,7 @@ import org.onap.sdc.workflow.services.types.WorkflowVersion;
 import org.onap.sdc.workflow.services.types.WorkflowVersionState;
 import org.springframework.mock.web.MockMultipartFile;
 
-@RunWith(MockitoJUnitRunner.class)
+@ExtendWith(MockitoExtension.class)
 public class WorkflowVersionManagerImplTest {
 
     private static final String ITEM1_ID = "item_id_1";
@@ -93,53 +93,61 @@ public class WorkflowVersionManagerImplTest {
     private VersionMapper versionMapperMock;
     @Mock
     private VersionStateMapper versionStateMapperMock;
-    @Rule
-    public ExpectedException exceptionRule = ExpectedException.none();
     @Spy
     @InjectMocks
     private WorkflowVersionManagerImpl workflowVersionManager;
 
-    @Test(expected = EntityNotFoundException.class)
+    @Test
     public void shouldThrowExceptionWhenVersionDontExist() {
-        doThrow(new RuntimeException()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
-        workflowVersionManager.get(ITEM1_ID, VERSION1_ID);
+        assertThrows(EntityNotFoundException.class, () -> {
+            doThrow(new RuntimeException()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+            workflowVersionManager.get(ITEM1_ID, VERSION1_ID);
+        });
     }
 
-    @Test(expected = WorkflowModificationException.class)
+    @Test
     public void shouldThrowExceptionWhenCreatingVersionForArchivedWorkflow() {
-        InternalItem mockItem = new InternalItem();
-        mockItem.setId(ITEM1_ID);
-        mockItem.setStatus(ItemStatus.ARCHIVED);
-        doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
-        workflowVersionManager.create(ITEM1_ID, null, new WorkflowVersion(VERSION1_ID));
+        assertThrows(WorkflowModificationException.class, () -> {
+            InternalItem mockItem = new InternalItem();
+            mockItem.setId(ITEM1_ID);
+            mockItem.setStatus(ItemStatus.ARCHIVED);
+            doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
+            workflowVersionManager.create(ITEM1_ID, null, new WorkflowVersion(VERSION1_ID));
+        });
     }
 
-    @Test(expected = WorkflowModificationException.class)
+    @Test
     public void shouldThrowExceptionWhenUpdatingVersionForArchivedWorkflow() {
-        InternalItem mockItem = new InternalItem();
-        mockItem.setId(ITEM1_ID);
-        mockItem.setStatus(ItemStatus.ARCHIVED);
-        doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
-        workflowVersionManager.update(ITEM1_ID, new WorkflowVersion(VERSION1_ID));
+        assertThrows(WorkflowModificationException.class, () -> {
+            InternalItem mockItem = new InternalItem();
+            mockItem.setId(ITEM1_ID);
+            mockItem.setStatus(ItemStatus.ARCHIVED);
+            doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
+            workflowVersionManager.update(ITEM1_ID, new WorkflowVersion(VERSION1_ID));
+        });
     }
 
-    @Test(expected = WorkflowModificationException.class)
+    @Test
     public void shouldThrowExceptionWhenUploadingArtifactForArchivedWorkflow() {
-        InternalItem mockItem = new InternalItem();
-        mockItem.setId(ITEM1_ID);
-        mockItem.setStatus(ItemStatus.ARCHIVED);
-        doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
-        MockMultipartFile mockFile = new MockMultipartFile("data", "filename.txt", "text/plain", "some xml".getBytes());
-        workflowVersionManager.uploadArtifact(ITEM1_ID, VERSION1_ID, mockFile);
+        assertThrows(WorkflowModificationException.class, () -> {
+            InternalItem mockItem = new InternalItem();
+            mockItem.setId(ITEM1_ID);
+            mockItem.setStatus(ItemStatus.ARCHIVED);
+            doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
+            MockMultipartFile mockFile = new MockMultipartFile("data", "filename.txt", "text/plain", "some xml".getBytes());
+            workflowVersionManager.uploadArtifact(ITEM1_ID, VERSION1_ID, mockFile);
+        });
     }
 
-    @Test(expected = WorkflowModificationException.class)
+    @Test
     public void shouldThrowExceptionWhenDeletingArtifactForArchivedWorkflow() {
-        InternalItem mockItem = new InternalItem();
-        mockItem.setId(ITEM1_ID);
-        mockItem.setStatus(ItemStatus.ARCHIVED);
-        doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
-        workflowVersionManager.deleteArtifact(ITEM1_ID, VERSION1_ID);
+        assertThrows(WorkflowModificationException.class, () -> {
+            InternalItem mockItem = new InternalItem();
+            mockItem.setId(ITEM1_ID);
+            mockItem.setStatus(ItemStatus.ARCHIVED);
+            doReturn(mockItem).when(itemManagerMock).get(ITEM1_ID);
+            workflowVersionManager.deleteArtifact(ITEM1_ID, VERSION1_ID);
+        });
     }
 
     @Test
@@ -277,35 +285,41 @@ public class WorkflowVersionManagerImplTest {
         workflowVersionManager.create(ITEM1_ID, null, versionRequest);
     }
 
-    @Test(expected = VersionCreationException.class)
+    @Test
     public void shouldTrowExceptionWhenDraftVersionExists() {
-        doNothing().when(workflowVersionManager).validateWorkflowStatus(ITEM1_ID);
-        WorkflowVersion versionRequestDto = new WorkflowVersion();
+        assertThrows(VersionCreationException.class, () -> {
+            doNothing().when(workflowVersionManager).validateWorkflowStatus(ITEM1_ID);
+            WorkflowVersion versionRequestDto = new WorkflowVersion();
 
-        Version baseVersion = createRetrievedVersion(VERSION2_ID, VersionStatus.Draft);
-        List<Version> versions = Collections.singletonList(baseVersion);
-        doReturn(versions).when(versioningManagerMock).list(ITEM1_ID);
+            Version baseVersion = createRetrievedVersion(VERSION2_ID, VersionStatus.Draft);
+            List<Version> versions = Collections.singletonList(baseVersion);
+            doReturn(versions).when(versioningManagerMock).list(ITEM1_ID);
 
-        workflowVersionManager.create(ITEM1_ID, VERSION2_ID, versionRequestDto);
+            workflowVersionManager.create(ITEM1_ID, VERSION2_ID, versionRequestDto);
+        });
     }
 
-    @Test(expected = VersionCreationException.class)
+    @Test
     public void shouldTrowExceptionWhenInputsSupplied() {
-        doNothing().when(workflowVersionManager).validateWorkflowStatus(ITEM1_ID);
-        WorkflowVersion versionRequestDto = new WorkflowVersion();
-        versionRequestDto.setInputs(Collections.singleton(new ParameterEntity()));
-        InternalVersion baseVersion = createRetrievedVersion(VERSION2_ID, VersionStatus.Draft);
-        List<Version> versions = Collections.singletonList(baseVersion);
-        doReturn(versions).when(versioningManagerMock).list(ITEM1_ID);
-
-        workflowVersionManager.create(ITEM1_ID, VERSION2_ID, versionRequestDto);
+        assertThrows(VersionCreationException.class, () -> {
+            doNothing().when(workflowVersionManager).validateWorkflowStatus(ITEM1_ID);
+            WorkflowVersion versionRequestDto = new WorkflowVersion();
+            versionRequestDto.setInputs(Collections.singleton(new ParameterEntity()));
+            InternalVersion baseVersion = createRetrievedVersion(VERSION2_ID, VersionStatus.Draft);
+            List<Version> versions = Collections.singletonList(baseVersion);
+            doReturn(versions).when(versioningManagerMock).list(ITEM1_ID);
+
+            workflowVersionManager.create(ITEM1_ID, VERSION2_ID, versionRequestDto);
+        });
     }
 
 
-    @Test(expected = EntityNotFoundException.class)
+    @Test
     public void getStateOfNonExisting() {
-        doThrow(new RuntimeException()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
-        workflowVersionManager.getState(ITEM1_ID, VERSION1_ID);
+        assertThrows(EntityNotFoundException.class, () -> {
+            doThrow(new RuntimeException()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+            workflowVersionManager.getState(ITEM1_ID, VERSION1_ID);
+        });
     }
 
     @Test
@@ -318,43 +332,49 @@ public class WorkflowVersionManagerImplTest {
         assertEquals(CERTIFIED, state);
     }
 
-    @Test(expected = EntityNotFoundException.class)
+    @Test
     public void updateStateOfNonExisting() {
-        doThrow(new RuntimeException()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
-        workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, CERTIFIED);
+        assertThrows(EntityNotFoundException.class, () -> {
+            doThrow(new RuntimeException()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+            workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, CERTIFIED);
+        });
     }
 
-    @Test(expected = VersionStateModificationException.class)
+    @Test
     public void updateStateToCurrentState() {
-        InternalVersion version = createRetrievedVersion(VERSION1_ID, VersionStatus.Draft);
-        doReturn(version).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
-        doReturn(DRAFT).when(versionStateMapperMock).versionStatusToWorkflowVersionState(version.getStatus());
+        assertThrows(VersionStateModificationException.class, () -> {
+            InternalVersion version = createRetrievedVersion(VERSION1_ID, VersionStatus.Draft);
+            doReturn(version).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+            doReturn(DRAFT).when(versionStateMapperMock).versionStatusToWorkflowVersionState(version.getStatus());
 
-        workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, DRAFT);
+            workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, DRAFT);
+        });
     }
 
-    @Test(expected = VersionStateModificationMissingArtifactException.class)
+    @Test
     public void updateStateWhenCertified() {
-        InternalVersion version = createRetrievedVersion(VERSION1_ID, VersionStatus.Certified);
-        doReturn(version).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
-        doReturn(CERTIFIED).when(versionStateMapperMock).versionStatusToWorkflowVersionState(version.getStatus());
+        assertThrows(VersionStateModificationMissingArtifactException.class, () -> {
+            InternalVersion version = createRetrievedVersion(VERSION1_ID, VersionStatus.Certified);
+            doReturn(version).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+            doReturn(CERTIFIED).when(versionStateMapperMock).versionStatusToWorkflowVersionState(version.getStatus());
 
-        workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, CERTIFIED);
+            workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, CERTIFIED);
+        });
     }
 
     @Test
     public void shouldFailUpdateStateWhenNoArtifact() {
-        InternalVersion retrievedVersion = createRetrievedVersion(VERSION1_ID, VersionStatus.Draft);
-        doReturn(retrievedVersion).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
-        doReturn(DRAFT).when(versionStateMapperMock).versionStatusToWorkflowVersionState(VersionStatus.Draft);
-
-        exceptionRule.expect(VersionStateModificationMissingArtifactException.class);
-        exceptionRule.expectMessage(String.format(
+        Throwable exception = assertThrows(VersionStateModificationMissingArtifactException.class, () -> {
+            InternalVersion retrievedVersion = createRetrievedVersion(VERSION1_ID, VersionStatus.Draft);
+            doReturn(retrievedVersion).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+            doReturn(DRAFT).when(versionStateMapperMock).versionStatusToWorkflowVersionState(VersionStatus.Draft);
+            workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, CERTIFIED);
+
+            verify(versioningManagerMock).updateStatus(ITEM1_ID, VERSION1_ID, VersionStatus.Certified, anyString());
+        });
+        assertTrue(exception.getMessage().contains(String.format(
                 VersionStateModificationMissingArtifactException.WORKFLOW_MODIFICATION_STATE_MISSING_ARTIFACT_TEMPLATE,
-                ITEM1_ID, VERSION1_ID, DRAFT, CERTIFIED));
-        workflowVersionManager.updateState(ITEM1_ID, VERSION1_ID, CERTIFIED);
-
-        verify(versioningManagerMock).updateStatus(ITEM1_ID, VERSION1_ID, VersionStatus.Certified, anyString());
+                ITEM1_ID, VERSION1_ID, DRAFT, CERTIFIED)));
     }
 
     @Test
@@ -384,12 +404,14 @@ public class WorkflowVersionManagerImplTest {
         verify(artifactRepositoryMock).update(eq(ITEM1_ID), eq(VERSION1_ID), any(ArtifactEntity.class));
     }
 
-    @Test(expected = EntityNotFoundException.class)
+    @Test
     public void shouldThrowExceptionWhenArtifactNotFound() {
-        doReturn(new Version()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+        assertThrows(EntityNotFoundException.class, () -> {
+            doReturn(new Version()).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
 
-        doReturn(Optional.empty()).when(artifactRepositoryMock).get(ITEM1_ID, VERSION1_ID);
-        workflowVersionManager.getArtifact(ITEM1_ID, VERSION1_ID);
+            doReturn(Optional.empty()).when(artifactRepositoryMock).get(ITEM1_ID, VERSION1_ID);
+            workflowVersionManager.getArtifact(ITEM1_ID, VERSION1_ID);
+        });
     }
 
     @Test
@@ -403,15 +425,17 @@ public class WorkflowVersionManagerImplTest {
         assertEquals(artifactMock, returnedArtifact);
     }
 
-    @Test(expected = VersionModificationException.class)
+    @Test
     public void shouldThrowExceptionInDeleteArtifactWhenVersionIsCertified() {
-        doNothing().when(workflowVersionManager).validateWorkflowStatus(ITEM1_ID);
-        Version version = new Version();
-        doReturn(version).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
-        WorkflowVersion workflowVersion = new WorkflowVersion(VERSION1_ID);
-        workflowVersion.setState(WorkflowVersionState.CERTIFIED);
-        doReturn(workflowVersion).when(versionMapperMock).fromVersion(version);
-        workflowVersionManager.deleteArtifact(ITEM1_ID, VERSION1_ID);
+        assertThrows(VersionModificationException.class, () -> {
+            doNothing().when(workflowVersionManager).validateWorkflowStatus(ITEM1_ID);
+            Version version = new Version();
+            doReturn(version).when(versioningManagerMock).get(ITEM1_ID, VERSION1_ID);
+            WorkflowVersion workflowVersion = new WorkflowVersion(VERSION1_ID);
+            workflowVersion.setState(WorkflowVersionState.CERTIFIED);
+            doReturn(workflowVersion).when(versionMapperMock).fromVersion(version);
+            workflowVersionManager.deleteArtifact(ITEM1_ID, VERSION1_ID);
+        });
     }
 
     @Test
index 8bffc3e..f8eb790 100644 (file)
 
 package org.onap.sdc.workflow.services.impl.mappers;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import java.util.Date;
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
 import org.onap.sdc.common.versioning.persistence.types.InternalVersion;
 import org.onap.sdc.common.versioning.services.types.Version;
 import org.onap.sdc.common.versioning.services.types.VersionStatus;
@@ -29,11 +28,9 @@ import org.onap.sdc.workflow.services.types.WorkflowVersionState;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.context.annotation.ComponentScan;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.test.context.ContextConfiguration;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringJUnitConfig;
 
-@ContextConfiguration(classes = VersionMapperTest.VersionMapperSpringTestConfig.class)
-@RunWith(SpringJUnit4ClassRunner.class)
+@SpringJUnitConfig(classes = VersionMapperTest.VersionMapperSpringTestConfig.class)
 public class VersionMapperTest {
 
     @Configuration
index c0804eb..dfe2ccc 100644 (file)
 
 package org.onap.sdc.workflow.services.impl.mappers;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
 import org.onap.sdc.workflow.services.types.WorkflowVersionState;
 import org.onap.sdc.common.versioning.services.types.VersionStatus;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.context.annotation.ComponentScan;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.test.context.ContextConfiguration;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringJUnitConfig;
 
-@ContextConfiguration(classes = VersionStateMapperTest.VersionStatusMapperSpringTestConfig.class)
-@RunWith(SpringJUnit4ClassRunner.class)
+@SpringJUnitConfig(classes = VersionStateMapperTest.VersionStatusMapperSpringTestConfig.class)
 public class VersionStateMapperTest {
 
     @Configuration
index ef84190..7f5d9fe 100644 (file)
 package org.onap.sdc.workflow.services.impl.mappers;
 
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.onap.sdc.workflow.TestUtil.createItem;
 import static org.onap.sdc.workflow.TestUtil.createWorkflow;
 
-import org.junit.Test;
-import org.junit.runner.RunWith;
+import org.junit.jupiter.api.Test;
 import org.onap.sdc.common.versioning.services.types.Item;
 import org.onap.sdc.common.versioning.services.types.ItemStatus;
 import org.onap.sdc.workflow.services.impl.ItemType;
@@ -31,11 +30,9 @@ import org.onap.sdc.workflow.services.types.Workflow;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.context.annotation.ComponentScan;
 import org.springframework.context.annotation.Configuration;
-import org.springframework.test.context.ContextConfiguration;
-import org.springframework.test.context.junit4.SpringJUnit4ClassRunner;
+import org.springframework.test.context.junit.jupiter.SpringJUnitConfig;
 
-@ContextConfiguration(classes = WorkflowMapperTest.WorkflowMapperSpringTestConfig.class)
-@RunWith(SpringJUnit4ClassRunner.class)
+@SpringJUnitConfig(classes = WorkflowMapperTest.WorkflowMapperSpringTestConfig.class)
 public class WorkflowMapperTest {
 
     @Configuration