Skip to content

Commit 62c5aba

Browse files
authored
Merge 874e973 into eb3380e
2 parents eb3380e + 874e973 commit 62c5aba

File tree

2 files changed

+2
-20
lines changed

2 files changed

+2
-20
lines changed

firebase-firestore/src/androidTest/java/com/google/firebase/firestore/AggregationTest.java

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,6 @@
3737
import androidx.test.ext.junit.runners.AndroidJUnit4;
3838
import com.google.android.gms.tasks.Task;
3939
import com.google.common.truth.Truth;
40-
import com.google.firebase.firestore.model.DatabaseId;
4140
import com.google.firebase.firestore.testutil.IntegrationTestUtil;
4241
import java.util.Collections;
4342
import java.util.Map;
@@ -904,15 +903,7 @@ public void testAggregateErrorMessageShouldContainConsoleLinkIfMissingIndex() {
904903

905904
Throwable cause = throwable.getCause();
906905
Truth.assertThat(cause).hasMessageThat().ignoringCase().contains("index");
907-
// TODO(b/316359394) Remove this check for the default databases once cl/582465034 is rolled
908-
// out to production.
909-
if (collection
910-
.firestore
911-
.getDatabaseId()
912-
.getDatabaseId()
913-
.equals(DatabaseId.DEFAULT_DATABASE_ID)) {
914-
Truth.assertThat(cause).hasMessageThat().contains("https://console.firebase.google.com");
915-
}
906+
Truth.assertThat(cause).hasMessageThat().contains("https://console.firebase.google.com");
916907
}
917908

918909
@Test

firebase-firestore/src/androidTest/java/com/google/firebase/firestore/CountTest.java

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,6 @@
3333

3434
import androidx.test.ext.junit.runners.AndroidJUnit4;
3535
import com.google.android.gms.tasks.Task;
36-
import com.google.firebase.firestore.model.DatabaseId;
3736
import com.google.firebase.firestore.testutil.IntegrationTestUtil;
3837
import java.util.Collections;
3938
import org.junit.After;
@@ -277,14 +276,6 @@ public void testCountErrorMessageShouldContainConsoleLinkIfMissingIndex() {
277276

278277
Throwable cause = throwable.getCause();
279278
assertThat(cause).hasMessageThat().ignoringCase().contains("index");
280-
// TODO(b/316359394) Remove this check for the default databases once cl/582465034 is rolled
281-
// out to production.
282-
if (collection
283-
.firestore
284-
.getDatabaseId()
285-
.getDatabaseId()
286-
.equals(DatabaseId.DEFAULT_DATABASE_ID)) {
287-
assertThat(cause).hasMessageThat().contains("https://console.firebase.google.com");
288-
}
279+
assertThat(cause).hasMessageThat().contains("https://console.firebase.google.com");
289280
}
290281
}

0 commit comments

Comments
 (0)