Settings People CrOS: Display supervised users' names correctly.

chrome/test/data/extensions/api_test/users_private/test.js

Index: chrome/test/data/extensions/api_test/users_private/test.js
diff --git a/chrome/test/data/extensions/api_test/users_private/test.js b/chrome/test/data/extensions/api_test/users_private/test.js
index 62832e472c52fe8abfc537dff84d64a412191749..843fc08dd3732b6620864ac3984cc481f41cd2db 100644
--- a/chrome/test/data/extensions/api_test/users_private/test.js
+++ b/chrome/test/data/extensions/api_test/users_private/test.js
@@ -15,6 +15,8 @@ function callbackResult(result) {
 
 var kEmail1 = 'asdf@gmail.com';
 var kEmail2 = 'asdf2@gmail.com';
+var kName1 = kEmail1;

[Devlin, 2016/08/20 01:23:04]

it'd probably be good to make this something different so that we'd catch a bug
if we *were* returning the same value for the name and email.

[tommycli, 2016/08/20 01:35:57]

So... they would only be different for locally managed users with an email that
looks like: 0@locally-managed.localhost

There's no provision for adding that in here. I'd have to figure out how to do
that on the .cc file side.

Is that worth?

+var kName2 = kEmail2;
 
 var availableTests = [
   function addUser() {
@@ -26,7 +28,7 @@ var availableTests = [
           chrome.usersPrivate.getWhitelistedUsers(function(users) {
             var foundUser = false;
             users.forEach(function(user) {
-              if (user.email == kEmail1) {
+              if (user.email == kEmail1 && user.name == kName1) {
                 foundUser = true;
               }
             });
@@ -54,8 +56,8 @@ var availableTests = [
                         chrome.usersPrivate.getWhitelistedUsers(
                             function(users) {
                               chrome.test.assertTrue(users.length == 1);
-                              chrome.test.assertEq(
-                                  kEmail2, users[0].email);
+                              chrome.test.assertEq(kEmail2, users[0].email);
+                              chrome.test.assertEq(kName2, users[0].name);
                               chrome.test.succeed();
                             });
 
@@ -78,4 +80,3 @@ var testToRun = window.location.search.substring(1);
 chrome.test.runTests(availableTests.filter(function(op) {
   return op.name == testToRun;
 }));
-