| 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')
|
|
|