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 08:22:21 UTC

[GH] (texera/gh-readonly-queue/main/pr-6094-bc5accff3797d538d165b7c0acd7f5112eabf031): Workflow run "Required Checks" failed!

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

Head commit for run:
d5f18a68304fce45b09259b2a6e8ec0214ad4910 / Xinyuan Lin <xi...@uci.edu>
test(config): add unit test coverage for the remaining config objects (#6094)

### What changes were proposed in this PR?

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

| File | Codecov before | What the tests pin |
| --- | --- | --- |
| `UdfConfig.scala` | 0% | python/R path + log handler defaults from
udf.conf |
| `DefaultsConfig.scala` | 0% | the `reinit` flag and the flattened
`allDefaults` short-key/value map from default.conf |
| `ComputingUnitConfig.scala` | 0% | local/sharing enabled flags |
| `LLMConfig.scala` | 0% | LiteLLM base URL + master key |
| `PekkoConfig.scala` | 0% | actor/serialization, remote/artery, and
cluster/failure-detector settings from cluster.conf (no ActorSystem
started) |
| `PythonUtils.scala` | 0% | `getPythonExecutable` blank→`python3`
fallback and trimmed-path branch |

Reading each value forces resolution from the backing `.conf`;
env/system-property-overridable values are guarded on the override being
unset (mirroring `StorageConfigSpec`).

### 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 *UdfConfigSpec *DefaultsConfigSpec
*ComputingUnitConfigSpec *LLMConfigSpec *PekkoConfigSpec
*PythonUtilsSpec"` — 13 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/28700078480

With regards,
GitHub Actions via GitBox