kursjan / petitparser2

A high-performance top-down parser
MIT License
41 stars 17 forks source link

new test method: testCurrentContext #64

Closed mabdi closed 3 weeks ago

mabdi commented 4 years ago

I submit this pull request to suggest a test method PP2NoopVisitorTest >> testCurrentContext.

We noticed that the method #currentContext is never executed by any of the tests in PP2NoopVisitorTest.

Note that these suggestions are adapted from a test amplification tool called SmallAmp (https://github.com/mabdi/small-amp). SmallAmp executes existing tests, sees which parts of the class under test are not covered and then suggests improvements on the test methods.

I hope you will accept this pull request. It would illustrate that SmallAmp makes relevant suggestions.

Mehrdad Abdi.