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

New interface refactor (sdk-api-kotlin) #238

Merged
merged 1 commit into from
Mar 7, 2024

Conversation

slinkydeveloper
Copy link
Contributor

No description provided.

Copy link
Contributor

github-actions bot commented Mar 6, 2024

Test Results

 92 files  ±0   92 suites  ±0   4m 12s ⏱️ +4s
 73 tests ±0   48 ✅ ±0  25 💤 ±0  0 ❌ ±0 
196 runs  ±0  131 ✅ ±0  65 💤 ±0  0 ❌ ±0 

Results for commit b29f181. ± Comparison against base commit d2ec402.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Mar 6, 2024

Unit Test Results

  42 files  +    1    42 suites  +1   1m 39s ⏱️ +4s
609 tests +253  579 ✔️ +246  30 💤 +7  0 ±0 
610 runs  +254  580 ✔️ +247  30 💤 +7  0 ±0 

Results for commit b29f181. ± Comparison against base commit d2ec402.

This pull request removes 9 and adds 262 tests. Note that renamed tests count towards both.
dev.restate.sdk.common.CoreSerdesTest ‑ Byte: 53
dev.restate.sdk.common.CoreSerdesTest ‑ Double: 0.3591363038313148
dev.restate.sdk.common.CoreSerdesTest ‑ Float: 0.860855
dev.restate.sdk.common.CoreSerdesTest ‑ Integer: 1627005003
dev.restate.sdk.common.CoreSerdesTest ‑ Long: 6475307996852761772
dev.restate.sdk.common.CoreSerdesTest ‑ Short: -9631
dev.restate.sdk.http.vertx.HttpVertxTests ‑ 91: [HttpVertxTestExecutor][dev.restate.sdk.http.vertx.HttpVertxTests$VertxExecutorsTest] CheckBlockingComponentTrampolineExecutor
dev.restate.sdk.lambda.LambdaHandlerTest ‑ [1] dev.restate.sdk.lambda.testservices.JavaCounterService
dev.restate.sdk.serde.jackson.JacksonSerdesTest ‑ [Till, Francesco]
dev.restate.sdk.common.CoreSerdesTest ‑ Byte: 99
dev.restate.sdk.common.CoreSerdesTest ‑ Double: 0.71937798776541
dev.restate.sdk.common.CoreSerdesTest ‑ Float: 0.7905205
dev.restate.sdk.common.CoreSerdesTest ‑ Integer: 957144728
dev.restate.sdk.common.CoreSerdesTest ‑ Long: -145241248904440343
dev.restate.sdk.common.CoreSerdesTest ‑ Short: 27046
dev.restate.sdk.http.vertx.HttpVertxTests ‑ 100: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] ReverseAwaitOrder: A2 and A1 completed later
dev.restate.sdk.http.vertx.HttpVertxTests ‑ 101: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] ReverseAwaitOrder: Only A2 completed
dev.restate.sdk.http.vertx.HttpVertxTests ‑ 102: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] ReverseAwaitOrder: Only A1 completed
dev.restate.sdk.http.vertx.HttpVertxTests ‑ 103: [HttpVertxTestExecutor][dev.restate.sdk.kotlin.DeferredTest] AwaitTwiceTheSameAwaitable
…

♻️ This comment has been updated with latest results.

@slinkydeveloper slinkydeveloper force-pushed the new-interface-refactor-sdk-api-kotlin branch 3 times, most recently from 9bd79a0 to def0dc1 Compare March 7, 2024 11:30
@slinkydeveloper slinkydeveloper linked an issue Mar 7, 2024 that may be closed by this pull request
@slinkydeveloper slinkydeveloper force-pushed the new-interface-refactor-sdk-api-kotlin branch from def0dc1 to b29f181 Compare March 7, 2024 13:27
@slinkydeveloper slinkydeveloper merged commit 3952fe5 into main Mar 7, 2024
5 checks passed
@slinkydeveloper slinkydeveloper deleted the new-interface-refactor-sdk-api-kotlin branch March 7, 2024 13:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Re-enable sdk-api-kotlin with the new interface system
1 participant