Provide exceptions for li to row and ul to radiogroup in the recommended config for no-noninteractive-element-to-interactive-role

Author: jessebeach

__tests__/src/rules/no-noninteractive-element-to-interactive-role-test.js

@@ -433,16 +433,19 @@ ruleTester.run(`${ruleName}:recommended`, rule, {
     ...alwaysValid,
     { code: '<ul role="menu" />;' },
     { code: '<ul role="menubar" />;' },
+    { code: '<ul role="radiogroup" />;' },
     { code: '<ul role="tablist" />;' },
     { code: '<ul role="tree" />;' },
     { code: '<ul role="treegrid" />;' },
     { code: '<ol role="menu" />;' },
     { code: '<ol role="menubar" />;' },
+    { code: '<ol role="radiogroup" />;' },
     { code: '<ol role="tablist" />;' },
     { code: '<ol role="tree" />;' },
     { code: '<ol role="treegrid" />;' },
     { code: '<li role="tab" />;' },
     { code: '<li role="menuitem" />;' },
+    { code: '<li role="row" />;' },
     { code: '<li role="treeitem" />;' },
   ]
     .map(ruleOptionsMapperFactory(recommendedOptions))
@@ -462,16 +465,19 @@ ruleTester.run(`${ruleName}:strict`, rule, {
     ...neverValid,
     { code: '<ul role="menu" />;', errors: [expectedError] },
     { code: '<ul role="menubar" />;', errors: [expectedError] },
+    { code: '<ul role="radiogroup" />;', errors: [expectedError] },
     { code: '<ul role="tablist" />;', errors: [expectedError] },
     { code: '<ul role="tree" />;', errors: [expectedError] },
     { code: '<ul role="treegrid" />;', errors: [expectedError] },
     { code: '<ol role="menu" />;', errors: [expectedError] },
     { code: '<ol role="menubar" />;', errors: [expectedError] },
+    { code: '<ol role="radiogroup" />;', errors: [expectedError] },
     { code: '<ol role="tablist" />;', errors: [expectedError] },
     { code: '<ol role="tree" />;', errors: [expectedError] },
     { code: '<ol role="treegrid" />;', errors: [expectedError] },
     { code: '<li role="tab" />;', errors: [expectedError] },
     { code: '<li role="menuitem" />;', errors: [expectedError] },
+    { code: '<li role="row" />;', errors: [expectedError] },
     { code: '<li role="treeitem" />;', errors: [expectedError] },
   ].map(parserOptionsMapper),
 });

src/index.js

@@ -72,9 +72,9 @@ module.exports = {
         'jsx-a11y/no-noninteractive-element-to-interactive-role': [
           'error',
           {
-            ul: ['menu', 'menubar', 'tablist', 'tree', 'treegrid'],
-            ol: ['menu', 'menubar', 'tablist', 'tree', 'treegrid'],
-            li: ['menuitem', 'tab', 'treeitem'],
+            ul: ['listbox', 'menu', 'menubar', 'radiogroup', 'tablist', 'tree', 'treegrid'],
+            ol: ['listbox', 'menu', 'menubar', 'radiogroup', 'tablist', 'tree', 'treegrid'],
+            li: ['menuitem', 'option', 'row', 'tab', 'treeitem'],
             table: ['grid'],
             td: ['gridcell'],
           },