Settings People CrOS: Display supervised users' names correctly.

chrome/test/data/extensions/api_test/users_private/test.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.
 
 // This just tests the interface. It does not test for specific results, only
 // that callbacks are correctly invoked, expected parameters are correct,
 // and failures are detected.
 
 function callbackResult(result) {
   if (chrome.runtime.lastError)
     chrome.test.fail(chrome.runtime.lastError.message);
   else if (result == false)
     chrome.test.fail('Failed: ' + 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() {
     chrome.usersPrivate.addWhitelistedUser(
         kEmail1,
         function(result) {
           callbackResult(result);
 
           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;
               }
             });
             chrome.test.assertTrue(foundUser);
             chrome.test.succeed();
           });
         });
   },
 
   function addAndRemoveUsers() {
     chrome.usersPrivate.addWhitelistedUser(
         kEmail1,
         function(result1) {
           callbackResult(result1);
 
           chrome.usersPrivate.addWhitelistedUser(
               kEmail2,
               function(result2) {
                 callbackResult(result2);
 
                   chrome.usersPrivate.removeWhitelistedUser(
                       kEmail1,
                       function(result3) {
 
                         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();
                             });
 
                       });
               });
         });
 
   },
 
   function isOwner() {
     chrome.usersPrivate.isCurrentUserOwner(function(isOwner) {
       // Since we are testing with --stub-cros-settings this should be true.
       chrome.test.assertTrue(isOwner);
       chrome.test.succeed();
     });
   },
 ];
 
 var testToRun = window.location.search.substring(1);
 chrome.test.runTests(availableTests.filter(function(op) {
   return op.name == testToRun;
 }));
-