Closed Bug 823703 Opened 7 years ago Closed 7 years ago

Create and use an android-specific xpcshell manifest

Categories

(Firefox for Android :: General, defect)

x86
Android
defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 20

People

(Reporter: gbrown, Assigned: gbrown)

References

Details

Attachments

(1 file)

To run android xpcshell tests in continuous integration, we want to start with a very small set of tests that we know will pass. As time permits, we will investigate failures and grow the list of supported tests.

We could use skip-if = os == 'android' in the main xpcshell.ini, but the preferred short-term solution is to create and use our own manifest, like xpcshell_b2g.ini.
Blocks: 811793
I know that gfx/tests passes and has been well-behaved for many weeks now.

Sample run from a pandaboard:

mozdev@mozdev-virtual-machine:/media/extra/objdir-native-droid$ make -C gfx/tests xpcshell-tests-remote
make: Entering directory `/media/extra/objdir-native-droid/gfx/tests'
/media/extra/objdir-native-droid/_virtualenv/bin/python -u /home/mozdev/src/config/pythonpath.py \
	  -I/home/mozdev/src/build \
	  -I/home/mozdev/src/build/mobile \
	  -I/home/mozdev/src/testing/mozbase/mozdevice/mozdevice \
	  /home/mozdev/src/testing/xpcshell/remotexpcshelltests.py \
	  --symbols-path=../../dist/crashreporter-symbols \
	  --build-info-json=../../mozinfo.json \
	  --testing-modules-dir=../../_tests/modules \
	   \
	  --dm_trans=sut \
	  --deviceIP=192.168.0.111 \
	  --objdir=../.. \
	  ../../_tests/xpcshell/gfx/tests/unit
using APK: ../../dist/fennec-20.0a1.en-US.android-arm.apk
Pushing libmozalloc.so..
Pushing libxul.so..
This is a big file, it could take a while.
Pushing libplc4.so..
Pushing libnss3.so..
Pushing libnssutil3.so..
Pushing libplds4.so..
Pushing libxpcom.so..
Pushing libsmime3.so..
Pushing libfreebl3.so..
Pushing libnssckbi.so..
Pushing libssl3.so..
Pushing libsoftokn3.so..
Pushing libmozsqlite3.so..
Pushing libnspr4.so..
pushing directory: ../../_tests/modules to /mnt/sdcard/tests/xpcshell/m
pushing ../../_tests/xpcshell (attempt 1 of 10)
pushing directory: ../../_tests/xpcshell to /mnt/sdcard/tests/xpcshell
TEST-INFO | /media/extra/objdir-native-droid/_tests/xpcshell/gfx/tests/unit/test_nsIScriptableRegion.js | running test ...
TEST-PASS | /media/extra/objdir-native-droid/_tests/xpcshell/gfx/tests/unit/test_nsIScriptableRegion.js | test passed (time: 797.169ms)
INFO | Result summary:
INFO | Passed: 1
INFO | Failed: 0
INFO | Todo: 0
make: Leaving directory `/media/extra/objdir-native-droid/gfx/tests'
There's *no* rush on this review -- we'll sort it out next year!
Attachment #694670 - Flags: review?(jmaher)
Comment on attachment 694670 [details] [diff] [review]
create and use xpcshell_android.ini

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

awesome!  My only concern here is that the make target might confuse folks; I think we should stick with it and if tests are missing they can ensure the specific tests are passing ;)
Attachment #694670 - Flags: review?(jmaher) → review+
https://hg.mozilla.org/mozilla-central/rev/2954ab7bab89
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 20
You need to log in before you can comment on or make changes to this bug.