Can now pass environment variables into daprd container during integration tests - #1871
Merged
Conversation
…ntegration testing with `Dapr.Testcontainers` Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #1871 +/- ##
==========================================
+ Coverage 62.14% 62.15% +0.01%
==========================================
Files 334 334
Lines 9719 9727 +8
Branches 1132 1133 +1
==========================================
+ Hits 6040 6046 +6
- Misses 3361 3363 +2
Partials 318 318
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
3 tasks
WhitWaldo
added a commit
that referenced
this pull request
Jul 13, 2026
…1873) * Fixing workflow tracing + adding observability integration testing (#1870) * Implementing better trace support in Dapr.Workflow + adding e2e testing to `Dapr.Testcontainers` to more properly validate it. * Added functionality to `Dapr.Testcontainers` to support capturing telemetry from the Zipkin endpoint on the Dapr runtime to add to the integration testing suite * Added a series of integration tests to validate proper telemetry capture across various scenarios: - Activity runtime-to-SDK span continuity - Child workflow activity tracing - Retry attempts - Fan-out/fan-in parallel activities - Timer plus external event before activity execution - Multi-app remote activity - Activity failure span status - Continue-as-new - Multi-app child workflow - Downstream custom ActivitySource parented under the workflow activity span - Suspend/resume (validate activity after resume) - Re-run workflow (validating spans for rerun instance) - History propagation with OwnHistory - History propagation with Lineage - Parallel child workflows, validating each child activity trace - Terminate (validating for work completed before termination and terminated history) - Caught activity failure/compensation (validate activity span is marked error - Workflow failure after activity failure (validate trace when activity failure is not caught and workflow ends as failed) - Child workflow + activity in both parent and child (validates activity in the child) * Corrected to validate that the `Dapr.Workflow` activity span exists with the expected display name, the span has the same trace id as the runtime span, the span's ParentSpanId equals the runtime span Id and that the runtime exports a Zipkin activity span with the expected runtime name Signed-off-by: Whit Waldo <whit.waldo@innovian.net> * Improved existing orchestration tests to make them more closely resemble accurate scenarios and requirements --------- Signed-off-by: Whit Waldo <whit.waldo@innovian.net> * Added mechanism to pass environment variables into daprd as part of integration testing with `Dapr.Testcontainers` (#1871) Signed-off-by: Whit Waldo <whit.waldo@innovian.net> --------- Signed-off-by: Whit Waldo <whit.waldo@innovian.net>
This was referenced Jul 26, 2026
Closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Added endpoint to fluently pass environment variables into daprd container as part of integration testing with
Dapr.Testcontainers.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: #1860
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: