Build out examples/documentation for unit testing Workflows/activities#1708
Conversation
daanschutte
force-pushed
the
daanschutte/workflow-tests
branch
from
1fc211a to
5936958
Compare
Signed-off-by: Daan Schutte <daan.schutte@icloud.com>
Signed-off-by: Daan Schutte <daan.schutte@icloud.com>
Signed-off-by: Daan Schutte <daan.schutte@icloud.com>
Signed-off-by: Daan Schutte <daan.schutte@icloud.com>
Signed-off-by: Daan Schutte <daan.schutte@icloud.com>
Signed-off-by: Daan Schutte <daan.schutte@icloud.com>
Signed-off-by: Daan Schutte <daan.schutte@icloud.com>
daanschutte
force-pushed
the
daanschutte/workflow-tests
branch
from
41c5400 to
ffd3857
Compare
WhitWaldo
left a comment
WhitWaldo
left a comment
There was a problem hiding this comment.
First of all, thank you for the contribution! Examples that improve understanding of how to test workflows are definitely lacking in the repository, so I appreciate you spotting and remedying that!
If you could fix the spelling typo, I'd be happy to merge this.
* Collapsed available strategies to just Numeric, Date and SemVer and normalized the options between each for similar support. Updated tests accordingly. * Added missing prefix support for date version strategy --------- Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
* Added support for a property that can be placed in the entry application csproj that enables the automatic discovery and import of workflows across multiple projects * Added built-in package reference to `Dapr.Workflow.Abstractions` --------- Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
* Updating to exclude Dapr.Workflow.Versioning.Abstractions,Dapr.Workflow.Versioning.Generators, and Dapr.Workflow.Versioning.Runtime projects from being deployed to NuGet. * Removed requirement that to discover packages requires a release tag, but left this in place for Publish Packages --------- Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
|
Fixed the typo, don't have the time now but will look at the merge conflict later! |
Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
|
@holopin-bot @daanschutte sdk-badge Thank you! Here's a digital badge as a small token of appreciation. |
|
Congratulations @daanschutte, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/cmqbf6kvu000ljl04v5pjh8nx This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account. |
3 participants
Description
Improves testing examples for Dapr Workflows by adding test coverage for additional concepts and fixing documentation issues.
TestHighCostOrderApproved: Mock external events (WaitForExternalEventAsync) and verify custom statusTestHighCostOrderApprovalTimeout: Simulate timeouts withTaskCanceledExceptionTestActivityException: Simulate activity failures withWorkflowTaskFailedExceptionIssue reference
Addresses #1532 by improving test coverage. Additional improvements may be needed - open to suggestions.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: