fix(rule): Layout-table does not match presentation / none roles (#828)

* fix(layout-table): check for presentation, none roles and isfocusable for matches

* fix(layout-table): tests for layout table matches for focusable and non-presentation/none tables

Author: waabid

lib/rules/layout-table-matches.js

@@ -1 +1,5 @@
-return !axe.commons.table.isDataTable(node);
+var role = (node.getAttribute('role') || '').toLowerCase();
+
+return  !((role === 'presentation' || role === 'none') &&
+        !axe.commons.dom.isFocusable(node)) &&
+        !axe.commons.table.isDataTable(node);

test/rule-matches/layout-table-matches.js

@@ -0,0 +1,52 @@
+describe('layout-table-matches', function () {
+	'use strict';
+
+	var fixture = document.getElementById('fixture');
+	var fixtureSetup = axe.testUtils.fixtureSetup;
+	var rule;
+
+	beforeEach(function () {
+		rule = axe._audit.rules.find(function (rule) {
+			return rule.id === 'layout-table';
+		});
+	});
+
+	afterEach(function () {
+		fixture.innerHTML = '';
+	});
+
+	it('returns false for element that is not focusable and has presentation role', function () {
+		fixtureSetup('<table role="presentation"></table>');
+		var target = fixture.querySelector('table');
+
+		assert.isFalse(rule.matches(target));
+	});
+
+	it('returns false for element that is not focusable and has none role', function () {
+		fixtureSetup('<table role="none"></table>');
+		var target = fixture.querySelector('table');
+
+		assert.isFalse(rule.matches(target));
+	});
+
+	it('returns trie for element that is a table without presentation/none role', function () {
+		fixtureSetup('<table></table>');
+		var target = fixture.querySelector('table');
+
+		assert.isTrue(rule.matches(target));
+	});
+
+	it('returns true for element that is focusable and has none role', function () {
+		fixtureSetup('<table role="none" tabindex="0"></table>');
+		var target = fixture.querySelector('table');
+
+		assert.isTrue(rule.matches(target));
+	});
+
+	it('returns true for element that is focusable and has presentation role', function () {
+		fixtureSetup('<table role="presentation" tabindex="0"></table>');
+		var target = fixture.querySelector('table');
+
+		assert.isTrue(rule.matches(target));
+	});
+});