You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@texera.apache.org by GitBox <gi...@apache.org> on 2026/07/04 06:38:37 UTC

[GH] (texera/gh-readonly-queue/main/pr-6084-ad7e2883ba07bd95daee5f8da6b263aecb9fe49e): Workflow run "Required Checks" is working again!

The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-6084-ad7e2883ba07bd95daee5f8da6b263aecb9fe49e has succeeded.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
a3f22bbe57096a7e6598cead668ed3bc560a993e / Xinyuan Lin <xi...@uci.edu>
test(config): add unit test coverage for KubernetesConfig, UserSystemConfig, and AuthConfig (#6084)

### What changes were proposed in this PR?

Add unit test coverage for three more `common/config` objects, selected
from the Codecov report (all 0%). No production-code changes.

| File | Codecov before | What the tests pin |
| --- | --- | --- |
| `KubernetesConfig.scala` | 0% | the fixed port, string/numeric/boolean
settings, and comma-separated CPU/memory/GPU limit-option parsing
(trimmed, non-empty) |
| `UserSystemConfig.scala` | 0% | admin/google/smtp/project defaults,
the version-collapse interval (name≠key), and the `appDomain` `None`
path for a blank domain |
| `AuthConfig.scala` | 0% | `jwtExpirationMinutes`, the configured
lowercased secret + memoization, and `getRandomHexString` (32-char
lowercase hex, fresh per call) via reflection |

Reading each value forces resolution from the backing `.conf`;
env-overridable values are guarded on the env var being unset (mirroring
`StorageConfigSpec`). The random-secret branch is exercised directly
rather than through the env-dependent `"random"` path.

### Any related issues, documentation, discussions?

Follow-up to the review feedback on #6043: prioritize tests that fill
uncovered code paths.

### How was this PR tested?

- `sbt "Config/testOnly *KubernetesConfigSpec *UserSystemConfigSpec
*AuthConfigSpec"` — 10 tests, all green
- `sbt "Config/Test/scalafmtCheck"` and `sbt "Config/scalafixAll
--check"` — clean

### Was this PR authored or co-authored using generative AI tooling?

Generated-by: Claude Code (Opus 4.8 [1M context])

Report URL: https://github.com/apache/texera/actions/runs/28697724773

With regards,
GitHub Actions via GitBox