search

yurtsiv / mooncake

🌜 A functional, dynamic programming language
MIT License
12 stars 4 forks source link

Comment float test cases and add test cases for interpreter #8

Closed mateidanut closed 3 years ago

mateidanut commented 3 years ago

The operations on floating point numbers are not yet implemented, but these test cases contain such operations. I commented them so that the test suite now passes completely.

Also, I added tests for invalid programs that will pass the Parser stage but not the Interpreter one.

This PR closes #3

mateidanut commented 3 years ago

Done :)

yurtsiv commented 3 years ago

@mateidanut One minor thing I noticed is that testProgramm needs to be renamed to testCorrectProgramm. Just for consistency. Thanks :)

mateidanut commented 3 years ago

Sure thing, I renamed it

yurtsiv commented 3 years ago

@mateidanut Thanks. Please, fix the conflict and I'll merge

yurtsiv commented 3 years ago

Hah, pushed to your fork instead of the original repo :)

mateidanut commented 3 years ago

It happens sometimes :)

yurtsiv commented 3 years ago

@mateidanut Are you finished with renaming? Currently, there's an error when running the tests

mateidanut commented 3 years ago

@yurtsiv Sorry for the delay, there was one more line that I hadn't payed attention to when renaming. It should work properly now

yurtsiv commented 3 years ago

@mateidanut Great, thanks!