commit e68da374094f0695d9c5cf19bfb6f801aceea884
author Mike Frysinger <vapier@chromium.org> Tue Feb 04 03:10:45 2014 -0500
committer chrome-internal-fetch <chrome-internal-fetch@google.com> Tue Feb 04 23:05:12 2014 +0000
tree 6fed622525acb010c2cb5af7096578d157dda116
parent 55ea86f47b0c064c0e6c87d17a87b31630f55fc1

pushimage: fix tobesigned state file

We write out some debugging states to the tobesigned files.  The python
rewrite added a bug where newlines were omitted between the entries.
This isn't a big problem as the content isn't used directly (it's meant
for humans trying to debug stuff), but let's fix it anyway.  For the
humans.

BUG=None
TEST=`./scripts/pushimage_unittest.py` passes

Change-Id: I7950e66ee08b811a6e06c41e94d9748f579b3f96
Reviewed-on: https://chromium-review.googlesource.com/184810
Reviewed-by: Mike Frysinger <vapier@chromium.org>
Commit-Queue: Mike Frysinger <vapier@chromium.org>
Tested-by: Mike Frysinger <vapier@chromium.org>

scripts/pushimage_unittest.py

diff --git a/scripts/pushimage_unittest.py b/scripts/pushimage_unittest.py
index aeb91f5..3cd9849 100755
--- a/scripts/pushimage_unittest.py
+++ b/scripts/pushimage_unittest.py
@@ -154,9 +154,10 @@
     pushimage.MarkImageToBeSigned(self.ctx, '', '', 50)
     # We assume the first call is the one we care about.
     self.assertTrue(m.called)
-    content = '\n'.join(m.call_args_list[0][0][1])
+    content = m.call_args_list[0][0][1]
     self.assertIn('USER=', content)
     self.assertIn('HOSTNAME=', content)
+    self.assertIn('\n', content)
 
   def testTbsUpload(self):
     """Make sure we actually try to upload the file"""