fix(nav-bar): add test for handling when a nav item's controller isn't found (#11964)

* add unit test
* add missing JSDoc

Co-authored-by: Tomasz Grabowski <[email protected]>
Co-authored-by: Michael Prentice <[email protected]>

Author: 3 people

src/components/navBar/navBar.js

@@ -328,7 +328,7 @@ MdNavBarController.prototype.getFocusedTab = function() {
  * Find a tab that matches the specified function, starting from the first tab.
  * @param {Function} fn
  * @param {number=} startIndex index to start at. Defaults to 0.
- * @returns {MdNavItemController}
+ * @returns {MdNavItemController|null}
  * @private
  */
 MdNavBarController.prototype._findTab = function(fn, startIndex) {

src/components/navBar/navBar.spec.js

@@ -254,6 +254,17 @@ describe('mdNavBar', function() {
       expect(ctrl._updateInkBarStyles)
         .not.toHaveBeenCalled();
     });
+
+    it('does not update selected tab if controller is undefined', function() {
+      $scope.selectedTabRoute = 'tab1';
+
+      spyOn(Object.getPrototypeOf(ctrl), '_updateInkBarStyles');
+      spyOn(Object.getPrototypeOf(ctrl), '_findTab').and.returnValue(null);
+      createTabs();
+
+      expect(ctrl._updateInkBarStyles)
+      .toHaveBeenCalledWith(null, -1);
+    });
   });
 
   describe('inkbar', function() {