Skip to content

Commit 69ae0b9

Browse files
committed
Fix broken test
Change-Id: Idb54fa4936e6c8af5db8f21e9d1c2ae0d9996729
1 parent 9646b63 commit 69ae0b9

File tree

3 files changed

+16
-7
lines changed

3 files changed

+16
-7
lines changed

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

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -62,8 +62,8 @@ public GoogleProvider(FragmentActivity activity, IDPProviderParcel parcel, @Null
6262
.requestIdToken(mClientId);
6363

6464
// Add additional scopes
65-
String[] scopeStrings = activity.getResources().getStringArray(R.array.google_permissions);
66-
for (String scopeString : scopeStrings) {
65+
String[] extraScopes = getExtraScopes();
66+
for (String scopeString : extraScopes) {
6767
builder.requestScopes(new Scope(scopeString));
6868
}
6969

@@ -154,5 +154,9 @@ public void onClick(View view) {
154154
public void onConnectionFailed(@NonNull ConnectionResult connectionResult) {
155155
Log.w(TAG, "onConnectionFailed:" + connectionResult);
156156
}
157+
158+
private String[] getExtraScopes() {
159+
return mActivity.getResources().getStringArray(R.array.google_permissions);
160+
}
157161
}
158162

auth/src/test/java/com/firebase/ui/auth/test_helpers/GoogleProviderShadow.java

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,12 +14,8 @@
1414

1515
package com.firebase.ui.auth.test_helpers;
1616

17-
import static org.mockito.Mockito.mock;
18-
import static org.mockito.Mockito.when;
19-
2017
import android.app.Activity;
2118
import android.os.Bundle;
22-
import android.util.Log;
2319

2420
import com.firebase.ui.auth.provider.GoogleProvider;
2521
import com.firebase.ui.auth.provider.IDPProvider;
@@ -30,6 +26,9 @@
3026
import org.robolectric.annotation.Implementation;
3127
import org.robolectric.annotation.Implements;
3228

29+
import static org.mockito.Mockito.mock;
30+
import static org.mockito.Mockito.when;
31+
3332

3433
@Implements(GoogleProvider.class)
3534
public class GoogleProviderShadow {
@@ -54,7 +53,7 @@ public GoogleProviderShadow() {
5453
public void __constructor__(Activity activity, IDPProviderParcel parcel, String email) {}
5554

5655

57-
@Implementation
56+
@Implementation
5857
public void setAuthenticationCallback(IDPProvider.IDPCallback idpCallback) {
5958
mCallback = idpCallback;
6059
}
@@ -63,4 +62,9 @@ public void setAuthenticationCallback(IDPProvider.IDPCallback idpCallback) {
6362
public void startLogin(Activity activity) {
6463
mCallback.onSuccess(mMockIdpResponse);
6564
}
65+
66+
@Implementation
67+
public String[] getExtraScopes() {
68+
return new String[]{};
69+
}
6670
}

auth/src/test/java/com/firebase/ui/auth/ui/idp/AuthMethodPickerActivityTest.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,7 @@ public void testAllProvidersArePopulated() {
8181

8282
AuthMethodPickerActivity authMethodPickerActivity =
8383
createActivity(providers);
84+
8485
assertEquals(providers.size(),
8586
((LinearLayout) authMethodPickerActivity.findViewById(R.id.btn_holder))
8687
.getChildCount());

0 commit comments

Comments
 (0)