mengw15 opened a new pull request, #6329: URL: https://github.com/apache/texera/pull/6329
### What changes were proposed in this PR? Add unit test coverage for `KubernetesClient.generatePodName`, a pure string helper (`computing-unit-<cuid>`) that needs no cluster. Covers the base case and the `cuid=0` boundary. No production-code changes. The client-backed methods (`getPodByName`, `createPod`, …) and `generatePodURI` (which embeds `KubernetesConfig` values) need a fabric8 mock / config control and stay out of scope, per the issue. ### Any related issues, documentation, discussions? Closes #6327. ### How was this PR tested? ``` sbt "ComputingUnitManagingService/testOnly *KubernetesClientSpec" ``` All 2 pass. `scalafmt` and `scalafix --check` are clean. The spec touches no cluster — `generatePodName` is a pure string helper, and loading `object KubernetesClient` builds the fabric8 client offline (config resolved from the classpath, no connection). ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Code (Opus 4.8 [1M context]) -- 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]
