716732 - Robocop: fennec_ids.txt not always generated

Status: RESOLVED FIXED

(Testing :: Mochitest, defect)

Description

[Geoff Brown [:gbrown]]

Both :kats and I have had trouble with fennec_ids.txt today:
 - an old version of the file might be used (it is not cleaned up at the start of a test)
 - it seems to be re-generated by make package-tests, which (I think) would not otherwise be required to run make mochitest-robotium (it would be better if it were generated as part of the mochitest-robotium rule, if possible)

Comment 1

[Geoff Brown [:gbrown]]

Also, I see this message from 'make mochitest-robotium':

cannot stat '/home/mozdev/src/objdir-native-droid/build/mobile/robocop/robocop.ini/fennec_ids.txt': Not a directory

Note the "robocop.ini" -- what's that doing in there?

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: allow for generation of fennec_ids.txt during make mochitest-robotium (1.0)

this patch cleans up a handful of things related to fennec_ids.txt.  First off it is generated and copied to a different place.  Also we clean up any instance of it on the device before running tests.

Comment 3

[Kartikaya Gupta (email:[e-mail])]

Comment on attachment 587564 [details] [diff] [review]
allow for generation of fennec_ids.txt during make mochitest-robotium (1.0)

Review of attachment 587564 [details] [diff] [review]:
-----------------------------------------------------------------

r+ with nit addressed

::: testing/testsuite-targets.mk
@@ +80,3 @@
>      --console-level=INFO --log-file=./$@.log --file-level=INFO $(DM_FLAGS) --dm_trans=adb \
>      --app=$(TEST_PACKAGE_NAME) --deviceIP=${TEST_DEVICE} --xre-path=${MOZ_HOST_BIN} \
> +    --robocop $(DEPTH)/build/mobile/robocop/robocop.ini $(SYMBOLS_PATH) $(TEST_PATH_ARG) $(EXTRA_TEST_ARGS)

nit: For consistency this should have an equals instead of a space between --robocop and $(DEPTH)? It seems like all the other options use the form --key=value rather than --key value.

@@ +286,5 @@
>  	rm -rf $(PKG_STAGE) && $(NSINSTALL) -D $(PKG_STAGE) && $(NSINSTALL) -D $(PKG_STAGE)/bin && $(NSINSTALL) -D $(PKG_STAGE)/bin/components && $(NSINSTALL) -D $(PKG_STAGE)/certs && $(NSINSTALL) -D $(PKG_STAGE)/jetpack && $(NSINSTALL) -D $(PKG_STAGE)/firebug && $(NSINSTALL) -D $(PKG_STAGE)/peptest && $(NSINSTALL) -D $(PKG_STAGE)/mozbase
>  
> +robotium-id-map:
> +ifeq ($(MOZ_BUILD_APP),mobile/android)
> +	$(PYTHON) $(DIST)/../build/mobile/robocop/parse_ids.py -i $(DEPTH)/mobile/android/base/R.java -o $(DEPTH)/build/mobile/robocop/fennec_ids.txt

Why does this use $(DIST)/../ instead of $(DEPTH) to find parse_ids.py? Shouldn't they always be the same?

Comment 4

[Geoff Brown [:gbrown]]

Works for me: generates fennec_ids.txt without package-tests, and no visible errors.


Can you also clean-up:

http://mxr.mozilla.org/mozilla-central/source/build/mobile/robocop/Makefile.in#114

parse_ids.txt looks like a typo to me.

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

inbound:
https://hg.mozilla.org/integration/mozilla-inbound/rev/d56f08ec0225

Comment 6

[Matt Brubeck (:mbrubeck)]

https://hg.mozilla.org/mozilla-central/rev/d56f08ec0225