testing-library/jest-dom

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

Open

#564 geöffnet am 27. Dez. 2023

Auf GitHub ansehen
 (2 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (416 Forks)github user discovery
buggood first issue

Repository-Metriken

Stars
 (4.594 Stars)
PR-Merge-Metriken
 (PR-Metriken ausstehend)

Beschreibung

  • @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.

Contributor Guide

Research-Richtung
Überprüfe die Implementierung des toHaveStyle Matchers in jest dom, insbesondere wie er numerische Werte für CSS Eigenschaften wie font size behandelt. Suche nach der Parsing Logik und dem Vergleich.
Tech Stack
javascript
Domain
testing
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptjest
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75