apache / apache/pulsar

Missing Test Coverage: Add embedded test for Proxy's PulsarHandlers

Open
#19,624 1 comment 0 reactions 0 assignees View on GitHub
Stale type/enhancement
Dominant language
Java
Stars
15.3k
Forks
3.8k
Avg merge
1d 14h
Merged PRs (30d)
160

Description

### Search before asking

- [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.

### Motivation

The `ServerCnxTest` class is extremely valuable and makes it easy to verify certain low level behaviors. We do not have the same tests for the proxy.

### Solution

Add these tests.

### Alternatives

_No response_

### Anything else?

I am happy to implement this at some point, but for now, I am unable to.

### Are you willing to submit a PR?

- [ ] I'm willing to submit a PR!

Contributor guide

Open the contributing guide

Research direction

Start by reading the existing ServerCnxTest class and locating the proxy's PulsarHandlers implementation and related embedded proxy tests. Identify the low-level behaviors already covered for ServerCnx, then add equivalent proxy coverage. Done means the embedded tests run successfully and cover the relevant PulsarHandlers behaviors.

Written by the indexing model from the issue text.

Assessment

Tech stack
java
Domain
backend, testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.