restatedev / sdk-java

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

Verify that there can't be two handlers with same name #326

Closed slinkydeveloper closed 1 month ago

slinkydeveloper commented 1 month ago

Fix #322

github-actions[bot] commented 1 month ago

Test Results

 48 files  ±0   48 suites  ±0   3m 5s :stopwatch: -2s 738 tests ±0  730 :white_check_mark: +1  8 :zzz:  - 1  0 :x: ±0  738 runs   - 1  730 :white_check_mark: ±0  8 :zzz:  - 1  0 :x: ±0 

Results for commit 824a5cf4. ± Comparison against base commit 2b5d8e42.

This pull request removes 7 and adds 7 tests. Note that renamed tests count towards both. ``` dev.restate.e2e.NodeWorkflowAPITest ‑ Workflow cannot be submitted more than once dev.restate.sdk.common.CoreSerdesTest ‑ Byte: -111 dev.restate.sdk.common.CoreSerdesTest ‑ Double: 0.5184783207844548 dev.restate.sdk.common.CoreSerdesTest ‑ Float: 0.62707436 dev.restate.sdk.common.CoreSerdesTest ‑ Integer: 1461953937 dev.restate.sdk.common.CoreSerdesTest ‑ Long: -4178086973975213529 dev.restate.sdk.common.CoreSerdesTest ‑ Short: 13800 ``` ``` dev.restate.sdk.common.CoreSerdesTest ‑ Byte: -45 dev.restate.sdk.common.CoreSerdesTest ‑ Double: 0.8948854648588914 dev.restate.sdk.common.CoreSerdesTest ‑ Float: 0.8062008 dev.restate.sdk.common.CoreSerdesTest ‑ Integer: 1743726599 dev.restate.sdk.common.CoreSerdesTest ‑ Long: -8028845254367141477 dev.restate.sdk.common.CoreSerdesTest ‑ Short: 19725 dev.restate.sdk.serde.jackson.JacksonSerdesTest ‑ [Till, Francesco] ```

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