Updated Robolectric documentation.

Updated Robolectric documentation.

Now that change to expose resources to Robolectric tests has
landed, I wanted to update the documentation to point out to
people they need to specify "package_name" GN variable to get
it to work (since that is definitely not obvious).

You can also (theoretically) now specify the AndroidManifest.xml
file to Robolectric (Robolectric will grab the package name from
there). However, no tests currently do this.

Review-Url: https://codereview.chromium.org/2882243003
Cr-Commit-Position: refs/heads/master@{#472156}
Committed: https://chromium.googlesource.com/chromium/src/+/5e8650321a3058567af36d42faf2618d93743e59

[mikecase (-- gone --), 2017-05-15 22:38:23]

mikecase@chromium.org changed reviewers:
+ jbudorick@chromium.org, yolandyan@chromium.org

[jbudorick, 2017-05-15 22:56:53]

lgtm w/ nits

https://codereview.chromium.org/2882243003/diff/1/testing/android/docs/junit.md
File testing/android/docs/junit.md (right):

https://codereview.chromium.org/2882243003/diff/1/testing/android/docs/junit....
testing/android/docs/junit.md:57: your own. They work like this; whenever an
object is instantiated within a
nit: this; -> this:

https://codereview.chromium.org/2882243003/diff/1/testing/android/docs/junit....
testing/android/docs/junit.md:58: Robolectric test, Robolectric looks for a
cooresponding shadow class (marked by
nit: cooresponding -> corresponding

https://codereview.chromium.org/2882243003/diff/1/testing/android/docs/junit....
testing/android/docs/junit.md:59: `@Implements(ClassBeingShadowed.class)`). If
found, anytime a method is invoked
nit: any time

https://codereview.chromium.org/2882243003/diff/1/testing/android/docs/junit....
testing/android/docs/junit.md:60: on the object, the shadow class'
implementation of the method is invoked first.
nit: class's, I believe, as class isn't plural

[mikecase (-- gone --), 2017-05-16 17:37:01]

Fixed all the nits.

From now on, going to write documentation in Docs (apparently I badly need
spellcheck) and will copy/paste over.

[mikecase (-- gone --), 2017-05-16 17:37:03]

The CQ bit was checked by mikecase@chromium.org

[mikecase (-- gone --), 2017-05-16 17:37:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org
Link to the patchset: https://codereview.chromium.org/2882243003/#ps40001
(title: "Updated Robolectric documentation.")

[commit-bot: I haz the power, 2017-05-16 17:38:24]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-05-16 18:05:18]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1494956223198260,
"parent_rev": "ae01bc6a3a50e49466bd2c1455ff011f2d006bf4", "commit_rev":
"5e8650321a3058567af36d42faf2618d93743e59"}

[commit-bot: I haz the power, 2017-05-16 18:05:31]

Description was changed from

==========
Updated Robolectric documentation.

Now that change to expose resources to Robolectric tests has
landed, I wanted to update the documentation to point out to
people they need to specify "package_name" GN variable to get
it to work (since that is definitely not obvious).

You can also (theoretically) now specify the AndroidManifest.xml
file to Robolectric (Robolectric will grab the package name from
there). However, no tests currently do this.
==========

to

==========
Updated Robolectric documentation.

Now that change to expose resources to Robolectric tests has
landed, I wanted to update the documentation to point out to
people they need to specify "package_name" GN variable to get
it to work (since that is definitely not obvious).

You can also (theoretically) now specify the AndroidManifest.xml
file to Robolectric (Robolectric will grab the package name from
there). However, no tests currently do this.

Review-Url: https://codereview.chromium.org/2882243003
Cr-Commit-Position: refs/heads/master@{#472156}
Committed:
https://chromium.googlesource.com/chromium/src/+/5e8650321a3058567af36d42faf2...
==========

[commit-bot: I haz the power, 2017-05-16 18:05:32]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/5e8650321a3058567af36d42faf2...