Issue 4202 - Fix failing AndroidWebRequestTest.testXMLHttpRequest

libadblockplus-android-tests/src/org/adblockplus/libadblockplus/tests/AndroidWebRequestTest.java

 /*
  * This file is part of Adblock Plus <https://adblockplus.org/>,
  * Copyright (C) 2006-2016 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
  *
  * You should have received a copy of the GNU General Public License
  * along with Adblock Plus.  If not, see <http://www.gnu.org/licenses/>.
  */
 
 package org.adblockplus.libadblockplus.tests;
 
 import org.adblockplus.android.AndroidWebRequest;
+import org.adblockplus.libadblockplus.FilterEngine;
 import org.adblockplus.libadblockplus.JsValue;
 import org.adblockplus.libadblockplus.ServerResponse;
 
 import org.junit.Test;
 
 public class AndroidWebRequestTest extends BaseJsTest
 {
     @Override
     protected void setUp() throws Exception
     {
@@ skipping 30 matching lines @@
         assertEquals(
             "[Adblock Plus ",
             jsEngine.evaluate("foo.responseText.substr(0, 14)").asString());
         JsValue jsHeaders = jsEngine.evaluate("foo.responseHeaders");
         assertNotNull(jsHeaders);
         assertFalse(jsHeaders.isUndefined());
         assertFalse(jsHeaders.isNull());
         assertTrue(jsHeaders.isObject());
         assertEquals(
             "text/plain",
-            jsEngine.evaluate("foo.responseHeaders['Content-Type'].substr(0,10)").asString());
+            jsEngine.evaluate("foo.responseHeaders['content-type'].substr(0,10)").asString());
         assertTrue(jsEngine.evaluate("foo.responseHeaders['location']").isUndefined());
     }
 
     @Test
     public void testXMLHttpRequest()
     {
+        // creating not used anywhere FilterEngine object is not as useless as it seems:
+        // it loads compat.js JsEngine to add XMLHttpRequest class support
+        new FilterEngine(jsEngine);
+
         jsEngine.evaluate(
             "var result;\n" +
             "var request = new XMLHttpRequest();\n" +
             "request.open('GET', 'https://easylist-downloads.adblockplus.org/easylist.txt');\n" +
             "request.setRequestHeader('X', 'Y');\n" +
             "request.setRequestHeader('X2', 'Y2');\n" +
             "request.overrideMimeType('text/plain');\n" +
             "request.addEventListener('load',function() {result=request.responseText;}, false);\n" +
             "request.addEventListener('error',function() {result='error';}, false);\n" +
             "request.send(null);");
@@ skipping 14 matching lines @@
             jsEngine.evaluate("request.channel.status").asLong());
 
         assertEquals(200l, jsEngine.evaluate("request.status").asLong());
         assertEquals("[Adblock Plus ", jsEngine.evaluate("result.substr(0, 14)").asString());
         assertEquals(
             "text/plain",
             jsEngine.evaluate("request.getResponseHeader('Content-Type').substr(0,10)").asString());
         assertTrue(jsEngine.evaluate("request.getResponseHeader('Location')").isNull());
     }
 }