{"action":"created","issue":{"url":"https://api.github.com/repos/Test-More/test-more/issues/554","labels_url":"https://api.github.com/repos/Test-More/test-more/issues/554/labels{/name}","comments_url":"https://api.github.com/repos/Test-More/test-more/issues/554/comments","events_url":"https://api.github.com/repos/Test-More/test-more/issues/554/events","html_url":"https://github.com/Test-More/test-more/issues/554","id":62637613,"number":554,"title":"Separate \"manual\" diag messages and the \"Failed [...]\" messages (Test::Builder)","user":{"login":"biffen","id":954005,"avatar_url":"https://avatars.githubusercontent.com/u/954005?v=3","gravatar_id":"","url":"https://api.github.com/users/biffen","html_url":"https://github.com/biffen","followers_url":"https://api.github.com/users/biffen/followers","following_url":"https://api.github.com/users/biffen/following{/other_user}","gists_url":"https://api.github.com/users/biffen/gists{/gist_id}","starred_url":"https://api.github.com/users/biffen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/biffen/subscriptions","organizations_url":"https://api.github.com/users/biffen/orgs","repos_url":"https://api.github.com/users/biffen/repos","events_url":"https://api.github.com/users/biffen/events{/privacy}","received_events_url":"https://api.github.com/users/biffen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-18T09:32:08Z","updated_at":"2015-03-18T15:10:32Z","closed_at":null,"body":"Using `Test::Builder`'s `ok()` I get a \"*Failed test [...] at [...] line [...]*\" for every failed test. I don't want those. Since they are internally printed using `diag()` (see code below) I can get rid of them by disabling  diagnostic messages (`no_diag()` (but that still prints an extra `\\n`)), but I would still like to be able to use `diag()` myself for other things.\r\n\r\nWould it be possible to have a separate setting for whether these \"*Failed [...]*\" messages get printed?\r\n\r\nI (think I) found the code in `Test/Builder.pm:878`:\r\n\r\n```perl\r\n    unless($test) {\r\n        my $msg = $self->in_todo ? \"Failed (TODO)\" : \"Failed\";\r\n        $self->_print_to_fh( $self->_diag_fh, \"\\n\" ) if $ENV{HARNESS_ACTIVE};\r\n \r\n        my( undef, $file, $line ) = $self->caller;\r\n        if( defined $name ) {\r\n            $self->diag(qq[  $msg test '$name'\\n]);\r\n            $self->diag(qq[  at $file line $line.\\n]);\r\n        }\r\n        else {\r\n            $self->diag(qq[  $msg test at $file line $line.\\n]);\r\n        }\r\n    }\r\n```\r\n\r\nIf only that `unless` would also check `$self->something` to determine whether to print those messages."},"comment":{"url":"https://api.github.com/repos/Test-More/test-more/issues/comments/83016866","html_url":"https://github.com/Test-More/test-more/issues/554#issuecomment-83016866","issue_url":"https://api.github.com/repos/Test-More/test-more/issues/554","id":83016866,"user":{"login":"exodist","id":73914,"avatar_url":"https://avatars.githubusercontent.com/u/73914?v=3","gravatar_id":"","url":"https://api.github.com/users/exodist","html_url":"https://github.com/exodist","followers_url":"https://api.github.com/users/exodist/followers","following_url":"https://api.github.com/users/exodist/following{/other_user}","gists_url":"https://api.github.com/users/exodist/gists{/gist_id}","starred_url":"https://api.github.com/users/exodist/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/exodist/subscriptions","organizations_url":"https://api.github.com/users/exodist/orgs","repos_url":"https://api.github.com/users/exodist/repos","events_url":"https://api.github.com/users/exodist/events{/privacy}","received_events_url":"https://api.github.com/users/exodist/received_events","type":"User","site_admin":false},"created_at":"2015-03-18T15:10:32Z","updated_at":"2015-03-18T15:10:32Z","body":"I would be willing to accept a patch for something like this. I would call it no_error_diag(). Please write your patch against current master, which is much much different from the stable currently on cpan. The new guts will be going live fairly soon."}}