• Do not register here on develop.twiki.org, login with your twiki.org account.
• Use View topic Item7848 for generic doc work for TWiki-6.1.1. Use View topic Item7851 for doc work on extensions that are not part of a release. More... Close
• Anything you create or change in standard webs (Main, TWiki, Sandbox etc) will be automatically reverted on every SVN update.
Does this site look broken?. Use the LitterTray web for test cases.

Item3666: Not declaring $/ local in unit test cases and DelimitedFile.pm causes errors in perl 5.6.1

Item Form Data

AppliesTo: Component: Priority: CurrentState: WaitingFor: TargetRelease ReleasedIn
Engine   Normal Closed   patch 4.1.2

Edit Form Data

Summary:
Reported By:
Codebase:
Applies To:
Component:
Priority:
Current State:
Waiting For:
Target Release:
Released In:
 

Detail

In perl 5.6.1 running the unit test cases with TWikiSuite seems to require better separation between the individual test cases than what we know from Perl 5.8.

When StoreSmokeTest runs - in the verify_releaselocksonsave $/ is unset to make the test read an entire file.

Same in several other unit test cases. And one test case calls bulk registration which uses DelimitedFile.pm and this makes a different test case fail later.

This input record separator remains in effect in later testcases in Perl 5.6.1 resulting in mysterious errors and creation of garbage directories.

You must always declare $/ locally when unsetting it

   local $/ = undef;

The issue in DelimitedFile.pm could potentially also cause errors in Perl 5.8 with mod_perl or similar.

-- TWiki:Main/KennethLavrsen - 20 Feb 2007

Fixed. Waiting for release as it also involves a code fix so I want this in release note.

-- TWiki:Main/KennethLavrsen - 21 Feb 2007

Closed with release of 4.1.2

KJL

ItemTemplate
Summary Not declaring $/ local in unit test cases and DelimitedFile.pm causes errors in perl 5.6.1
ReportedBy TWiki:Main.KennethLavrsen
Codebase ~twiki4
SVN Range TWiki-4.1.1, Tue, 20 Feb 2007, build 12932
AppliesTo Engine
Component

Priority Normal
CurrentState Closed
WaitingFor

Checkins 12946 12947
TargetRelease patch
ReleasedIn 4.1.2
Edit | Attach | Watch | Print version | History: r5 < r4 < r3 < r2 < r1 | Backlinks | Raw View | Raw edit | More topic actions
Topic revision: r5 - 2007-03-04 - KennethLavrsen
 
This site is powered by the TWiki collaboration platform Powered by PerlCopyright © 2008-2019 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback