Opened 16 months ago

Closed 15 months ago

Last modified 14 months ago

#6887 closed change (fixed)

Run qunit tests in adblockpluschrome in chrome

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

https://codereview.adblockplus.org/29866577

Description (last modified by tlucas)

Background

In #6717, we added a feature to automatically run the integrated qunit tests in a headless Firefox instance with npm test. After we switched to mocha as our test suite (#6884), we can now easily setup the same test for multiple browsers.

What to change

  • Add chrome support to adblockpluschromes nodejs environment.
  • Change the "test" script in package.json to build a devenv for the target browser(s).
  • Let npm test (with no arguments) run the qunit tests on both Firefox(headless) and Chrome.
  • Let npm test's target Browser be switchable with arguments analog to the targets in our buildtools, analog to mocha's usage i.e.:
 $ npm test #Run tests in all supported browsers
 $ npm test -- -g gecko #Run tests in Firefox only
 $ npm test -- -g chrome #Run tests in Chrome only
 $ npm test -- -g "[gecko|chrome]" #Run tests in both Firefox and Chrome only

Change History (7)

comment:1 Changed 16 months ago by tlucas

  • Description modified (diff)

comment:2 Changed 16 months ago by tlucas

  • Description modified (diff)

comment:3 Changed 16 months ago by tlucas

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

comment:4 Changed 16 months ago by tlucas

  • Blocking 6890 added

comment:5 Changed 15 months ago by abpbot

A commit referencing this issue has landed:
Issue 6887 - add Chrome to "npm test"

comment:6 Changed 15 months ago by tlucas

  • Resolution set to fixed
  • Status changed from reviewing to closed

comment:7 Changed 14 months ago by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Done. Working as expected.

adblockpluschrome / master / 2175:02d136073262

Note: See TracTickets for help on using tickets.