Improve make_interpret test coverage; Define behaviour for non explicitly interpreted stateless grammar

[aszymaniak]

I have added extra tests to improve test coverage of make_interpreter.
While improving the test coverage of stateless interpreter, I noticed the fallback for non-explicitly supported stateless grammar was not calling emit_eval even though emit_eval has an implemented fallback for that scenario. I am not sure what should be the idea on how to handle this case. For now, I suggested to call emit_eval in the commit: 791c618

[ReubenJ]

Hey! Thanks for the contribution.

We no longer use the dev branch. We merge everything directly into master. Please switch your PR to point to master and make sure your branch is up to date with everything there. You can either rebase on top of the latest master or merge from the latest master into your own branch.

Once you've done so, we can take a look again at the PR. Thanks again! :)

[aszymaniak]

@ReubenJ I have changed the target branch

[ReubenJ]

The test additions look great, thanks!

I'm not sure I understand the emit_eval bit. Maybe @THinnerichs would be a better pair of eyes for this.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.09%. Comparing base (b5a6c5c) to head (a70d297).

Additional details and impacted files

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
+ Coverage   69.11%   76.09%   +6.97%     
==========================================
  Files           3        3              
  Lines         204      205       +1     
==========================================
+ Hits          141      156      +15     
+ Misses         63       49      -14     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.