apache / apache/dubbo

[Bug] Some test cases still rely on the REST protocol

Open
#14,340 3 comments 0 reactions 0 assignees View on GitHub
component/need-triage type/need-triage
Dominant language
Java
Stars
41.6k
Forks
26.4k
Avg merge
15h 13m
Merged PRs (30d)
4

Description

### Pre-check

- [X] I am sure that all the content I provide is in English.

### Search before asking

- [X] I had searched in the [issues](https://github.com/apache/dubbo/issues?q=is%3Aissue) and found no similar issues.

### Apache Dubbo Component

Java SDK (apache/dubbo)

### Dubbo Version

Dubbo 3.3

### Steps to reproduce this issue

Rest Protocol has been move to dubbo-spi-extension, But some test cases still rely on the REST protocol.
![image](https://github.com/apache/dubbo/assets/22697462/eee49a6b-8f2d-4d8e-8a19-690eda281ac4)

### What you expected to happen

Hope to pass the test case.
Which was better?
1. Add org.apache.dubbo.extensions:dubbo-rpc-rest dependency to the test case
2. Replace the rest protocol in the test case with triple

### Anything else

_No response_

### Are you willing to submit a pull request to fix on your own?

- [ ] Yes I am willing to submit a pull request on my own!

### Code of Conduct

- [X] I agree to follow this project's [Code of Conduct](https://www.apache.org/foundation/policies/conduct)

Contributor guide

Open the contributing guide

Research direction

Locate the test cases in the Java SDK that still reference the REST protocol, then inspect how the REST extension was moved to dubbo-spi-extension. Compare the available dependency and protocol replacement options, run the affected tests, and consider the issue done when the tests pass without an unintended REST dependency.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
distributed-systems
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.