Skip to content

Commit

Permalink
Allow creating a stub of an update only interface
Browse files Browse the repository at this point in the history
  • Loading branch information
Quinn-With-Two-Ns committed Jan 9, 2024
1 parent 1f7a59c commit a64d92c
Show file tree
Hide file tree
Showing 2 changed files with 79 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,10 +47,7 @@
import io.temporal.serviceclient.MetricsTag;
import io.temporal.serviceclient.WorkflowServiceStubs;
import io.temporal.worker.WorkerFactory;
import io.temporal.workflow.Functions;
import io.temporal.workflow.QueryMethod;
import io.temporal.workflow.SignalMethod;
import io.temporal.workflow.WorkflowMethod;
import io.temporal.workflow.*;
import java.lang.annotation.Annotation;
import java.lang.reflect.Method;
import java.lang.reflect.Proxy;
Expand Down Expand Up @@ -175,7 +172,12 @@ public <T> T newWorkflowStub(Class<T> workflowInterface, String workflowId) {
@Override
public <T> T newWorkflowStub(
Class<T> workflowInterface, String workflowId, Optional<String> runId) {
checkAnnotation(workflowInterface, WorkflowMethod.class, QueryMethod.class, SignalMethod.class);
checkAnnotation(
workflowInterface,
WorkflowMethod.class,
QueryMethod.class,
SignalMethod.class,
UpdateMethod.class);
if (Strings.isNullOrEmpty(workflowId)) {
throw new IllegalArgumentException("workflowId is null or empty");
}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,72 @@
package io.temporal.workflow.updateTest;

import static org.junit.Assert.assertEquals;

import io.temporal.api.common.v1.WorkflowExecution;
import io.temporal.client.WorkflowClient;
import io.temporal.client.WorkflowOptions;
import io.temporal.testing.internal.SDKTestOptions;
import io.temporal.testing.internal.SDKTestWorkflowRule;
import io.temporal.workflow.UpdateMethod;
import io.temporal.workflow.Workflow;
import io.temporal.workflow.WorkflowInterface;
import io.temporal.workflow.WorkflowMethod;
import java.util.Optional;
import org.junit.Rule;
import org.junit.Test;

public class UpdateAnnotationTest {
@Rule
public SDKTestWorkflowRule testWorkflowRule =
SDKTestWorkflowRule.newBuilder().setWorkflowTypes(WorkflowImpl.class).build();

@Test
public void testUpdateOnlyInterface() {
// Verify a stub to an interface with only @UpdateMethod can be created.
WorkflowClient workflowClient = testWorkflowRule.getWorkflowClient();
WorkflowOptions options =
SDKTestOptions.newWorkflowOptionsWithTimeouts(testWorkflowRule.getTaskQueue()).toBuilder()
.build();
WorkflowTestInterface workflow =
workflowClient.newWorkflowStub(WorkflowTestInterface.class, options);

WorkflowExecution execution = WorkflowClient.start(workflow::execute);

UpdateWorkflowInterface updateOnlyWorkflow =
workflowClient.newWorkflowStub(UpdateWorkflowInterface.class, execution.getWorkflowId());
updateOnlyWorkflow.update();

String result =
testWorkflowRule
.getWorkflowClient()
.newUntypedWorkflowStub(execution, Optional.empty())
.getResult(String.class);
assertEquals("success", result);
}

public interface UpdateWorkflowInterface {
@UpdateMethod
void update();
}

@WorkflowInterface
public interface WorkflowTestInterface extends UpdateWorkflowInterface {
@WorkflowMethod
String execute();
}

public static class WorkflowImpl implements WorkflowTestInterface {
boolean complete = false;

@Override
public String execute() {
Workflow.await(() -> complete);
return "success";
}

@Override
public void update() {
complete = true;
}
}
}

0 comments on commit a64d92c

Please sign in to comment.