merlimat opened a new pull request, #25646: URL: https://github.com/apache/pulsar/pull/25646
## Summary The test was converted to `SharedPulsarBaseTest` in [#25392](https://github.com/apache/pulsar/pull/25392), but kept the original `assertTrue(stats.isEmpty())` assertion. That assertion only held when each test class got a fresh broker — on the shared cluster, prior tests have already exercised BookKeeper, so `generate()` returns non-empty stats. The test's stated intent is just to verify `generate()` doesn't throw NPE or other exceptions. Keep that, but replace the emptiness check with `assertNotNull`. ## Test plan - [x] Test passes locally with the change applied - [ ] CI is green -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
