Nexus caller timeouts

[bergundy]

What was changed

• Added support for ScheduleToStartTimeout and StartToCloseTimeout for Nexus operations in the Java SDK and test server.
• In the test server:
  • Cap secondary timeouts to scheduleToClose timeout
  • Support all three timeout types in timeoutNexusOperation()
  • Calculate OPERATION_TIMEOUT header dynamically in pollNexusTaskQueue() and use the correct header format
• Added tests for the new functionality

NOTE: Do not merge until the API is tagged.

[Quinn-With-Two-Ns]

Can we add some tests like we have for activity

sdk-java/temporal-sdk/src/test/java/io/temporal/workflow/activityTests/ActivityTimeoutTest.java

Line 51 in 2322bd0

public class ActivityTimeoutTest {

to verify the parameter is properly wired through and respected

[bergundy]

Can we add some tests like we have for activity

sdk-java/temporal-sdk/src/test/java/io/temporal/workflow/activityTests/ActivityTimeoutTest.java

Line 51 in 2322bd0

public class ActivityTimeoutTest {

to verify the parameter is properly wired through and respected

We have coverage in NexusWorkflowTest.java.

[Quinn-With-Two-Ns]

Should be able to use testWorkflowRule.sleep here to avoid waiting real time in the time skipping test server, but I would prefer the test not use a sleep and just wait for the condition to be true. Should be possible to wait for the operation to fail by looking at the workflow describe output no?

[Quinn-With-Two-Ns]

We have coverage in NexusWorkflowTest.java.

NexusWorkflowTest.java doesn't use the Java SDK so it is not providing coverage of the SDK

[Quinn-With-Two-Ns]

you also need to update constructPendingNexusOperationInfo in the test server I think, unless I missed that.