Add a test_name to the Job obj returned from frontend.
Add a test_name for suites to pick up on so that we can properly
dump keyval pairs including job id + test name in a straight forward
way.
TEST=Unittests
BUG=chromium-os:26857
Change-Id: I71963892be338f453dcd0072aa46155d9dd0e2b0
Reviewed-on: https://gerrit.chromium.org/gerrit/17063
Reviewed-by: Chris Masone <cmasone@chromium.org>
Reviewed-by: Scott Zawalski <scottz@chromium.org>
Tested-by: Scott Zawalski <scottz@chromium.org>
Commit-Ready: Scott Zawalski <scottz@chromium.org>
diff --git a/server/cros/dynamic_suite_unittest.py b/server/cros/dynamic_suite_unittest.py
index fa3d348..e44abf8 100755
--- a/server/cros/dynamic_suite_unittest.py
+++ b/server/cros/dynamic_suite_unittest.py
@@ -21,6 +21,7 @@
"""Faked out RPC-client-side Job object."""
def __init__(self, id=0, statuses=[]):
self.id = id
+ self.owner = 'tester'
self.name = 'Fake Job %d' % self.id
self.statuses = statuses
@@ -359,7 +360,7 @@
mox.StrContains(test.name)),
control_type=mox.IgnoreArg(),
meta_hosts=[dynamic_suite.VERSION_PREFIX + self._BUILD],
- dependencies=[])
+ dependencies=[]).AndReturn(FakeJob())
def testScheduleTests(self):
@@ -386,6 +387,8 @@
suite._record_scheduled_jobs()
self.mox.ReplayAll()
suite.schedule()
+ for job in suite._jobs:
+ self.assertTrue(hasattr(job, 'test_name'))
def testScheduleStableTests(self):