NotificationsYou must be signed in to change notification settings
Fork6
Star31

Commit90251ff

committed

Fix places in TestLib.pm in need of adaptation to the output of Msys perl

Contrary to the output of native perl, Msys perl generates outputs withCRLFs characters. There are already places in the TAP code where CRLFs(\r\n) are automatically converted to LF (\n) on Msys, but we missed acouple of places when running commands and using their output forcomparison, that would lead to failures.This problem has been found thanks to the test added in5adb067 usingTestLib::command_checks_all(), but after a closer look more code pathswere missing a filter.This is backpatched all the way down to prevent any surprises if a newtest is introduced in stable branches.Reviewed-by: Andrew Dunstan, Álvaro HerreraDiscussion:https://postgr.es/m/1252480.1631829409@sss.pgh.pa.usBackpatch-through: 9.6

1 parent2ad5f96 commit90251ffCopy full SHA for 90251ff

File tree

1 file changed

-0

lines changed

src/test/perl
- TestLib.pm

1 file changed

-0

lines changed

`‎src/test/perl/TestLib.pm`

Lines changed: 5 additions & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -414,6 +414,7 @@ sub run_command`
`414`	`414`	`my ($cmd) =@_;`
`415`	`415`	`my ($stdout,$stderr);`
`416`	`416`	`my$result = IPC::Run::run$cmd,'>', \$stdout,'2>', \$stderr;`
	`417`	`+foreach ($stderr,$stdout) {s/\r\n/\n/gif$Config{osname}eq'msys'; }`
`417`	`418`	`chomp($stdout);`
`418`	`419`	`chomp($stderr);`
`419`	`420`	`return ($stdout,$stderr);`
`@@ -858,6 +859,7 @@ sub command_like`
`858`	`859`	`my$result = IPC::Run::run$cmd,'>', \$stdout,'2>', \$stderr;`
`859`	`860`	`ok($result,"$test_name: exit code 0");`
`860`	`861`	`is($stderr,'',"$test_name: no stderr");`
	`862`	`+$stdout =~s/\r\n/\n/gif$Config{osname}eq'msys';`
`861`	`863`	`like($stdout,$expected_stdout,"$test_name: matches");`
`862`	`864`	`return;`
`863`	`865`	`}`
`@@ -910,6 +912,7 @@ sub command_fails_like`
`910`	`912`	`print("# Running:" .join("", @{$cmd}) ."\n");`
`911`	`913`	`my$result = IPC::Run::run$cmd,'>', \$stdout,'2>', \$stderr;`
`912`	`914`	`ok(!$result,"$test_name: exit code not 0");`
	`915`	`+$stderr =~s/\r\n/\n/gif$Config{osname}eq'msys';`
`913`	`916`	`like($stderr,$expected_stderr,"$test_name: matches");`
`914`	`917`	`return;`
`915`	`918`	`}`
`@@ -954,6 +957,8 @@ sub command_checks_all`
`954`	`957`	`if$ret & 127;`
`955`	`958`	`$ret =$ret >> 8;`
`956`	`959`
	`960`	`+foreach ($stderr,$stdout) {s/\r\n/\n/gif$Config{osname}eq'msys'; }`
	`961`	`+`
`957`	`962`	`# check status`
`958`	`963`	`ok($ret ==$expected_ret,`
`959`	`964`	`"$test_name status (got$ret vs expected$expected_ret)");`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit90251ff

File tree

1 file changed

1 file changed

`‎src/test/perl/TestLib.pm`

0 commit comments