validate: launcher: Initialize Test start time outside of wait_process
authorRamiro Polla <ramiro.polla@collabora.co.uk>
Fri, 16 Jan 2015 18:00:25 +0000 (19:00 +0100)
committerThibault Saunier <tsaunier@gnome.org>
Thu, 5 Feb 2015 14:18:38 +0000 (15:18 +0100)
wait_process will be moved to TestManager, so the values used to track
process update must remain inside Test.

Patch 2/4 to make TestManager handle waiting for processes instead of
expecting each Test to do it.

validate/launcher/baseclasses.py

index 69af9f5..393e401 100644 (file)
@@ -192,9 +192,6 @@ class Test(Loggable):
         return Result.NOT_RUN
 
     def wait_process(self):
-        self.last_val = 0
-        self.last_change_ts = time.time()
-        self.start_ts = time.time()
         while True:
             # Check process every second for timeout
             self.thread.join(1.0)
@@ -305,6 +302,10 @@ class Test(Loggable):
         self.thread = threading.Thread(target=self.thread_wrapper)
         self.thread.start()
 
+        self.last_val = 0
+        self.last_change_ts = time.time()
+        self.start_ts = time.time()
+
         try:
             self.wait_process()
         except KeyboardInterrupt: