Fix concurrent tests

embrace-io · Sep 23, 2024 · ec9d3c3 · ec9d3c3
1 parent eaca61e
commit ec9d3c3
Showing 1 changed file with 0 additions and 9 deletions.
diff --git a/...ace/android/embracesdk/internal/comms/delivery/EmbraceCacheServiceConcurrentAccessTest.kt b/...ace/android/embracesdk/internal/comms/delivery/EmbraceCacheServiceConcurrentAccessTest.kt
@@ -13,7 +13,6 @@ import io.embrace.android.embracesdk.internal.payload.SessionPayload
 import org.junit.Assert.assertEquals
 import org.junit.Assert.assertNull
 import org.junit.Before
-import org.junit.Ignore
 import org.junit.Test
 
 internal class EmbraceCacheServiceConcurrentAccessTest {
@@ -40,7 +39,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         )
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `concurrent write attempts to the same non-existent session file should lead to last finished persist`() {
         val type = Envelope.sessionEnvelopeType
@@ -59,7 +57,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         )
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `accessing sessions with different names should not block`() {
         val type = Envelope.sessionEnvelopeType
@@ -72,7 +69,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         )
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `reading a session should not block other reads to same session`() {
         val type = Envelope.sessionEnvelopeType
@@ -85,7 +81,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         )
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `reads should block writes`() {
         val type = Envelope.sessionEnvelopeType
@@ -106,7 +101,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         )
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `reading a file that is being written to should block and succeed`() {
         var readSession: Envelope<SessionPayload>? = null
@@ -121,7 +115,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         assertEquals(executionCoordinator.getErrorMessage(), testSessionEnvelope.getSessionId(), readSession?.getSessionId())
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `reading a file that is being rewritten to should block and succeed`() {
         var readSession: Envelope<SessionPayload>? = null
@@ -143,7 +136,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         )
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `interrupting a session write should not leave a file`() {
         val type = Envelope.sessionEnvelopeType
@@ -157,7 +149,6 @@ internal class EmbraceCacheServiceConcurrentAccessTest {
         assertNull(embraceCacheService.loadObject(FILENAME, type))
     }
 
-    @Ignore("Failing in CI after switching to different image")
     @Test
     fun `interrupting a session rewrite should not overwrite the file`() {
         val type = Envelope.sessionEnvelopeType