Skip to content

Adding Dapr.Cryptography E2E tests + improving test coverage#1765

Merged
WhitWaldo merged 1 commit intodapr:masterfrom
WhitWaldo:crypto-e2e-tests
Apr 4, 2026
Merged

Adding Dapr.Cryptography E2E tests + improving test coverage#1765
WhitWaldo merged 1 commit intodapr:masterfrom
WhitWaldo:crypto-e2e-tests

Conversation

@WhitWaldo
Copy link
Copy Markdown
Contributor

Description

Significantly broadened unit test coverage for Dapr.Cryptography + enabled crypto E2E tests on TestContainers implementation and added integration tests (mimicking the existing example for a small file, large generated file and string value).

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>
@WhitWaldo WhitWaldo self-assigned this Apr 4, 2026
@WhitWaldo WhitWaldo requested review from a team as code owners April 4, 2026 19:31
@WhitWaldo WhitWaldo changed the title Added more unit tests -> 13% to 93% test coverage, fixing typo Adding Dapr.Cryptography E2E tests + improving test coverage Apr 4, 2026
@WhitWaldo WhitWaldo merged commit d23cd35 into dapr:master Apr 4, 2026
259 of 261 checks passed
@WhitWaldo WhitWaldo deleted the crypto-e2e-tests branch April 4, 2026 19:46
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