Skip to content

Workflow test coverage#1754

Merged
WhitWaldo merged 4 commits intodapr:masterfrom
WhitWaldo:workflow-test-coverage
Mar 18, 2026
Merged

Workflow test coverage#1754
WhitWaldo merged 4 commits intodapr:masterfrom
WhitWaldo:workflow-test-coverage

Conversation

@WhitWaldo
Copy link
Copy Markdown
Contributor

Description

Improved unit test coverage across Dapr.Workflow and Dapr.Workflow.Abstractions projects (bringing the latter up to 100% and the former to 91%), plus removed a few unused variables and unreferenced test classes.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
…ker`

Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
…sed variables

Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
@WhitWaldo WhitWaldo self-assigned this Mar 18, 2026
@WhitWaldo WhitWaldo requested a review from a team as a code owner March 18, 2026 23:38
@WhitWaldo WhitWaldo requested a review from a team as a code owner March 18, 2026 23:38
@WhitWaldo WhitWaldo merged commit cfe667d into dapr:master Mar 18, 2026
277 of 279 checks passed
@WhitWaldo WhitWaldo deleted the workflow-test-coverage branch March 18, 2026 23:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant