Noissue - Fixed test pages integration tests after change of subscription URL

test/wrappers/pages.js

Index: test/wrappers/pages.js
===================================================================
--- a/test/wrappers/pages.js
+++ b/test/wrappers/pages.js
@@ -17,7 +17,7 @@
 
 "use strict";
 
-const TEST_PAGES_URL = "https://testpages.adblockplus.org/";
+const TEST_PAGES_URL = "https://testpages.adblockplus.org/en/";

[Sebastian Noack, 2018/09/25 16:12:06]

Today a change landed for the test pages that caused all URLs to be constructed
rather than hard-coded redundantly. As a side effect the subscription URL
changed from http://testpages.adblockplus.org/abp-testcase-subscription.txt to
https://testpages.adblockplus.org/en/abp-testcase-subscription.txt.

 
 const assert = require("assert");
 const Jimp = require("jimp");
@@ -79,8 +79,7 @@
                                        ignoreDisabled: true,
                                        downloadable: true}).then(subs =>
             subs.some(s =>
-              s.url.endsWith("${TEST_PAGES_URL.replace(/^[^:]+/, "")}" +
-                             "abp-testcase-subscription.txt")
+              s.url == "${TEST_PAGES_URL}abp-testcase-subscription.txt"

[Sebastian Noack, 2018/09/25 16:12:06]

This was neccessary before, but no longer, because previously the protocol the
URL in the subscription link used http:// while the test pages themselves were
delivered over https://.

             )
           ).then(
             res => callback([res, null]),