Merge lp:~kissiel/checkbox/fix-1392123 into lp:checkbox

[Maciej Kisielewski]

This MR fixes zeroing estimated_time.

This was caused by using min() instead of max() when handling negative estimates.
The MR also removes unused module in inv_run.py

[lp:~kissiel/checkbox/fix-1392123]

3698. By Maciej Kisielewski on 2015-04-16

plainbox:commands:inv_run: fix wrong reporting of estimated_time

This patch fixes reporting of time left in the session.

Signed-off-by: Maciej Kisielewski <email address hidden>

3699. By Maciej Kisielewski on 2015-04-16

plainbox:commands:inv_run: remove unused math module

Signed-off-by: Maciej Kisielewski <email address hidden>

[Zygmunt Krynicki (zyga)]

+1, fun :-)

review: Approve

=== modified file 'plainbox/plainbox/impl/commands/inv_run.py'
--- plainbox/plainbox/impl/commands/inv_run.py	2015-04-16 10:16:50 +0000
+++ plainbox/plainbox/impl/commands/inv_run.py	2015-04-16 12:46:00 +0000
@@ -30,7 +30,6 @@
 import io
 import itertools
 import logging
-import math
 import os
 import sys
 import time
@@ -726,7 +725,7 @@
                 print(self.C.header(
                     _('Running job {} / {}. Estimated time left: {}').format(
                         job_no, len(jobs_to_run),
-                        seconds_to_human_duration(min(0, estimated_time))),
+                        seconds_to_human_duration(max(0, estimated_time))),
                     fill='-'))
                 self.run_single_job(job)
                 estimated_time -= job.estimated_duration