Skip to content

Commit 2b8a2d5

Browse files
authored
fix(to-have-style): fix bug in to-have-style (#98)
* fix(to-have-style): fix bug in to-have-style * fix lint error
1 parent 585086d commit 2b8a2d5

File tree

2 files changed

+7
-0
lines changed

2 files changed

+7
-0
lines changed

src/__tests__/lib/rules/prefer-to-have-style.js

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,10 @@ ruleTester.run("prefer-to-have-style", rule, {
1818
}, [foo]);`,
1919
],
2020
invalid: [
21+
{
22+
code: `expect(a.style).toHaveProperty('transform')`,
23+
errors
24+
},
2125
{
2226
code: `expect(el.style.foo).toBe("bar")`,
2327
errors,

src/rules/prefer-to-have-style.js

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -193,6 +193,9 @@ export const create = (context) => ({
193193
node: node.property,
194194
message: "Use toHaveStyle instead of asserting on element style",
195195
fix(fixer) {
196+
if (!styleValue) {
197+
return null;
198+
}
196199
return [
197200
fixer.removeRange([node.object.range[1], node.property.range[1]]),
198201
fixer.replaceText(matcher, "toHaveStyle"),

0 commit comments

Comments
 (0)