Skip to content

test: migrate durable task TLS tests - #1802

Open
NikStack20 wants to merge 1 commit into
dapr:masterfrom
NikStack20:fix/1604-durabletask-tls-test
Open

test: migrate durable task TLS tests#1802
NikStack20 wants to merge 1 commit into
dapr:masterfrom
NikStack20:fix/1604-durabletask-tls-test

Conversation

@NikStack20

Copy link
Copy Markdown

Description

Migrates the commented-out Durable Task gRPC TLS test to the current Java SDK test pattern.

Changes

  • Replaced the legacy certificate-generation based test implementation.
  • Added pre-generated TLS test certificates as test resources.
  • Added coverage for TLS, CA certificates, mTLS, invalid TLS configuration, insecure TLS, and plaintext connections.
  • Ensured the Durable Task client is closed after each test.

Validation

  • DurableTaskGrpcClientTlsTest: 7 tests passed.
  • durabletask-client test suite: 94 tests passed.
  • git diff --check: clean.

Related to #1604.

@NikStack20
NikStack20 requested review from a team as code owners September 2, 2026 20:17
Signed-off-by: NikStack20 <codingnik20@gmail.com>
@NikStack20
NikStack20 force-pushed the fix/1604-durabletask-tls-test branch from 22ba524 to 68d6923 Compare September 2, 2026 20:19
@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.08%. Comparing base (4192ca9) to head (68d6923).
⚠️ Report is 3 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1802      +/-   ##
============================================
- Coverage     77.15%   77.08%   -0.07%     
+ Complexity     2317     2316       -1     
============================================
  Files           245      245              
  Lines          7186     7186              
  Branches        750      750              
============================================
- Hits           5544     5539       -5     
- Misses         1281     1285       +4     
- Partials        361      362       +1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant