Skip to content

Commit a43d3bc

Browse files
fix(user-profile-component): Fix user profile image (#576)
1 parent 4fece57 commit a43d3bc

File tree

2 files changed

+10
-5
lines changed

2 files changed

+10
-5
lines changed

__tests__/tests/components/UserProfile.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,9 @@ describe('<UserProfile />', () => {
4444
);
4545

4646
const result = wrapper.instance().getUserUri();
47-
const expectedResult = `${initialUser.avatar_url}&lastModified=${initialUser.updated_at}`;
47+
const expectedResult = {
48+
uri: `${initialUser.avatar_url}&lastModified=${initialUser.updated_at}`,
49+
};
4850

4951
expect(result).toEqual(expectedResult);
5052
});
@@ -62,7 +64,9 @@ describe('<UserProfile />', () => {
6264
);
6365

6466
const result = wrapper.instance().getUserUri();
65-
const expectedResult = `${user.avatar_url}&lastModified=${user.updated_at}`;
67+
const expectedResult = {
68+
uri: `${user.avatar_url}&lastModified=${user.updated_at}`,
69+
};
6670

6771
expect(result).toEqual(expectedResult);
6872
});

src/components/user-profile.component.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -101,10 +101,11 @@ export class UserProfile extends Component {
101101

102102
getUserUri = () => {
103103
const { initialUser, user } = this.props;
104-
105-
return initialUser.avatar_url
104+
const image = initialUser.avatar_url
106105
? `${initialUser.avatar_url}&lastModified=${initialUser.updated_at}`
107106
: `${user.avatar_url}&lastModified=${user.updated_at}`;
107+
108+
return { uri: image };
108109
};
109110

110111
render() {
@@ -130,7 +131,7 @@ export class UserProfile extends Component {
130131
>
131132
<View style={styles.profile}>
132133
<ImageZoom
133-
uri={this.getUserUri}
134+
uri={this.getUserUri()}
134135
style={[
135136
styles.avatar,
136137
(initialUser.type === 'User' || user.type === 'User') &&

0 commit comments

Comments
 (0)