[Bug 1034017] Review Request: perl-Test-Name-FromLine - Auto fill test names from caller line

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1034017

Mathieu Bridon <bochecha@xxxxxxxxxxxxxxxxx> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEW



--- Comment #2 from Mathieu Bridon <bochecha@xxxxxxxxxxxxxxxxx> ---
Thanks for the feedback Petr.

All points should be fixed in this new package.

Spec URL:
http://bochecha.fedorapeople.org/packages/perl-Test-Name-FromLine.spec
SRPM URL:
http://bochecha.fedorapeople.org/packages/perl-Test-Name-FromLine-0.11-2.fc20.src.rpm

----------

On the summary/description, I agree that the original one was pretty terrible.
The new one should be a bit better, but it's not very clear either.

The idea is that you should name your tests when using Test::More:
  http://search.cpan.org/~rjbs/Test-Simple-1.001002/lib/Test/More.pm#Test_names

With this package, tests are automatically named, based on the line of code
that defines them.

For example, the following test file:

  use Test::More;

  is(1, 1);
  done_testing();

Would output:

  ok 1

This is because the test is not named. Using Test::Name::FromLine, the output
would become:

  ok 1 - L3: is 1, 1;

That is, as if the test had been defined as follows:

  use Test::More;

  is(1, 1, "L3: is 1, 1;");
  done_testing();

Now, I hope the new summary and description make that a bit more clear, but if
you still find it lacking (and/or if you have ideas to improve it), I'll try
harder. :)

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list
package-review@xxxxxxxxxxxxxxxxxxxxxxx
https://admin.fedoraproject.org/mailman/listinfo/package-review





[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]