Opened 2 years ago

Last modified 2 years ago

#6887 closed change

Run qunit tests in adblockpluschrome in chrome — at Version 2

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: 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 (2)

comment:1 Changed 2 years ago by tlucas

  • Description modified (diff)

comment:2 Changed 2 years ago by tlucas

  • Description modified (diff)
Note: See TracTickets for help on using tickets.