jnioche opened a new pull request, #8450: URL: https://github.com/apache/storm/pull/8450
## Summary - Ports 2 of 5 tests from `metrics_test.clj` and all 5 tests from `nimbus_auth_test.clj` to Java/JUnit 5 - Clojure `defbolt count-acks` replaced with Java `CountAcksBolt` inner class - JAAS digest config copied to storm-server test resources for classpath access ### Tests ported **MetricsIntegrationTest** (storm-core, 2 tests): | Test | Coverage | |---|---| | `testCustomMetric` | CountMetric with FakeMetricConsumer, simulated time | | `testCustomMetricWithMultiTasks` | Same with 2 tasks per bolt (all grouping) | Skipped: 3 Python multilang metrics tests (require Python shell bolt/spout) **NimbusAuthTest** (storm-server, 5 tests): | Test | Coverage | |---|---| | `testSimpleAuthentication` | SimpleTransportPlugin with no authorizer | | `testNoopAuthorizationWithSimpleTransport` | NoopAuthorizer, mocked cluster state | | `testDenyAuthorizationWithSimpleTransport` | DenyAuthorizer, 17 denied Nimbus operations | | `testNoopAuthorizationWithSaslDigest` | DigestSaslTransportPlugin with JAAS config | | `testDenyAuthorizationWithSaslDigest` | DenyAuthorizer with SASL digest, 17 denied ops | Part of the Clojure test migration effort (Phase 5 of #8445). ## Test plan - [x] `mvn test -pl storm-core -Dtest=org.apache.storm.metric.MetricsIntegrationTest` — 2 tests pass - [x] `mvn test -pl storm-server -Dtest=org.apache.storm.security.auth.NimbusAuthTest` — 5 tests pass - [ ] Verify existing tests still pass 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- 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]
