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

https://codereview.adblockplus.org/29378674/

Description

Environment

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

elemHideEmulation.js

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

Expected behaviour

Test go through.

Diagnostic

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
  file:////home/hub/source/eyeo/adblockpluscore/chrome/content/elemHideEmulation.js:86

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
  applyElemHideEmulation@file:////home/hub/source/eyeo/adblockpluscore/test/browser/elemHideEmulation.js:96:48
  file:////home/hub/source/eyeo/adblockpluscore/test/browser/elemHideEmulation.js:129:27

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, for..in)

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
Action
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.