Use here-doc for stdin if both stdin and arguments are used

dodona-edu / universal-judge

Universal judge for educational software testing

https://docs.dodona.be/en/tested

MIT License

9 stars 4 forks source link

Use here-doc for stdin if both stdin and arguments are used #485

Closed niknetniko closed 6 months ago

niknetniko commented 6 months ago

Currently, we just add the stdin below the command that is executed, which is not clear at all. We now use here-doc to display stdin in that case.

If only stdin is used, nothing changes: only stdin is shown.

It looks like this:

The colouring is sometimes a bit wonky, both Rouge and Pygments are not that good at highlighting here-docs.

Fixes #484.

pdawyndt commented 6 months ago

Colouring wasn't reliable either in the current representation (text highlighted as if it were a shell command), so that's no reason not to make the change. Indeed looks like an improvement to me.