Skip to content

Commit e97f325

Browse files
committed
Merge pull request #160 from amandle/testing3
Remove AccountLinkInitActivity, add more Robolectric tests
2 parents 50c9143 + d0d68a9 commit e97f325

21 files changed

+761
-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: 8 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,9 @@ 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())
123+
.setResultCallback(this);
121124
}
122125

123126
@Override
@@ -148,7 +151,6 @@ public void onConnectionFailed(@NonNull ConnectionResult connectionResult) {
148151

149152
@Override
150153
public void onResult(@NonNull Status status) {
151-
152154
if (status.isSuccess()) {
153155
finish(RESULT_OK, getIntent());
154156
} else {
@@ -184,7 +186,9 @@ public void onActivityResult(int requestCode, int resultCode, Intent data) {
184186
} else if (requestCode == RC_UPDATE_SERVICE) {
185187
if (resultCode == RESULT_OK) {
186188
Credential credential = new Credential.Builder(mEmail).setPassword(mPassword).build();
187-
Auth.CredentialsApi.save(mCredentialsApiClient, credential).setResultCallback(this);
189+
mActivityHelper.getCredentialsApi()
190+
.save(mCredentialsApiClient, credential)
191+
.setResultCallback(this);
188192
} else {
189193
Log.e(TAG, "SAVE: Canceled by user");
190194
finish(RESULT_FIRST_USER, getIntent());

0 commit comments

Comments
 (0)