Merge pull request #707 from jsx-eslint/690-fix

Additional test case for no-redundant-roles

Author: jessebeach

__tests__/src/rules/no-redundant-roles-test.js

@@ -50,6 +50,7 @@ ruleTester.run(`${ruleName}:recommended`, rule, {
 });
 
 const noNavExceptionsOptions = { nav: [] };
+const listException = { ul: ['list'], ol: ['list'] };
 
 ruleTester.run(`${ruleName}:recommended`, rule, {
   valid: alwaysValid
@@ -62,3 +63,17 @@ ruleTester.run(`${ruleName}:recommended`, rule, {
     .map(ruleOptionsMapperFactory(noNavExceptionsOptions))
     .map(parserOptionsMapper),
 });
+
+ruleTester.run(`${ruleName}:recommended (valid list role override)`, rule, {
+  valid: [
+    { code: '<ul role="list" />' },
+    { code: '<ol role="list" />' },
+  ]
+    .map(ruleOptionsMapperFactory(listException))
+    .map(parserOptionsMapper),
+  invalid: [
+    { code: '<ul role="list" />', errors: [expectedError('ul', 'list')] },
+    { code: '<ol role="list" />', errors: [expectedError('ol', 'list')] },
+  ]
+    .map(parserOptionsMapper),
+});