kubeflow/trainer
test(statusserver): add unit tests for token parsing and status URL helpers
クローズ
#3,404 opened on 2026/03/31
area/testinggood first issuehelp wantedkind/feature
Repository metrics
- Stars
- (2,112 個のスター)
- PR merge metrics
- (PR metrics pending)
説明
What you would like to be added?
I’d like to add focused unit tests for helper functions in pkg/statusserver, specifically:
extractRawTokenTokenAudienceStatusUrl
I also noticed that extractRawToken currently uses exact string splitting on "Bearer <token>", which is a bit brittle for extra whitespace or lowercase bearer. If maintainers agree, I’d like to make that parsing slightly more robust while keeping the behavior simple.
Why is this needed?
These helpers are small but important for status server auth and request path generation. Adding direct unit tests would improve confidence in the status server code and make this logic easier to maintain.