MD Settings: Implement search URLs.

chrome/test/data/webui/settings/settings_main_test.js

 // Copyright 2016 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.
 
 cr.define('settings_main_page', function() {
   /**
    * @implements {SearchManager}
+   * @extends {settings.TestBrowserProxy}

[tommycli, 2016/10/27 21:07:17]

If I understand correctly? : The TestSearchManager isn't actually mocking a
browser proxy... it's just leveraging the TestBrowserProxy whenCalled machinery?
If so, it would be worth adding a comment explaining what's going on.

One more thing -- I'm not sure you actually need the whole methodCalled Promise
business, since I think everything is synchronous (correct me if there's any
async bits). Maybe you only need to set a lastQuery variable.

[dpapad, 2016/10/28 20:23:39]

Added comment. Even though TestBrowserProxy has been traditionally used with
BrowserProxy classes, it is actually able to act as a proxy for any external
dependency. I am using it here to ensure that the code being tested is making
the right calls to SearchManager, which is not part of this unit test.
The call to SearchManager#search is happening within a setTimeout() see
https://cs.chromium.org/chromium/src/chrome/browser/resources/settings/settin....

    */
   var TestSearchManager = function() {
+    settings.TestBrowserProxy.call(this, [
+      'search',
+    ]);
+
     /** @private {boolean} */
     this.matchesFound_ = true;
 
     /** @private {?settings.SearchRequest} */
     this.searchRequest_ = null;
   }
 
   TestSearchManager.prototype = {
+    __proto__: settings.TestBrowserProxy.prototype,
+
     /**
      * @param {boolean} matchesFound
      */
     setMatchesFound: function(matchesFound) {
       this.matchesFound_ = matchesFound;
     },
 
     /** @override */
     search: function(text, page) {
+      this.methodCalled('search', text);
+
       if (this.searchRequest_ == null || !this.searchRequest_.isSame(text)) {
         this.searchRequest_ = new settings.SearchRequest(text);
         this.searchRequest_.finished = true;
         this.searchRequest_.updateMatches(this.matchesFound_);
         this.searchRequest_.resolver.resolve(this.searchRequest_);
       }
       return this.searchRequest_.resolver.promise;
     },
   };
 
@@ skipping 18 matching lines @@
         settings.setSearchManagerForTesting(searchManager);
         PolymerTest.clearBody();
         settingsMain = document.createElement('settings-main');
         settingsMain.prefs = settingsPrefs.prefs;
         settingsMain.toolbarSpinnerActive = false;
         document.body.appendChild(settingsMain);
       });
 
       teardown(function() { settingsMain.remove(); });
 
+      /**
+       * Triggers a navigation to a URL that has a 'search' parameter for the
+       * given query.
+       * @param {string} query
+       */
+      function triggerSearchByUrl(query) {
+        settings.navigateTo(
+            settings.Route.BASIC,
+            new URLSearchParams(`search=${query}`));
+      }
+
+      test('search URL parameter triggers search', function() {
+        Polymer.dom.flush();
+
+        var expectedQuery1 = 'foo';
+        var expectedQuery2 = 'bar';
+        var expectedQuery3 = '';
+
+        triggerSearchByUrl(expectedQuery1);
+        return searchManager.whenCalled('search').then(function(query) {
+          assertEquals(expectedQuery1, query);
+
+          searchManager.resetResolver('search');
+          triggerSearchByUrl(expectedQuery2);
+          return searchManager.whenCalled('search');
+        }).then(function(query) {
+          assertEquals(expectedQuery2, query);
+
+          searchManager.resetResolver('search');
+          triggerSearchByUrl(expectedQuery3);
+          return searchManager.whenCalled('search');
+        }).then(function(query) {
+          assertEquals(expectedQuery3, query);
+        });
+      });
+
       test('no results page shows and hides', function() {
         Polymer.dom.flush();
         var noSearchResults = settingsMain.$.noSearchResults;
         assertTrue(!!noSearchResults);
         assertTrue(noSearchResults.hidden);
 
         var toggleContainer = settingsMain.$$('#toggleContainer');
         assertTrue(!!toggleContainer);
         assertNotEquals('none', toggleContainer.style.display);
 
         searchManager.setMatchesFound(false);
-        return settingsMain.searchContents('Query1').then(function() {
+        return settingsMain.searchContents_('Query1').then(function() {
           assertFalse(noSearchResults.hidden);
           assertEquals('none', toggleContainer.style.display);
 
           searchManager.setMatchesFound(true);
-          return settingsMain.searchContents('Query2');
+          return settingsMain.searchContents_('Query2');
         }).then(function() {
           assertTrue(noSearchResults.hidden);
         });
       });
 
       // Ensure that when the user clears the search box, the "no results" page
       // is hidden and the "advanced page toggle" is visible again.
       test('no results page hides on clear', function() {
         Polymer.dom.flush();
         var noSearchResults = settingsMain.$.noSearchResults;
         assertTrue(!!noSearchResults);
         assertTrue(noSearchResults.hidden);
 
         var toggleContainer = settingsMain.$$('#toggleContainer');
         assertTrue(!!toggleContainer);
         assertNotEquals('none', toggleContainer.style.display);
 
         searchManager.setMatchesFound(false);
         // Clearing the search box is effectively a search for the empty string.
-        return settingsMain.searchContents('').then(function() {
+        return settingsMain.searchContents_('').then(function() {
           Polymer.dom.flush();
           assertTrue(noSearchResults.hidden);
           assertNotEquals('none', toggleContainer.style.display);
         });
       });
 
       /**
        * Asserts the visibility of the basic and advanced pages after exiting
        * search mode.
        * @param {string} Expected 'display' value for the basic page.
        * @param {string} Expected 'display' value for the advanced page.
        * @return {!Promise}
        */
       function assertPageVisibilityAfterSearch(
           expectedBasic, expectedAdvanced) {
         searchManager.setMatchesFound(true);
-        return settingsMain.searchContents('Query1').then(function() {
+        return settingsMain.searchContents_('Query1').then(function() {
           searchManager.setMatchesFound(false);
-          return settingsMain.searchContents('');
+          return settingsMain.searchContents_('');
         }).then(function() {
           Polymer.dom.flush();
           assertEquals(
               expectedBasic,
               settingsMain.$$('settings-basic-page').style.display);
           assertEquals(
               expectedAdvanced,
               settingsMain.$$('settings-advanced-page').style.display);
         });
       }
@@ skipping 15 matching lines @@
       });
 
       // Ensure that searching, then entering a subpage, then going back
       // lands the user in a page where both basic and advanced sections are
       // visible, because the page is still in search mode.
       test('returning from subpage to search results', function() {
         settings.navigateTo(settings.Route.BASIC);
         Polymer.dom.flush();
 
         searchManager.setMatchesFound(true);
-        return settingsMain.searchContents('Query1').then(function() {
+        return settingsMain.searchContents_('Query1').then(function() {
           // Simulate navigating into a subpage.
           settings.navigateTo(settings.Route.SEARCH_ENGINES);
           settingsMain.$$('settings-basic-page').fire('subpage-expand');
           Polymer.dom.flush();
 
           // Simulate clicking the left arrow to go back to the search results.
           settingsMain.currentRouteChanged(settings.Route.BASIC);
           Polymer.dom.flush();
           assertEquals(
               '', settingsMain.$$('settings-basic-page').style.display);
@@ skipping 27 matching lines @@
 
         assertTrue(settingsMain.showPages_.advanced);
       });
     });
   }
 
   return {
     registerTests: registerTests,
   };
 });