Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1718)

Unified Diff: test/mjsunit/element-accessor.js

Issue 2431223005: Implement DefineOwnProperty for TypedArrays (Closed)
Patch Set: Correct use of assertThrows Created 4 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: test/mjsunit/element-accessor.js
diff --git a/test/mjsunit/element-accessor.js b/test/mjsunit/element-accessor.js
index 5fba359c2a769cd5b46598bacc385c519ffa7762..7c22f146d4dbcafb31e6cd4c8eea77727afae9c6 100644
--- a/test/mjsunit/element-accessor.js
+++ b/test/mjsunit/element-accessor.js
@@ -29,9 +29,16 @@
})();
(function () {
- var o = new Int32Array();
- Object.defineProperty(o, "0", {get: function(){}});
- assertEquals(undefined, Object.getOwnPropertyDescriptor(o, "0"));
+ var o = new Int32Array(1);
+ assertThrows(function() {
+ Object.defineProperty(o, '0', {get: function() {}})
+ }, TypeError);
caitp 2016/11/21 22:02:36 nit: you can use arrow function to make this more
Henrique Ferreiro 2016/11/21 22:38:11 Done.
+ assertEquals({
+ value: 0,
+ writable: true,
+ enumerable: true,
+ configurable: false
+ }, Object.getOwnPropertyDescriptor(o, "0"));
})();
(function() {
« no previous file with comments | « src/objects.cc ('k') | test/test262/test262.status » ('j') | test/test262/test262.status » ('J')

Powered by Google App Engine
This is Rietveld 408576698