Settings People Revamp: Add tests for ChromeOS Change Picture.

chrome/browser/resources/settings/people_page/change_picture.js

Index: chrome/browser/resources/settings/people_page/change_picture.js
diff --git a/chrome/browser/resources/settings/people_page/change_picture.js b/chrome/browser/resources/settings/people_page/change_picture.js
index 5fe931b3e17d7af1aa6931e1980dd019a396f92a..34343bf8804dfccae1796c120baec3464827d221 100644
--- a/chrome/browser/resources/settings/people_page/change_picture.js
+++ b/chrome/browser/resources/settings/people_page/change_picture.js
@@ -2,6 +2,14 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
+var changePictureOptions = changePictureOptions || {

[dpapad, 2016/01/19 19:44:59]

Is there a way we can avoid this global variable? Can we turn it into a member
variable of the custom element instead?

[tommycli, 2016/01/20 00:30:27]

I could not figure out an alternative. I followed the example here:
https://code.google.com/p/chromium/codesearch#chromium/src/chrome/browser/res...

Is there a way to declare static variables on Polymer elements? This relies on
the fact that the test has a chance to declare the global BEFORE polymer does.

[dpapad, 2016/01/20 01:09:29]

Two ways I can think of.
1) Don't test that the URL actually changes. Instead just test that
receiveSelectedImage() is being called with the new URL.
2) Keep testing selectedImageUrl_ via the change notification but define a
namespace for the global config var,
   cr.define('settings_test', function() {
     var changePictureOptions = ......;
     return {changePictureOptions: changePictureOptions};
   });

[tommycli, 2016/01/20 22:38:12]

Done.

Okay cool! i was able to namespace it to settings_test! In the future I might
choose to take Option a) instead.

Thanks!

+  /**
+   * True if property changes should fire events for testing purposes.
+   * @type {boolean}
+   */
+  notifyPropertyChangesForTest: false,
+};
+
 /**
  * @fileoverview
  * 'settings-change-picture' is the settings subpage containing controls to
@@ -22,7 +30,10 @@ Polymer({
      * The currently selected profile image URL. May be a data URL.
      * @private {string}
      */
-    selectedImageUrl_: String,
+    selectedImageUrl_: {
+      type: String,
+      notify: changePictureOptions.notifyPropertyChangesForTest,
+    },
 
     /**
      * The url of the 'old' image, which is the existing image sourced from