fix: don't flag invalid roles in unsupportedrole

lib/checks/aria/unsupportedrole.js

@@ -1,3 +1 @@
-return !axe.commons.aria.isValidRole(node.getAttribute('role'), {
-	flagUnsupported: true
-});
+return axe.commons.aria.isUnsupportedRole(node.getAttribute('role'));

lib/commons/aria/roles.js

@@ -21,6 +21,19 @@ aria.isValidRole = function(
 	return allowAbstract ? true : roleDefinition.type !== 'abstract';
 };
 
+/**
+ * Check if a given role is unsupported
+ * @method isUnsupportedRole
+ * @memberof axe.commons.aria
+ * @instance
+ * @param {String} role The role to check
+ * @return {Boolean}
+ */
+aria.isUnsupportedRole = function(role) {
+	const roleDefinition = aria.lookupTable.role[role];
+	return roleDefinition ? roleDefinition.unsupported : false;
+};
+
 /**
  * Get the roles that get name from the element's contents
  * @method getRolesWithNameFromContents

test/checks/aria/unsupportedrole.js

@@ -12,7 +12,7 @@ describe('unsupportedrole', function() {
 			type: 'widget',
 			unsupported: true
 		};
-		fixture.innerHTML = '<div id="target" role="mccheddarton">Contents</div>';
+		fixture.innerHTML = '<div id="target" role="mcheddarton">Contents</div>';
 		var node = fixture.querySelector('#target');
 		assert.isTrue(checks.unsupportedrole.evaluate(node));
 	});
@@ -22,4 +22,10 @@ describe('unsupportedrole', function() {
 		var node = fixture.querySelector('#target');
 		assert.isFalse(checks.unsupportedrole.evaluate(node));
 	});
+
+	it('should return false if applied to an invalid role', function() {
+		fixture.innerHTML = '<input id="target" role="foo">';
+		var node = fixture.querySelector('#target');
+		assert.isFalse(checks.unsupportedrole.evaluate(node));
+	});
 });

test/commons/aria/roles.js

@@ -25,6 +25,39 @@ describe('aria.isValidRole', function() {
 	});
 });
 
+describe('aria.isUnsupportedRole', function() {
+	'use strict';
+
+	it('should return true if the role is unsupported', function() {
+		var orig = axe.commons.aria.lookupTable.role;
+		axe.commons.aria.lookupTable.role = {
+			cats: {
+				unsupported: true
+			}
+		};
+		assert.isTrue(axe.commons.aria.isUnsupportedRole('cats'));
+		axe.commons.aria.lookupTable.role = orig;
+	});
+
+	it('should return false if the role is supported', function() {
+		var orig = axe.commons.aria.lookupTable.role;
+		axe.commons.aria.lookupTable.role = {
+			cats: {
+				unsupported: false
+			}
+		};
+		assert.isFalse(axe.commons.aria.isUnsupportedRole('cats'));
+		axe.commons.aria.lookupTable.role = orig;
+	});
+
+	it('should return false if the role is invalid', function() {
+		var orig = axe.commons.aria.lookupTable.role;
+		axe.commons.aria.lookupTable.role = {};
+		assert.isFalse(axe.commons.aria.isUnsupportedRole('cats'));
+		axe.commons.aria.lookupTable.role = orig;
+	});
+});
+
 describe('aria.getRolesWithNameFromContents', function() {
 	'use strict';