More Flowable5 unit test fixes

flowable · Aug 3, 2023 · 00c8f81 · 00c8f81
1 parent da277f6
commit 00c8f81
Show file tree

Hide file tree

Showing 8 changed files with 90 additions and 66 deletions.
diff --git a/...able-dmn-engine/src/test/java/org/flowable/dmn/engine/test/deployment/DeploymentTest.java b/...able-dmn-engine/src/test/java/org/flowable/dmn/engine/test/deployment/DeploymentTest.java
@@ -382,64 +382,77 @@ public void multipleSameDeploymentsInTenant() throws Exception {
 
     @Test
     public void multipleDeploymentsInTenant() throws Exception {
-        repositoryService.createDeployment()
-                .name("deploymentA")
-                .addClasspathResource("org/flowable/dmn/engine/test/deployment/simple.dmn")
-                .tenantId("myTenant")
-                .enableDuplicateFiltering()
-                .deploy();
-
-        org.flowable.dmn.api.DmnDeployment deployment = repositoryService.createDeploymentQuery()
-                .deploymentName("deploymentA")
-                .deploymentTenantId("myTenant")
-                .singleResult();
-        assertThat(deployment).isNotNull();
-
-        List<DmnDecision> decisions = repositoryService.createDecisionQuery()
-                .decisionKey("decision")
-                .decisionTenantId("myTenant")
-                .list();
-        assertThat(decisions).hasSize(1);
-
-        repositoryService.createDeployment()
-                .name("deploymentA")
-                .addClasspathResource("org/flowable/dmn/engine/test/deployment/simple2.dmn")
-                .tenantId("myTenant")
-                .enableDuplicateFiltering()
-                .deploy();
-
-        List<org.flowable.dmn.api.DmnDeployment> deployments = repositoryService.createDeploymentQuery()
-                .deploymentName("deploymentA")
-                .deploymentTenantId("myTenant")
-                .list();
-        assertThat(deployments).hasSize(2);
-
-        decisions = repositoryService.createDecisionQuery()
-                .decisionKey("anotherDecision")
-                .decisionTenantId("myTenant")
-                .list();
-        assertThat(decisions).hasSize(1);
-
-        repositoryService.createDeployment()
-                .name("deploymentA")
-                .addClasspathResource("org/flowable/dmn/engine/test/deployment/simple2.dmn")
-                .tenantId("myTenant")
-                .enableDuplicateFiltering()
-                .deploy();
-
-        deployments = repositoryService.createDeploymentQuery()
-                .deploymentName("deploymentA")
-                .deploymentTenantId("myTenant")
-                .list();
-        assertThat(deployments).hasSize(2);
-
-        decisions = repositoryService.createDecisionQuery()
-                .decisionKey("anotherDecision")
-                .decisionTenantId("myTenant")
-                .list();
-        assertThat(decisions).hasSize(1);
-
-        deleteDeployments();
+        try {
+            repositoryService.createDeployment()
+                    .name("deploymentA")
+                    .addClasspathResource("org/flowable/dmn/engine/test/deployment/simple.dmn")
+                    .tenantId("myTenant")
+                    .enableDuplicateFiltering()
+                    .deploy();
+
+            org.flowable.dmn.api.DmnDeployment deployment = repositoryService.createDeploymentQuery()
+                    .deploymentName("deploymentA")
+                    .deploymentTenantId("myTenant")
+                    .singleResult();
+            assertThat(deployment).isNotNull();
+
+            List<DmnDecision> decisions = repositoryService.createDecisionQuery()
+                    .decisionKey("decision")
+                    .decisionTenantId("myTenant")
+                    .list();
+            assertThat(decisions).hasSize(1);
+
+            Calendar deployCal = new GregorianCalendar();
+            deployCal.add(Calendar.DATE, 2);
+            dmnEngineConfiguration.getClock().setCurrentTime(deployCal.getTime());
+
+            repositoryService.createDeployment()
+                    .name("deploymentA")
+                    .addClasspathResource("org/flowable/dmn/engine/test/deployment/simple2.dmn")
+                    .tenantId("myTenant")
+                    .enableDuplicateFiltering()
+                    .deploy();
+
+            List<org.flowable.dmn.api.DmnDeployment> deployments = repositoryService.createDeploymentQuery()
+                    .deploymentName("deploymentA")
+                    .deploymentTenantId("myTenant")
+                    .list();
+            assertThat(deployments).hasSize(2);
+
+            decisions = repositoryService.createDecisionQuery()
+                    .decisionKey("anotherDecision")
+                    .decisionTenantId("myTenant")
+                    .list();
+            assertThat(decisions).hasSize(1);
+
+            deployCal = new GregorianCalendar();
+            deployCal.add(Calendar.DATE, 4);
+            dmnEngineConfiguration.getClock().setCurrentTime(deployCal.getTime());
+
+            repositoryService.createDeployment()
+                    .name("deploymentA")
+                    .addClasspathResource("org/flowable/dmn/engine/test/deployment/simple2.dmn")
+                    .tenantId("myTenant")
+                    .enableDuplicateFiltering()
+                    .deploy();
+
+            deployments = repositoryService.createDeploymentQuery()
+                    .deploymentName("deploymentA")
+                    .deploymentTenantId("myTenant")
+                    .list();
+            assertThat(deployments).hasSize(2);
+
+            decisions = repositoryService.createDecisionQuery()
+                    .decisionKey("anotherDecision")
+                    .decisionTenantId("myTenant")
+                    .list();
+            assertThat(decisions).hasSize(1);
+
+            deleteDeployments();
+
+        } finally {
+            dmnEngineConfiguration.getClock().reset();
+        }
     }
 
     @Test

diff --git a/...ble5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/MaxValueReachedFault.java b/...ble5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/MaxValueReachedFault.java
@@ -12,7 +12,7 @@
  */
 package org.activiti.engine.impl.webservice;
 
-import javax.xml.ws.WebFault;
+import jakarta.xml.ws.WebFault;
 
 /**
  * An implementation of a fault returned by the counter WS

diff --git a/...5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/WebServiceDataStructure.java b/...5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/WebServiceDataStructure.java
@@ -14,8 +14,8 @@
 
 import java.util.Date;
 
-import javax.xml.bind.annotation.XmlElement;
-import javax.xml.bind.annotation.XmlSchemaType;
+import jakarta.xml.bind.annotation.XmlElement;
+import jakarta.xml.bind.annotation.XmlSchemaType;
 
 public class WebServiceDataStructure {
 

diff --git a/.../flowable5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/WebServiceMock.java b/.../flowable5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/WebServiceMock.java
@@ -14,9 +14,9 @@
 
 import java.util.Date;
 
-import javax.jws.WebParam;
-import javax.jws.WebResult;
-import javax.jws.WebService;
+import jakarta.jws.WebParam;
+import jakarta.jws.WebResult;
+import jakarta.jws.WebService;
 
 /**
  * A simple WS for unit test purpose

diff --git a/...wable5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/WebServiceMockImpl.java b/...wable5-cxf-test/src/test/java/org/activiti/engine/impl/webservice/WebServiceMockImpl.java
@@ -14,7 +14,7 @@
 
 import java.util.Date;
 
-import javax.jws.WebService;
+import jakarta.jws.WebService;
 
 /**
  * An implementation of a Counter WS

diff --git a/modules/flowable5-spring-test/pom.xml b/modules/flowable5-spring-test/pom.xml
@@ -180,6 +180,11 @@
             <artifactId>slf4j-log4j12</artifactId>
             <scope>test</scope>
         </dependency>
+        <dependency>
+            <groupId>com.github.h-thurow</groupId>
+            <artifactId>simple-jndi</artifactId>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
     <profiles>

diff --git a/...les/flowable5-spring-test/src/test/java/org/activiti/spring/test/email/JndiEmailTest.java b/...les/flowable5-spring-test/src/test/java/org/activiti/spring/test/email/JndiEmailTest.java
@@ -23,6 +23,7 @@
 import org.flowable.engine.runtime.ProcessInstance;
 import org.flowable.engine.test.Deployment;
 import org.junit.BeforeClass;
+import org.junit.Test;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 import org.springframework.test.context.ContextConfiguration;
@@ -55,9 +56,10 @@ public void setUp() throws NoSuchProviderException, NamingException {
                 .bind("Session", mailSession);
     }
 
+    @Test
     @Deployment(resources = { "org/activiti/spring/test/email/EmailTaskUsingJndi.bpmn20.xml" })
     public void testEmailUsingJndi() {
-        Map<String, Object> variables = new HashMap<String, Object>();
+        Map<String, Object> variables = new HashMap<>();
         ProcessInstance processInstance = runtimeService.startProcessInstanceByKey("EmailJndiProcess", variables);
         assertEquals(0, runtimeService.createProcessInstanceQuery().processInstanceId(processInstance.getId()).count());
     }

diff --git a/modules/flowable5-spring-test/src/test/resources/jndi.properties b/modules/flowable5-spring-test/src/test/resources/jndi.properties
@@ -0,0 +1,4 @@
+java.naming.factory.initial = org.osjava.sj.SimpleJndiContextFactory
+org.osjava.sj.delimiter = .
+jndi.syntax.separator = /
+org.osjava.sj.jndi.shared = true