Skip to content

Commit d5f21b3

Browse files
connormeredithgnapse
authored andcommitted
fix: Issue with lowercasing toHaveStyle values. (#164)
1 parent c12a476 commit d5f21b3

File tree

2 files changed

+6
-2
lines changed

2 files changed

+6
-2
lines changed

src/__tests__/to-have-style.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ describe('.toHaveStyle', () => {
1717
color: white;
1818
float: left;
1919
transition: opacity 0.2s ease-out, top 0.3s cubic-bezier(1.175, 0.885, 0.32, 1.275);
20+
transform: translateX(0px);
2021
}
2122
`
2223
document.body.appendChild(style)
@@ -49,6 +50,10 @@ describe('.toHaveStyle', () => {
4950
expect(container.querySelector('.label')).toHaveStyle(`
5051
Align-items: center;
5152
`)
53+
54+
expect(container.querySelector('.label')).toHaveStyle(`
55+
transform: translateX(0px);
56+
`)
5257
})
5358

5459
test('handles negative test cases', () => {

src/to-have-style.js

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,7 @@ function getStyleDeclaration(document, css) {
1919
function isSubset(styles, computedStyle) {
2020
return Object.entries(styles).every(
2121
([prop, value]) =>
22-
computedStyle.getPropertyValue(prop.toLowerCase()) ===
23-
value.toLowerCase(),
22+
computedStyle.getPropertyValue(prop.toLowerCase()) === value,
2423
)
2524
}
2625

0 commit comments

Comments
 (0)