Opened 2 years ago

Closed 2 years ago

Last modified 2 years ago

#6715 closed defect (fixed)

The adblockpluschrome unit tests are failing

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

https://codereview.adblockplus.org/29795579/

Description (last modified by kzar)

Environment

Adblock Plus master 3.2 (code freeze) (rev 7122a201b554) on Chrome.

How to reproduce

  1. build the devenv for Chrome
  2. load the extension in Chrome
  3. Run the qunit test:
    1. open the option page
    2. In the JS console enter location.href = "qunit/index.html";

Observed behaviour

Test show failure (missing getDecodedHostname() and stringifyURL(); punycode in URL).

Expected behaviour

All test pass.

Notes

  • Not tested on Firefox. I don't think it matters.
  • This is related to issue #6647: we removed the missing functions and we no longer convert URLs with Unicode.

Hints for testers

No need to test this one, it's just a change to the unit tests which aren't included in the release.

Change History (7)

comment:1 Changed 2 years ago by hfiguiere

  • Review URL(s) modified (diff)

comment:2 Changed 2 years ago by hfiguiere

  • Status changed from new to reviewing

comment:3 Changed 2 years ago by hfiguiere

  • Description modified (diff)

comment:4 Changed 2 years ago by kzar

  • Cc kzar sebastian Ross added
  • Component changed from Adblock-Plus-for-Chromium to Platform
  • Description modified (diff)
  • Milestone set to Adblock-Plus-3.2-for-Chrome-Opera-Firefox
  • Priority changed from Unknown to P1
  • Ready set
  • Summary changed from WebExtension qunit tests fail to The adblockpluschrome unit tests are failing

comment:5 Changed 2 years ago by abpbot

A commit referencing this issue has landed:
Issue 6715 - Fix the url qunit tests.

comment:6 Changed 2 years ago by hfiguiere

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

comment:7 Changed 2 years ago by Ross

  • Tester changed from Unknown to Ross
  • Verified working set

These pass again.

adblockpluschrome / master / 2069:56f54c897e3a
Chrome 67 / Windows 10

Note: See TracTickets for help on using tickets.