beeware / batavia

A JavaScript implementation of the Python virtual machine.
http://pybee.org/batavia
Other
1.39k stars 425 forks source link

Add list tests #817

Closed alexjdw closed 4 years ago

alexjdw commented 5 years ago

Noticed an incorrect error message in builtins/list, then noticed that test_list had no tests, so I made some. Working on getting them passing in a separate commit.

Let me know if you can think of any more cases.

martica commented 5 years ago

It looks like a lot of these tests are duplicated in tests/datatypes/test_list.py to some extent, at least the parts that are passing. How do you feel about moving them into that file? At least the ones that don't directly concern the list() builtin function.

phildini commented 4 years ago

Hi there! It looks like this PR might be dead, so we're closing it for now. Feel free to re-open it if you'd like to continue, or think about directing your efforts to https://github.com/beeware/briefcase or https://github.com/beeware/toga. Both of these have more active development right now. 😄