Auth current authenticated user enhancement

packages/aws-amplify-react/__tests__/Auth/Greetings-test.js

@@ -64,6 +64,25 @@ describe('Greetings', () => {
                 expect(wrapper).toMatchSnapshot();
             }
         });
+
+        test('render name from attributes', () => {
+            const wrapper = shallow(<Greetings/>);
+            wrapper.setProps({
+                authState: 'signedIn',
+                theme: 'theme'
+            });
+
+            wrapper.setState({
+                authData: {
+                    attributes: {
+                        name: 'name'
+                    }
+                },
+                authState: 'signedIn'
+            })  
+
+            expect(wrapper).toMatchSnapshot();
+        })
     });
 
 

packages/aws-amplify-react/__tests__/Auth/__snapshots__/Greetings-test.js.snap

@@ -2,6 +2,34 @@
 
 exports[`Greetings normal case render correctly with authState signedIn 1`] = `""`;
 
+exports[`Greetings normal case render name from attributes 1`] = `
+<NavBar
+  theme="theme"
+>
+  <Nav
+    theme="theme"
+  >
+    <NavRight
+      theme="theme"
+    >
+      <span>
+        <NavItem
+          theme="theme"
+        >
+          Hello name
+        </NavItem>
+        <NavButton
+          onClick={[Function]}
+          theme="theme"
+        >
+          Sign Out
+        </NavButton>
+      </span>
+    </NavRight>
+  </Nav>
+</NavBar>
+`;
+
 exports[`Greetings render corrently with other authStates 1`] = `""`;
 
 exports[`Greetings render corrently with other authStates 2`] = `""`;

packages/aws-amplify-react/src/Auth/Greetings.jsx

@@ -127,7 +127,14 @@ export default class Greetings extends AuthPiece {
     userGreetings(theme) {
         const user = this.state.authData;
         const greeting = this.props.inGreeting || this.inGreeting;
-        const name = user.name || user.username;
+        // get name from attributes first
+        const nameFromAttr = user.attributes? 
+            (user.attributes.name || 
+            (user.attributes.given_name? 
+                (user.attributes.given_name + ' ' + user.attributes.family_name) : undefined))
+            : undefined;
+
+        const name = nameFromAttr || user.name || user.username;
         const message = (typeof greeting === 'function')? greeting(name) : greeting;
         return (
             <span>

packages/aws-amplify/__tests__/Auth/auth-unit-test.ts

@@ -891,7 +891,8 @@ describe('auth unit test', () => {
                 userPoolId: undefined,
                 userPoolWebClientId: "awsUserPoolsWebClientId",
                 region: "region",
-                identityPoolId: "awsCognitoIdentityPoolId"
+                identityPoolId: "awsCognitoIdentityPoolId",
+                mandatorySignIn: false
             });
 
             expect.assertions(1);
@@ -918,8 +919,14 @@ describe('auth unit test', () => {
                     });
                 });
 
+            const spyon2 = jest.spyOn(Auth.prototype, 'userAttributes').mockImplementationOnce(() => {
+                return Promise.resolve([{
+                    Name: 'name',
+                    Value: 'val' 
+                }]);
+            });
             expect.assertions(1);
-            expect(await auth.currentAuthenticatedUser()).toEqual(user);
+            expect(await auth.currentAuthenticatedUser()).toEqual({"attributes": {"name": "val"}});
 
             spyon.mockClear();
 

packages/aws-amplify/src/Auth/Auth.ts

@@ -713,7 +713,9 @@ export default class AuthClass {
         } else {
             logger.debug('get current authenticated userpool user');
             try {
-                this.user = await this.currentUserPoolUser();
+                const user = await this.currentUserPoolUser();
+                const attributes = this.attributesToObject(await this.userAttributes(user));
+                this.user = Object.assign({}, user, { attributes });
                 return this.user;
             } catch (e) {
                 return Promise.reject('not authenticated');