Merge lp:~kissiel/checkbox/fix-1501500-resume-after-tp-selected into lp:checkbox

[Maciej Kisielewski]

This MR fixes a resume session problem for sessions suspended just after test plan selection screen.

3083714 checkbox-touch: ignore resumable sessions having no app_blob

As we're trying to blindly deserialize app_blob from the last valid incomplete session, we need to make sure that we actually can do this. This patch adds checking if the items returned as resumable sessions by session assistant contain app_blob.

This patch doesn't change anything in the session assistant, as a session  checkpoint without app_blob is not invalid.

[Zygmunt Krynicki (zyga)]

+1, thanks

review: Approve

=== modified file 'checkbox-touch/py/checkbox_touch.py'
--- checkbox-touch/py/checkbox_touch.py	2015-10-28 08:38:46 +0000
+++ checkbox-touch/py/checkbox_touch.py	2015-11-01 15:11:34 +0000
@@ -223,6 +223,8 @@
     def is_session_resumable(self):
         """Check whether there is a session that can be resumed."""
         for session_id, session_md in self.assistant.get_resumable_sessions():
+            if session_md.app_blob is None:
+                continue
             # we're interested in the latest session only, this is why we
             # return early
             self._latest_session = session_id