restatedev / sdk-java

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

Fix getOutput return type #342

Closed slinkydeveloper closed 1 month ago

github-actions[bot] commented 1 month ago

Test Results

682 tests   - 80   675 :white_check_mark:  - 80   11s :stopwatch: - 7m 0s  14 suites  - 35     7 :zzz: ± 0   14 files    - 35     0 :x: ± 0 

Results for commit 787f9570. ± Comparison against base commit 5bba5003.

This pull request removes 374 and adds 294 tests. Note that renamed tests count towards both. ``` dev.restate.e2e.JavaAwaitTimeoutTest ‑ Test Awaitable#await(Duration) dev.restate.e2e.JavaAwakeableTest ‑ generate(Client) 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(Client) 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 ‑ 116: [MockSingleThread][dev.restate.sdk.PromiseTest] IsCompletedPromise#run: Not completed dev.restate.sdk.JavaBlockingTests ‑ 117: [MockMultiThreaded][dev.restate.sdk.PromiseTest] IsCompletedPromise#run: Not completed dev.restate.sdk.JavaBlockingTests ‑ 118: [MockSingleThread][dev.restate.sdk.PromiseTest] ResolvePromise#run: resolve succeeds dev.restate.sdk.JavaBlockingTests ‑ 119: [MockMultiThreaded][dev.restate.sdk.PromiseTest] ResolvePromise#run: resolve succeeds dev.restate.sdk.JavaBlockingTests ‑ 120: [MockSingleThread][dev.restate.sdk.PromiseTest] ResolvePromise#run: resolve fails dev.restate.sdk.JavaBlockingTests ‑ 121: [MockMultiThreaded][dev.restate.sdk.PromiseTest] ResolvePromise#run: resolve fails dev.restate.sdk.JavaBlockingTests ‑ 122: [MockSingleThread][dev.restate.sdk.PromiseTest] RejectPromise#run: resolve succeeds dev.restate.sdk.JavaBlockingTests ‑ 123: [MockMultiThreaded][dev.restate.sdk.PromiseTest] RejectPromise#run: resolve succeeds dev.restate.sdk.JavaBlockingTests ‑ 124: [MockSingleThread][dev.restate.sdk.PromiseTest] RejectPromise#run: resolve fails dev.restate.sdk.JavaBlockingTests ‑ 125: [MockMultiThreaded][dev.restate.sdk.PromiseTest] RejectPromise#run: resolve fails … ```
This pull request removes 3 skipped tests and adds 3 skipped tests. Note that renamed tests count towards both. ``` dev.restate.sdk.http.vertx.HttpVertxTests ‑ 138: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown dev.restate.sdk.http.vertx.HttpVertxTests ‑ 139: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown: Fires timeout dev.restate.sdk.http.vertx.HttpVertxTests ‑ 175: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.StateTest] Unknown ``` ``` dev.restate.sdk.http.vertx.HttpVertxTests ‑ 137: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown dev.restate.sdk.http.vertx.HttpVertxTests ‑ 138: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] Unknown: Fires timeout dev.restate.sdk.http.vertx.HttpVertxTests ‑ 174: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.StateTest] Unknown ```

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