{"action":"created","issue":{"url":"https://api.github.com/repos/kern/minitest-reporters/issues/159","labels_url":"https://api.github.com/repos/kern/minitest-reporters/issues/159/labels{/name}","comments_url":"https://api.github.com/repos/kern/minitest-reporters/issues/159/comments","events_url":"https://api.github.com/repos/kern/minitest-reporters/issues/159/events","html_url":"https://github.com/kern/minitest-reporters/pull/159","id":85274356,"number":159,"title":"Fixed redundant spacing when using fast_fail option","user":{"login":"ojak","id":15408,"avatar_url":"https://avatars.githubusercontent.com/u/15408?v=3","gravatar_id":"","url":"https://api.github.com/users/ojak","html_url":"https://github.com/ojak","followers_url":"https://api.github.com/users/ojak/followers","following_url":"https://api.github.com/users/ojak/following{/other_user}","gists_url":"https://api.github.com/users/ojak/gists{/gist_id}","starred_url":"https://api.github.com/users/ojak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ojak/subscriptions","organizations_url":"https://api.github.com/users/ojak/orgs","repos_url":"https://api.github.com/users/ojak/repos","events_url":"https://api.github.com/users/ojak/events{/privacy}","received_events_url":"https://api.github.com/users/ojak/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-06-04T20:57:59Z","updated_at":"2015-06-08T02:55:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/kern/minitest-reporters/pulls/159","html_url":"https://github.com/kern/minitest-reporters/pull/159","diff_url":"https://github.com/kern/minitest-reporters/pull/159.diff","patch_url":"https://github.com/kern/minitest-reporters/pull/159.patch"},"body":"Output when using fast_fail options with skipped tests prints out redundant newlines.  For example:\r\n\r\n```ruby\r\n# Running tests with run options --seed 58301:\r\n\r\n..............S....................S.........................................SSSSSSSSSSSSSSSS..\r\n\r\nFinished tests in 22.954128s, 4.1387 tests/s, 7.1447 assertions/s.\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n\r\n95 tests, 164 assertions, 0 failures, 0 errors, 18 skips\r\n```\r\n\r\nLooks like the newline in the default reporter is out of place.  Also, `message_for` should probably allow for empty messages so that it can be easily overridden without redundant newlines.  I've patched it so the new output looks as follows:\r\n\r\n```ruby\r\n# Running tests with run options --seed 5569:\r\n\r\n.............S....................S.........................................SSSSSSSSSSSSSSSS...\r\n\r\nFinished tests in 23.166914s, 4.1007 tests/s, 7.0791 assertions/s.\r\n\r\n\r\n95 tests, 164 assertions, 0 failures, 0 errors, 18 skips\r\n```\r\n"},"comment":{"url":"https://api.github.com/repos/kern/minitest-reporters/issues/comments/109844866","html_url":"https://github.com/kern/minitest-reporters/pull/159#issuecomment-109844866","issue_url":"https://api.github.com/repos/kern/minitest-reporters/issues/159","id":109844866,"user":{"login":"ojak","id":15408,"avatar_url":"https://avatars.githubusercontent.com/u/15408?v=3","gravatar_id":"","url":"https://api.github.com/users/ojak","html_url":"https://github.com/ojak","followers_url":"https://api.github.com/users/ojak/followers","following_url":"https://api.github.com/users/ojak/following{/other_user}","gists_url":"https://api.github.com/users/ojak/gists{/gist_id}","starred_url":"https://api.github.com/users/ojak/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ojak/subscriptions","organizations_url":"https://api.github.com/users/ojak/orgs","repos_url":"https://api.github.com/users/ojak/repos","events_url":"https://api.github.com/users/ojak/events{/privacy}","received_events_url":"https://api.github.com/users/ojak/received_events","type":"User","site_admin":false},"created_at":"2015-06-08T02:55:48Z","updated_at":"2015-06-08T02:55:48Z","body":"Here's the final output:\r\n\r\n`{ fast_fail: true, detailed_skip: false }`\r\n```\r\n.....F\r\nFailure:\r\nMyTest#test_six [/Users/.../my_test.rb:10]\r\nMinitest::Assertion: Failed assertion, no message given.\r\n........S.......F...\r\n```\r\n----\r\n`{ fast_fail: true, detailed_skip: true }`\r\n```\r\n.....F\r\nFailure:\r\nMyTest#test_six [/Users/.../my_test.rb:10]\r\nMinitest::Assertion: Failed assertion, no message given.\r\n........S\r\nSkipped:\r\nMyTest#test_fourteen [/Users/.../my_test.rb:15]:\r\nSkip me.\r\n......\r\n```\r\n----\r\n`{ fast_fail: false }`\r\n```\r\n.....F........S...\r\n```\r\n\r\n"}}