make grace period configurable and reduce test duration

Author: BillCarsonFr

vector/src/main/java/im/vector/app/UISIDetector.kt

@@ -52,7 +52,7 @@ data class E2EMessageDetected(
     }
 }
 
-class UISIDetector : LiveEventListener {
+class UISIDetector(private val timeoutMillis: Long = 30_000L) : LiveEventListener {
 
     interface UISIDetectorCallback {
         val enabled: Boolean
@@ -66,7 +66,6 @@ class UISIDetector : LiveEventListener {
     private val trackedEvents = mutableMapOf<String, TimerTask>()
     private val executor = Executors.newSingleThreadExecutor()
     private val timer = Timer()
-    private val timeoutMillis = 30_000L
     private val enabled: Boolean get() = callback?.enabled.orFalse()
 
     override fun onEventDecrypted(event: Event, clearEvent: JsonDict) {

vector/src/test/java/im/vector/app/features/crypto/UISIDetectorTest.kt

@@ -38,7 +38,8 @@ class UISIDetectorTest {
 
     @Test
     fun `trigger detection after grace period`() {
-        val uisiDetector = UISIDetector()
+        val gracePeriod = 5_000L
+        val uisiDetector = UISIDetector(gracePeriod)
         var detectedEvent: E2EMessageDetected? = null
 
         uisiDetector.callback = object : UISIDetector.UISIDetectorCallback {
@@ -60,15 +61,16 @@ class UISIDetectorTest {
         uisiDetector.onEventDecryptionError(event, fakeCryptoError())
 
         runBlocking {
-            delay(40_000)
+            delay((gracePeriod * 1.2).toLong())
         }
         Assert.assertEquals(eventId, detectedEvent?.eventId)
     }
 
     @Test
     fun `If event decrypted during grace period should not trigger detection`() {
         val scope = CoroutineScope(SupervisorJob())
-        val uisiDetector = UISIDetector()
+        val gracePeriod = 5_000L
+        val uisiDetector = UISIDetector(gracePeriod)
 
         uisiDetector.callback = object : UISIDetector.UISIDetectorCallback {
             override val enabled = true
@@ -89,12 +91,12 @@ class UISIDetectorTest {
 
         // the grace period is 30s
         scope.launch(Dispatchers.Default) {
-            delay(10_000)
+            delay((gracePeriod * 0.5).toLong())
             uisiDetector.onEventDecrypted(event, emptyMap())
         }
 
         runBlocking {
-            delay(60_000)
+            delay((gracePeriod * 1.2).toLong())
         }
     }