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

Unified Diff: chrome/test/data/webui/settings/certificate_manager_page_test.js

Issue 1784553004: MD Settings: Certificate manager, delete confirmation dialog. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@certificates_catrustedit
Patch Set: Addressing comments. Created 4 years, 9 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: chrome/test/data/webui/settings/certificate_manager_page_test.js
diff --git a/chrome/test/data/webui/settings/certificate_manager_page_test.js b/chrome/test/data/webui/settings/certificate_manager_page_test.js
index b43390d12c1d4e8f4175db13a7996b19f028fbbb..0578fc3559464b64e5a688fd7f5b4782805344ca 100644
--- a/chrome/test/data/webui/settings/certificate_manager_page_test.js
+++ b/chrome/test/data/webui/settings/certificate_manager_page_test.js
@@ -15,8 +15,9 @@ cr.define('certificate_manager_page', function() {
/** @private {!Map<string, !PromiseResolver>} */
this.resolverMap_ = new Map();
var wrapperMethods = [
- 'getCaCertificateTrust',
+ 'deleteCertificate',
'editCaCertificateTrust',
+ 'getCaCertificateTrust',
];
wrapperMethods.forEach(this.resetResolver, this);
@@ -62,6 +63,12 @@ cr.define('certificate_manager_page', function() {
});
return Promise.resolve();
},
+
+ /** @override */
+ deleteCertificate: function(id) {
+ this.resolverMap_.get('deleteCertificate').resolve(id);
+ return Promise.resolve();
+ },
};
/** @return {!CertificateSubnode} */
@@ -136,7 +143,51 @@ cr.define('certificate_manager_page', function() {
});
}
+ function registerDeleteDialogTests() {
+ /** @type {?SettingsCertificateDeleteConfirmationDialogElement} */
+ var dialog = null;
+
+ /** @type {?TestCertificatesBrowserProxy} */
+ var browserProxy = null;
+
+ /** @type {!CertificateSubnode} */
+ var model = createSampleCertificateSubnode();
+
+ suite('CertificateDeleteConfirmationDialogTests', function() {
+ setup(function() {
+ browserProxy = new TestCertificatesBrowserProxy();
+ settings.CertificatesBrowserProxyImpl.instance_ = browserProxy;
+ PolymerTest.clearBody();
+ dialog = document.createElement(
+ 'settings-certificate-delete-confirmation-dialog');
+ dialog.model = model;
+ dialog.certificateType = settings.CertificateType.PERSONAL;
+ document.body.appendChild(dialog);
+ });
+
+ teardown(function() { dialog.remove(); });
+
+ test('DeleteSuccess', function() {
+ assertTrue(dialog.$.dialog.opened);
+ // Check that the dialog title includes the certificate name.
+ var titleEl = Polymer.dom(dialog.$.dialog).querySelector('.title');
+ assertTrue(titleEl.textContent.includes(model.name));
+
+ // Simulate clicking 'OK'.
+ MockInteractions.tap(dialog.$.ok);
+
+ return browserProxy.whenCalled('deleteCertificate').then(
+ function(id) {
+ assertEquals(model.id, id);
+ // Check that the dialog is closed.
+ assertFalse(dialog.$.dialog.opened);
+ });
+ });
+ });
+ }
+
return {
registerCaTrustEditDialogTests: registerCaTrustEditDialogTests,
+ registerDeleteDialogTests: registerDeleteDialogTests,
};
});

Powered by Google App Engine
This is Rietveld 408576698