The default bug view has changed. See this FAQ.

assertion errors in worker tests

RESOLVED FIXED in mozilla16

Status

()

Core
JavaScript Engine
--
critical
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Benjamin, Assigned: Benjamin)

Tracking

unspecified
mozilla16
x86_64
Linux
Points:
---
Bug Flags:
in-testsuite +

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
$ ./configure --enable-debug --disable-optimize --enable-threadsafe --with-system-nspr
$ tests/jstests.py _DBG.OBJ/js worker -o
    rc = 0, run time = 0.034421
 PASSED! worker-error

jstest: undefined bug:  result: PASSED type: shell description: worker-error reason: 

    rc = 0, run time = 0.052837
 PASSED! worker-init

jstest: undefined bug:  result: PASSED type: shell description: worker-init reason: 

    rc = -11, run time = 0.049652
Assertion failure: JSVAL_IS_DOUBLE(v), at ../../jsapi.h:1927
REGRESSION - js1_8_5/extensions/worker-terminate.js
    rc = -11, run time = 0.064559
Assertion failure: JSVAL_IS_DOUBLE(v), at ../../jsapi.h:1927
REGRESSION - js1_8_5/extensions/worker-fib.js
    rc = 0, run time = 0.063370
 PASSED! worker-simple

jstest: undefined bug:  result: PASSED type: shell description: worker-simple reason: 

    rc = 0, run time = 0.059212
 PASSED! typeof event.message

jstest: undefined bug:  result: PASSED type: shell description: typeof event.message reason: 

    rc = 6, run time = 1.040414
--- NOTE: IN THIS TESTCASE, WE EXPECT EXIT CODE 6 ---
Script runs for too long, terminating.
REGRESSIONS
    js1_8_5/extensions/worker-terminate.js
    js1_8_5/extensions/worker-fib.js
FAIL
(Assignee)

Comment 1

5 years ago
Created attachment 632083 [details] [diff] [review]
fix

Looks like a simple oversight. (Are the tests for this thing never run?)
Attachment #632083 - Flags: review?(jorendorff)
(Assignee)

Comment 2

5 years ago
Evidently bustage from bug 697537.
Attachment #632083 - Flags: review?(jorendorff) → review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/1eb234066998
Assignee: general → bpeterson
Flags: in-testsuite+
Keywords: checkin-needed
Target Milestone: --- → mozilla16
https://hg.mozilla.org/mozilla-central/rev/1eb234066998
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.