testbot: When restarting a task, reset all the fields related to its previous run.

Francois Gouget fgouget at codeweavers.com
Tue Mar 26 04:55:12 CDT 2013


Some fields like the start and stop time would still be displayed and could be confusing. The 'TestFailures' field could even not get reset if the task then is canceled or fails to run and thus report incorrect results.
---
 testbot/lib/WineTestBot/Jobs.pm |    4 ++++
 1 file changed, 4 insertions(+)

diff --git a/testbot/lib/WineTestBot/Jobs.pm b/testbot/lib/WineTestBot/Jobs.pm
index cd6c427..c73dea1 100644
--- a/testbot/lib/WineTestBot/Jobs.pm
+++ b/testbot/lib/WineTestBot/Jobs.pm
@@ -231,6 +231,10 @@ sub Restart
         system("rm", "-rf", "$JobDir/" . $Step->No . "/" . $Task->No);
       }
       $Task->Status("queued");
+      $Task->ChildPid(undef);
+      $Task->Started(undef);
+      $Task->Ended(undef);
+      $Task->TestFailures(undef);
     }
     # Subsequent steps only contain files generated by the previous steps
     system("rm", "-rf", "$JobDir/" . $Step->No) if (!$FirstStep);
-- 
1.7.10.4




More information about the wine-patches mailing list