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..96e571314ed0e19c7f4d9054ccd365ff3ee6afc1 100644
--- a/chrome/test/data/extensions/api_test/users_private/test.js
+++ b/chrome/test/data/extensions/api_test/users_private/test.js
@@ -26,7 +26,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 == kEmail1) {

[Oren Blasberg, 2016/08/19 18:47:48]

I think you're reusing the kEmail1 variable for user.name, but this makes the
test a bit confusing (I thought it was a mistake until I thought about it a bit
more). Please make a new "kName1" variable and just use that here and in the
test below instead.

[tommycli, 2016/08/19 19:15:58]

Done.

                 foundUser = true;
               }
             });
@@ -54,8 +54,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(kEmail2, users[0].name);

[Oren Blasberg, 2016/08/19 18:47:48]

So this would be kName2

[tommycli, 2016/08/19 19:15:58]

Done.

                               chrome.test.succeed();
                             });
 
@@ -78,4 +78,3 @@ var testToRun = window.location.search.substring(1);
 chrome.test.runTests(availableTests.filter(function(op) {
   return op.name == testToRun;
 }));
-