restatedev / sdk-java

Restate SDK for JVM Languages
https://restate.dev
MIT License
22 stars 4 forks source link

New workflow api implementation #312

Closed slinkydeveloper closed 4 months ago

slinkydeveloper commented 4 months ago
github-actions[bot] commented 4 months ago

Test Results

665 tests  ± 0   658 :white_check_mark: +2   10s :stopwatch: - 3m 9s  14 suites  - 34     7 :zzz:  - 2   14 files    - 34     0 :x: ±0 

Results for commit 6d231695. ± Comparison against base commit d08025ab.

This pull request removes 326 and adds 326 tests. Note that renamed tests count towards both. ``` dev.restate.e2e.JavaAwaitTimeoutTest ‑ Test Awaitable#await(Duration) dev.restate.e2e.JavaAwakeableTest ‑ generate(IngressClient) dev.restate.e2e.JavaCancelInvocationTest ‑ cancel blocked invocation on AWAKEABLE dev.restate.e2e.JavaCancelInvocationTest ‑ cancel blocked invocation on CALL dev.restate.e2e.JavaCancelInvocationTest ‑ cancel blocked invocation on SLEEP dev.restate.e2e.JavaCoordinatorWithNodeReceiverServiceToServiceCallTest ‑ synchronousCall(IngressClient) dev.restate.e2e.JavaErrorsTest ‑ Test calling method that fails terminally dev.restate.e2e.JavaErrorsTest ‑ Test calling method that fails terminally multiple times dev.restate.e2e.JavaErrorsTest ‑ Test invocations are retried until they succeed dev.restate.e2e.JavaErrorsTest ‑ Test propagate failure from another service … ``` ``` dev.restate.sdk.JavaBlockingTests ‑ 100: [MockSingleThread][dev.restate.sdk.PromiseTest] AwaitPromise#run: Completed with failure dev.restate.sdk.JavaBlockingTests ‑ 101: [MockMultiThreaded][dev.restate.sdk.PromiseTest] AwaitPromise#run: Completed with failure dev.restate.sdk.JavaBlockingTests ‑ 102: [MockSingleThread][dev.restate.sdk.PromiseTest] PeekPromise#run: Completed with success dev.restate.sdk.JavaBlockingTests ‑ 103: [MockMultiThreaded][dev.restate.sdk.PromiseTest] PeekPromise#run: Completed with success dev.restate.sdk.JavaBlockingTests ‑ 104: [MockSingleThread][dev.restate.sdk.PromiseTest] PeekPromise#run: Completed with failure dev.restate.sdk.JavaBlockingTests ‑ 105: [MockMultiThreaded][dev.restate.sdk.PromiseTest] PeekPromise#run: Completed with failure dev.restate.sdk.JavaBlockingTests ‑ 106: [MockSingleThread][dev.restate.sdk.PromiseTest] PeekPromise#run: Completed with null dev.restate.sdk.JavaBlockingTests ‑ 107: [MockMultiThreaded][dev.restate.sdk.PromiseTest] PeekPromise#run: Completed with null dev.restate.sdk.JavaBlockingTests ‑ 108: [MockSingleThread][dev.restate.sdk.PromiseTest] IsCompletedPromise#run: Completed with success dev.restate.sdk.JavaBlockingTests ‑ 109: [MockMultiThreaded][dev.restate.sdk.PromiseTest] IsCompletedPromise#run: Completed with success … ```
This pull request removes 5 skipped tests and adds 3 skipped tests. Note that renamed tests count towards both. ``` dev.restate.e2e.NodeWorkflowAPITest ‑ Set and resolve durable promise leads to completion dev.restate.e2e.NodeWorkflowAPITest ‑ Workflow cannot be submitted more than once dev.restate.sdk.http.vertx.HttpVertxTests ‑ 122: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown dev.restate.sdk.http.vertx.HttpVertxTests ‑ 123: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown: Fires timeout dev.restate.sdk.http.vertx.HttpVertxTests ‑ 156: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.StateTest] Unknown ``` ``` dev.restate.sdk.http.vertx.HttpVertxTests ‑ 134: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown dev.restate.sdk.http.vertx.HttpVertxTests ‑ 135: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown: Fires timeout dev.restate.sdk.http.vertx.HttpVertxTests ‑ 168: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.StateTest] Unknown ```

:recycle: This comment has been updated with latest results.