temporalio / sdk-java / #410
71%
master: 75%

LAST BUILD BRANCH: main
DEFAULT BRANCH: master
Ran
Jobs 1
Files 931
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 70.128% (+0.3%) from 69.806%
#410

push

github

web-flow 
Add transfer type converters (#3073)

* WIP Transfer type converters

* Handle missing transfer result types

* Test transfer type converters across SDK paths

* Add transfer type support to workflow streams

* Use transfer-type converter internally

* Remove transfer-type converters from workflow streams

* Consolidate client data converter creation into factory

* Separate configured and internal client converters

Pass transfer-aware converters to Activity and Schedule invokers without rebuilding their configured options. Preserve serialization context for task-token heartbeats and propagate the configured converter to test Nexus clients.

* Validate transfer converter results during argument decoding

* Expose transfer-aware conversion in clients and failures

* Document transfer type converters in changelog

* Test transfer-aware converter in activity context

* Address transfer type converter review findings

Make the default converter transfer-aware while preserving its public API.
Fix reflection-array decoding, callback exception handling, converter
construction diagnostics, generic class declarations, and cache absence.

Simplify client invoker and worker configuration. Strengthen direct and
wrapped converter tests and correct Nexus operation-context coverage.

* Pass explicit dependencies to client invokers

* Remove unrelated Worker constructor cleanup

* Separate workflow client options from converter plumbing

* Preserve test client options in cloud configuration

Keep test-specific converters and identities when applying the cloud profile. Add regression coverage for both configuration paths and remove the obsolete public converter wrapper test.

8091 of 13636 branches covered (59.34%)

Branch coverage included in aggregate %.

254 of 296 new or added lines in 19 files covered. (85.81%)

17 existing lines in 10 files now uncovered.

32621 of 44418 relevant lines covered (73.44%)

0.73 hits per line

Uncovered Changes

Lines Coverage ∆ File
17
4.08
1.02% temporal-sdk/src/main/java/io/temporal/internal/client/RootScheduleClientInvoker.java
11
42.39
0.38% temporal-sdk/src/main/java/io/temporal/internal/client/RootActivityClientInvoker.java
8
92.27
temporal-sdk/src/main/java/io/temporal/internal/common/converter/TemporalTransferTypeDataConverter.java
3
50.0
50.0% temporal-sdk/src/main/java/io/temporal/client/NexusClientImpl.java
3
18.05
15.23% temporal-sdk/src/main/java/io/temporal/internal/client/ScheduleProtoUtil.java

Coverage Regressions

Lines Coverage ∆ File
4
66.44
-1.38% temporal-envconfig/src/main/java/io/temporal/envconfig/ClientConfigProfile.java
4
85.21
-4.14% temporal-sdk/src/main/java/io/temporal/internal/worker/ShutdownManager.java
2
88.89
-1.96% temporal-serviceclient/src/main/java/io/temporal/internal/payload/visitor/Traversal.java
1
90.72
-1.03% temporal-sdk/src/main/java/io/temporal/common/converter/PayloadAndFailureDataConverter.java
1
86.0
-2.0% temporal-sdk/src/main/java/io/temporal/internal/client/external/ManualActivityCompletionClientFactoryImpl.java
1
18.05
15.23% temporal-sdk/src/main/java/io/temporal/internal/client/ScheduleProtoUtil.java
1
90.49
-0.58% temporal-sdk/src/main/java/io/temporal/internal/sync/DeterministicRunnerImpl.java
1
81.62
-1.47% temporal-sdk/src/main/java/io/temporal/internal/worker/BasePoller.java
1
78.57
-1.79% temporal-sdk/src/main/java/io/temporal/internal/worker/MultiThreadedPoller.java
1
79.51
-1.86% temporal-testing/src/main/java/io/temporal/testing/TestActivityEnvironmentInternal.java
Jobs
ID Job ID Ran Files Coverage
1 #410.1 931
70.13
Source Files on build #410