Add expression parsing tests - Githubissues

sugarlabs / chart

A graphing sugar activity

GNU General Public License v3.0

1 stars 7 forks source link

Add expression parsing tests #9

Open quozl opened 4 years ago

quozl commented 4 years ago

In reusing the expression parsing code in the Finance activity, we found a few problems.

https://github.com/sugarlabs/finance-activity/pull/24#issuecomment-589877397 and https://github.com/sugarlabs/finance-activity/pull/24/commits/4dca761340466023a1537d9742b0b43be697b1e8 explains further.

Suggested actions;

[ ] separate the expression parser into a different file,
[ ] add the tests to the parser, and test,
[ ] minimise the differences between Finance and Chart expression parser.