#287 — Some @onlyStrict tests broken on command-line harness

bug_id: 287
creation_ts: 2012-03-01 16:47:00 -0800
short_desc: Some @onlyStrict tests broken on command-line harness
delta_ts: 2012-03-06 19:30:38 -0800
product: Test262
component: ECMA-262 Tests
version: unspecified
rep_platform: All
op_sys: All
bug_status: RESOLVED
resolution: FIXED
priority: Normal
bug_severity: normal
everconfirmed: true
reporter: Dave Fugate
assigned_to: Dave Fugate
cc: utatane.tea

commentid: 717
comment_count: 0
who: Dave Fugate
bug_when: 2012-03-01 16:47:41 -0800

<Thanks>Gavin Barraclough</Thanks>

In the following snippet, "@onlyStrict" should be replaced with "@noStrict" indicating that test262's command-line runner is not allowed to dynamically inject a "use strict" (thus invalidating the testcase). I have a feeling there's quite a few identical issues under 15.3.5.4.

/**
* @path ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-76gs.js
* @description Strict mode - checking access to strict function caller from non-strict function (non-strict function declaration called by strict eval)
* @onlyStrict
*/
function f() { return gNonStrict();};
(function () {"use strict"; return eval("f();"); })();

function gNonStrict() {
return gNonStrict.caller;
}

Note that this doesn't affect test262's browser harness.

commentid: 720
comment_count: 1
who: Yusuke Suzuki
bug_when: 2012-03-03 16:15:46 -0800

I found follwing listed test cases are broken.

test/suite/ch10/10.4/10.4.3/10.4.3-1-101-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-101gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-45-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-45gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-46-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-46gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-47-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-47gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-48-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-48gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-49-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-49gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-50-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-50gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-51-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-51gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-52-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-52gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-53-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-53gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-81-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-81gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-82-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-82gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-83-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-83gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-84-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-84gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-85-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-85gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-86-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-86gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-87-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-87gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-90-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-90gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-91-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-91gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-92-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-92gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-95-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-95gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-96-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-96gs.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-97-s.js
test/suite/ch10/10.4/10.4.3/10.4.3-1-97gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-12gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-14gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-75gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-76gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-77gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-78gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-79gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-80gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-81gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-82gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-83gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-84gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-85gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-86gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-87gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-88gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-90gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-91gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-92gs.js
test/suite/ch15/15.3/15.3.5/15.3.5.4/15.3.5.4_2-93gs.js

commentid: 725
comment_count: 2
who: Dave Fugate
bug_when: 2012-03-06 08:53:08 -0800

Thanks Yusuke! Your list looks accurate and you saved me quite a bit of time. The fix was checked into Hg a few minutes ago.

Gavin, please let me know if any files were missed.

commentid: 726
comment_count: 3
who: Yusuke Suzuki
bug_when: 2012-03-06 19:30:38 -0800

Great, thanks!

archives

#287 — Some @onlyStrict tests broken on command-line harness