Properly render multiline output when a V2 task fails

[Eric-Arellano]

Currently we render stdout and stderr as byte strings when a V2 task fails, which means that the \n literal gets rendered, so multiline strings are hard to read.

We already have made the assumption that stdout and stderr are UTF-8 encoded, so this is safe to do here.

pants/src/python/pants/backend/python/rules/python_test_runner.py

Lines 164 to 168 in 3c17cd2

	yield TestResult(
	status=status,
	stdout=result.stdout.decode('utf-8'),
	stderr=result.stderr.decode('utf-8'),
	)

[stuhood]

Thanks.

[blorente]

Cool! These seemingly small changes make the difference in the eyes of users.

[Eric-Arellano]

These seemingly small changes make the difference in the eyes of users.

Agreed! I suspect as we turn on V2 for unit tests (in order to land remoting), we will continue to find really easy wins like this to improve the UX.