From 45a1207cdf3471c0e32af64f28f2de7dd207aa41 Mon Sep 17 00:00:00 2001 From: Nihanth Subramanya Date: Wed, 24 Feb 2016 22:45:12 -0800 Subject: [PATCH] Bug 1201437 - Make cert override tests check for STATE_CERT_USER_OVERRIDDEN. r=keeler MozReview-Commit-ID: G6KQPXHbEPL --HG-- extra : rebase_source : 9ed61d521996d96d2d18f5d602439bedc46393c0 --- security/manager/ssl/tests/unit/head_psm.js | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/security/manager/ssl/tests/unit/head_psm.js b/security/manager/ssl/tests/unit/head_psm.js index 9230a74fc33a..1b1f24f5e609 100644 --- a/security/manager/ssl/tests/unit/head_psm.js +++ b/security/manager/ssl/tests/unit/head_psm.js @@ -662,7 +662,11 @@ function add_cert_override(aHost, aExpectedBits, aSecurityInfo) { function add_cert_override_test(aHost, aExpectedBits, aExpectedError) { add_connection_test(aHost, aExpectedError, null, add_cert_override.bind(this, aHost, aExpectedBits)); - add_connection_test(aHost, PRErrorCodeSuccess); + add_connection_test(aHost, PRErrorCodeSuccess, null, aSecurityInfo => { + Assert.ok(aSecurityInfo.securityState & + Ci.nsIWebProgressListener.STATE_CERT_USER_OVERRIDDEN, + "Cert override flag should be set on the security state"); + }); } // Helper function for add_prevented_cert_override_test. This is much like