dapr / dapr/java-sdk

Add new test for TLS on durable task module after merging DT to SDK

Open
#1,604 0 comments 0 reactions 0 assignees View on GitHub
kind/bug
Dominant language
Java
Stars
300
Forks
230
Avg merge
1d 20h
Merged PRs (30d)
4

Description

## Expected Behavior

The commented test: durabletask-client/src/test/java/io/dapr/durabletask/DurableTaskGrpcClientTlsTest.java
needs to be migrated to follow the same shape as the test that is already in the SDK.

We do NOT need to add this test back as it is, as it brings a lot of nasty dependencies and behavior that doesn't make sense to test inside the SDK. We need to follow this PR: https://github.com/dapr/java-sdk/commit/e25f52a13a5c47c166cfea51e09412220a98ed02

Notice the description FIX CVE!

## Actual Behavior

The test is commented out, but we don't need to re enable it, we just need to migrate it to look like the existing test in the SDK.

## Steps to Reproduce the Problem

## Release Note

RELEASE NOTE:

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.