MD Bookmarks: Implement search and selection in new data flow system

chrome/test/data/webui/md_bookmarks/reducers_test.js

 // Copyright 2017 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.
 
 suite('closed folder state', function() {
   var nodes;
   var initialState;
 
   setup(function() {
     nodes = testTree(createFolder('0', [
@@ skipping 107 matching lines @@
   test('updates when a node is deleted', function() {
     var action = bookmarks.actions.removeBookmark('3', '1', 1);
     var nextState = bookmarks.NodeState.updateNodes(initialState, action);
 
     assertDeepEquals(['2', '4'], nextState['1'].children);
 
     // TODO(tsergeant): Deleted nodes should be removed from the nodes map
     // entirely.
   });
 });
+
+suite('search state', function() {
+  var initialState;
+
+  setup(function() {
+    // Search touches a few different things, so we test using the entire state:
+    initialState = bookmarks.util.createEmptyState();
+    initialState.nodes = testTree(createFolder('0', [
+      createFolder(
+          '1',
+          [
+            createItem('2'),
+          ]),
+    ]));
+  });
+
+  test('updates when search is started and finished', function() {
+    var action;
+    var nextState;
+
+    action = bookmarks.actions.selectFolder('0');

[calamity, 2017/03/13 04:50:36]

Hmm. Can we select a real folder here. The root isn't selectable in the real
world. We should probably actually throw something if the root folder gets
selected.

[tsergeant, 2017/03/14 02:40:36]

Done. I've added an assert to the action to make sure you can't select the root
folder, and updated testTree to add in a root folder automatically.

[calamity, 2017/03/14 03:32:18]

#Like+1.

+    nextState = bookmarks.reduceAction(initialState, action);

[calamity, 2017/03/13 04:50:36]

Initialize nextState with initialState so that this line isn't subtly different.
In fact, it might be a good cargo-cultable pattern to just have 'var state;' at
the top level and then have state = initialState in the setup. (Preferably apply
this to all tests that use an entire state.)

[tsergeant, 2017/03/14 02:40:36]

For this test, I've combined initialState and nextState into a single `state`
variable, since the distinction wasn't necessary.

+
+    action = bookmarks.actions.setSearchTerm('test');
+    nextState = bookmarks.reduceAction(nextState, action);
+
+    assertEquals('test', nextState.search.term);
+    assertTrue(nextState.search.inProgress);
+    // UI should not have changed yet:
+    assertEquals('0', nextState.selectedFolder);
+    assertDeepEquals(['1'], bookmarks.util.getDisplayedList(nextState));
+
+    action = bookmarks.actions.setSearchResults(['1', '2']);
+    var searchedState = bookmarks.reduceAction(nextState, action);
+
+    assertFalse(searchedState.search.inProgress);
+    // UI changes once search results arrive:

[calamity, 2017/03/13 04:50:36]

nit: Newlines before these comments.

[tsergeant, 2017/03/14 02:40:36]

Done.

+    assertEquals(null, searchedState.selectedFolder);
+    assertDeepEquals(
+        ['1', '2'], bookmarks.util.getDisplayedList(searchedState));
+
+    // Case 1: Clear search by setting an empty search term.
+    action = bookmarks.actions.setSearchTerm('');
+    var clearedState = bookmarks.reduceAction(searchedState, action);
+
+    assertEquals('1', clearedState.selectedFolder);
+    assertDeepEquals(['2'], bookmarks.util.getDisplayedList(clearedState));
+    assertEquals('', clearedState.search.term);
+    assertDeepEquals([], clearedState.search.results);
+
+    // Case 2: Clear search by selecting a new folder.
+    action = bookmarks.actions.selectFolder('0');
+    var selectedState = bookmarks.reduceAction(searchedState, action);

[calamity, 2017/03/13 04:50:36]

Cool.

+
+    assertEquals('0', selectedState.selectedFolder);
+    assertDeepEquals(['1'], bookmarks.util.getDisplayedList(selectedState));
+    assertEquals('', selectedState.search.term);
+    assertDeepEquals([], selectedState.search.results);
+  });
+});