kubeflow/trainer

test(statusserver): add unit tests for token parsing and status URL helpers

Chiusa

#3404 aperta il 31 mar 2026

Vedi su GitHubGuida contributor
 (7 commenti) (0 reazioni) (1 assegnatario)Go (964 fork)github user discovery
area/testinggood first issuehelp wantedkind/feature

Metriche repository

Star
 (2112 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

What you would like to be added?

I’d like to add focused unit tests for helper functions in pkg/statusserver, specifically:

  • extractRawToken
  • TokenAudience
  • StatusUrl

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.

Guida contributor

Direzione di ricerca
Esamina le funzioni helper esistenti in pkg/statusserver e i loro chiamanti. Guarda i file di test esistenti nello stesso pacchetto per pattern e convenzioni di testing. Implementa test unitari per le funzioni extractRawToken, TokenAudience e StatusUrl, coprendo casi limite come spazi extra o 'bearer' in minuscolo.
Tech stack
go
Dominio
backend
Tipo issue
Test
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Gotesting
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
80