Skip to content

Commit

Permalink
Remove errant Home and End assumption in combobox test
Browse files Browse the repository at this point in the history
  • Loading branch information
stalgiag committed Dec 4, 2024
1 parent 0a84f39 commit 75bf788
Showing 1 changed file with 0 additions and 16 deletions.
16 changes: 0 additions & 16 deletions test/tests/combobox_grid-combo.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,6 @@ const assertAriaLabelledby = require('../util/assertAriaLabelledby');
const assertAttributeValues = require('../util/assertAttributeValues');
const assertAttributeDNE = require('../util/assertAttributeDNE');
const assertAriaRoles = require('../util/assertAriaRoles');
const isMacOS = require('../util/isMacOS');
const exampleFile = 'content/patterns/combobox/examples/grid-combo.html';

const ex = {
Expand Down Expand Up @@ -874,16 +873,8 @@ ariaTest(
const combobox = t.context.session.findElement(By.css(ex.comboboxSelector));
await combobox.sendKeys('a', Key.ARROW_DOWN);

// "ARROW_HOME" is not valid on macos, equivalent is an FN + ARROW_LEFT key chord
// Selenium WebDriver does not support FN keys, so this test is not valid on macos
if (isMacOS()) {
t.pass('Skipping test on macOS - HOME key not supported');
return;
}

// Send key "ARROW_HOME"
await combobox.sendKeys(Key.HOME);

t.true(
await confirmCursorIndex(t, ex.comboboxSelector, 0),
'Cursor should be at index 0 after one ARROW_HOME key'
Expand All @@ -906,13 +897,6 @@ ariaTest(
const combobox = t.context.session.findElement(By.css(ex.comboboxSelector));
await combobox.sendKeys('a', Key.ARROW_DOWN);

// "ARROW_END" is not valid on macos, equivalent is an FN + ARROW_RIGHT key chord
// Selenium WebDriver does not support FN keys, so this test is not valid on macos
if (isMacOS()) {
t.pass('Skipping test on macOS - END key not supported');
return;
}

// Send key "END_ARROW"
await combobox.sendKeys(Key.END);

Expand Down

0 comments on commit 75bf788

Please sign in to comment.