JSAbrahams / mamba

🐍 The Mamba programming language, because we care about safety
MIT License
88 stars 4 forks source link

Test more functionality of `Node` #336

Closed JSAbrahams closed 1 year ago

JSAbrahams commented 2 years ago

The type checker depends on the node class being properly implemented.

It'd be nice to test every node variant before we move on to more advanced language features. This prevents us from deciphering non-nonsensical error messages which may be due to and implementation error here.

JSAbrahams commented 1 year ago

Hm coverage is quite high, I'm not sure we gain much from unit testing here. I'm more for just writing increasingly complex integration tests, which are also less finicky and susceptible to refactors.