tweak conflict policy error message

temporalio · Nov 30, 2024 · 514935b · 514935b
1 parent 84d1248
commit 514935b
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 8 deletions.
diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowClient.java
@@ -871,8 +871,7 @@ static <R> WorkflowUpdateHandle<R> startUpdateWithStart(
       Proc updateMethod,
       @Nonnull UpdateOptions<R> options,
       @Nonnull WithStartWorkflowOperation<?> startOperation) {
-    return WorkflowClientInternalImpl.startUpdateWithStart(
-        updateMethod, options, startOperation);
+    return WorkflowClientInternalImpl.startUpdateWithStart(updateMethod, options, startOperation);
   }
 
   /**
@@ -1207,8 +1206,7 @@ static <R> R executeUpdateWithStart(
       Functions.Proc updateMethod,
       @Nonnull UpdateOptions<R> options,
       @Nonnull WithStartWorkflowOperation<?> startOperation) {
-    return WorkflowClientInternalImpl.executeUpdateWithStart(
-        updateMethod, options, startOperation);
+    return WorkflowClientInternalImpl.executeUpdateWithStart(updateMethod, options, startOperation);
   }
 
   /**

diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowStub.java
@@ -164,7 +164,9 @@ <R> WorkflowUpdateHandle<R> getUpdateHandle(
    */
   @Experimental
   <R> WorkflowUpdateHandle<R> startUpdateWithStart(
-      UpdateOptions<R> updateOptions, Object[] updateArgs, WithStartWorkflowOperation<?> startOperation);
+      UpdateOptions<R> updateOptions,
+      Object[] updateArgs,
+      WithStartWorkflowOperation<?> startOperation);
 
   /**
    * Synchronously update a workflow execution by invoking its update handler, and start the
@@ -179,7 +181,9 @@ <R> WorkflowUpdateHandle<R> startUpdateWithStart(
    */
   @Experimental
   <R> R executeUpdateWithStart(
-      UpdateOptions<R> updateOptions, Object[] updateArgs, WithStartWorkflowOperation<?> startOperation);
+      UpdateOptions<R> updateOptions,
+      Object[] updateArgs,
+      WithStartWorkflowOperation<?> startOperation);
 
   WorkflowExecution signalWithStart(String signalName, Object[] signalArgs, Object[] startArgs);
 

diff --git a/temporal-sdk/src/main/java/io/temporal/client/WorkflowStubImpl.java b/temporal-sdk/src/main/java/io/temporal/client/WorkflowStubImpl.java
@@ -139,7 +139,7 @@ public <R> WorkflowUpdateHandle<R> startUpdateWithStart(
     }
     if (options.getWorkflowIdConflictPolicy() == null) {
       throw new IllegalStateException(
-          "Required parameter WorkflowIdConflictPolicy in WorkflowOptions is missing in WorkflowStub");
+          "WorkflowIdConflictPolicy is required in WorkflowOptions for Update-With-Start");
     }
     if (!startOp.markInvoked()) {
       throw new IllegalStateException("WithStartWorkflowOperation was already executed");

diff --git a/temporal-sdk/src/test/java/io/temporal/workflow/updateTest/UpdateWithStartTest.java b/temporal-sdk/src/test/java/io/temporal/workflow/updateTest/UpdateWithStartTest.java
@@ -791,7 +791,7 @@ public void failWhenConflictPolicyIsMissing() {
     } catch (IllegalStateException e) {
       assertEquals(
           e.getMessage(),
-          "Required parameter WorkflowIdConflictPolicy in WorkflowOptions is missing in WorkflowStub");
+          "WorkflowIdConflictPolicy is required in WorkflowOptions for Update-With-Start");
     }
   }