fix: fixed bug in toHaveAttribute check on style (#81)

testing-library · Sep 10, 2020 · 167137a · 167137a
1 parent d639db4
commit 167137a
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/__tests__/lib/rules/prefer-to-have-style.js b/src/__tests__/lib/rules/prefer-to-have-style.js
@@ -11,6 +11,11 @@ ruleTester.run("prefer-to-have-style", rule, {
     `expect(el.style).toMatchSnapshot()`,
     `expect(el.style).toEqual(foo)`,
     `expect(el).toHaveAttribute("style")`,
+    `React.useLayoutEffect(() => {
+      if (foo) {
+        document.body.setAttribute("style", "foo");
+      }
+    }, [foo]);`,
   ],
   invalid: [
     {

diff --git a/src/rules/prefer-to-have-style.js b/src/rules/prefer-to-have-style.js
@@ -109,7 +109,7 @@ export const create = (context) => ({
   },
 
   //expect(el).toHaveAttribute("style", "foo: bar");
-  [`CallExpression[callee.property.name][arguments.0.value=style][arguments.1]`](
+  [`CallExpression[callee.property.name=toHaveAttribute][arguments.0.value=style][arguments.1][callee.object.callee.name=expect]`](
     node
   ) {
     context.report({