add extra message in case of missing/spurious trailing newline

[dmouriss]

• 819232: eerste foute testgeval - oplossing heeft een newline te veel, maar feedbacktabel toont dit niet
• extra console.logs buiten functie stellen message in

Original issue by @dmouriss on Tue May 02 2017 at 14:33.

[pdawyndt]

@dmouriss extra console.logs buiten functie stellen message in

Dit was expliciet de bedoeling. Bovendien is het zo dat als bij het uitvoeren van de ingediende code een fout wordt opgemerkt (hetzij een exception, hetzij schrijven naar stdout of stderr), dan worden de testgevallen ook niet meer uitgevoerd. De testgevallen blijven altijd staan, maar we zouden nog een extensie van Dodona nodig hebben om deze testgevallen met status "unprocessed" te markeren (dit zou dan ook de default status worden).

Als je denkt dat het verwarrend/vervelend is voor de studenten dat ze niet met console.log kunnen uitschrijven bij het debuggen, dan hoor ik het graag. Maar ik vind het geen goede praktijk om console.logs in finale code te laten staan. Desnoods moeten ze maar een globale vlag DEBUG gebruiken en ervoor zorgen dat de console logs conditioneel uitgevoerd worden als DEBUG === true.

Original comment by @pdawyndt on Tue May 02 2017 at 15:39.

[pdawyndt]

@dmouriss Can you review the proposed solution with submission 819232? If OK, you can close this issue.

NOTE: I added the same error message to the evaluation of stdout/stderr.

Original comment by @pdawyndt on Tue May 02 2017 at 16:21.

[pdawyndt]

See https://github.ugent.be/dodona/dodona/issues/412 for what seems to be a serious bug in Dodona.

Original comment by @pdawyndt on Wed May 03 2017 at 19:57.