archives

« Bugzilla Issues Index

#2207 — results of “hasownproperty” treated with “==” instead of “===”.

• bug_id: 2207
• creation_ts: 2013-11-07 02:11:00 -0800
• short_desc: results of “hasownproperty” treated with “==” instead of “===”.
• delta_ts: 2015-12-04 09:36:44 -0800
• product: Test262
• component: ECMA-262 Tests
• version: unspecified
• rep_platform: All
• op_sys: All
• bug_status: RESOLVED
• resolution: FIXED
• priority: Normal
• bug_severity: enhancement
• everconfirmed: true
• reporter: Martin Bodin
• assigned_to: Brian Terlson
• cc: andrebargull

---

• commentid: 6481
• comment_count: 0
• who: Martin Bodin
• bug_when: 2013-11-07 02:11:57 -0800

Hi,
I may be looking at a wrong version but for instance in the test 10.6-12-2.js and some of its neighbouring tests, the results of “hasownproperty” are treated using “==” instead of “===”, while “hasownproperty” shall return a boolean anyway (according to Section 15.2.4.5 of the ECMAScript specification).

Here is the current code of the test 10.6-12-2.js:
function testcase() {

var desc = Object.getOwnPropertyDescriptor(arguments,"callee");
if(desc.configurable === true &&
desc.enumerable === false &&
desc.writable === true &&
desc.hasOwnProperty('get') == false &&
desc.hasOwnProperty('put') == false)
return true;
}
runTestCase(testcase);

I suggest to replace it by:
function testcase() {

var desc = Object.getOwnPropertyDescriptor(arguments,"callee");
if(desc.configurable === true &&
desc.enumerable === false &&
desc.writable === true &&
desc.hasOwnProperty('get') === false &&
desc.hasOwnProperty('put') === false)
return true;
}
runTestCase(testcase);

This is relatively minor of course, but I was surprise to see this use of the “==” operator.
Hoping it can help,
Martin.

---

• commentid: 14902
• comment_count: 1
• who: André Bargull
• bug_when: 2015-12-04 09:36:44 -0800

Fixed in https://github.com/tc39/test262/commit/27b234f70806be9a5cbd908558141d8e22d60176