Skip to content

Commit 3a47f2f

Browse files
authored
test: remove flakey IE integration test (#2796)
1 parent 7d22e62 commit 3a47f2f

File tree

1 file changed

+16
-19
lines changed
  • packages/integration-tests/src/components/events/test-mouseover/integration

1 file changed

+16
-19
lines changed

packages/integration-tests/src/components/events/test-mouseover/integration/mouseover.spec.js

Lines changed: 16 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -7,18 +7,18 @@
77
const assert = require('assert');
88
const URL = '/mouseover';
99

10-
describe('mouseover', () => {
11-
beforeEach(async () => {
12-
await browser.url(URL);
13-
});
10+
if (process.env.COMPAT === 'false') {
11+
describe('mouseover', () => {
12+
beforeEach(async () => {
13+
await browser.url(URL);
14+
});
1415

15-
it('should be able to trigger programmatic mouseover', async () => {
16-
const component = await browser.$('integration-mouseover');
17-
const elementToHover = await component.shadow$('.mouseover');
18-
// This might seem like a roundabout way to mouse over an element, but this puts ChromeDriver into the
19-
// code path where it calls elementsFromPoint, which is what we're trying to test:
20-
// https://github.com/bayandin/chromedriver/blob/ad6ede8/js/get_element_location.js#L122
21-
if (typeof browser.performActions === 'function') {
16+
it('should be able to trigger programmatic mouseover', async () => {
17+
const component = await browser.$('integration-mouseover');
18+
const elementToHover = await component.shadow$('.mouseover');
19+
// This might seem like a roundabout way to mouse over an element, but this puts ChromeDriver into the
20+
// code path where it calls elementsFromPoint, which is what we're trying to test:
21+
// https://github.com/bayandin/chromedriver/blob/ad6ede8/js/get_element_location.js#L122
2222
await browser.performActions([
2323
{
2424
type: 'pointer',
@@ -37,13 +37,10 @@ describe('mouseover', () => {
3737
],
3838
},
3939
]);
40-
} else {
41-
// IE driver does not support performActions, fall back to moveTo()
42-
await elementToHover.moveTo();
43-
}
4440

45-
const successElement = await component.shadow$('.hovering');
46-
const exists = await successElement.isExisting();
47-
assert.strictEqual(exists, true);
41+
const successElement = await component.shadow$('.hovering');
42+
const exists = await successElement.isExisting();
43+
assert.strictEqual(exists, true);
44+
});
4845
});
49-
});
46+
}

0 commit comments

Comments
 (0)