search

qunitjs / js-reporters

📋 Common Reporter Interface (CRI) for JavaScript testing frameworks.
MIT License
60 stars 18 forks source link

Reporter: TapReporter align actual with expected. #107

Closed rwjblue closed 4 years ago

rwjblue commented 6 years ago

When comparing two strings, it is significantly easier to spot the difference when they are aligned. This change pads the label for actual with a couple of spaces so that the actual value is printed at the same indentation as the expected value. Previously the expected value was right shifted by 2 characters...

Before:

2018-05-18_23-31-32_65aka_tmux image

After:

2018-05-18_23-29-33_ogp95_tmux image
rwjblue commented 6 years ago

Thoughts?

jzaefferer commented 4 years ago

This certainly looks like an improvement!