Closed kennknowles closed 11 years ago
I was hoping someone more qualified would have looked at this by now, but I'm fine with it. Maybe @puffnfresh, @taku0, or someone can give it a glance. If not I'll merge it.
Until we get a quickcheck implementation in roy like suggested in #21, I don't see a problem in using claire. The source may be written in ls, but what we'd get from npm would be js, so there's no extra dependency for it. Feel free to add it if it makes this easier/better.
It seems fine to me.
Hello,
In order to more easily isolate parsing failures when editing the grammar, I thought I would begin a test suite just for the type parser. This is just a skeleton but I hope it is useful anyhow.
Notes and limitations:
toString()
which is obviously fragile and must be replaced in the long term.compile.js
but am not entirely sure what that's about. Is it because it would take a layer of indirection from the generated parser to encapsulate the extra lexing logic?I would love to express all of these tests as the property
identity == parse . pretty
in a QuickCheck-like framework (like I did here).- Kenn