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

Unified Diff: scripts/slave/unittests/expect_tests/serialize.py

Issue 354913003: Add module discovery and autoloading to expect_tests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/tools/build
Patch Set: add --force_coverage option Created 6 years, 6 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
Index: scripts/slave/unittests/expect_tests/serialize.py
diff --git a/scripts/slave/unittests/expect_tests/serialize.py b/scripts/slave/unittests/expect_tests/serialize.py
index 2d498d9d1c647151a2b85c10a8e90bf675703083..940921aed7231930edc0a090d06d492b3ef63827 100644
--- a/scripts/slave/unittests/expect_tests/serialize.py
+++ b/scripts/slave/unittests/expect_tests/serialize.py
@@ -79,7 +79,7 @@ def GetCurrentData(test):
for ext in sorted(SUPPORTED_SERIALIZERS, key=lambda s: s != test.ext):
path = test.expect_path(ext)
if path is None:
- return None, True
+ return NonExistant, True
if ext not in SERIALIZERS and ext == test.ext:
raise Exception('The package to support %s is not installed.' % ext)

Powered by Google App Engine
This is Rietveld 408576698