Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Fix for RT#78398: t/singlepart.t fails #3

Closed
wants to merge 1 commit into from

2 participants

@mbeijen

Hi, I came across the issue that the test t/singlepart.t fails. At first I assumed it was a Win32 issue - more or less because that is where I discovered it first and I did not assume the module test to be broken - but I can also reproduce it on my Linux laptop with multiple different perls.

This commit fixes it for me; basically in the 'expected' string all line feeds were removed so they do not match the returned string.

@rjbs
Owner

I believe this bug has been fixed through other, though very similar, means. Please re-open it if I am mistaken!

@rjbs rjbs closed this
@mbeijen

You're right, you fixed it in this commit four days ago, I updated my repo and I can verify it works.
0aa9fde

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jul 20, 2012
  1. @mbeijen
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  t/singlepart.t
View
2  t/singlepart.t
@@ -38,7 +38,7 @@ It's base64 encoded.
END_BODY
# $expected_body =~ s/\n/\x0d\x0a/g;
-$expected_string =~ s/\n\z/\x0d\x0a/g;
+$expected_string =~ s/\n/\x0d\x0a/g;
is $email->as_string, $expected_string, 'as_string matches';
is $email->body, $expected_body, 'body matches';
Something went wrong with that request. Please try again.