The GitHub Actions job "Required Checks" on texera.git/main has succeeded.
Run started by GitHub user github-merge-queue[bot] (triggered by 
github-merge-queue[bot]).

Head commit for run:
aec180ef0297b539834b3c2ea8fe03f4e7b720d4 / Matthew B. <[email protected]>
test(pyamber): add unit tests for storage_config (#6643)

### What changes were proposed in this PR?
- Add `amber/src/test/python/core/storage/test_storage_config.py`, a new
pytest spec for the static StorageConfig holder, which previously had no
dedicated unit tests.
- Confirm initialize populates every Iceberg and S3 field from its args
and sets the _initialized flag.
- Confirm the string commit_batch_size is coerced to int, and that
re-initialization is rejected with RuntimeError without overwriting
existing fields.
- Confirm the class cannot be instantiated (TypeError); a fixture resets
the process-wide holder per test.
### Any related issues, documentation, discussions?
Closes: #6642
### How was this PR tested?
- Run: `cd amber && python -m pytest
src/test/python/core/storage/test_storage_config.py`, expect all 6 tests
passing.
- Test-only change; no production code is modified.
### Was this PR authored or co-authored using generative AI tooling?
Co-authored with Claude Opus 4.8 in compliance with ASF

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

With regards,
GitHub Actions via GitBox

Reply via email to