develooper Front page | perl.perl5.porters | Postings from October 2017

[perl #132276] [PATCH] Use library functions instead of manual TAPin t/io/tell.t

Thread Previous
James E Keenan via RT
October 12, 2017 20:43
[perl #132276] [PATCH] Use library functions instead of manual TAPin t/io/tell.t
Message ID:
On Thu, 12 Oct 2017 13:38:38 GMT, wrote:
> This is a bug report for perl from,
> generated with the help of perlbug 1.40 running under perl 5.27.5.
> -----------------------------------------------------------------
> [Please describe your issue here]
> Use library functions instead of manual TAP in t/io/tell.t

While I agree that t/io/tell.t falls under the rubric of the Porting/todo.pod "Migrate t/ from custom TAP generation" suggestion, I don't believe this patch should be applied as is.

If we're going to reformulate each unit test in this file, then we should provide descriptions for each such test, viz., a 2nd argument for ok() and a 3rd argument for is().  (See, e.g., t/io/argv.t.)  Otherwise, we will continue to have only a string of 'ok' and 'not ok' statements without any description as to what each test is doing.

Can you re-work?

Thank you very much.

James E Keenan (

via perlbug:  queue: perl5 status: new

Thread Previous Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at | Group listing | About