Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] TimeSkipping test server hangs when testing Update #72

Open
mnichols opened this issue Jun 23, 2024 · 1 comment
Open

[Bug] TimeSkipping test server hangs when testing Update #72

mnichols opened this issue Jun 23, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@mnichols
Copy link

What are you really trying to do?

I was trying to test WorkflowUpdate in my own code and was using StartTimeSkippingAsync . My test was hanging each time even though the samples in this repo were succeeding.
I changed the WorkflowUpdate sample in this repo to use StartTimeSkippingAsync instead of StartLocalAsync and got the same behavior.

Here is logging output after test times out:

A total of 1 test files matched the specified pattern.
[xUnit.net 00:00:30.59]     TemporalioSamples.Tests.WorkflowUpdate.WorkflowUpdateTests.SimpleRun_Succeed [FAIL]
  Failed TemporalioSamples.Tests.WorkflowUpdate.WorkflowUpdateTests.SimpleRun_Succeed [30 s]
  Error Message:
   Temporalio.Exceptions.RpcException : Timeout expired
  Stack Trace:
     at Temporalio.Bridge.Client.CallAsync[T](RpcService service, String rpc, IMessage req, MessageParser`1 resp, Boolean retry, IEnumerable`1 metadata, Nullable`1 timeout, Nullable`1 cancellationToken)
   at Temporalio.Client.TemporalConnection.InvokeRpcAsync[T](RpcService service, String rpc, IMessage req, MessageParser`1 resp, RpcOptions options)
   at Temporalio.Client.TemporalClient.Impl.StartWorkflowUpdateAsync[TResult](StartWorkflowUpdateInput input)
   at Temporalio.Client.WorkflowHandle`1.ExecuteUpdateAsync[TUpdateResult](Expression`1 updateCall, WorkflowUpdateOptions options)
   at TemporalioSamples.Tests.WorkflowUpdate.WorkflowUpdateTests.<>c__DisplayClass1_0.<<SimpleRun_Succeed>b__0>d.MoveNext() in /Users/mnichols/dev/samples-dotnet/tests/WorkflowUpdate/WorkflowUpdateTests.cs:line 31
--- End of stack trace from previous location ---
   at Temporalio.Worker.TemporalWorker.ExecuteInternalAsync(Func`1 untilComplete, CancellationToken stoppingToken)
   at Temporalio.Worker.TemporalWorker.ExecuteInternalAsync(Func`1 untilComplete, CancellationToken stoppingToken)
   at TemporalioSamples.Tests.WorkflowUpdate.WorkflowUpdateTests.SimpleRun_Succeed() in /Users/mnichols/dev/samples-dotnet/tests/WorkflowUpdate/WorkflowUpdateTests.cs:line 25
   at TemporalioSamples.Tests.WorkflowUpdate.WorkflowUpdateTests.SimpleRun_Succeed() in /Users/mnichols/dev/samples-dotnet/tests/WorkflowUpdate/WorkflowUpdateTests.cs:line 25
--- End of stack trace from previous location ---

Failed!  - Failed:     1, Passed:     0, Skipped:     0, Total:     1, Duration: < 1 ms - TemporalioSamples.Tests.dll (net8.0)

Minimal Reproduction

Just change WorkflowEnvironment.StartLocalAsync to WorkflowEnvironment.StartTimeSkippingAsync and run

dotnet test --filter "TemporalioSamples.Tests.WorkflowUpdate.WorkflowUpdateTests.SimpleRun_Succeed"

Environment/Versions

  • OS and processor: M1 Mac
  • Temporal Version: 1.1.2
  • DOTNET 8.0
@mnichols mnichols added the bug Something isn't working label Jun 23, 2024
@cretz
Copy link
Member

cretz commented Jun 24, 2024

This is a bug on the test server not this SDK specifically: temporalio/sdk-java#1903. Also see temporalio/sdk-dotnet#179.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants