Opened on 09/25/2018 at 07:51:34 PM

Closed on 09/26/2018 at 10:24:19 PM

Last modified on 10/08/2019 at 06:05:46 PM

#6985 closed defect (fixed)

Running npm test fails on Windows for adblockpluschrome

Reported by: geo Assignee: geo
Priority: P3 Milestone: Adblock-Plus-3.4-for-Chrome-Opera-Firefox
Module: Automation Keywords:
Cc: sebastian, Ross Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Ross Verified working: yes
Review URL(s):

https://codereview.adblockplus.org/29892585/

Description

Environment

Windows 10
revision: git f2d0c16 hg e7abbf2e9d9b
nodejs: v8.11.3

How to reproduce

  1. Run npm install from PowerShell
  2. Run npm test from PowerShell

Observed behaviour

The script throws the following errors:

1) chrome
       "before all" hook:
     Error: The ChromeDriver could not be found on the current PATH. Please download the latest version of the ChromeDriver from http://chromedriver.storage.googleapis.com/index.html and ensure it can be found on your PATH.
      at new ServiceBuilder (node_modules\selenium-webdriver\chrome.js:215:13)
      at getDefaultService (node_modules\selenium-webdriver\chrome.js:304:22)
      at Function.createSession (node_modules\selenium-webdriver\chrome.js:752:44)
      at createDriver (node_modules\selenium-webdriver\index.js:170:33)
      at Builder.build (node_modules\selenium-webdriver\index.js:642:16)
      at Object.exports.getDriver (test\browsers\chromium.js:49:6)
      at Promise.all.then (test\all.js:53:30)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  2) chrome
       "after all" hook:
     TypeError: Cannot read property 'quit' of undefined
      at Context.<anonymous> (test\all.js:80:19)

  3) gecko
       "before all" hook:
     Error: The geckodriver.exe executable could not be found on the current PATH. Please download the latest version from https://github.com/mozilla/geckodriver/releases/ and ensure it can be found on your PATH.
      at findGeckoDriver (node_modules\selenium-webdriver\firefox\index.js:354:11)
      at new ServiceBuilder (node_modules\selenium-webdriver\firefox\index.js:446:22)
      at Function.createSession (node_modules\selenium-webdriver\firefox\index.js:516:21)
      at createDriver (node_modules\selenium-webdriver\index.js:170:33)
      at Builder.build (node_modules\selenium-webdriver\index.js:645:16)
      at Object.exports.getDriver (test\browsers\firefox.js:43:6)
      at Promise.all.then (test\all.js:53:30)
      at <anonymous>
      at process._tickCallback (internal/process/next_tick.js:188:7)

  4) gecko
       "after all" hook:
     TypeError: Cannot read property 'quit' of undefined
      at Context.<anonymous> (test\all.js:80:19)
npm ERR! code ELIFECYCLE
npm ERR! errno 4
npm ERR! adblockpluschrome@ test-only: `mocha`
npm ERR! Exit status 4
npm ERR!
npm ERR! Failed at the adblockpluschrome@ test-only script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

Expected behaviour

The script should run without errors.

Attachments (0)

Change History (10)

comment:1 Changed on 09/25/2018 at 07:59:03 PM by sebastian

  • Cc Ross added
  • Priority changed from Unknown to P3
  • Ready set

Ross, this varies from the error you got last time. Can you try again, and let us know if you get the same error now, or which error you get instead (as of current master)?

comment:2 Changed on 09/26/2018 at 12:31:58 PM by sebastian

  • Component changed from Platform to Automation

comment:3 Changed on 09/26/2018 at 03:59:06 PM by geo

  • Owner set to geo

comment:4 Changed on 09/26/2018 at 09:30:38 PM by geo

  • Review URL(s) modified (diff)

comment:5 Changed on 09/26/2018 at 10:24:03 PM by abpbot

comment:6 Changed on 09/26/2018 at 10:24:19 PM by sebastian

  • Milestone set to Adblock-Plus-3.4-for-Chrome-Opera-Firefox
  • Resolution set to fixed
  • Status changed from new to closed

comment:7 Changed on 10/01/2018 at 12:02:42 PM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Fixed. Now working for both Chrome and Firefox on Windows. Thanks!

comment:8 Changed on 03/15/2019 at 06:18:50 AM by takken3

spam

Last edited on 10/08/2019 at 06:05:43 PM by kzar

comment:9 Changed on 05/24/2019 at 05:42:31 AM by whiteson

spam

Last edited on 10/08/2019 at 06:05:46 PM by kzar

comment:10 Changed on 09/26/2019 at 04:40:01 AM by moffu

spam

Last edited on 10/08/2019 at 06:05:41 PM by kzar

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from geo.
 
Note: See TracTickets for help on using tickets.