Opened on 03/07/2017 at 05:15:20 PM

Closed on 03/09/2017 at 02:16:42 PM

#4962 closed defect (fixed)

adblockpluscore test hang

Reported by: hfiguiere Assignee: hfiguiere
Priority: Unknown Milestone:
Module: Core Keywords:
Cc: Blocked By:
Blocking: #3143 Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):



This in on stock Fedora 25. I don't have a macOS env yet to test.

How to reproduce

  1. npm test

Observed behaviour

Test hang:

% npm test

> adblockpluscore@ test /home/hub/source/eyeo/adblockpluscore
> node test_runner.js


I have seen it sometime not hang, but then it doesn't seem, to run any test.

Expected behaviour

Test go through.


It seems to go on a loop where applyElemHideEmulation() gets called infinitely, because loading script for ElemHideEmulation seems to fail.

If I revert to revision c7ae7532578a, the test fail differently, with

Test failed: global failure: 
Failed assertion: SyntaxError: Expected an identifier but found 'rules' instead

Test failed: Element hiding emulation: Verbatim property selector: 
Failed assertion: Died on test #1 global code@file:////home/hub/source/eyeo/adblockpluscore/test/browser/elemHideEmulation.js:123:11: Can't find variable: ElemHideEmulation

I'll look at that as it should work. (with a fix)

Attachments (0)

Change History (6)

comment:1 Changed on 03/07/2017 at 05:51:50 PM by hfiguiere

Revision 2fc6eeda3478 is causing the breakage as it uses syntax that does not go through with the tests... (let,

comment:2 Changed on 03/07/2017 at 06:00:41 PM by hfiguiere

Revision c7ae7532578a cause another breakage by not updating the test after an API change.

comment:3 Changed on 03/07/2017 at 06:20:22 PM by hfiguiere

  • Blocking 3143 added

comment:4 Changed on 03/07/2017 at 06:36:33 PM by hfiguiere

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

comment:5 Changed on 03/09/2017 at 02:13:07 PM by abpbot

A commit referencing this issue has landed:
Issue 4962 - Fix the tests.

comment:6 Changed on 03/09/2017 at 02:16:42 PM by hfiguiere

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

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.