changes from review

Author: camden-king

GoogleSignIn/Sources/GIDAuthStateMigration/Fake/GIDFakeAuthStateMigration.m

@@ -27,10 +27,6 @@ - (instancetype)initWithKeychainStore:(GTMKeychainStore *)keychainStore {
   return self;
 }
 
-- (instancetype)init {
-  return self;
-}
-
 - (void)migrateIfNeededWithTokenURL:(NSURL *)tokenURL
                        callbackPath:(NSString *)callbackPath
                        keychainName:(NSString *)keychainName

GoogleSignIn/Sources/GIDSignIn.m

@@ -491,7 +491,7 @@ + (GIDSignIn *)sharedInstance {
     GTMKeychainStore *keychainStore =
         [[GTMKeychainStore alloc] initWithItemName:kGTMAppAuthKeychainName];
     GIDAuthStateMigration *authStateMigrationService =
-          [[GIDAuthStateMigration alloc] initWithKeychainStore:keychainStore];
+        [[GIDAuthStateMigration alloc] initWithKeychainStore:keychainStore];
 #if TARGET_OS_IOS && !TARGET_OS_MACCATALYST
     if (@available(iOS 14.0, *)) {
       GIDAppCheck *appCheck = [GIDAppCheck appCheckUsingAppAttestProvider];
@@ -501,7 +501,8 @@ + (GIDSignIn *)sharedInstance {
     }
 #endif // TARGET_OS_IOS && !TARGET_OS_MACCATALYST
     if (!sharedInstance) {
-      sharedInstance = [[self alloc] initWithKeychainStore:keychainStore                                        authStateMigrationService:authStateMigrationService];
+      sharedInstance = [[self alloc] initWithKeychainStore:keychainStore
+                                 authStateMigrationService:authStateMigrationService];
     }
   });
   return sharedInstance;
@@ -566,9 +567,9 @@ - (instancetype)initWithKeychainStore:(GTMKeychainStore *)keychainStore
     _keychainStore = keychainStore;
     // Perform migration of auth state from old versions of the SDK if needed.
     [authStateMigrationService migrateIfNeededWithTokenURL:_appAuthConfiguration.tokenEndpoint
-                              callbackPath:kBrowserCallbackPath
-                              keychainName:kGTMAppAuthKeychainName
-                            isFreshInstall:isFreshInstall];
+                                              callbackPath:kBrowserCallbackPath
+                                              keychainName:kGTMAppAuthKeychainName
+                                            isFreshInstall:isFreshInstall];
   }
   return self;
 }
@@ -577,7 +578,8 @@ - (instancetype)initWithKeychainStore:(GTMKeychainStore *)keychainStore
 - (instancetype)initWithKeychainStore:(GTMKeychainStore *)keychainStore
             authStateMigrationService:(GIDAuthStateMigration *) authStateMigrationService
                              appCheck:(GIDAppCheck *)appCheck {
-  self = [self initWithKeychainStore:keychainStore authStateMigrationService:authStateMigrationService];
+  self = [self initWithKeychainStore:keychainStore
+           authStateMigrationService:authStateMigrationService];
   if (self) {
     _appCheck = appCheck;
     _configureAppCheckCalled = NO;

GoogleSignIn/Tests/Unit/GIDSignInTest.m

@@ -502,14 +502,13 @@ - (void)testInitWithKeychainStore_attemptsMigration {
 
   XCTestExpectation *expectation = [self expectationWithDescription:@"Callback should be called."];
   _authStateMigrationService.migrationInvokedCallback =
-  ^(NSURL *tokenURL, NSString *callbackPath, NSString *keychainName, BOOL isFreshInstall) {
-    XCTAssertFalse(isFreshInstall);
-    [expectation fulfill];
-  };
+    ^(NSURL *tokenURL, NSString *callbackPath, NSString *keychainName, BOOL isFreshInstall) {
+      XCTAssertFalse(isFreshInstall);
+      [expectation fulfill];
+    };
 
   GTMKeychainStore *store = [[GTMKeychainStore alloc] initWithItemName:expectedKeychainName];
-  GIDSignIn *signIn;
-  signIn = [[GIDSignIn alloc] initWithKeychainStore:store
+  GIDSignIn *signIn = [[GIDSignIn alloc] initWithKeychainStore:store
                           authStateMigrationService:_authStateMigrationService];
 
   [self waitForExpectationsWithTimeout:1 handler:nil];