[PM-23710] Fixed logic to getServerConfig and added new test on Authenticator (#5518)

data/src/main/kotlin/com/bitwarden/data/repository/ServerConfigRepositoryImpl.kt

@@ -35,11 +35,11 @@ internal class ServerConfigRepositoryImpl(
     override suspend fun getServerConfig(forceRefresh: Boolean): ServerConfig? {
         val localConfig = configDiskSource.serverConfig
         val needsRefresh = localConfig == null ||
-            Instant
-                .ofEpochMilli(localConfig.lastSync)
-                .isAfter(
-                    clock.instant().plusSeconds(MINIMUM_CONFIG_SYNC_INTERVAL_SEC),
-                )
+            clock.instant().isAfter(
+                Instant
+                    .ofEpochMilli(localConfig.lastSync)
+                    .plusSeconds(MINIMUM_CONFIG_SYNC_INTERVAL_SEC),
+            )
 
         if (needsRefresh || forceRefresh) {
             configService

data/src/test/kotlin/com/bitwarden/data/repository/ServerConfigRepositoryTest.kt

@@ -119,6 +119,30 @@ class ServerConfigRepositoryTest {
             assertEquals(fakeConfigDiskSource.serverConfig, awaitItem())
         }
     }
+
+    @Suppress("MaxLineLength")
+    @Test
+    fun `serverConfigStateFlow should fetch new server configurations when minimum config sync interval is reached `() =
+        runTest {
+
+            val testConfig = SERVER_CONFIG.copy(
+                lastSync = fixedClock.instant().minusSeconds(60 * 60 + 1).toEpochMilli(),
+                serverData = CONFIG_RESPONSE_JSON.copy(
+                    version = "old version!!",
+                ),
+            )
+            fakeConfigDiskSource.serverConfig = testConfig
+
+            coEvery {
+                configService.getConfig()
+            } returns CONFIG_RESPONSE_JSON.asSuccess()
+
+            repository.getServerConfig(forceRefresh = false)
+
+            repository.serverConfigStateFlow.test {
+                assertNotEquals(testConfig, awaitItem())
+            }
+        }
 }
 
 private val SERVER_CONFIG = ServerConfig(