finished the test

Author: dconeybe

Firestore/Example/Tests/Integration/API/FIRQueryTests.mm

@@ -1211,12 +1211,13 @@ - (void)testResumingAQueryShouldUseExistenceFilterToDetectDeletes {
     }
     createdDocuments = [createdDocumentsAccumulator copy];
   }
+  XCTAssertEqual(createdDocuments.count, 100, @"createdDocuments has the wrong size");
 
   // Delete 50 of the 100 documents. Do this in a transaction, rather than
   // [FIRDocumentReference deleteDocument], to avoid affecting the local cache.
   NSSet<NSString*>* deletedDocumentIds;
   {
-    NSMutableSet<NSString*>* deletedDocumentIdsAccumulator = [[NSMutableSet alloc] init];
+    NSMutableArray<NSString*>* deletedDocumentIdsAccumulator = [[NSMutableArray alloc] init];
     deletedDocumentIds = [deletedDocumentIdsAccumulator copy];
     XCTestExpectation *expectation = [self expectationWithDescription:@"DeleteTransaction"];
     [collRef.firestore runTransactionWithBlock:
@@ -1234,7 +1235,9 @@ - (void)testResumingAQueryShouldUseExistenceFilterToDetectDeletes {
         }
     ];
     [self awaitExpectation:expectation];
+    deletedDocumentIds = [NSSet setWithArray:deletedDocumentIdsAccumulator];
   }
+  XCTAssertEqual(deletedDocumentIds.count, 50, @"deletedDocumentIds has the wrong size");
 
   // Wait for 10 seconds, during which Watch will stop tracking the query and will send an existence
   // filter rather than "delete" events when the query is resumed.
@@ -1252,26 +1255,26 @@ - (void)testResumingAQueryShouldUseExistenceFilterToDetectDeletes {
   if (!([FSTIntegrationTestCase isRunningAgainstEmulator] && querySnapshot2.count == 100)) {
     NSSet<NSString*>* actualDocumentIds;
     {
-      NSMutableSet<NSString*>* actualDocumentIdsAccumulator = [[NSMutableSet alloc] init];
+      NSMutableArray<NSString*>* actualDocumentIdsAccumulator = [[NSMutableArray alloc] init];
       for (FIRDocumentSnapshot* documentSnapshot in querySnapshot2.documents) {
         [actualDocumentIdsAccumulator addObject:documentSnapshot.documentID];
       }
-      actualDocumentIds = [actualDocumentIdsAccumulator copy];
+      actualDocumentIds = [NSSet setWithArray:actualDocumentIdsAccumulator];
     }
     NSSet<NSString*>* expectedDocumentIds;
     {
-      NSMutableSet<NSString*>* expectedDocumentIdsAccumulator = [[NSMutableSet alloc] init];
+      NSMutableArray<NSString*>* expectedDocumentIdsAccumulator = [[NSMutableArray alloc] init];
       for (FIRDocumentReference* documentRef in createdDocuments) {
         if (![deletedDocumentIds containsObject:documentRef.documentID]) {
           [expectedDocumentIdsAccumulator addObject:documentRef.documentID];
         }
       }
-      expectedDocumentIds = [expectedDocumentIdsAccumulator copy];
+      expectedDocumentIds = [NSSet setWithArray:expectedDocumentIdsAccumulator];
     }
     if (! [actualDocumentIds isEqualToSet:expectedDocumentIds]) {
       NSArray<NSString*>* unexpectedDocumentIds = SortedStringsNotIn(actualDocumentIds, expectedDocumentIds);
       NSArray<NSString*>* missingDocumentIds = SortedStringsNotIn(expectedDocumentIds, actualDocumentIds);
-      XCTFail(@"The snapshot contained %@ documents (expected %@), %@ unexpected and %@ missing", @(actualDocumentIds.count), @(expectedDocumentIds.count), @(unexpectedDocumentIds.count), @(missingDocumentIds.count));
+      XCTFail(@"The snapshot contained %@ documents (expected %@): %@ unexpected and %@ missing; unexpected documents: %@; missing documents: %@", @(actualDocumentIds.count), @(expectedDocumentIds.count), @(unexpectedDocumentIds.count), @(missingDocumentIds.count), [unexpectedDocumentIds componentsJoinedByString:@", "], [missingDocumentIds componentsJoinedByString:@", "]);
     }
   }
 }