rename tests 'error message is good' -> 'error message contains console link'

Author: dconeybe

packages/firestore/test/integration/api/aggregation.test.ts

@@ -132,7 +132,7 @@ apiDescribe('Count queries', persistence => {
   // and will, therefore, never fail in this situation.
   // eslint-disable-next-line no-restricted-properties
   (USE_EMULATOR ? it.skip : it)(
-    'getCountFromServer error message is good if missing index',
+    'getCountFromServer error message contains console link if missing index',
     () => {
       return withEmptyTestCollection(persistence, async coll => {
         const query_ = query(
@@ -342,7 +342,7 @@ apiDescribe('Aggregation queries', persistence => {
   // and will, therefore, never fail in this situation.
   // eslint-disable-next-line no-restricted-properties
   (USE_EMULATOR ? it.skip : it)(
-    'getAggregateFromServer error message is good if missing index',
+    'getAggregateFromServer error message contains console link good if missing index',
     () => {
       return withEmptyTestCollection(persistence, async coll => {
         const query_ = query(

packages/firestore/test/lite/integration.test.ts

@@ -2406,7 +2406,7 @@ describe('Count queries', () => {
   // and will, therefore, never fail in this situation.
   // eslint-disable-next-line no-restricted-properties
   (USE_EMULATOR ? it.skip : it)(
-    'getCount error message is good if missing index',
+    'getCount error message contains console link if missing index',
     () => {
       return withTestCollection(async coll => {
         const query_ = query(
@@ -2713,7 +2713,7 @@ describe('Aggregate queries', () => {
   // and will, therefore, never fail in this situation.
   // eslint-disable-next-line no-restricted-properties
   (USE_EMULATOR ? it.skip : it)(
-    'getAggregate error message is good if missing index',
+    'getAggregate error message contains console link if missing index',
     () => {
       return withTestCollection(async coll => {
         const query_ = query(