Skip to content

Conversation

jakobht
Copy link
Member

@jakobht jakobht commented Nov 28, 2024

What changed?
Added tests for LocalActivityOptions so we are now at 89% coverage for this file

Why?
Increase coverage

How did you test it?

Potential risks

Release notes

Documentation Changes

@jakobht jakobht force-pushed the TestLocalActivityOptions branch from 4e75b0f to 0945169 Compare November 28, 2024 09:47

final String expected =
"LocalActivityOptions{"
+ "scheduleToCloseTimeout=PT2M3S, "
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would expect it to be more human-readable, but that's OK as well :-)

@jakobht jakobht merged commit 4f39f33 into cadence-workflow:master Nov 28, 2024
7 checks passed
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.

2 participants