Load Passwords and Autofill in the corresponding sub page.

chrome/test/data/webui/settings/settings_passwords_section_browsertest.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 Runs the Polymer Password Settings tests. */
 
 /** @const {string} Path to root from chrome/test/data/webui/settings/. */
 var ROOT_PATH = '../../../../../';
 
 // Polymer BrowserTest fixture.
@@ skipping 16 matching lines @@
   /** @override */
   browsePreload:
       'chrome://md-settings/passwords_and_forms_page/passwords_section.html',
 
   /** @override */
   extraLibraries: PolymerTest.getLibraries(ROOT_PATH),
 
   /** @override */
   setUp: function() {
     PolymerTest.prototype.setUp.call(this);
-    PolymerTest.clearBody();

[Dan Beam, 2017/02/15 23:19:39]

why are you removing this?

[hcarmona, 2017/02/16 00:00:01]

Moved it to Mocha's setup so it runs between each test.
Updated in the other tests to be consistent.

-
     // Test is run on an individual element that won't have a page language.
     this.accessibilityAuditConfig.auditRulesToIgnore.push('humanLangMissing');
   },
 
+};
+
+/** This test will validate that the section is loaded with data. */
+TEST_F('SettingsPasswordSectionBrowserTest', 'uiTests', function() {
   /**
    * Helper method that validates a that elements in the password list match
    * the expected data.
    * @param {!Element} listElement The iron-list element that will be checked.
    * @param {!Array<!chrome.passwordsPrivate.PasswordUiEntry>} passwordList The
    *     expected data.
    * @private
    */
-  validatePasswordList: function(listElement, passwordList) {
+  function validatePasswordList(listElement, passwordList) {
     assertEquals(passwordList.length, listElement.items.length);
     if (passwordList.length > 0) {
       // The first child is a template, skip and get the real 'first child'.
       var node = Polymer.dom(listElement).children[1];
       assert(node);
       var passwordInfo = passwordList[0];
       assertEquals(passwordInfo.loginPair.originUrl,
           node.querySelector('#originUrl').textContent.trim());
       assertEquals(passwordInfo.linkUrl,
           node.querySelector('#originUrl').href);
       assertEquals(passwordInfo.loginPair.username,
           node.querySelector('#username').textContent);
       assertEquals(passwordInfo.numCharactersInPassword,
           node.querySelector('#password').value.length);
     }
-  },
+  }
 
   /**
    * Helper method that validates a that elements in the exception list match
    * the expected data.
    * @param {!Array<!Element>} nodes The nodes that will be checked.
    * @param {!Array<!chrome.passwordsPrivate.ExceptionPair>} exceptionList The
    *     expected data.
    * @private
    */
-  validateExceptionList_: function(nodes, exceptionList) {
+  function validateExceptionList(nodes, exceptionList) {
     assertEquals(exceptionList.length, nodes.length);
     for (var index = 0; index < exceptionList.length; ++index) {
       var node = nodes[index];
       var exception = exceptionList[index];
       assertEquals(exception.exceptionUrl,
                    node.querySelector('#exception').textContent);
       assertEquals(exception.linkUrl,
                    node.querySelector('#exception').href);
     }
-  },
+  }
 
   /**
    * Returns all children of an element that has children added by a dom-repeat.
    * @param {!Element} element
    * @return {!Array<!Element>}
    * @private
    */
-  getDomRepeatChildren_: function(element) {
+  function getDomRepeatChildren(element) {
     var nodes = element.querySelectorAll('.list-item:not([id])');
     return nodes;
-  },
+  }
+
+  /**
+   * Allow the iron-list to be sized properly.
+   * @param {!Object} passwordsSection
+   * @private
+   */
+  function flushPasswordSection(passwordsSection) {
+    passwordsSection.$.passwordList.notifyResize();
+    Polymer.dom.flush();
+  }
 
   /**
    * Helper method used to create a password section for the given lists.
+   * @param {!PasswordManager} passwordManager
    * @param {!Array<!chrome.passwordsPrivate.PasswordUiEntry>} passwordList
    * @param {!Array<!chrome.passwordsPrivate.ExceptionPair>} exceptionList
    * @return {!Object}
    * @private
    */
-  createPasswordsSection_: function(passwordList, exceptionList) {
+  function createPasswordsSection(passwordManager, passwordList,
+      exceptionList) {
+    // Override the PasswordManager data for testing.
+    passwordManager.data.passwords = passwordList;
+    passwordManager.data.exceptions = exceptionList;
+
     // Create a passwords-section to use for testing.
     var passwordsSection = document.createElement('passwords-section');
-    passwordsSection.savedPasswords = passwordList;
-    passwordsSection.passwordExceptions = exceptionList;
     document.body.appendChild(passwordsSection);
-    this.flushPasswordSection_(passwordsSection);
+    flushPasswordSection(passwordsSection);
     return passwordsSection;
-  },
+  }
 
   /**
    * Helper method used to create a password editing dialog.
    * @param {!chrome.passwordsPrivate.PasswordUiEntry} passwordItem
    * @return {!Object}
    * @private
    */
-  createPasswordDialog_: function(passwordItem) {
+  function createPasswordDialog(passwordItem) {
     var passwordDialog = document.createElement('password-edit-dialog');
     passwordDialog.item = passwordItem;
     document.body.appendChild(passwordDialog);
     Polymer.dom.flush();
     return passwordDialog;
-  },
+  }
 
   /**
    * Helper method used to test for a url in a list of passwords.
    * @param {!Array<!chrome.passwordsPrivate.PasswordUiEntry>} passwordList
    * @param {string} url The URL that is being searched for.
    */
-  listContainsUrl(passwordList, url) {
+  function listContainsUrl(passwordList, url) {
     for (var i = 0; i < passwordList.length; ++i) {
       if (passwordList[i].loginPair.originUrl == url)
         return true;
     }
     return false;
-  },
+  }
 
   /**
    * Helper method used to test for a url in a list of passwords.
    * @param {!Array<!chrome.passwordsPrivate.ExceptionPair>} exceptionList
    * @param {string} url The URL that is being searched for.
    */
-  exceptionsListContainsUrl(exceptionList, url) {
+  function exceptionsListContainsUrl(exceptionList, url) {
     for (var i = 0; i < exceptionList.length; ++i) {
       if (exceptionList[i].exceptionUrl == url)
         return true;
     }
     return false;
-  },
-
-  /**
-   * Allow the iron-list to be sized properly.
-   * @param {!Object} passwordsSection
-   * @private
-   */
-  flushPasswordSection_: function(passwordsSection) {
-    passwordsSection.$.passwordList.notifyResize();
-    Polymer.dom.flush();
-  },
-};
-
-/** This test will validate that the section is loaded with data. */
-TEST_F('SettingsPasswordSectionBrowserTest', 'uiTests', function() {
-  var self = this;
+  }
 
   suite('PasswordsSection', function() {
+    /** @type {TestPasswordManager} */
+    var passwordManager = null;
+
+    setup(function() {
+      PolymerTest.clearBody();
+      // Override the PasswordManagerImpl for testing.
+      passwordManager = new TestPasswordManager();
+      PasswordManagerImpl.instance_ = passwordManager;
+    });
+
     test('verifyNoSavedPasswords', function() {
-      var passwordsSection = self.createPasswordsSection_([], []);
+      var passwordsSection = createPasswordsSection(passwordManager, [], []);
 
-      self.validatePasswordList(passwordsSection.$.passwordList, []);
+      validatePasswordList(passwordsSection.$.passwordList, []);
 
       assertFalse(passwordsSection.$.noPasswordsLabel.hidden);
       assertTrue(passwordsSection.$.savedPasswordsHeading.hidden);
     });
 
     test('verifySavedPasswordLength', function() {
       var passwordList = [
         FakeDataMaker.passwordEntry('site1.com', 'luigi', 1),
         FakeDataMaker.passwordEntry('longwebsite.com', 'peach', 7),
         FakeDataMaker.passwordEntry('site2.com', 'mario', 70),
         FakeDataMaker.passwordEntry('site1.com', 'peach', 11),
         FakeDataMaker.passwordEntry('google.com', 'mario', 7),
         FakeDataMaker.passwordEntry('site2.com', 'luigi', 8),
       ];
 
-      var passwordsSection = self.createPasswordsSection_(passwordList, []);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, passwordList, []);
 
       // Assert that the data is passed into the iron list. If this fails,
       // then other expectations will also fail.
       assertEquals(passwordList, passwordsSection.$.passwordList.items);
 
-      self.validatePasswordList(passwordsSection.$.passwordList, passwordList);
+      validatePasswordList(passwordsSection.$.passwordList, passwordList);
 
       assertTrue(passwordsSection.$.noPasswordsLabel.hidden);
       assertFalse(passwordsSection.$.savedPasswordsHeading.hidden);
     });
 
     // Test verifies that removing a password will update the elements.
     test('verifyPasswordListRemove', function() {
       var passwordList = [
         FakeDataMaker.passwordEntry('anotherwebsite.com', 'luigi', 1),
         FakeDataMaker.passwordEntry('longwebsite.com', 'peach', 7),
         FakeDataMaker.passwordEntry('website.com', 'mario', 70)
       ];
 
-      var passwordsSection = self.createPasswordsSection_(passwordList, []);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, passwordList, []);
 
-      self.validatePasswordList(passwordsSection.$.passwordList, passwordList);
+      validatePasswordList(passwordsSection.$.passwordList, passwordList);
       // Simulate 'longwebsite.com' being removed from the list.
       passwordsSection.splice('savedPasswords', 1, 1);
-      self.flushPasswordSection_(passwordsSection);
+      flushPasswordSection(passwordsSection);
 
-      assertFalse(self.listContainsUrl(passwordsSection.savedPasswords,
+      assertFalse(listContainsUrl(passwordsSection.savedPasswords,
                                        'longwebsite.com'));
-      assertFalse(self.listContainsUrl(passwordList, 'longwebsite.com'));
+      assertFalse(listContainsUrl(passwordList, 'longwebsite.com'));
 
-      self.validatePasswordList(passwordsSection.$.passwordList, passwordList);
+      validatePasswordList(passwordsSection.$.passwordList, passwordList);
     });
 
     // Test verifies that pressing the 'remove' button will trigger a remove
     // event. Does not actually remove any passwords.
     test('verifyPasswordItemRemoveButton', function(done) {
       var passwordList = [
         FakeDataMaker.passwordEntry('one', 'six', 5),
         FakeDataMaker.passwordEntry('two', 'five', 3),
         FakeDataMaker.passwordEntry('three', 'four', 1),
         FakeDataMaker.passwordEntry('four', 'three', 2),
         FakeDataMaker.passwordEntry('five', 'two', 4),
         FakeDataMaker.passwordEntry('six', 'one', 6),
       ];
 
-      var passwordsSection = self.createPasswordsSection_(passwordList, []);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, passwordList, []);
 
       // The first child is a template, skip and get the real 'first child'.
       var firstNode = Polymer.dom(passwordsSection.$.passwordList).children[1];
       assert(firstNode);
       var firstPassword = passwordList[0];
 
-      // Listen for the remove event. If this event isn't received, the test
-      // will time out and fail.
-      passwordsSection.addEventListener('remove-saved-password',
-                                        function(event) {
+      passwordManager.onRemoveSavedPassword = function(detail) {
         // Verify that the event matches the expected value.
-        assertEquals(firstPassword.loginPair.originUrl,
-                     event.detail.originUrl);
-        assertEquals(firstPassword.loginPair.username,
-                     event.detail.username);
+        assertEquals(firstPassword.loginPair.originUrl, detail.originUrl);
+        assertEquals(firstPassword.loginPair.username, detail.username);
+
+        // Clean up after self.
+        passwordManager.onRemoveSavedPassword = null;
+
         done();
-      });
+      };
 
       // Click the remove button on the first password.
       MockInteractions.tap(firstNode.querySelector('#passwordMenu'));
       MockInteractions.tap(passwordsSection.$.menuRemovePassword);
     });
 
     test('verifyFilterPasswords', function() {
       var passwordList = [
         FakeDataMaker.passwordEntry('one.com', 'show', 5),
         FakeDataMaker.passwordEntry('two.com', 'shower', 3),
         FakeDataMaker.passwordEntry('three.com/show', 'four', 1),
         FakeDataMaker.passwordEntry('four.com', 'three', 2),
         FakeDataMaker.passwordEntry('five.com', 'two', 4),
         FakeDataMaker.passwordEntry('six-show.com', 'one', 6),
       ];
 
-      var passwordsSection = self.createPasswordsSection_(passwordList, []);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, passwordList, []);
       passwordsSection.filter = 'show';
       Polymer.dom.flush();
 
       var expectedList = [
         FakeDataMaker.passwordEntry('one.com', 'show', 5),
         FakeDataMaker.passwordEntry('two.com', 'shower', 3),
         FakeDataMaker.passwordEntry('three.com/show', 'four', 1),
         FakeDataMaker.passwordEntry('six-show.com', 'one', 6),
       ];
 
-      self.validatePasswordList(passwordsSection.$.passwordList, expectedList);
+      validatePasswordList(passwordsSection.$.passwordList, expectedList);
     });
 
     test('verifyFilterPasswordExceptions', function() {
       var exceptionList = [
         FakeDataMaker.exceptionEntry('docsshow.google.com'),
         FakeDataMaker.exceptionEntry('showmail.com'),
         FakeDataMaker.exceptionEntry('google.com'),
         FakeDataMaker.exceptionEntry('inbox.google.com'),
         FakeDataMaker.exceptionEntry('mapsshow.google.com'),
         FakeDataMaker.exceptionEntry('plus.google.comshow'),
       ];
 
-      var passwordsSection = self.createPasswordsSection_([], exceptionList);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, [], exceptionList);
       passwordsSection.filter = 'show';
       Polymer.dom.flush();
 
       var expectedExceptionList = [
         FakeDataMaker.exceptionEntry('docsshow.google.com'),
         FakeDataMaker.exceptionEntry('showmail.com'),
         FakeDataMaker.exceptionEntry('mapsshow.google.com'),
         FakeDataMaker.exceptionEntry('plus.google.comshow'),
       ];
 
-      self.validateExceptionList_(
-          self.getDomRepeatChildren_(passwordsSection.$.passwordExceptionsList),
+      validateExceptionList(
+          getDomRepeatChildren(passwordsSection.$.passwordExceptionsList),
           expectedExceptionList);
     });
 
     test('verifyNoPasswordExceptions', function() {
-      var passwordsSection = self.createPasswordsSection_([], []);
+      var passwordsSection = createPasswordsSection(passwordManager, [], []);
 
-      self.validateExceptionList_(
-          self.getDomRepeatChildren_(passwordsSection.$.passwordExceptionsList),
+      validateExceptionList(
+          getDomRepeatChildren(passwordsSection.$.passwordExceptionsList),
           []);
 
       assertFalse(passwordsSection.$.noExceptionsLabel.hidden);
     });
 
     test('verifyPasswordExceptions', function() {
       var exceptionList = [
         FakeDataMaker.exceptionEntry('docs.google.com'),
         FakeDataMaker.exceptionEntry('mail.com'),
         FakeDataMaker.exceptionEntry('google.com'),
         FakeDataMaker.exceptionEntry('inbox.google.com'),
         FakeDataMaker.exceptionEntry('maps.google.com'),
         FakeDataMaker.exceptionEntry('plus.google.com'),
       ];
 
-      var passwordsSection = self.createPasswordsSection_([], exceptionList);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, [], exceptionList);
 
-      self.validateExceptionList_(
-          self.getDomRepeatChildren_(passwordsSection.$.passwordExceptionsList),
+      validateExceptionList(
+          getDomRepeatChildren(passwordsSection.$.passwordExceptionsList),
           exceptionList);
 
       assertTrue(passwordsSection.$.noExceptionsLabel.hidden);
     });
 
     // Test verifies that removing an exception will update the elements.
     test('verifyPasswordExceptionRemove', function() {
       var exceptionList = [
         FakeDataMaker.exceptionEntry('docs.google.com'),
         FakeDataMaker.exceptionEntry('mail.com'),
         FakeDataMaker.exceptionEntry('google.com'),
         FakeDataMaker.exceptionEntry('inbox.google.com'),
         FakeDataMaker.exceptionEntry('maps.google.com'),
         FakeDataMaker.exceptionEntry('plus.google.com'),
       ];
 
-      var passwordsSection = self.createPasswordsSection_([], exceptionList);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, [], exceptionList);
 
-      self.validateExceptionList_(
-          self.getDomRepeatChildren_(passwordsSection.$.passwordExceptionsList),
+      validateExceptionList(
+          getDomRepeatChildren(passwordsSection.$.passwordExceptionsList),
           exceptionList);
 
       // Simulate 'mail.com' being removed from the list.
       passwordsSection.splice('passwordExceptions', 1, 1);
-      assertFalse(self.exceptionsListContainsUrl(
+      assertFalse(exceptionsListContainsUrl(
             passwordsSection.passwordExceptions, 'mail.com'));
-      assertFalse(self.exceptionsListContainsUrl(exceptionList, 'mail.com'));
-      self.flushPasswordSection_(passwordsSection);
+      assertFalse(exceptionsListContainsUrl(exceptionList, 'mail.com'));
+      flushPasswordSection(passwordsSection);
 
-      self.validateExceptionList_(
-          self.getDomRepeatChildren_(passwordsSection.$.passwordExceptionsList),
+      validateExceptionList(
+          getDomRepeatChildren(passwordsSection.$.passwordExceptionsList),
           exceptionList);
     });
 
     // Test verifies that pressing the 'remove' button will trigger a remove
     // event. Does not actually remove any exceptions.
     test('verifyPasswordExceptionRemoveButton', function(done) {
       var exceptionList = [
         FakeDataMaker.exceptionEntry('docs.google.com'),
         FakeDataMaker.exceptionEntry('mail.com'),
         FakeDataMaker.exceptionEntry('google.com'),
         FakeDataMaker.exceptionEntry('inbox.google.com'),
         FakeDataMaker.exceptionEntry('maps.google.com'),
         FakeDataMaker.exceptionEntry('plus.google.com'),
       ];
 
-      var passwordsSection = self.createPasswordsSection_([], exceptionList);
+      var passwordsSection = createPasswordsSection(
+          passwordManager, [], exceptionList);
 
       var exceptions =
-          self.getDomRepeatChildren_(passwordsSection.$.passwordExceptionsList);
+          getDomRepeatChildren(passwordsSection.$.passwordExceptionsList);
 
       // The index of the button currently being checked.
       var index = 0;
 
       var clickRemoveButton = function() {
         MockInteractions.tap(
             exceptions[index].querySelector('#removeExceptionButton'));
       };
 
-      // Listen for the remove event. If this event isn't received, the test
-      // will time out and fail.
-      passwordsSection.addEventListener('remove-password-exception',
-                                        function(event) {
+      passwordManager.onRemoveException = function(detail) {
         // Verify that the event matches the expected value.
         assertTrue(index < exceptionList.length);
-        assertEquals(exceptionList[index].exceptionUrl, event.detail);
+        assertEquals(exceptionList[index].exceptionUrl, detail);
 
-        if (++index < exceptionList.length)
+        if (++index < exceptionList.length) {
           clickRemoveButton();  // Click 'remove' on all passwords, one by one.
-        else
+        } else {
+          // Clean up after self.
+          passwordManager.onRemoveException = null;
+
           done();
-      });
+        }
+      };
 
       // Start removing.
       clickRemoveButton();
     });
 
     test('showSavedPassword', function() {
       var PASSWORD = 'bAn@n@5';
       var item = FakeDataMaker.passwordEntry('goo.gl', 'bart', PASSWORD.length);
-      var passwordDialog = self.createPasswordDialog_(item);
+      var passwordDialog = createPasswordDialog(item);
 
       passwordDialog.password = PASSWORD;
       passwordDialog.showPassword = true;
 
       Polymer.dom.flush();
 
       assertEquals(PASSWORD,
                    passwordDialog.$.passwordInput.value);
       // Password should be visible.
       assertEquals('text',
                    passwordDialog.$.passwordInput.type);
     });
 
     // Test will timeout if event is not received.
     test('onShowSavedPassword', function(done) {
       var item = FakeDataMaker.passwordEntry('goo.gl', 'bart', 1);
-      var passwordDialog = self.createPasswordDialog_(item);
+      var passwordDialog = createPasswordDialog(item);
 
       passwordDialog.addEventListener('show-password', function(event) {
         assertEquals(item.loginPair.originUrl, event.detail.originUrl);
         assertEquals(item.loginPair.username, event.detail.username);
         done();
       });
 
       MockInteractions.tap(passwordDialog.$.showPasswordButton);
     });
   });
 
   mocha.run();
 });