Closed TimSangster closed 5 years ago
Merging #374 into master will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #374 +/- ##
=======================================
Coverage 94.08% 94.08%
=======================================
Files 26 26
Lines 2113 2113
=======================================
Hits 1988 1988
Misses 125 125
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 9d76560...f1a5b8c. Read the comment docs.
Because code coverage did not hit the new lines, I investigated and found visiting the Return node should already be handled here: https://github.com/datacamp/pythonwhat/blob/2e69aa22941c75ae731d395be26879622568249d/pythonwhat/parsing.py#L769-L771
The test could still be useful. I wonder what SCT behaviour made us think it wasn't supported, probably a different parsing improvement. Now that we have _debug
it should be easier to localise problems in the future.
the parametrize isn't needed here
You're right, thanks!
Add test for function calls in return statements