Serialize AST #2123

iamdefinitelyahuman · 2020-07-29T10:04:48Z

Ensure AST nodes as dicts are serializable. Fixes #2121

Add unique to_dict and __init__ logic on the Decimal and Bytes nodes.
Expand the test case around this behavior to also include creating and loading a JSON string based on the AST-dict.

Run tests.

vyper/ast/nodes.py

tests/parser/ast_utils/test_ast_dict.py

fubuloubu approved these changes Jul 29, 2020

View reviewed changes

fix: ensure AST nodes as dicts are serializable

3be0ecf

iamdefinitelyahuman force-pushed the fix-serialize-ast branch from 50d32fc to ce11489 Compare July 29, 2020 14:14

fubuloubu reviewed Jul 31, 2020

View reviewed changes

vyper/ast/nodes.py Show resolved Hide resolved

fubuloubu reviewed Jul 31, 2020

View reviewed changes

tests/parser/ast_utils/test_ast_dict.py Outdated Show resolved Hide resolved

test: convert ast-dict to and from json

869fa78

iamdefinitelyahuman force-pushed the fix-serialize-ast branch from ce11489 to 869fa78 Compare July 31, 2020 16:44

fubuloubu approved these changes Jul 31, 2020

View reviewed changes

fubuloubu merged commit 444666a into vyperlang:master Jul 31, 2020

iamdefinitelyahuman mentioned this pull request Aug 3, 2020

JSON Encoder error with Decimals eth-brownie/brownie#703

Closed

iamdefinitelyahuman deleted the fix-serialize-ast branch October 9, 2020 17:36

Provide feedback