Skip Menu |
 

This queue is for tickets about the SOAP-WSDL CPAN distribution.

Report information
The Basics
Id: 29782
Status: resolved
Priority: 0/
Queue: SOAP-WSDL

People
Owner: Nobody in particular
Requestors: talexb [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Important
Broken in: 1.25
Fixed in: (no value)



Subject: Four tests fail if module is unpacked to /tmp and tested -- with patches
Download (untitled) / with headers
text/plain 722b
I downloaded and built this module in /tmp, but found that the tests were all failing to get to the test files. Also, the path to the test files was getting mangled .. the tests were looking for "file://mp/SOAP-WSDL-1.25/t/acceptance/helloworld.asmx.xml" when they should have been looking for "file://tmp/SOAP-WSDL-1.25/t/acceptance/helloworld.asmx.xml". So I replaced "$dir=~s|/t/?||;" with "$dir=~s|(.)/t/?|$1|;" in 1_performance.t (line 24), 2_helloworld.NET.t (line 37), 3_various.t (line 21) and 4_auto_set_port.t (line 35). After these changes, all tests passed. (OK, 2_helloworld.NET.t used / instead of | -- might was well make that the same as the others.) I'm running Fedora Core 6, kernel 2.6.18-1.2868.fc6.
Download (untitled) / with headers
text/plain 244b
Fixed in SVN and will be fixed in next revision. However, I did not use your suggestion (.), but anchored the regex at the end |...$|, which should work on non-unixoid systems as well, while ./ probably does not. Thanks for reporting, Martin
Subject: Re: [rt.cpan.org #29782] Four tests fail if module is unpacked to /tmp and tested -- with patches
Date: Fri, 5 Oct 2007 09:10:44 -0400
To: bug-SOAP-WSDL [...] rt.cpan.org
From: "Alex Beamish" <talexb [...] gmail.com>
Download (untitled) / with headers
text/plain 642b
On 10/5/07, Martin Kutter via RT <bug-SOAP-WSDL@rt.cpan.org> wrote: Show quoted text
> > > <URL: http://rt.cpan.org/Ticket/Display.html?id=29782 > > > Fixed in SVN and will be fixed in next revision.
Excellent! However, I did not use your suggestion (.), but anchored the regex at Show quoted text
> the end |...$|, which should work on non-unixoid systems as well, while > ./ probably does not.
Yes, I'm glad you didn't use my fix 'as-is' .. I just did the minimum needed to get the installation to work, and didn't think through any test cases. :) Any thoughts on how soon the 2.x version will come out of Alpha? Thanks, -- Alex Beamish Toronto, Ontario aka talexb
There's only two open items on my TODO list, so it should be soon - a week or two, I guess...


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.