Issue 4206 - Add opportunity to run specific test/method only

Issue 4206 - Add opportunity to run specific test/method only

[anton, 2016-06-30 07:44:22]

Added custom ant target and updated README.

Since the changes are minor and Rene is pretty busy now Felix feel free to
review it instead of Rene.

[Felix Dahlke, 2016-06-30 09:14:34]

Sure, I can do it. I'm rusty, but this is simple enough :) Moved René to CC - if
he leaves a comment he'll automatically be a reviewer again.

Looks good, just found a nit.

https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
File libadblockplus-android-tests/build.xml (right):

https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
libadblockplus-android-tests/build.xml:110: <arg value="class" />
Nit: Seems like the indentation is off here - shouldn't it be indented to the
same level as the line above? Same for the line below.

[anton, 2016-06-30 09:18:03]

https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
File libadblockplus-android-tests/build.xml (right):

https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
libadblockplus-android-tests/build.xml:110: <arg value="class" />
On 2016/06/30 09:14:34, Felix Dahlke wrote:
> Nit: Seems like the indentation is off here - shouldn't it be indented to the
> same level as the line above? Same for the line below.

since "-e" stands for "then will follow key and value" i keep the same code
style like in android ndk's build.xml where the following key/value have
additional indentation of few spaces.

[anton, 2016-06-30 09:20:43]

On 2016/06/30 09:18:03, anton wrote:
>
https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
> File libadblockplus-android-tests/build.xml (right):
> 
>
https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
> libadblockplus-android-tests/build.xml:110: <arg value="class" />
> On 2016/06/30 09:14:34, Felix Dahlke wrote:
> > Nit: Seems like the indentation is off here - shouldn't it be indented to
the
> > same level as the line above? Same for the line below.
> 
> since "-e" stands for "then will follow key and value" i keep the same code
> style like in android ndk's build.xml where the following key/value have
> additional indentation of few spaces.

https://android.googlesource.com/platform/sdk/+/98b631c6b3765be96b3707ba7aa54...

[Felix Dahlke, 2016-06-30 09:27:50]

LGTM

https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
File libadblockplus-android-tests/build.xml (right):

https://codereview.adblockplus.org/29347154/diff/29347155/libadblockplus-andr...
libadblockplus-android-tests/build.xml:110: <arg value="class" />
On 2016/06/30 09:18:03, anton wrote:
> On 2016/06/30 09:14:34, Felix Dahlke wrote:
> > Nit: Seems like the indentation is off here - shouldn't it be indented to
the
> > same level as the line above? Same for the line below.
> 
> since "-e" stands for "then will follow key and value" i keep the same code
> style like in android ndk's build.xml where the following key/value have
> additional indentation of few spaces.

Oh right, I remember :)