[2.1] Telephony tests fail with "InternalError: too much recursion"

RESOLVED FIXED

Status

Firefox OS
Certification Suite
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jgriffin, Assigned: jgriffin)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
On a 2.1 Flame, the telephony tests mostly fail with:

InternalError: too much recursion
stacktrace: 
   execute_script @telephony_test.py, line 45 
   inline javascript, line 176 src: " app = null;"
(Assignee)

Comment 1

3 years ago
This occurs because the TelephonyCall object now includes a TelephonyCallId field which in turn contains a reference to the window object, which is unserializable with Marionette..  I'll have to adjust the tests to return a subset of fields from these objects instead.
(Assignee)

Updated

3 years ago
Assignee: nobody → jgriffin
Status: NEW → ASSIGNED
(Assignee)

Comment 2

3 years ago
Created attachment 8550470 [details] [review]
github PR
Attachment #8550470 - Flags: review?(atsai)

Comment 3

3 years ago
Comment on attachment 8550470 [details] [review]
github PR

r=me, good to merge.
Attachment #8550470 - Flags: review?(atsai) → review+

Comment 4

3 years ago
merged
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.