testing-library/jest-dom

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

Open

#564 aberto em 27 de dez. de 2023

Ver no GitHub
 (2 comments) (1 reaction) (0 assignees)JavaScript (416 forks)github user discovery
buggood first issue

Métricas do repositório

Stars
 (4.594 stars)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

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

Guia do colaborador

Direção de pesquisa
Inspecione a implementação do matcher toHaveStyle no jest dom, especificamente como ele lida com valores numéricos para propriedades CSS como font size. Procure pela lógica de parsing e comparação.
Pilha de tecnologia
javascript
Domain
testing
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptjest
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
75