Opened on 06/01/2018 at 05:56:13 PM

Closed on 06/05/2018 at 03:45:54 AM

Last modified on 07/25/2018 at 08:45:30 AM

#6720 closed defect (fixed)

qunit tests are packaged in the extension (Chrome and Firefox)

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

Description (last modified by kzar)


With adblockpluschrome master (rev 9dd0a8081209), the qunit tests are packaged in the extension archive file.

How to reproduce

  1. In an adblockpluschrome checkout, do ./ build -t gecko or ./ build -t chrome
  2. Look at the content of the respective archives: using unzip -l | grep qunit

Observed behaviour

Both qunit/tests.js and qunit/ are packaged.

Expected behaviour

I'd expect them to not be packaged.


In buildtools/, the create_bundles function creates the qunit/tests.js bundle for the webpack configuration, unconditionally, based on the manifest file. Then again, if bundle_tests is true, it adds another bundle in the webpack configuration, by listing the qunit/tests directory content.

Notes for testers

Check that the extension file (xpi or zip) doesn't have qunit/* in it.

Attachments (0)

Change History (7)

comment:1 Changed on 06/01/2018 at 06:03:19 PM by hfiguiere

  • Cc kzar sebastian tlucas added
  • Owner set to hfiguiere

comment:2 Changed on 06/01/2018 at 06:04:30 PM by hfiguiere

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

comment:3 Changed on 06/01/2018 at 06:06:20 PM by hfiguiere

  • Description modified (diff)

comment:4 Changed on 06/04/2018 at 05:44:25 PM by kzar

  • Description modified (diff)
  • Priority changed from Unknown to P2
  • Ready set

Well spotted, thanks the issue is good too. I can reproduce as described.

comment:5 Changed on 06/05/2018 at 03:44:41 AM by abpbot

A commit referencing this issue has landed:
Issue 6720 - Do not package the qunit test files.

comment:6 Changed on 06/05/2018 at 03:45:54 AM by hfiguiere

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

comment:7 Changed on 07/25/2018 at 08:45:30 AM by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

Fixed. The created archives no longer contain quint dir/files.

adblockpluschrome / 2093:20ad2e20df42

Add Comment

Modify Ticket

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