From d36c752c72b71fd22280ada11089018e7f4e3622 Mon Sep 17 00:00:00 2001
From: austinried <4966622+austinried@users.noreply.github.com>
Date: Fri, 25 Sep 2026 08:37:16 +0000
Subject: [PATCH] test the tab row reflows in a single direction during a
 switch

Drives the pager across the Artists to Playlists transition and asserts the
Songs icon only moves one way. Fails on the min-width clamp version (738-727-745)
and passes on the natural-48dp version. LibraryTabs is internal so the test can
drive its PagerState.
---
 .../com/subtracks/ui/library/LibraryScreen.kt |  2 +-
 .../com/subtracks/ui/LibraryScreenTest.kt     | 33 +++++++++++++++++++
 2 files changed, 34 insertions(+), 1 deletion(-)

diff --git a/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt b/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt
index b0e18988..c11d189c 100644
--- a/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt
+++ b/app/src/main/kotlin/com/subtracks/ui/library/LibraryScreen.kt
@@ -660,7 +660,7 @@ private fun SearchField(
 
 @OptIn(ExperimentalMaterial3Api::class)
 @Composable
-private fun LibraryTabs(
+internal fun LibraryTabs(
     pagerState: PagerState,
     onTabSelected: (LibraryTab) -> Unit,
     onOpenSettings: () -> Unit,
diff --git a/app/src/test/kotlin/com/subtracks/ui/LibraryScreenTest.kt b/app/src/test/kotlin/com/subtracks/ui/LibraryScreenTest.kt
index bb56b81f..505755b6 100644
--- a/app/src/test/kotlin/com/subtracks/ui/LibraryScreenTest.kt
+++ b/app/src/test/kotlin/com/subtracks/ui/LibraryScreenTest.kt
@@ -1,6 +1,7 @@
 package com.subtracks.ui
 
 import androidx.activity.ComponentActivity
+import androidx.compose.foundation.pager.PagerState
 import androidx.compose.runtime.remember
 import androidx.compose.ui.test.junit4.v2.createAndroidComposeRule
 import androidx.compose.ui.test.onAllNodesWithTag
@@ -24,6 +25,7 @@ import com.subtracks.data.model.SongListItem
 import com.subtracks.ui.library.ALBUM_COVER_TAG
 import com.subtracks.ui.library.LibraryScreen
 import com.subtracks.ui.library.LibraryTab
+import com.subtracks.ui.library.LibraryTabs
 import com.subtracks.ui.theme.SubtracksTheme
 import kotlinx.coroutines.flow.flowOf
 import org.junit.Assert.assertTrue
@@ -107,6 +109,37 @@ class LibraryScreenTest {
         assertTrue("an empty library should offer a sync", synced)
     }
 
+    @Test
+    fun theTabRowReflowsInOneDirectionDuringASwitch() {
+        val pagerState = PagerState(currentPage = 1, pageCount = { LibraryTab.entries.size })
+        composeRule.setContent {
+            SubtracksTheme {
+                LibraryTabs(
+                    pagerState = pagerState,
+                    onTabSelected = {},
+                    onOpenSettings = {},
+                    artwork = null,
+                )
+            }
+        }
+        composeRule.waitForIdle()
+
+        val samples =
+            (0..10).map { step ->
+                composeRule.runOnIdle { pagerState.requestScrollToPage(1, step * 0.05f) }
+                composeRule.waitForIdle()
+                composeRule
+                    .onNodeWithContentDescription(LibraryTab.Songs.label)
+                    .fetchSemanticsNode()
+                    .boundsInRoot.left
+            }
+
+        val monotonic =
+            samples.zipWithNext().all { (a, b) -> b >= a } ||
+                samples.zipWithNext().all { (a, b) -> b <= a }
+        assertTrue("the tab row should reflow in one direction, but the icons moved $samples", monotonic)
+    }
+
     private val settledLoadStates =
         LoadStates(
             refresh = LoadState.NotLoading(endOfPaginationReached = true),