[JS] Removing deprecation warning for tagName and unused imports (Sel…

…eniumHQ#8516) Co-authored-by: Diego Molina <diemol@users.noreply.github.com>
quehanwei · Jul 13, 2020 · 00b70d8 · 00b70d8
1 parent c79fc0c
commit 00b70d8
Show file tree

Hide file tree

Showing 4 changed files with 1 addition and 11 deletions.
diff --git a/javascript/node/selenium-webdriver/lib/by.js b/javascript/node/selenium-webdriver/lib/by.js
@@ -222,7 +222,6 @@ class By {
    *
    * @param {string} name The tag name to search for.
    * @return {!By} The new locator.
-   * @deprecated Use {@link By.css() By.css(tagName)} instead.
    */
   static tagName(name) {
     return By.css(name);

diff --git a/javascript/node/selenium-webdriver/lib/test/fileserver.js b/javascript/node/selenium-webdriver/lib/test/fileserver.js
@@ -17,7 +17,6 @@
 
 'use strict';
 
-const fs = require('fs');
 const http = require('http');
 const path = require('path');
 const url = require('url');

diff --git a/javascript/node/selenium-webdriver/lib/test/index.js b/javascript/node/selenium-webdriver/lib/test/index.js
@@ -17,16 +17,10 @@
 
 'use strict';
 
-const assert = require('assert');
-
 const build = require('./build');
 const fileserver = require('./fileserver');
-const firefox = require('../../firefox');
 const logging = require('../../lib/logging');
-const remote = require('../../remote');
-const safari = require('../../safari');
 const testing = require('../../testing');
-const webdriver = require('../../');
 
 const NO_BUILD = /^1|true$/i.test(process.env['SELENIUM_NO_BUILD']);
 

diff --git a/javascript/node/selenium-webdriver/test/execute_script_test.js b/javascript/node/selenium-webdriver/test/execute_script_test.js
@@ -18,8 +18,6 @@
 'use strict';
 
 const assert = require('assert');
-const {fail} = require('assert');
-
 const {Browser, By, WebElement, error} = require('..');
 const {Pages, ignore, suite} = require('../lib/test');
 
@@ -324,7 +322,7 @@ suite(function(env) {
       it('fails if script took too long', function() {
         return executeTimeOutScript(TOO_LONG_WAIT)
           .then(function() {
-            fail('it should have timed out');
+            assert.fail('it should have timed out');
           }).catch(function(e) {
             if (env.browser.name === Browser.SAFARI) {
               assert.equal(e.name, error.TimeoutError.name);