jnioche opened a new pull request, #8446: URL: https://github.com/apache/storm/pull/8446
## Summary - Ports 8 of 10 tests from `storm-core/test/clj/org/apache/storm/cluster_test.clj` to `storm-server/src/test/java/org/apache/storm/cluster/ClusterStateTest.java` - Tests cover IStateStorage (ZKStateStorage) and IStormClusterState (StormClusterStateImpl) using an in-process ZooKeeper - 2 tests skipped: one was an empty TODO, the other relies on Clojure-specific static mocking infrastructure already covered by `StormClusterStateImplTest` ### Tests ported | Java test | Coverage | |---|---| | `testBasics` | IStateStorage set/get/delete/mkdirs operations | | `testMultiState` | Two IStateStorage instances sharing ZK state | | `testEphemeral` | Ephemeral nodes removed on creator close | | `testCallbacks` | ZKStateChangedCallback for data/children changes | | `testStormClusterStateBasics` | Assignments, stormBase, credentials, blobstore, nimbuses | | `testStormClusterStateErrors` | Error reporting with simulated time | | `testSupervisorState` | Supervisor heartbeat and ephemeral disconnect | | `testClusterAuthentication` | ZK digest auth via CuratorUtils | Part of the Clojure test migration effort ([Phase 3](https://github.com/apache/storm/issues/8438)). ## Test plan - [x] `mvn test -pl storm-server -Dtest=org.apache.storm.cluster.ClusterStateTest` — all 8 tests pass - [ ] Verify existing tests still pass: `mvn test -pl storm-server` 🤖 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]
