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

Side by Side Diff: chrome/browser/ui/webui/options/certificate_manager_browsertest.js

Issue 2434473004: Disable many tests with flaky accessibility audits. (Closed)
Patch Set: Disable a bunch of flaky tests Created 4 years, 2 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Mac and Windows go to native certificate manager, and certificate manager 5 // Mac and Windows go to native certificate manager, and certificate manager
6 // isn't implemented if OpenSSL is used. 6 // isn't implemented if OpenSSL is used.
7 GEN('#if defined(USE_NSS_CERTS)'); 7 GEN('#if defined(USE_NSS_CERTS)');
8 8
9 GEN_INCLUDE(['options_browsertest_base.js']); 9 GEN_INCLUDE(['options_browsertest_base.js']);
10 10
(...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after
261 261
262 CertificateManagerSettingsWebUITest.prototype = { 262 CertificateManagerSettingsWebUITest.prototype = {
263 __proto__: CertificateManagerWebUITest.prototype, 263 __proto__: CertificateManagerWebUITest.prototype,
264 264
265 /** 265 /**
266 * Browse to the certificate manager dialog in the Settings page. 266 * Browse to the certificate manager dialog in the Settings page.
267 */ 267 */
268 browsePreload: CERTIFICATE_MANAGER_SETTINGS_PAGE_URL, 268 browsePreload: CERTIFICATE_MANAGER_SETTINGS_PAGE_URL,
269 }; 269 };
270 270
271
272 // TODO(crbug.com/657514) Flakes on linux and chromeos.
273 GEN('#if defined(OS_LINUX) || defined(OS_CHROMEOS)');
274 GEN('#define MAYBE_testViewAndDeleteCert ' +
275 'DISABLED_testViewAndDeleteCert');
276 GEN('#else');
277 GEN('#define MAYBE_testViewAndDeleteCert testViewAndDeleteCert');
278 GEN('#endif');
271 TEST_F('CertificateManagerSettingsWebUITest', 279 TEST_F('CertificateManagerSettingsWebUITest',
272 'testViewAndDeleteCert', function() { 280 'MAYBE_testViewAndDeleteCert', function() {
273 assertEquals(this.browsePreload, document.location.href); 281 assertEquals(this.browsePreload, document.location.href);
274 282
275 this.mockHandler.expects(once()).viewCertificate(['c1']); 283 this.mockHandler.expects(once()).viewCertificate(['c1']);
276 284
277 expectTrue($('personalCertsTab-view').disabled); 285 expectTrue($('personalCertsTab-view').disabled);
278 expectTrue($('personalCertsTab-backup').disabled); 286 expectTrue($('personalCertsTab-backup').disabled);
279 expectTrue($('personalCertsTab-delete').disabled); 287 expectTrue($('personalCertsTab-delete').disabled);
280 expectFalse($('personalCertsTab-import').disabled); 288 expectFalse($('personalCertsTab-import').disabled);
281 if (this.isChromeOS) 289 if (this.isChromeOS)
282 expectFalse($('personalCertsTab-import-and-bind').disabled); 290 expectFalse($('personalCertsTab-import-and-bind').disabled);
(...skipping 113 matching lines...) Expand 10 before | Expand all | Expand 10 after
396 caCertsTab.querySelector('div.tree-item').click(); 404 caCertsTab.querySelector('div.tree-item').click();
397 var certs = caCertsTab.querySelectorAll('div.tree-item div.tree-item'); 405 var certs = caCertsTab.querySelectorAll('div.tree-item div.tree-item');
398 406
399 // There should be exactly three certificates displayed. 407 // There should be exactly three certificates displayed.
400 expectEquals(certs.length, 3); 408 expectEquals(certs.length, 3);
401 }); 409 });
402 410
403 GEN('#endif // defined(OS_CHROMEOS)'); 411 GEN('#endif // defined(OS_CHROMEOS)');
404 412
405 GEN('#endif // defined(USE_NSS_CERTS)'); 413 GEN('#endif // defined(USE_NSS_CERTS)');
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698