Closed
Bug 1058107
Opened 10 years ago
Closed 10 years ago
Possible race condition in the Loop call functional test
Categories
(Hello (Loop) :: Client, defect, P3)
Hello (Loop)
Client
Tracking
(Not tracked)
RESOLVED
WORKSFORME
backlog | tech-debt |
People
(Reporter: Paolo, Unassigned)
References
Details
(Whiteboard: [tech-debt])
There might be a race condition in the functional tests that results in the following stack:
Traceback (most recent call last):
File "/Users/paolo/m-c/testing/marionette/client/marionette/marionette_test.py", line 171, in run
testMethod()
File "/Users/paolo/m-c/browser/components/loop/test/functional/test_1_browser_call.py", line 160, in test_1_browser_call
self.hangup_call_and_verify_feedback()
File "/Users/paolo/m-c/browser/components/loop/test/functional/test_1_browser_call.py", line 128, in hangup_call_and_verify_feedback
self.marionette.set_context("chrome")
File "/Users/paolo/m-c/testing/marionette/client/marionette/marionette.py", line 949, in set_context
return self._send_message('setContext', 'ok', value=context)
File "/Users/paolo/m-c/testing/marionette/client/marionette/decorators.py", line 35, in _
return func(*args, **kwargs)
File "/Users/paolo/m-c/testing/marionette/client/marionette/marionette.py", line 615, in _send_message
response = self.client.send(message)
File "/Users/paolo/m-c/testing/marionette/transport/marionette_transport/transport.py", line 101, in send
response = self.receive()
File "/Users/paolo/m-c/testing/marionette/transport/marionette_transport/transport.py", line 58, in receive
raise IOError(self.connection_lost_msg)
Updated•10 years ago
|
Whiteboard: [loop-fn-test]
Updated•10 years ago
|
Whiteboard: [loop-fn-test] → [tech-debt]
Updated•10 years ago
|
backlog: --- → Fx36+
Updated•10 years ago
|
backlog: Fx36+ → Fx37+
Comment 1•10 years ago
|
||
hi nils, are you still seeing this or has it been fixed elsewhere
Flags: needinfo?(drno)
Comment 2•10 years ago
|
||
I have not run the test often enough to judge if we still get race conditions. Once we have to test running on a server I assume we will be better able to judge this. So lets leave this open for a little bit.
Flags: needinfo?(drno)
Updated•10 years ago
|
backlog: Fx37+ → Fx38?
Updated•10 years ago
|
backlog: Fx38? → tech-debt
Priority: -- → P3
Comment 3•10 years ago
|
||
Still waiting for the test on the server to stabilize to have some data to judge if this is gone or not...
Comment 4•10 years ago
|
||
Again we haven't seen this once on the test server since the test is running stable their. Lets close and re-open if needed.
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•