Fix tests that failed in test lab

mrober · mrober · commit 9eda4b26e37d · 2024-07-26T15:15:32.000-04:00
diff --git a/firebase-crashlytics/src/androidTest/java/com/google/firebase/crashlytics/internal/metadata/MetaDataStoreTest.java b/firebase-crashlytics/src/androidTest/java/com/google/firebase/crashlytics/internal/metadata/MetaDataStoreTest.java
@@ -104,9 +104,10 @@ public void testWriteUserData_null() {
     assertNull(userData.getUserId());
   }
 
-  public void testWriteUserData_emptyString() {
+  public void testWriteUserData_emptyString() throws Exception {
     storeUnderTest.writeUserData(SESSION_ID_1, metadataWithUserId(SESSION_ID_1, "").getUserId());
     UserMetadata userData = UserMetadata.loadFromExistingSession(SESSION_ID_1, fileStore, worker);
+    worker.await();
     assertEquals("", userData.getUserId());
   }
 
@@ -117,10 +118,11 @@ public void testWriteUserData_unicode() {
     assertEquals(UNICODE, userData.getUserId());
   }
 
-  public void testWriteUserData_escaped() {
+  public void testWriteUserData_escaped() throws Exception {
     storeUnderTest.writeUserData(
         SESSION_ID_1, metadataWithUserId(SESSION_ID_1, ESCAPED).getUserId());
     UserMetadata userData = UserMetadata.loadFromExistingSession(SESSION_ID_1, fileStore, worker);
+    worker.await();
     assertEquals(ESCAPED.trim(), userData.getUserId());
   }
 

Original file line number	Diff line number	Diff line change
`@@ -104,9 +104,10 @@ public void testWriteUserData_null() {`
`104`	`104`	`assertNull(userData.getUserId());`
`105`	`105`	`}`
`106`	`106`
`107`		`- public void testWriteUserData_emptyString() {`
	`107`	`+ public void testWriteUserData_emptyString() throws Exception {`
`108`	`108`	`storeUnderTest.writeUserData(SESSION_ID_1, metadataWithUserId(SESSION_ID_1, "").getUserId());`
`109`	`109`	`UserMetadata userData = UserMetadata.loadFromExistingSession(SESSION_ID_1, fileStore, worker);`
	`110`	`+ worker.await();`
`110`	`111`	`assertEquals("", userData.getUserId());`
`111`	`112`	`}`
`112`	`113`
`@@ -117,10 +118,11 @@ public void testWriteUserData_unicode() {`
`117`	`118`	`assertEquals(UNICODE, userData.getUserId());`
`118`	`119`	`}`
`119`	`120`
`120`		`- public void testWriteUserData_escaped() {`
	`121`	`+ public void testWriteUserData_escaped() throws Exception {`
`121`	`122`	`storeUnderTest.writeUserData(`
`122`	`123`	`SESSION_ID_1, metadataWithUserId(SESSION_ID_1, ESCAPED).getUserId());`
`123`	`124`	`UserMetadata userData = UserMetadata.loadFromExistingSession(SESSION_ID_1, fileStore, worker);`
	`125`	`+ worker.await();`
`124`	`126`	`assertEquals(ESCAPED.trim(), userData.getUserId());`
`125`	`127`	`}`
`126`	`128`