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

Side by Side Diff: chrome/test/data/webui/media_router/media_router_elements_browsertest.js

Issue 1744563003: Splitting media router browser tests into separate test cases. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 years, 10 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
« no previous file with comments | « chrome/test/data/webui/media_router/media_router_container_tests.js ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 /** @fileoverview Runs the Media Router Polymer elements tests. */ 5 /** @fileoverview Runs the Media Router Polymer elements tests. */
6 6
7 /** @const {string} Path to source root. */ 7 /** @const {string} Path to source root. */
8 var ROOT_PATH = '../../../../../'; 8 var ROOT_PATH = '../../../../../';
9 9
10 // Polymer BrowserTest fixture. 10 // Polymer BrowserTest fixture.
(...skipping 10 matching lines...) Expand all
21 MediaRouterElementsBrowserTest.prototype = { 21 MediaRouterElementsBrowserTest.prototype = {
22 __proto__: PolymerTest.prototype, 22 __proto__: PolymerTest.prototype,
23 23
24 /** @override */ 24 /** @override */
25 browsePreload: 'chrome://media-router/', 25 browsePreload: 'chrome://media-router/',
26 26
27 commandLineSwitches: [{ 27 commandLineSwitches: [{
28 switchName: 'media-router', switchValue: '1' 28 switchName: 'media-router', switchValue: '1'
29 }], 29 }],
30 30
31 // List tests for individual elements. 31 // List tests for individual elements.
apacible 2016/02/26 22:02:39 nit: Update this comment since the container's tes
32 extraLibraries: PolymerTest.getLibraries(ROOT_PATH).concat([ 32 extraLibraries: PolymerTest.getLibraries(ROOT_PATH).concat([
33 'issue_banner_tests.js', 33 'issue_banner_tests.js',
34 'media_router_container_tests.js', 34 'media_router_container_tests.js',
35 'media_router_container_filter_tests.js',
35 'media_router_header_tests.js', 36 'media_router_header_tests.js',
36 'media_router_search_highlighter.js', 37 'media_router_search_highlighter.js',
37 'route_details_tests.js', 38 'route_details_tests.js',
38 ]), 39 ]),
39 }; 40 };
40 41
41 // Runs all tests. 42 // Runs all tests.
apacible 2016/02/26 22:02:40 nit: Remove this comment since tests are separated
42 // TODO(crbug.com/590177, crbug.com/582649): This test is disabled, as it 43 TEST_F('MediaRouterElementsBrowserTest', 'MediaRouterElementsTestIssue',
imcheng 2016/02/26 21:56:42 naming suggestion: IssueBanner. Similarly for belo
43 // consistently fails on Linux Tests, and is flaky on Win7 and other bots.
44 TEST_F('MediaRouterElementsBrowserTest', 'DISABLED_MediaRouterElementsTest',
45 function() { 44 function() {
46 // Register mocha tests for each element. 45 // Register mocha tests for the issue banner.
47 issue_banner.registerTests(); 46 issue_banner.registerTests();
47
48 // Run all registered tests.
49 mocha.run();
50 });
51
52 TEST_F('MediaRouterElementsBrowserTest', 'MediaRouterElementsTestContainer',
53 function() {
54 // Register mocha tests for the container.
48 media_router_container.registerTests(); 55 media_router_container.registerTests();
56
57 // Run all registered tests.
58 mocha.run();
59 });
60
61 TEST_F('MediaRouterElementsBrowserTest',
62 'MediaRouterElementsTestContainerFilter',
63 function() {
64 // Register mocha tests for the container filter.
65 media_router_container_filter.registerTests();
66
67 // Run all registered tests.
68 mocha.run();
69 });
70
71 TEST_F('MediaRouterElementsBrowserTest', 'MediaRouterElementsTestHeader',
72 function() {
73 // Register mocha tests for the header.
49 media_router_header.registerTests(); 74 media_router_header.registerTests();
75
76 // Run all registered tests.
77 mocha.run();
78 });
79
80 TEST_F('MediaRouterElementsBrowserTest',
81 'MediaRouterElementsTestSearchHighlighter',
82 function() {
83 // Register mocha tests for the search highlighter.
50 media_router_search_highlighter.registerTests(); 84 media_router_search_highlighter.registerTests();
85
86 // Run all registered tests.
87 mocha.run();
88 });
89
90 TEST_F('MediaRouterElementsBrowserTest', 'MediaRouterElementsTestRouteDetails',
91 function() {
92 // Register mocha tests for the route details.
51 route_details.registerTests(); 93 route_details.registerTests();
52 94
53 // Run all registered tests. 95 // Run all registered tests.
54 mocha.run(); 96 mocha.run();
55 }); 97 });
OLDNEW
« no previous file with comments | « chrome/test/data/webui/media_router/media_router_container_tests.js ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698