Issue 4031 - Implement tests for libadblockplus-android

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
@@ skipping 51 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());

[Felix Dahlke, 2016/09/12 13:52:13]

Nit: Missing space before comma and `10`? Same thing again below.

[Felix Dahlke, 2016/09/13 08:30:14]

What about this comment?

[Felix Dahlke, 2016/09/13 10:51:44]

You don't seem to have noticed this comment, hm?

[anton, 2016/09/13 11:00:22]

My bad, sorry. Going to fix it right now

+      jsEngine.evaluate("foo.responseHeaders['Content-Type'].substr(0, 10)").asString());
     assertTrue(jsEngine.evaluate("foo.responseHeaders['location']").isUndefined());
   }
 
   @Test
   public void testXMLHttpRequest()
   {
     jsEngine.evaluate(
       "var result;\n" +
       "var request = new XMLHttpRequest();\n" +
       "request.open('GET', 'https://easylist-downloads.adblockplus.org/easylist.txt');\n" +
@@ skipping 17 matching lines @@
     } while (jsEngine.evaluate("result").isUndefined());
 
     assertEquals(
       ServerResponse.NsStatus.OK.getStatusCode(),
       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());
+      jsEngine.evaluate("request.getResponseHeader('Content-Type').substr(0, 10)").asString());
     assertTrue(jsEngine.evaluate("request.getResponseHeader('Location')").isNull());
   }
 }