Skip to content

Commit 8215a1e

Browse files
committed
[javascript] Removing comment as we are opening a browser per test
It is not ideal to open a browser per test but this seems to make tests in Chrome run more stable.
1 parent 8301f18 commit 8215a1e

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

javascript/node/selenium-webdriver/test/actions_test.js

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,6 @@
1818
'use strict'
1919

2020
const assert = require('assert')
21-
const error = require('../lib/error')
2221
const fileServer = require('../lib/test/fileserver')
2322
const { ignore, Pages, suite } = require('../lib/test')
2423
const { Key, Origin } = require('../lib/input')
@@ -107,12 +106,6 @@ suite(function (env) {
107106
assert.strictEqual(await box.getAttribute('class'), 'blue')
108107
})
109108

110-
// For some reason for Chrome 75 we need to wrap this test in an extra
111-
// describe for the afterEach hook above to properly clear action sequences.
112-
// This appears to be a quirk of the timing around mocha tests and not
113-
// necessarily a bug in the chromedriver.
114-
// TODO(jleyba): dig into this more so we can remove this hack.
115-
// describe('dragAndDrop()', function () {
116109
it('dragAndDrop()', async function () {
117110
await driver.get(fileServer.whereIs('/data/actions/drag.html'))
118111

@@ -130,7 +123,6 @@ suite(function (env) {
130123
assert.strictEqual(await slide.getCssValue('left'), '206px')
131124
assert.strictEqual(await slide.getCssValue('top'), '1px')
132125
})
133-
// })
134126

135127
it('move()', async function () {
136128
await driver.get(fileServer.whereIs('/data/actions/drag.html'))

0 commit comments

Comments
 (0)