Fix and enable ChakraCore/test/es6/ES6Function_bugs.js after r215072
authormsaboff@apple.com <msaboff@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 8 Apr 2017 00:56:24 +0000 (00:56 +0000)
committermsaboff@apple.com <msaboff@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 8 Apr 2017 00:56:24 +0000 (00:56 +0000)
https://bugs.webkit.org/show_bug.cgi?id=170580

Reviewed by JF Bastien.

Commented out subtest that expects defineProperty("length") to a getter on a
sealed function shouldn't throw.

* ChakraCore.yaml:
* ChakraCore/test/es6/ES6Function_bugs.js:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@215133 268f45cc-cd09-0410-ab3c-d52691b4dbfc

JSTests/ChakraCore.yaml
JSTests/ChakraCore/test/es6/ES6Function_bugs.js
JSTests/ChangeLog

index f34ad80..e09f873 100644 (file)
   # Different behavior.
   cmd: runChakra :skip, "NoException", "proxytest9.baseline", []
 - path: ChakraCore/test/es6/ES6Function_bugs.js
-  # Different behavior with Object.seal().
-  cmd: runChakra :skip, "NoException", "", ["../UnitTestFramework/UnitTestFramework.js"]
+  cmd: runChakra :pass, "NoException", "", ["../UnitTestFramework/UnitTestFramework.js"]
 - path: ChakraCore/test/es6/OS_2700778.js
   cmd: runChakra :pass, "NoException", "", []
 - path: ChakraCore/test/es6/bug_OS_2184795.js
index d7c0050..aa85130 100644 (file)
@@ -28,7 +28,10 @@ var tests = [
                     get: function () { }
                     });
             }
-            assert.doesNotThrow(function () { g('length') }, "assertion failure on defineProperty 'length' with getter after sealing a function object");
+/* This check seems wrong in that we are trying to change length to a getter, which should throw.
+ * Verified that V8 also throws here.
+ *          assert.doesNotThrow(function () { g('length') }, "assertion failure on defineProperty 'length' with getter after sealing a function object");
+ */
             assert.throws(function () { g('arguments') }, TypeError, "Cannot redefine non-configurable property 'arguments'");
             assert.throws(function () { g('caller') }, TypeError, "Cannot redefine non-configurable property 'caller'");
          }
index eef9fc0..6497cbb 100644 (file)
@@ -1,3 +1,16 @@
+2017-04-07  Michael Saboff  <msaboff@apple.com>
+
+        Fix and enable ChakraCore/test/es6/ES6Function_bugs.js after r215072
+        https://bugs.webkit.org/show_bug.cgi?id=170580
+
+        Reviewed by JF Bastien.
+
+        Commented out subtest that expects defineProperty("length") to a getter on a
+        sealed function shouldn't throw.
+
+        * ChakraCore.yaml:
+        * ChakraCore/test/es6/ES6Function_bugs.js:
+
 2017-04-06  Michael Saboff  <msaboff@apple.com>
 
         Cannot Object.seal() or Object.freeze() global "this"