Enable frogium test in bots, and fix checked mode on co19 tests.

frog/scripts/buildbot_annotated_steps.py

Index: frog/scripts/buildbot_annotated_steps.py
diff --git a/frog/scripts/buildbot_annotated_steps.py b/frog/scripts/buildbot_annotated_steps.py
index e433c6b9c3f0dc607fda67fe5780f9a6deef8758..c6c4c6efe4c8f2a407753eb818590db194c052bc 100644
--- a/frog/scripts/buildbot_annotated_steps.py
+++ b/frog/scripts/buildbot_annotated_steps.py
@@ -95,12 +95,30 @@ def TestFrog(arch, mode):
           '-v',
           'leg_only']
   if flags:
-    cmd.append('--flag=' + flags)
+    cmd.append(flags)
 
   status = subprocess.call(cmd)
   if status != 0:
     return status
 
+  if arch == 'frogsh':
+    print ('@@@BUILD_STEP client browser tests@@@')
+    cmd  = ['xvfb-run', sys.executable,
+            '../tools/test.py',
+            '--mode=' + testpy_mode,
+            '--component=frogium',
+            '--time',
+            '--report',
+            '--progress=buildbot',
+            '-v',
+            'client']
+    if flags:
+      cmd.append(flags)

[ngeoffray, 2011/11/11 09:55:58]

The number of steps are getting out of control. We should probably start having
a helper method to avoid duplicated code like this :)

+
+    status = subprocess.call(cmd)
+    if status != 0:
+      return status
+
   print ('@@@BUILD_STEP frog co19 tests: %s@@@' %arch)
   cmd  = [sys.executable,
           '../tools/test.py',
@@ -112,7 +130,7 @@ def TestFrog(arch, mode):
           '-v',
           'co19']
   if flags:
-    cmd.append('--flag=' + flags)
+    cmd.append(flags)
 
   return subprocess.call(cmd)