testing-library/jest-dom

toHaveStyle for property `font-size` behaves incorrectly for number values

Open

#564 aperta il 27 dic 2023

Vedi su GitHub
 (2 commenti) (1 reazione) (0 assegnatari)JavaScript (416 fork)github user discovery
buggood first issue

Metriche repository

Star
 (4594 star)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

  • @testing-library/jest-dom version: 6.1.5
  • node version: 20.3.1
  • jest (or vitest) version: 0.34.1
  • npm (or yarn) version: 9.6.7

Relevant code or config:

render(<div data-testid="element" style={{ fontSize: 8 }} />)
expect(screen.getByTestId('element')).toHaveStyle({ fontSize: 1 })

What you did:

I expected the above assertion to fail

What happened:

The test passes instead!✅

Reproduction:

Problem description:

the assertion is clearly incorrect and can easily cause bugs

Suggested solution:

I have not thought of a solution yet but would like to investigate this issue during my off hours.

Guida contributor

Direzione di ricerca
Esamina l'implementazione del matcher toHaveStyle in jest dom, in particolare come gestisce i valori numerici per proprietà CSS come font size. Cerca la logica di parsing e confronto.
Tech stack
javascript
Dominio
testing
Tipo issue
Bug
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
JavaScriptjest
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75