yegor256/takes

Improve TkMeasuredTest Coverage (related to #1408)

Aperta

#1434 aperta il 24 lug 2025

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)Java (215 fork)github user discovery
enhancementhelp wanted

Metriche repository

Star
 (875 stelle)
Metriche merge PR
 (Merge medio 95g 2h) (22 PR mergiate in 30 g)

Descrizione

Refactor and extend TkMeasuredTest to improve coverage and robustness (related to #1408)

The current TkMeasuredTest verifies only the presence of a custom or default header. However, there are several aspects that could be improved:

  1. Add negative/edge test cases:

    • What happens if a null or empty header is passed?
    • Does TkMeasured properly override an existing timing header?

  2. Validate header value:

    • Currently, the tests only assert that the header exists, but not that it contains a valid numeric duration (in ms).

  3. Improve assertion messages:

    • Current failure messages are generic and could be improved with more informative output (e.g. actual value of the response).

This refactor would help ensure better long-term stability and maintainability of the test.

I'm happy to prepare a PR for this if approved.

Guida contributor

Direzione di ricerca
Rivedi il TkMeasuredTest corrente per vedere cosa copre, poi ispeziona la classe TkMeasured per capire come viene impostato l'header dei tempi. Cerca casi limite come header null o vuoti e come asserire valori numerici. Controlla le convenzioni di altri test e considera l'esecuzione dei test con Maven.
Tech stack
java
Dominio
backendtesting
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
JavaJUnit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75