Workflow Update in Test Environment should dedup updates by ID

[Quinn-With-Two-Ns]

Expected Behavior

Workflow Updates in Test Environment behave like Updates against a real server and either dupes for fails if an update with a duplicate ID is sent

Actual Behavior

Test Environment accepts update with duplicate ID

[yuandrew]

Is the ID here referring to UpdateID from the UpdateClientWorkflow options? I'm working through some other failures in setting up this workflow, want to double check I'm going down the right path.

[Quinn-With-Two-Ns]

Is the ID here referring to UpdateID from the UpdateClientWorkflow options

Conceptually yes, but the test environment does not use a client since it is all in memory in the same process

Here is an example of sending an update with the test environment https://github.com/temporalio/samples-go/blob/main/reqrespupdate/workflow_test.go#L41, here "test id" is the UpdateID

[yuandrew]

I'm playing around with the update go sample, and when I add UpdateID: we.GetID() to the UpdateWorkflowOptions, I'm not seeing the real server throw any error. Shouldn't this update fail?

[Quinn-With-Two-Ns]

Shouldn't this update fail?

No, the real server will dedup the update request by the update ID