[CRD iOS] Implementing save to keychain and user defaults for login info. Refactor remoting service.

remoting/client/ios/domain/user_info.mm

Index: remoting/client/ios/domain/user_info.mm
diff --git a/remoting/client/ios/domain/user_info.mm b/remoting/client/ios/domain/user_info.mm
index ece9064d063afd6d0d34499871244ce5bb8ad864..3257fd22551bbbc04c0cfe514aa0086f626e60d8 100644
--- a/remoting/client/ios/domain/user_info.mm
+++ b/remoting/client/ios/domain/user_info.mm
@@ -13,6 +13,7 @@
 @synthesize userId = _userId;
 @synthesize userFullName = _userFullName;
 @synthesize userEmail = _userEmail;
+@synthesize refreshToken = _refreshToken;
 
 // Parse jsonData into Host list.
 + (UserInfo*)parseListFromJSON:(NSMutableData*)data {
@@ -26,12 +27,26 @@
   user.userId = [json objectForKey:@"userId"];
   user.userFullName = [json objectForKey:@"userFullName"];
   user.userEmail = [json objectForKey:@"userEmail"];
+  user.refreshToken = [json objectForKey:@"refreshToken"];
 
   return user;
 }
 
+- (BOOL)isAuthenticated {
+  if (_userEmail && _userEmail.length > 0 && _refreshToken &&

[Yuwei, 2017/05/04 05:10:13]

Just return _userEmail && _userEmail.length > 0 && _refreshToken &&
_refreshToken.length > 0?

[nicholss, 2017/05/08 17:08:10]

I tend avoid return the result of a wacky logic test because it makes it hard to
understand the result.  This is for readability. 

+      _refreshToken.length > 0) {
+    return YES;
+  }
+  return NO;
+}
+
 - (NSComparisonResult)compare:(UserInfo*)user {
   return [self.userId compare:user.userId];
 }
 
+- (NSString*)description {
+  return [NSString stringWithFormat:@"UserInfo: userEmail=%@ refreshToken=%@",
+                                    _userEmail, _refreshToken];
+}
+
 @end