lalalastella opened a new pull request, #5790: URL: https://github.com/apache/texera/pull/5790
### What changes were proposed in this PR? Add `HeaderFieldSpec.scala` — a dedicated unit spec for `HeaderField` in `common/auth/src/main/scala/org/apache/texera/auth/util/`. `HeaderField` holds the canonical HTTP header names the auth layer reads from every request. Without a spec, accidental renames silently break header-based authentication. This PR pins three contracts: | Contract | Tests | |---|---| | Exact value | Each constant equals its expected string literal | | RFC-7230 format | Every constant matches `x-user-*` lowercase-hyphen namespace | | Distinctness | No two constants share the same value (no accidental aliasing) | No production-code changes. ### Any related issues, documentation, discussions? Closes #5662 ### How was this PR tested? New spec file: `HeaderFieldSpec.scala` (~50 lines, no build changes needed — ScalaTest is already a `% Test` dependency in `common/auth`). ``` sbt "Auth/testOnly org.apache.texera.auth.util.HeaderFieldSpec" ``` ### Was this PR authored or co-authored using generative AI tooling? Generated-by: Claude Sonnet 4.6 -- 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]
