Symbol.length is 0

fixes #648
mozilla · May 19, 2020 · b8c7672 · b8c7672
1 parent 734db6e
commit b8c7672
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/src/org/mozilla/javascript/NativeSymbol.java b/src/org/mozilla/javascript/NativeSymbol.java
@@ -159,7 +159,7 @@ protected void initPrototypeId(int id)
     {
         switch (id) {
         case Id_constructor:
-            initPrototypeMethod(CLASS_NAME, id, "constructor", 1);
+            initPrototypeMethod(CLASS_NAME, id, "constructor", 0);
             break;
         case Id_toString:
             initPrototypeMethod(CLASS_NAME, id, "toString", 0);

diff --git a/testsrc/jstests/es6/symbols.js b/testsrc/jstests/es6/symbols.js
@@ -88,7 +88,7 @@ TestPrototype()
 
 
 function TestConstructor() {
- // Rhino assertEquals(0, Symbol.length);
+  assertEquals(0, Symbol.length);
   assertSame(Function.prototype, Symbol.__proto__)
   assertFalse(Object === Symbol.prototype.constructor)
   assertFalse(Symbol === Object.prototype.constructor)

diff --git a/testsrc/test262.properties b/testsrc/test262.properties
@@ -908,7 +908,6 @@ built-ins/Symbol
     ! species/subclassing.js
     ! prototype/Symbol.toPrimitive/name.js
     ! prototype/Symbol.toPrimitive/prop-desc.js
-    ! Symbol/length.js
     ! this-val-non-obj.js
 
 built-ins/ThrowTypeError