Fix racy cache-hit assertions in CacheManagerTest (#11)

The cache-hit path deliberately fires a deduped background refresh, so
an assertion that the request count stays identical races the async job
and loses on slow CI machines. Assert the cached value and allow at
most one extra request instead.

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: yinnho

app/src/test/java/com/yinnho/upnpcast/internal/core/CacheManagerTest.kt

@@ -62,10 +62,16 @@ class CacheManagerTest {
         val first = cacheManager.getProgress(controller)
         assertEquals(Pair(10000L, 600000L), first)
 
+        // A cache hit answers synchronously; the by-design background
+        // refresh may fire at most one extra GetPositionInfo (deduped
+        // while a job is active), racing with this assertion on slow
+        // machines — accept either outcome
         val positionCallsAfterFirst = renderer.callsFor("GetPositionInfo").size
         val second = cacheManager.getProgress(controller)
         assertNotNull(second)
-        assertEquals(positionCallsAfterFirst, renderer.callsFor("GetPositionInfo").size)
+        assertEquals(600000L, second!!.second)
+        val positionCallsAfterSecond = renderer.callsFor("GetPositionInfo").size
+        assertTrue(positionCallsAfterSecond - positionCallsAfterFirst <= 1)
     }
 
     @Test
@@ -89,9 +95,11 @@ class CacheManagerTest {
     fun volumeIsCachedWithMuteState() = runBlocking {
         assertEquals(Pair(30, false), cacheManager.getVolume(controller))
 
+        // Cache hit returns the cached pair; the background refresh may
+        // add at most one GetVolume call racing this assertion
         val volumeCallsAfterFirst = renderer.callsFor("GetVolume").size
         assertEquals(Pair(30, false), cacheManager.getVolume(controller))
-        assertEquals(volumeCallsAfterFirst, renderer.callsFor("GetVolume").size)
+        assertTrue(renderer.callsFor("GetVolume").size - volumeCallsAfterFirst <= 1)
     }
 
     @Test