Closed Bug 790677 Opened 8 years ago Closed 7 years ago

Update xpcshell test packaging to work for B2G


(Testing :: XPCShell Harness, defect, P1)



(blocking-basecamp:+, firefox18 fixed, firefox19 fixed)

blocking-basecamp +
Tracking Status
firefox18 --- fixed
firefox19 --- fixed


(Reporter: jgriffin, Assigned: jgriffin)




(1 file)

We need to update the files that get included for xpcshell in make package-tests, so we can run them for B2G.

At the same time, I'll fix the broken paths that were introduced when devicemanager was moved from build/mobile to testing/mozbase/mozdevice.
Turns out 'make package-tests' is totally broken on B2G; it tries to copy some Android apk's that don't exist.  I'll file a separate bug for this, and wait on this bug until that's fixed.
Just a bump, does the 'make package-tests' bug still need to be filed, or can that be covered by this one?
Noming since this blocks getting the existing updater xpcshell tests running for B2G
blocking-basecamp: --- → ?
(In reply to Marshall Culpepper [:marshall_law] from comment #2)
> Just a bump, does the 'make package-tests' bug still need to be filed, or
> can that be covered by this one?

That can be covered by this.
This adds all the necessary files to the xpcshell directory of a package to run the tests for B2G, and fixes paths for files that used to live in build/mobile to their new location in testing/mozbase/mozdevice.  This is needed to get xpcshell tests running in CI for B2G; running them locally is covered by the patch in bug 797162.  Note that before we can run these in CI from a package, bug 797153 will also have to be fixed.
Attachment #668244 - Flags: review?(ted.mielczarek)
With this patch:

ls objdir-gecko/dist/test-package-stage/xpcshell:   head.js            moz-spdy        node-spdy               test     mozinfo.json  tests
Try run for 8f915ff65d05 is complete.
Detailed breakdown of the results available here:
Results (out of 16 total builds):
    success: 15
    failure: 1
Builds (or logs if builds failed) available at:
The single try failure was a timeout:

DEBUG: doshell: chrootPath:/builds/mock_mozilla/mozilla-centos6-x86_64/root/, uid:500, gid:494
DEBUG: doshell environment: {'LANG': 'en_US.UTF-8', 'TERM': 'vt100', 'SHELL': '/bin/bash', 'TZ': 'EST5EDT', 'HOSTNAME': 'mock', 'HOME': '/builds', 'PATH': '/usr/bin:/bin:/usr/sbin:/sbin', 'TMPDIR': '/tmp'}
DEBUG: doshell: command: /usr/bin/env HG_SHARE_BASE_DIR="/builds/hg-shared" LC_ALL="C" CCACHE_COMPRESS="1" MOZ_SYMBOLS_EXTRA_BUILDID="linux64" SYMBOL_SERVER_HOST="" CCACHE_DIR="/builds/ccache" POST_SYMBOL_UPLOAD_CMD="/usr/local/bin/" MOZ_SIGN_CMD="python /builds/slave/try-lnx64/tools/release/signing/ --cachedir /builds/slave/try-lnx64/signing_cache -t /builds/slave/try-lnx64/token -n /builds/slave/try-lnx64/nonce -c /builds/slave/try-lnx64/tools/release/signing/host.cert -H" SYMBOL_SERVER_SSH_KEY="/home/mock_mozilla/.ssh/ffxbld_dsa" DISPLAY=":2" PATH="/tools/buildbot/bin:/usr/local/bin:/usr/lib64/ccache:/bin:/usr/bin:/usr/local/sbin:/usr/sbin:/sbin:/tools/git/bin:/tools/python27/bin:/tools/python27-mercurial/bin:/home/cltbld/bin" CCACHE_BASEDIR="/builds/slave/try-lnx64" TINDERBOX_OUTPUT="1" SYMBOL_SERVER_PATH="/mnt/netapp/breakpad/symbols_ffx/" MOZ_OBJDIR="obj-firefox" MOZ_CRASHREPORTER_NO_REPORT="1" SYMBOL_SERVER_USER="ffxbld" LD_LIBRARY_PATH="/tools/gcc-4.3.3/installed/lib64" CCACHE_UMASK="002" tools/codesighs/autosummary.unix.bash -o obj-firefox -s . ../codesize-auto.log ../codesize-auto-old.log ../codesize-auto-diff.log
DEBUG: child environment: {'LANG': 'en_US.UTF-8', 'TERM': 'vt100', 'SHELL': '/bin/bash', 'TZ': 'EST5EDT', 'HOSTNAME': 'mock', 'HOME': '/builds', 'PATH': '/usr/bin:/bin:/usr/sbin:/sbin', 'TMPDIR': '/tmp'}

command timed out: 1200 seconds without output, attempting to kill

I assume this has nothing to do with this patch, as I've seen the same error on other try runs today.
Blocks a P1/blocker. Marking this as a P1/blocker.
blocking-basecamp: ? → +
Priority: -- → P1
review ping
Comment on attachment 668244 [details] [diff] [review]
Fix xpcshell test packaging for B2G,

Review of attachment 668244 [details] [diff] [review]:

Sorry for the delay.
Attachment #668244 - Flags: review?(ted.mielczarek) → review+
Closed: 7 years ago
Resolution: --- → FIXED
Whiteboard: [automation-needed-in-aurora]
Flags: in-testsuite+
Whiteboard: [automation-needed-in-aurora]
You need to log in before you can comment on or make changes to this bug.