Index: client/site_tests/security_BundledCRXs/security_BundledCRXs.py |
diff --git a/client/site_tests/security_BundledCRXs/security_BundledCRXs.py b/client/site_tests/security_BundledCRXs/security_BundledCRXs.py |
index 1205cc3bee0200b392a5a2e50cb077e979bd589a..0a8fb961c8c156d821ca81d93f47ef02e85250ed 100644 |
--- a/client/site_tests/security_BundledCRXs/security_BundledCRXs.py |
+++ b/client/site_tests/security_BundledCRXs/security_BundledCRXs.py |
@@ -53,14 +53,11 @@ class security_BundledCRXs(test.test): |
for crx in diff: |
logging.error('New/unexpected bundled crx %s' % crx) |
- # Or, things in baseline are missing from the system. We log missing |
- # CRX's as warnings instead of test failures because we autotest some |
- # builds where extensions aren't bundled. See chrome-os-partner:2414. |
- # TODO(jimhebert) make these warnings fatal after 2414 is resolved. |
+ # Or, things in baseline are missing from the system: |
diff2 = baseline_set.difference(observed_set) |
if len(diff2) > 0: |
for crx in diff2: |
- logging.warning('Missing bundled crx %s' % crx) |
+ logging.error('Missing bundled crx %s' % crx) |
- if len(diff): |
+ if (len(diff) + len(diff2)) > 0: |
raise error.TestFail('Baseline mismatch') |