eslint-plugin-smarthr 0.5.9 → 0.5.11

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 All notable changes to this project will be documented in this file. See [standard-version](https://github.com/conventional-changelog/standard-version) for commit guidelines.
 
+### [0.5.11](https://github.com/kufu/eslint-plugin-smarthr/compare/v0.5.9...v0.5.11) (2024-06-06)
+
+
+### Bug Fixes
+
+*  <FormControl> 内に三項演算子がある場合に、入力要素が複数あると誤認されないようにする ([#139](https://github.com/kufu/eslint-plugin-smarthr/issues/139)) ([8a234c7](https://github.com/kufu/eslint-plugin-smarthr/commit/8a234c73e140725ff1926d4d23f64c263e46d5d6))
+* 適切にonClick属性を利用しているケースを通したい！ ([cc71417](https://github.com/kufu/eslint-plugin-smarthr/commit/cc7141768f222f45ffbff1aa1dadec1fcfb83dd2))
+
+### [0.5.10](https://github.com/kufu/eslint-plugin-smarthr/compare/v0.5.9...v0.5.10) (2024-04-26)
+
+
+### Bug Fixes
+
+*  <FormControl> 内に三項演算子がある場合に、入力要素が複数あると誤認されないようにする ([#139](https://github.com/kufu/eslint-plugin-smarthr/issues/139)) ([8a234c7](https://github.com/kufu/eslint-plugin-smarthr/commit/8a234c73e140725ff1926d4d23f64c263e46d5d6))
+
 ### [0.5.9](https://github.com/kufu/eslint-plugin-smarthr/compare/v0.5.7...v0.5.9) (2024-04-22)
 
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-smarthr",
-  "version": "0.5.9",
+  "version": "0.5.11",
   "author": "SmartHR",
   "license": "MIT",
   "description": "A sharable ESLint plugin for SmartHR",

package/rules/a11y-delegate-element-has-role-presentation/index.js

@@ -31,6 +31,7 @@ const EXPECTED_NAMES = {
   'Pagination$': 'Pagination$',
   'SideNav$': 'SideNav$',
   'AccordionPanel$': 'AccordionPanel$',
+  'FilterDropdown$': 'FilterDropdown$',
 }
 
 const UNEXPECTED_NAMES = {

package/rules/a11y-input-in-form-control/index.js

@@ -94,6 +94,7 @@ module.exports = {
     const checkAdditionalMultiInputComponents = (name) => additionalMultiInputComponents && name.match(additionalMultiInputComponents)
 
     let formControls = []
+    let conditionalformControls = []
     let checkboxFormControls = []
 
     return {
@@ -102,6 +103,7 @@ module.exports = {
         const nodeName = node.name.name || '';
         const isFormControlInput = nodeName.match(FORM_CONTROL_INPUTS_REGEX)
         const isAdditionalMultiInput = checkAdditionalMultiInputComponents(nodeName)
+        let conditionalExpressions = []
 
         if (isFormControlInput || isAdditionalMultiInput || checkAdditionalInputComponents(nodeName)) {
           let isInMap = false
@@ -162,15 +164,25 @@ module.exports = {
                 if (name) {
                   if (name.match(FROM_CONTROLS_REGEX)) {
                     const hit = formControls.includes(n)
+                    let conditionalHit = false
 
                     if (!hit) {
                       formControls.push(n)
 
+
                       if (isCheckbox) {
                         checkboxFormControls.push(n)
                       }
                     }
 
+                    if (conditionalExpressions.length > 0) {
+                      conditionalHit = conditionalformControls.includes(n)
+
+                      if (!conditionalHit) {
+                        conditionalformControls.push(n)
+                      }
+                    }
+
                     const isMultiInput = isPreMultiple || hit || isInMap
                     const matcherFormControl = name.match(FORM_CONTROL_REGEX)
 
@@ -184,7 +196,7 @@ module.exports = {
  - Fieldsetで同じname属性のラジオボタン全てを囲むことで正しくグループ化され、適切なタイトル・説明を追加出来ます` : ''}${isPureInput ? `
  - 可能なら${nodeName}は${convertComp}への変更を検討してください。難しい場合は ${nodeName} と結びつくlabel要素が必ず存在するよう、マークアップする必要があることに注意してください。` : ''}`,
                         });
-                      } else if (isMultiInput && !openingElement.attributes.find(findRoleGroup)) {
+                      } else if (isMultiInput && !conditionalHit && !openingElement.attributes.find(findRoleGroup)) {
                         context.report({
                           node: n,
                           message: `${name} が複数の入力要素を含んでいます。ラベルと入力要素の紐づけが正しく行われない可能性があるため、以下の方法のいずれかで修正してください。
@@ -245,6 +257,10 @@ module.exports = {
 
                 break
               }
+              case 'ConditionalExpression': {
+                conditionalExpressions.push(n)
+                break
+              }
               case 'VariableDeclarator': {
                 if (n.parent.parent?.type && n.parent.parent.type.match(IGNORE_INPUT_CHECK_PARENT_TYPE)) {
                   const name = n.id.name

package/test/a11y-input-in-form-control.js

@@ -135,6 +135,7 @@ ruleTester.run('a11y-input-in-form-control', rule, {
     { code: '<Fieldset><HogeRadioButtonPanels /></Fieldset>' },
     { code: '<Fieldset><HogeCheckBoxs /></Fieldset>' },
     { code: '<Fieldset><HogeCheckBoxes /></Fieldset>' },
+    { code: '<HogeFormControl>{ dateInput ? <DateInput /> : <Input /> }</HogeFormControl>'},
   ],
   invalid: [
     { code: `import hoge from 'styled-components'`, errors: [ { message: `styled-components をimportする際は、名称が"styled" となるようにしてください。例: "import styled from 'styled-components'"` } ] },
@@ -179,5 +180,7 @@ ruleTester.run('a11y-input-in-form-control', rule, {
     { code: '<HogeFormControl><HogeRadioButtons /></HogeFormControl>', errors: [ { message: invalidRadioInFormControl('HogeRadioButtons') } ] },
     { code: '<HogeFormControl><HogeCheckBoxs /></HogeFormControl>', errors: [ { message: invalidMultiInputsInFormControl() } ] },
     { code: '<HogeFormControl><HogeCheckBoxes /></HogeFormControl>', errors: [ { message: invalidMultiInputsInFormControl() } ] },
+    { code: "<HogeFormControl>{ dateInput ? <DateInput /> : <Input /> }<CheckBox /></HogeFormControl>", errors: [ { message: invalidMultiInputsInFormControl() } ]},
+    { code: "<HogeFormControl><CheckBox />{ dateInput ? <DateInput /> : <Input /> }</HogeFormControl>", errors: [ { message: invalidMultiInputsInFormControl() } ]},
   ]
 })