Closed
Bug 1315080
Opened 8 years ago
Closed 6 years ago
unable to run dom/media mochitests locally due to missing python library
Categories
(Core :: WebRTC: Networking, defect, P3)
Core
WebRTC: Networking
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: jmaher, Unassigned)
References
Details
elvis@JMAHER-WIN10 ~/mozilla-inbound
$ ./mach mochitest dom/media/tests/mochitest/test_getUserMedia_callbacks.html
######
### Now running mochitest-plain with subsuite media.
######
Checking for orphan ssltunnel processes...
Checking for orphan xpcshell processes...
SUITE-START | Running 1 tests
dir: dom/media/tests/mochitest/test_getUserMedia_callbacks.html
JMAHER: iteration: 1, repeat: 10, tests: [u'dom/media/tests/mochitest/test_getUserMedia_callbacks.html']
mozprofile.addons WARNING | Could not install c:\Users\elvis\mozilla-inbound\obj-i686-pc-mingw32\_tests\testing\mochitest\extensions\mozscreenshots: [Errno 2] No such file or directory: 'c:\\Users\\elvis\\mozilla-inbound\\obj-i686-pc-mingw32\\_tests\\testing\\mochitest\\extensions\\mozscreenshots\\install.rdf'
c:\Users\elvis\mozilla-inbound\obj-i686-pc-mingw32\dist\bin\pk12util.exe: PKCS12 IMPORT SUCCESSFUL
MochitestServer : launching [u'c:\\Users\\elvis\\mozilla-inbound\\obj-i686-pc-mingw32\\dist\\bin\\xpcshell.exe', '-g', u'c:\\Users\\elvis\\mozilla-inbound\\obj-i686-pc-mingw32\\dist\\bin', '-v', '170', '-f', u'c:\\Users\\elvis\\mozilla-inbound\\obj-i686-pc-mingw32\\dist\\bin\\components\\httpd.js', '-e', "const _PROFILE_PATH = 'c:\\\\users\\\\elvis\\\\appdata\\\\local\\\\temp\\\\tmp5f0m4d.mozrunner'; const _SERVER_PORT = '8888'; const _SERVER_ADDR = '127.0.0.1'; const _TEST_PREFIX = undefined; const _DISPLAY_RESULTS = false;", '-f', 'c:\\Users\\elvis\\mozilla-inbound\\obj-i686-pc-mingw32\\_tests\\testing\\mochitest\\server.js']
runtests.py | Server pid: 6108
runtests.py | Websocket server pid: 14612
runtests.py | websocket/process bridge pid: 11472
Traceback (most recent call last):
File "websocketprocessbridge\websocketprocessbridge.py", line 6, in <module>
from twisted.internet import protocol, reactor
ImportError: No module named twisted.internet
elvis@JMAHER-WIN10 ~/mozilla-inbound
$
Reporter | ||
Comment 1•8 years ago
|
||
I had also tried a:
* mach clobber
* mach build (artifact build)
* mach mochitest ...
still no luck.
Comment 2•8 years ago
|
||
You need to do "pip install -r testing/tools/websocketprocessbridge/websocketprocessbridge_requirements.txt".
There's a patch in bug 1290765 that fixes this, but it can't land because we are still using a very old version of pip in CI. I'll see if I can get a version of that patch that works with older pip's landed.
Depends on: 1290765
Comment 3•8 years ago
|
||
This shouldn't be preventing that test from running.
Reporter | ||
Comment 4•8 years ago
|
||
this does prevent tests from running- but :ahal's command allowed me to move forward and run tests locally.
Comment 5•8 years ago
|
||
It could be it's only blocked on Windows if that module isn't installed. If bug 1290765 is fixed, this bug may become moot (or less important)
Rank: 25
Component: Audio/Video: MediaStreamGraph → WebRTC: Networking
Priority: -- → P2
Comment 6•8 years ago
|
||
Byron, we start the websocketProcessBridge anytime any test that is part of the "media" subsuite gets run:
https://dxr.mozilla.org/mozilla-central/source/testing/mochitest/runtests.py#1130
So though that test may not use the server, we are starting it anyway. It may be worthwhile to look into a better way of detecting whether the server needs to be started or not (probably using a key in the test manifests).
Anyway, I put up a patch in bug 1290765. It fixes the ImportError for me, though the harness still hangs and times out when running:
./mach mochitest dom/media/tests/mochitest/test_getUserMedia_callbacks.html
I'm not sure what I'm doing wrong.. but if the patches there fix the issue for everyone here, feel free to RESOLVE DUPLICATE this.
Comment 7•7 years ago
|
||
Mass change P2->P3 to align with new Mozilla triage process.
Priority: P2 → P3
Comment 8•6 years ago
|
||
jmaher: wfm on m-c from this morning. still an issue for you?
Flags: needinfo?(jmaher)
Reporter | ||
Comment 9•6 years ago
|
||
thanks for asking, this works great.
Status: NEW → RESOLVED
Closed: 6 years ago
Flags: needinfo?(jmaher)
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•