launchpad-reviewers team mailing list archive

[Māris Fogels <mfogels@xxxxxxxxx>]

Māris Fogels has proposed merging lp:~mars/launchpad/fix-ec2test-on-production-devel into lp:~launchpad-pqm/launchpad/production-devel.

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)


This branch builds on top of the Python 2.5 fixes in https://code.edge.launchpad.net/~stub/launchpad/production/+merge/37482.  It adds a fix for utf-8 encoded strings inside the ec2 test suite.  As far as I can tell this particular error was unrelated to the running Python version (strange.)  This new branch should fix prod_lp.
-- 
https://code.launchpad.net/~mars/launchpad/fix-ec2test-on-production-devel/+merge/37526
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~mars/launchpad/fix-ec2test-on-production-devel into lp:~launchpad-pqm/launchpad/production-devel.

=== modified file 'lib/devscripts/ec2test/tests/test_remote.py'
--- lib/devscripts/ec2test/tests/test_remote.py	2010-09-20 22:44:49 +0000
+++ lib/devscripts/ec2test/tests/test_remote.py	2010-10-04 21:31:13 +0000
@@ -986,10 +986,14 @@
         result = self.make_failing_result()
         logger.got_result(result)
         [user_message] = log
+        error_result_string = request.format_result(
+                result, logger._start_time, logger._end_time)
+        # Stringify the utf8-encoded MIME text message part containing the
+        # test run summary.
+        summary_text = user_message.get_payload(0).get_payload(decode=True)
         self.assertEqual(
-            request.format_result(
-                result, logger._start_time, logger._end_time),
-            self.get_body_text(user_message).decode('quoted-printable'))
+            error_result_string,
+            summary_text)
 
     def test_gzip_of_full_log_attached(self):
         # The full log is attached to the email.