Merge pull request #81 from JCEmmons/master

JCEmmons · JCEmmons · commit 7d1c043552e9 · 2015-10-02T13:40:23.000-05:00
Update test suite to work on ECMAScript 2015 implementations. Fixes #80
diff --git a/tests/intl402/harness/testBuiltInObject.js b/tests/intl402/harness/testBuiltInObject.js
@@ -63,7 +63,9 @@ define([ 'intern!object', 'intern/chai!assert' ],
 				var desc = Object.getOwnPropertyDescriptor(obj, "length");
 				assert.isFalse(desc.writable, "The length property of a built-in function must not be writable.");
 				assert.isFalse(desc.enumerable,"The length property of a built-in function must not be enumerable.");
+				/* Since the length property changed from ECMA 5 to ECMA 6, we won't bother to test for it here.
 				assert.isFalse(desc.configurable,"The length property of a built-in function must not be configurable.");
+				*/
 			}
 
 			properties.forEach(function(prop) {

Original file line number	Diff line number	Diff line change
`@@ -63,7 +63,9 @@ define([ 'intern!object', 'intern/chai!assert' ],`
`63`	`63`	`var desc = Object.getOwnPropertyDescriptor(obj, "length");`
`64`	`64`	`assert.isFalse(desc.writable, "The length property of a built-in function must not be writable.");`
`65`	`65`	`assert.isFalse(desc.enumerable,"The length property of a built-in function must not be enumerable.");`
	`66`	`+ /* Since the length property changed from ECMA 5 to ECMA 6, we won't bother to test for it here.`
`66`	`67`	`assert.isFalse(desc.configurable,"The length property of a built-in function must not be configurable.");`
	`68`	`+ */`
`67`	`69`	`}`
`68`	`70`
`69`	`71`	`properties.forEach(function(prop) {`