gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Bug 40675: Add unit tests for SIP line ending normalization 

This patch adds comprehensive unit tests for the add_field function
to verify that all types of line endings (Unix \n, Windows \r\n,
Mac \r, and mixed) are properly normalized to spaces to prevent
SIP message corruption.

Tests cover:
- Unix line endings (\n)
- Windows line endings (\r\n)
- Mac line endings (\r)
- Mixed line ending combinations
- Normal text without line endings
- Empty values

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
(cherry picked from commit b535f20596)
Signed-off-by: Paul Derscheid <paul.derscheid@lmscloud.de>
(cherry picked from commit 6260f6c40d)
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
This commit is contained in:
Martin Renvoize 2025-09-16 10:08:56 +01:00 committed by Fridolin Somers
parent 10b6823ff7
commit a0f4709180
No known key found for this signature in database
GPG Key ID: 9A8E73E2A8B96537
1 changed files with 30 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
t/SIP/Sip.t
View File

@ -17,11 +17,11 @@
use Modern::Perl;
use Test::More tests => 10;
use Test::More tests => 11;
use Test::Warn;
BEGIN {
use_ok('C4::SIP::Sip', qw( timestamp ));
use_ok( 'C4::SIP::Sip', qw( timestamp add_field ) );
}
my $date_time = C4::SIP::Sip::timestamp();
@ -113,3 +113,31 @@ subtest 'remove_password_from_message' => sub {
"11 Checkout - empty AC and AND"
);
};
subtest 'add_field line ending normalization' => sub {
plan tests => 6;
# Test Unix line endings (\n)
my $field_unix = C4::SIP::Sip::add_field( "AF", "Line one\nLine two\nLine three" );
is( $field_unix, "AFLine one Line two Line three|", "Unix line endings (\\n) converted to spaces" );
# Test Windows line endings (\r\n)
my $field_windows = C4::SIP::Sip::add_field( "AF", "Line one\r\nLine two\r\nLine three" );
is( $field_windows, "AFLine one Line two Line three|", "Windows line endings (\\r\\n) converted to spaces" );
# Test Mac line endings (\r)
my $field_mac = C4::SIP::Sip::add_field( "AF", "Line one\rLine two\rLine three" );
is( $field_mac, "AFLine one Line two Line three|", "Mac line endings (\\r) converted to spaces" );
# Test mixed line endings
my $field_mixed = C4::SIP::Sip::add_field( "AF", "Line one\r\nLine two\nLine three\rLine four" );
is( $field_mixed, "AFLine one Line two Line three Line four|", "Mixed line endings converted to spaces" );
# Test no line endings (should remain unchanged)
my $field_normal = C4::SIP::Sip::add_field( "AF", "Single line message" );
is( $field_normal, "AFSingle line message|", "Normal text without line endings unchanged" );
# Test empty and undefined values
my $field_empty = C4::SIP::Sip::add_field( "AF", "" );
is( $field_empty, "AF|", "Empty value handled correctly" );
};