fixed tests

conductor-oss · Nov 19, 2024 · f807490 · f807490
1 parent 29ac65c
commit f807490
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/common/src/test/java/com/netflix/conductor/common/tasks/TaskDefTest.java b/common/src/test/java/com/netflix/conductor/common/tasks/TaskDefTest.java
@@ -82,6 +82,7 @@ public void testTaskDefInvalidEmail() {
         taskDef.setName("test-task");
         taskDef.setRetryCount(1);
         taskDef.setTimeoutSeconds(1000);
+        taskDef.setTotalTimeoutSeconds(2000);
         taskDef.setResponseTimeoutSeconds(1);
 
         Set<ConstraintViolation<Object>> result = validator.validate(taskDef);
@@ -102,6 +103,7 @@ public void testTaskDefValidEmail() {
         taskDef.setRetryCount(1);
         taskDef.setTimeoutSeconds(1000);
         taskDef.setResponseTimeoutSeconds(1);
+        taskDef.setTotalTimeoutSeconds(2000);
         taskDef.setOwnerEmail("[email protected]");
 
         Set<ConstraintViolation<Object>> result = validator.validate(taskDef);

diff --git a/common/src/test/java/com/netflix/conductor/common/tasks/TaskTest.java b/common/src/test/java/com/netflix/conductor/common/tasks/TaskTest.java
@@ -98,7 +98,7 @@ public void testDeepCopyTask() {
         final Task task = new Task();
         // In order to avoid forgetting putting inside the copy method the newly added fields check
         // the number of declared fields.
-        final int expectedTaskFieldsNumber = 41;
+        final int expectedTaskFieldsNumber = 42;
         final int declaredFieldsNumber = task.getClass().getDeclaredFields().length;
 
         assertEquals(expectedTaskFieldsNumber, declaredFieldsNumber);
@@ -133,6 +133,7 @@ public void testDeepCopyTask() {
         task.setIsolationGroupId("groupId");
         task.setStartTime(12L);
         task.setEndTime(20L);
+        task.setFirstStartTime(12L);
         task.setScheduledTime(7L);
         task.setRetried(false);
         task.setReasonForIncompletion("");