parsingdata/metal

Include verification in ParameterizedParse tests if stream is fully parsed.

Closed this issue · 1 comments

The ParameterizedParse tests test if we are able (or not able) to parse a given token on a given stream. However, this test does not verify if we have also parsed the whole stream, in case it succeeds. We should include this check in the test. This makes it much easier to verify parsing results for new features.

jvdb commented

Oh this is a really good idea! I hope it doesn't interfere with any existing tests though (but then we should probably just fix them).