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

Unified Diff: chrome/test/data/extensions/api_test/networking_private/chromeos/test.js

Issue 2891453002: Introduce networkingPrivate.getCertificateLists (Closed)
Patch Set: . Created 3 years, 7 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/extensions/api_test/networking_private/chromeos/test.js
diff --git a/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js b/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
index 388390dd53bf0cae6f36e707462163e369155eaa..965730d966695182e705f2b3768cb591c5c63355 100644
--- a/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
+++ b/chrome/test/data/extensions/api_test/networking_private/chromeos/test.js
@@ -799,6 +799,15 @@ var availableTests = [
chrome.networkingPrivate.onDeviceStateListChanged.addListener(listener);
chrome.networkingPrivate.disableNetworkType('WiFi');
},
+ function onCertificateListsChangedEvent() {
+ var listener = callbackPass(function() {
tbarzic 2017/05/17 22:35:57 chrome.test.listenOnce should do something similar
stevenjb 2017/05/18 16:30:57 Done.
+ chrome.networkingPrivate.onCertificateListsChanged.removeListener(
+ listener);
+ });
+ chrome.networkingPrivate.onCertificateListsChanged.addListener(listener);
+ // The test delegate will send a change event when we request the certs.
+ chrome.networkingPrivate.getCertificateLists(function() {console.info('GOT CERTS');});
tbarzic 2017/05/17 22:35:57 remove log (and add callbackPass so the test fail
stevenjb 2017/05/18 16:30:57 Done.
+ },
function verifyDestination() {
chrome.networkingPrivate.verifyDestination(
verificationProperties,
@@ -971,6 +980,13 @@ var availableTests = [
'tetherName1', 50, 'tetherCarrier1', 75, true);
}));
},
+ function getCertificateLists() {
+ chrome.networkingPrivate.getCertificateLists(
+ callbackPass(function(certificateLists) {
+ assertEq(1, certificateLists.server_ca_certificates.length);
+ assertEq(0, certificateLists.user_certificates.length);
+ }));
+ },
];
chrome.test.getConfig(function(config) {

Powered by Google App Engine
This is Rietveld 408576698