Issue 4031 - Implement tests for libadblockplus-android

README.md

 Adblock Plus Library for Android
 ========================
 
 An Android library project that runs a proxy to block ads.
 
 Updating the dependencies
 -------------------------
 
 Adblock Plus Library for Android has dependencies that aren't in this repository.
 To update those, call:
 
     ./ensure_dependencies.py
 
+## Library
+
 Building with Ant
 ------------------
 
 ### Requirements
 
-- [The Android SDK](http://developer.android.com/sdk)
-- [The Android NDK](https://developer.android.com/tools/sdk/ndk)
-- [Ant](http://ant.apache.org)
+* [The Android SDK](http://developer.android.com/sdk)
+* [The Android NDK](https://developer.android.com/tools/sdk/ndk)
+* [Ant](http://ant.apache.org)
 
 ### Building
 
 In the 'libadblockplus-android' directory create the file _local.properties_ and set
 _sdk.dir_ and _ndk.dir_ to where you installed it, e.g.:
 
     sdk.dir = /some/where/sdk
     ndk.dir = /some/where/ndk
 
 Then run:
 
     ant debug
 
 Building with Maven
 -------------------
 
 ### Requirements
 
 All 'Building with Ant' requirements and additional requirements:
 
-- [Maven](https://maven.apache.org)
+* [Maven](https://maven.apache.org)
 
 ### Building
 
 Set environment variable ANDROID_HOME to your Android SDK directory or pass it in command-line (below).
 In the 'libadblockplus-android' directory run:
 
         mvn clean install [-Dandroid.sdk.path=/some/where/sdk]
 
 This will generate *.aar library artifact in the 'target' directory. 
+
+## Library tests
+
+### Requirements
+
+Make sure _Library_ requirements are present.
+
+### Building
+
+Set ANDROID_HOME environment variable to your Android SDK directory.
+
+In the 'libadblockplus-android-tests' run:

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

Nit: "In the 'libadblockplus-android-tests' directory"?

[anton, 2016/09/13 05:49:08]

Yes, it's "Library tests" README section and it's about the tests. We're having
"libadblockplus-android" directory with jni-glue files and
"libadblockplus-android-tests" with the tests. We need to run in
"libadblockplus-android-tests" to build both tests and dependencies (jni glue)
so it's not mistyping

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

Just to clarify, what I meant here is that the sentence sounds weird this way:
"In the $directory_name run", so I suggested to write "In the $directory_name
directory run". You also say "In the 'libadblockplus-android-tests' directory
run" below, so this sentence here seemed like it's just missing a "directory".

[anton, 2016/09/13 10:44:37]

Acknowledged.

+
+    ant instrument
+
+### Testing
+
+1. Connect an Android device or start the Android Emulator.
+2. In the 'libadblockplus-android-tests' directory run:
+
+    ant instrument install test
+
+to build instrumentation tests app and perform testing or run:
+
+    ant test
+
+to run installed instrumentation tests app

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

Nit: Missing period? :D

[anton, 2016/09/13 05:49:08]

did not catch that. it can be called "instrumentation tests app". it's an
application that has instrumented code and actually the whole app is tests

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

I know, I'm just saying that there appears to be a period (".") missing at the
end of the sentence.