Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(176)

Unified Diff: client/site_tests/factory_Verify/factory_Verify.py

Issue 3340013: Fix status reporting and auto-seq logic. (Closed) Base URL: ssh://git@gitrw.chromium.org:9222/autotest.git
Patch Set: address comment, and patch factory_Verify Created 10 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « client/bin/factory_ui ('k') | client/site_tests/suite_Factory/control » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: client/site_tests/factory_Verify/factory_Verify.py
diff --git a/client/site_tests/factory_Verify/factory_Verify.py b/client/site_tests/factory_Verify/factory_Verify.py
index 4a68666fbb87c04398876792943b466daff4a45d..a95021d9bef7a101449a68a613f17f44ff48060a 100644
--- a/client/site_tests/factory_Verify/factory_Verify.py
+++ b/client/site_tests/factory_Verify/factory_Verify.py
@@ -68,20 +68,19 @@ class factory_Verify(test.test):
return
# check if all previous tests are passed.
- status_map = factory.StatusMap(test_list, status_file)
- db = status_map.test_db
+ db = factory.TestDatabase(test_list)
+ status_map = factory.StatusMap(test_list, status_file, db)
failed_list = status_map.filter(ful.FAILED)
if failed_list:
- failed = ','.join([db.get_unique_details(t) for t in failed_list])
- raise error.TestFail('Some previous tests failed: %s' %
- failed)
+ failed = ','.join([db.get_unique_id_str(t) for t in failed_list])
+ raise error.TestFail('Some previous tests failed: %s' % failed)
# check if all Google Required Tests are passed
missing = []
for g in GOOGLE_REQUIRED_TESTS:
t = db.get_test_by_unique_name(g)
if status_map.lookup_status(t) != ful.PASSED:
- missing.append('%s(%s)' % (g, db.get_unique_details(t)))
+ missing.append('%s(%s)' % (g, db.get_unique_id_str(t)))
if missing:
missing_msg = ', '.join(missing)
raise error.TestFail('You need to execute following ' +
« no previous file with comments | « client/bin/factory_ui ('k') | client/site_tests/suite_Factory/control » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698