Skip to content

Commit 3eb46d9

Browse files
author
amandle
committed
Remove AccountLinkInitActivity, add more robolectric tests
1 parent e04e70d commit 3eb46d9

21 files changed

+744
-323
lines changed

auth/src/main/java/com/firebase/ui/auth/provider/FacebookProvider.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,13 @@
3939
import java.util.Arrays;
4040

4141
public class FacebookProvider implements IDPProvider, FacebookCallback<LoginResult> {
42+
public static final String ACCESS_TOKEN = "facebook_access_token";
4243

4344
protected static final String ERROR = "err";
4445
protected static final String ERROR_MSG = "err_msg";
4546

4647
private static final String TAG = "FacebookProvider";
47-
private static final String ACCESS_TOKEN = "facebook_access_token";
4848
private static final String APPLICATION_ID = "application_id";
49-
5049
private CallbackManager mCallbackManager;
5150
private IDPCallback mCallbackObject;
5251

auth/src/main/java/com/firebase/ui/auth/provider/GoogleProvider.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,10 @@
3131
import com.google.firebase.auth.GoogleAuthProvider;
3232

3333
public class GoogleProvider implements IDPProvider, OnClickListener {
34+
public static final String TOKEN_KEY = "token_key";
35+
3436
private static final int RC_SIGN_IN = 20;
3537
private static final String ERROR_KEY = "error";
36-
private static final String TOKEN_KEY = "token_key";
3738
private static final String CLIENT_ID_KEY = "client_id_key";
3839
private GoogleApiClient mGoogleApiClient;
3940
private Activity mActivity;

auth/src/main/java/com/firebase/ui/auth/ui/ActivityHelper.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,8 @@
2121
import android.support.annotation.NonNull;
2222
import android.support.annotation.StringRes;
2323

24+
import com.google.android.gms.auth.api.Auth;
25+
import com.google.android.gms.auth.api.credentials.CredentialsApi;
2426
import com.google.firebase.FirebaseApp;
2527
import com.google.firebase.auth.FirebaseAuth;
2628
import com.google.firebase.auth.FirebaseUser;
@@ -86,6 +88,10 @@ public FirebaseAuth getFirebaseAuth() {
8688
return FirebaseAuth.getInstance(getFirebaseApp());
8789
}
8890

91+
public CredentialsApi getCredentialsApi() {
92+
return Auth.CredentialsApi;
93+
}
94+
8995
public FirebaseUser getCurrentUser() {
9096
return getFirebaseAuth().getCurrentUser();
9197
}

auth/src/main/java/com/firebase/ui/auth/ui/account_link/AccountLinkInitActivity.java

Lines changed: 0 additions & 172 deletions
This file was deleted.

auth/src/main/java/com/firebase/ui/auth/ui/account_link/SaveCredentialsActivity.java

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,10 @@
2727
import com.firebase.ui.auth.BuildConfig;
2828
import com.firebase.ui.auth.R;
2929
import com.firebase.ui.auth.ui.ActivityHelper;
30+
import com.firebase.ui.auth.ui.AppCompatBase;
3031
import com.firebase.ui.auth.ui.ExtraConstants;
3132
import com.firebase.ui.auth.ui.FlowParameters;
3233
import com.firebase.ui.auth.util.FirebaseAuthWrapperFactory;
33-
import com.firebase.ui.auth.ui.AppCompatBase;
3434
import com.google.android.gms.auth.api.Auth;
3535
import com.google.android.gms.auth.api.credentials.Credential;
3636
import com.google.android.gms.auth.api.credentials.IdentityProviders;
@@ -63,6 +63,7 @@ protected void onCreate(Bundle savedInstanceState) {
6363
if (!FirebaseAuthWrapperFactory.getFirebaseAuthWrapper(mActivityHelper.getAppName())
6464
.isPlayServicesAvailable(this)) {
6565
finish(RESULT_FIRST_USER, getIntent());
66+
return;
6667
}
6768
mName = getIntent().getStringExtra(ExtraConstants.EXTRA_NAME);
6869
mEmail = getIntent().getStringExtra(ExtraConstants.EXTRA_EMAIL);
@@ -117,7 +118,8 @@ public void onConnected(@Nullable Bundle bundle) {
117118
if (mProfilePictureUri != null) {
118119
builder.setProfilePictureUri(Uri.parse(mProfilePictureUri));
119120
}
120-
Auth.CredentialsApi.save(mCredentialsApiClient, builder.build()).setResultCallback(this);
121+
mActivityHelper.getCredentialsApi()
122+
.save(mCredentialsApiClient, builder.build()).setResultCallback(this);
121123
}
122124

123125
@Override
@@ -148,7 +150,6 @@ public void onConnectionFailed(@NonNull ConnectionResult connectionResult) {
148150

149151
@Override
150152
public void onResult(@NonNull Status status) {
151-
152153
if (status.isSuccess()) {
153154
finish(RESULT_OK, getIntent());
154155
} else {
@@ -184,7 +185,8 @@ public void onActivityResult(int requestCode, int resultCode, Intent data) {
184185
} else if (requestCode == RC_UPDATE_SERVICE) {
185186
if (resultCode == RESULT_OK) {
186187
Credential credential = new Credential.Builder(mEmail).setPassword(mPassword).build();
187-
Auth.CredentialsApi.save(mCredentialsApiClient, credential).setResultCallback(this);
188+
mActivityHelper.getCredentialsApi()
189+
.save(mCredentialsApiClient, credential).setResultCallback(this);
188190
} else {
189191
Log.e(TAG, "SAVE: Canceled by user");
190192
finish(RESULT_FIRST_USER, getIntent());

0 commit comments

Comments
 (0)