Allow JSX spread attributes in style attribute checks (#62)

Author: utarwyn

CHANGELOG.md

@@ -18,6 +18,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 -   [#56](https://github.com/green-code-initiative/creedengo-javascript/issues/56) **BREAKING:** Rename plugin to creedengo-javascript
 -   [#44](https://github.com/green-code-initiative/creedengo-javascript/pull/44) Implement the rule GCI523 for React Native
 -   [#52](https://github.com/green-code-initiative/creedengo-javascript/pull/52) Remove trailing dots in Sonar rules descriptions
+-   [#62](https://github.com/green-code-initiative/creedengo-javascript/pull/62) Fix style attribute checks in GCI26 and GCI29
 -   Update Docker Compose configuration file to V2
 
 ### Deleted

eslint-plugin/lib/rules/avoid-css-animations.js

@@ -37,7 +37,7 @@ module.exports = {
     return {
       JSXOpeningElement(node) {
         const styleAttribute = node.attributes.find(
-          (attribute) => attribute.name.name === "style",
+          (attribute) => attribute.name?.name === "style",
         );
 
         if (styleAttribute?.value.expression) {

eslint-plugin/lib/rules/prefer-shorthand-css-notations.js

@@ -84,7 +84,7 @@ module.exports = {
     return {
       JSXOpeningElement(node) {
         const styleAttribute = node.attributes.find(
-          (attr) => attr.name.name === "style",
+          (attr) => attr.name?.name === "style",
         );
         if (styleAttribute) {
           const nodePropertyNames =

eslint-plugin/tests/lib/rules/avoid-css-animations.js

@@ -53,6 +53,8 @@ ruleTester.run("avoid-css-animations", rule, {
     `,
     `<div style={{ width: '100px', height: '100px' }}>Hello world</div>`,
     `<div style="border: 2px solid red">My red element</div>`,
+    // spread attributes should not throw an error (#49)
+    "<input {...inputProps} className={styles.input} onChange={handleChange}/>",
   ],
 
   invalid: [

eslint-plugin/tests/lib/rules/prefer-shorthand-css-notations.js

@@ -73,6 +73,8 @@ ruleTester.run("prefer-shorthand-css-notations", rule, {
       code: "<div style={{ animationName: 'example', animationDuration: '5s' }}/>",
       options: [{ disableProperties: ["animation"] }],
     },
+    // spread attributes should not throw an error (#49)
+    "<input {...inputProps} className={styles.input} onChange={handleChange}/>",
   ],
   invalid: [
     {