Skip to content

Commit

Permalink
fix(aria-roles): reverted test change as well
Browse files Browse the repository at this point in the history
test change was only for data tables, but we didn't want to keep that type change

Refs: #4371
  • Loading branch information
gaiety-deque committed Apr 25, 2024
1 parent 2d185f6 commit 417e6b1
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions test/commons/table/is-data-table.js
Expand Up @@ -32,14 +32,6 @@ describe('table.isDataTable', function () {
assert.isFalse(axe.commons.table.isDataTable(node));
});

it('should be false if the table has role=application, which is an astract role of structure', function () {
fixture.innerHTML = '<table role="application"></table>';

var node = fixture.querySelector('table');
axe.testUtils.flatTreeSetup(fixture.firstChild);
assert.isFalse(axe.commons.table.isDataTable(node));
});

it('should be true if the table is inside an editable area', function () {
fixture.innerHTML =
'<div contenteditable="true">' +
Expand Down Expand Up @@ -79,6 +71,13 @@ describe('table.isDataTable', function () {
});

describe('should be true if the table has a landmark role', function () {
it('application', function () {
fixture.innerHTML = '<table role="application"></table>';

var node = fixture.querySelector('table');
axe.testUtils.flatTreeSetup(fixture.firstChild);
assert.isTrue(axe.commons.table.isDataTable(node));
});
it('banner', function () {
fixture.innerHTML = '<table role="banner"></table>';

Expand Down

0 comments on commit 417e6b1

Please sign in to comment.