Fix len() for bytes, bytearray

katieamazing · Mar 12, 2017 · 9b6a333 · 9b6a333
1 parent 4823682
commit 9b6a333
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 6 deletions.
diff --git a/python/common/org/python/types/ByteArray.java b/python/common/org/python/types/ByteArray.java
@@ -624,7 +624,7 @@ public org.python.Iterable __iter__(java.util.List<org.python.Object> args, java
     @org.python.Method(
             __doc__ = ""
     )
-    public org.python.types.Int __len__(java.util.List<org.python.Object> args, java.util.Map<java.lang.String, org.python.Object> kwargs, java.util.List<org.python.Object> default_args, java.util.Map<java.lang.String, org.python.Object> default_kwargs) {
+    public org.python.types.Int __len__() {
         return new org.python.types.Int(this.value.length);
     }
 

diff --git a/python/common/org/python/types/Bytes.java b/python/common/org/python/types/Bytes.java
@@ -479,7 +479,7 @@ public org.python.Iterable __iter__(java.util.List<org.python.Object> args, java
     @org.python.Method(
             __doc__ = ""
     )
-    public org.python.types.Int __len__(java.util.List<org.python.Object> args, java.util.Map<java.lang.String, org.python.Object> kwargs, java.util.List<org.python.Object> default_args, java.util.Map<java.lang.String, org.python.Object> default_kwargs) {
+    public org.python.types.Int __len__() {
         return new org.python.types.Int(this.value.length);
     }
 

diff --git a/tests/builtins/test_len.py b/tests/builtins/test_len.py
@@ -9,8 +9,5 @@ class BuiltinLenFunctionTests(BuiltinFunctionTestCase, TranspileTestCase):
     functions = ["len"]
 
     not_implemented = [
-        'test_bytearray',
-        'test_bytes',
-        'test_class',
         'test_frozenset',
     ]
diff --git a/tests/utils.py b/tests/utils.py
@@ -672,7 +672,7 @@ def wrapper(*args, **kwargs):
         ],
     'set': [
             'set()',
-            '{1, 2.3456, "another"}',
+            'set({1, 2.3456, "another"})',
         ],
     'slice': [
             'slice(0)',