Almenon / AREPL-backend

TS interface to python evaluator for AREPL
MIT License
12 stars 8 forks source link

add python 3.11 tests #189

Closed Almenon closed 1 year ago

Almenon commented 1 year ago

3.11 is now generally available

Almenon commented 1 year ago

The following tests fail locally:

test_arepl_dump_not_in_modules
test_file_IO
test_builtinImportDeleted
test_pipImportDeleted
Almenon commented 1 year ago

3.11 changelog: https://docs.python.org/3/whatsnew/3.11.html

Relevant changes:

inspect.getframeinfo()

inspect.getouterframes()

inspect.getinnerframes(),

inspect.stack()

inspect.trace()"

Almenon commented 1 year ago

Jsonpickle has some 3.11 related changes: https://github.com/jsonpickle/jsonpickle/blob/main/CHANGES.rst

https://github.com/jsonpickle/jsonpickle/issues/411 https://github.com/jsonpickle/jsonpickle/issues/395

Almenon commented 1 year ago

Works w/ jsonpickle update, but I'll need to do regression tests as there is significant changes.

Almenon commented 1 year ago

There's a regression where file objects are encoded as null rather than _io.TextIOWrapper.

Looks like this is because of the following addition:

class TextIOHandler(BaseHandler):
    """Serialize file descriptors as None because we cannot roundtrip"""

    def flatten(self, obj, data):
        return None

Edit: Fixed

github-actions[bot] commented 1 year ago

:tada: This PR is included in version 2.1.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket: