Skip Menu |
 

This queue is for tickets about the Carp CPAN distribution.

Report information
The Basics
Id: 102784
Status: resolved
Priority: 0/
Queue: Carp

People
Owner: Nobody in particular
Requestors: nanis [...] runu.moc.invalid
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 1.35
Fixed in: (no value)



Subject: Spurious test failure due to hard coded test file name
Download (untitled) / with headers
text/plain 904b
With nmake, `\` are used as directory separators in paths. Therefore, `nmake test` fails with: t\Carp.t ............. 1/60 # Failed test 'we don't overshoot the top stack frame' # at t\Carp.t line 32. # got: 'foo at t\Carp.t line 31. # ' # expected: 'foo at t/Carp.t line 31. # ' # Looks like you failed 1 test of 60. Therefore: 31: my $str = Carp::longmess("foo"); is( $str, "foo at t/Carp.t line 31.\n", "we don't overshoot the top stack frame", ); should be 31: my $str = Carp::longmess("foo"); is( $str, "foo at $0 line 31.\n", "we don't overshoot the top stack frame", ); or even 31: my $line = __LINE__, my $str = Carp::longmess("foo"); is( $str, "foo at $0 line $line.\n", "we don't overshoot the top stack frame", ); to avoid further hard coding of information that may change. HTH, -- Sinan
Thanks for this report. I'll look into it in the next day or two. -- rjbs
This should be fixed in v1.36 Please let me know if you encounter further problems! -- rjbs


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.