Non-Local Join for Media Router and Presentation API

chrome/test/data/webui/media_router/route_details_tests.js

 // Copyright 2015 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 /** @fileoverview Suite of tests for route-details. */
 cr.define('route_details', function() {
   function registerTests() {
     suite('RouteDetails', function() {
       /**
        * Route Details created before each test.
@@ skipping 19 matching lines @@
         assertEquals(expected,
             details.$[elementId].querySelector('span').innerText);
       };
 
       // Checks the default route view is shown.
       var checkDefaultViewIsShown = function() {
         assertFalse(details.$['route-information'].hasAttribute('hidden'));
         assertTrue(details.$['custom-controller'].hasAttribute('hidden'));
       };
 
+      // Checks the default route view is shown.
+      var checkJoinButtonIsShown = function() {
+        assertFalse(details.$['join-route-button'].hasAttribute('hidden'));
+      };
+
+      // Checks the default route view is not shown.
+      var checkJoinButtonIsNotShown = function() {
+        assertTrue(details.$['join-route-button'].hasAttribute('hidden'));
+      };
+
       // Checks the custom controller is shown.
       var checkCustomControllerIsShown = function() {
         assertTrue(details.$['route-information'].hasAttribute('hidden'));
         assertFalse(details.$['custom-controller'].hasAttribute('hidden'));
       };
 
       // Checks whether |expected| and the text in the |elementId| element
       // are equal given an id.
       var checkElementTextWithId = function(expected, elementId) {
         assertEquals(expected, details.$[elementId].innerText);
       };
 
       // Import route_details.html before running suite.
       suiteSetup(function() {
         return PolymerTest.importHtml(
             'chrome://media-router/elements/route_details/' +
             'route_details.html');
       });
 
       // Initialize a route-details before each test.
       setup(function(done) {
         PolymerTest.clearBody();
         details = document.createElement('route-details');
         document.body.appendChild(details);
 
         // Initialize routes and sinks.
         fakeRouteOne = new media_router.Route('route id 1', 'sink id 1',
-            'Video 1', 1, true, 'chrome-extension://123/custom_view.html');
+            'Video 1', 1, true, false,
+            'chrome-extension://123/custom_view.html');
         fakeRouteTwo = new media_router.Route('route id 2', 'sink id 2',
-            'Video 2', 2, false);
+            'Video 2', 2, false, true);
+        fakeSinkOne = new media_router.Sink('sink id 1', 'Living Room',

[apacible, 2015/12/09 19:56:25]

What are fakeSinkOne and fakeSinkTwo used for?

[matt.boetger, 2015/12/16 00:21:08]

Nothing anymore.  Thanks for catching.

+            media_router.SinkIconType.CAST,
+            media_router.SinkStatus.ACTIVE, [0, 1, 2]);
+        fakeSinkTwo = new media_router.Sink('sink id 2', 'my device',
+            media_router.SinkIconType.CAST,
+            media_router.SinkStatus.ACTIVE, [0, 1, 2]);
 
         // Allow for the route details to be created and attached.
         setTimeout(done);
       });
 
       // Tests for 'close-route-click' event firing when the
       // 'close-route-button' button is clicked.
       test('close route button click', function(done) {
         details.addEventListener('close-route-click', function() {
           done();
         });
         MockInteractions.tap(details.$['close-route-button']);
       });
 
+      // Tests for 'close-route-click' event firing when the
+      // 'close-route-button' button is clicked.

[apacible, 2015/12/09 19:56:25]

nit: 'join-route-button'

[matt.boetger, 2015/12/16 00:21:08]

Done.

+      test('join route button click', function(done) {
+        details.addEventListener('join-route-click', function() {
+          done();
+        });
+        MockInteractions.tap(details.$['join-route-button']);
+      });
+
       // Tests the initial expected text.
       test('initial text setting', function() {
         // <paper-button> text is styled as upper case.
         checkSpanText(loadTimeData.getString('stopCastingButton')
             .toUpperCase(), 'close-route-button');
+        // unless it's hidden (join is hidden unless the route can be joined)

[apacible, 2015/12/09 19:56:25]

This is a fragment; is this a continuation from the previous comment? If so,
please make the comment standalone to not rely on the previous one.

[apacible, 2015/12/09 19:56:25]

nit: period at the end.

[matt.boetger, 2015/12/16 00:21:08]

Done.

[matt.boetger, 2015/12/16 00:21:08]

Done.

+        checkSpanText(loadTimeData.getString('joinButton'),
+                      'join-route-button');
         checkSpanText('', 'route-information');
       });
 
       // Tests when |route| is null or set.
       test('route is null or set', function() {
         // |route| is null.
         assertEquals(null, details.route);
         checkDefaultViewIsShown();
 
         // Set |route| to be non-null.
         details.route = fakeRouteOne;
         assertEquals(fakeRouteOne, details.route);
         checkSpanText(loadTimeData.getStringF('castingActivityStatus',
             fakeRouteOne.description), 'route-information');
         checkDefaultViewIsShown();
+        checkJoinButtonIsNotShown();
 
         // Set |route| to a different route.
         details.route = fakeRouteTwo;
         assertEquals(fakeRouteTwo, details.route);
         checkSpanText(loadTimeData.getStringF('castingActivityStatus',
             fakeRouteTwo.description), 'route-information');
         checkDefaultViewIsShown();
+        checkJoinButtonIsShown();
       });
 
       // Tests when |route| exists, has a custom controller, and it loads.
       test('route has custom controller and loading succeeds', function(done) {
         var loadInvoked = false;
         details.$['custom-controller'].load = function(url) {
           loadInvoked = true;
           assertEquals('chrome-extension://123/custom_view.html', url);
           return Promise.resolve();
         };
@@ skipping 21 matching lines @@
           done();
         });
       });
     });
   }
 
   return {
     registerTests: registerTests,
   };
 });