test: ensure initial file path cursor is visible in window (20 files, 5 visible)

Author: lazysegtree

src/internal/model_test.go

@@ -3,6 +3,7 @@ package internal
 import (
 	"flag"
 	"fmt"
+	tea "github.com/charmbracelet/bubbletea"
 	"os"
 	"path/filepath"
 	"testing"
@@ -116,6 +117,55 @@ func TestInitialFilePathPositionsCursor(t *testing.T) {
 	assert.Equal(t, m.fileModel.filePanels[1].getSelectedItem().location, file2)
 }
 
+func TestInitialFilePathPositionsCursorWindow(t *testing.T) {
+	curTestDir := filepath.Join(testDir, "TestInitialFilePathPositionsCursorWindow")
+	dir1 := filepath.Join(curTestDir, "dir1")
+
+	utils.SetupDirectories(t, curTestDir, dir1)
+	t.Cleanup(func() { _ = os.RemoveAll(curTestDir) })
+
+	// Create 20 files with zero-padded names so lexicographic == numeric order.
+	var targetFile string
+	for i := 1; i <= 20; i++ {
+		f := filepath.Join(dir1, fmt.Sprintf("file%02d.txt", i))
+		utils.SetupFiles(t, f)
+		if i == 8 {
+			targetFile = f
+		}
+	}
+
+	// Initialize model with the file path; cursor should land on file08
+	m := defaultTestModel(targetFile)
+
+	// Make panel show exactly 5 entries: mainPanelHeight = 8 => 8-3 = 5
+	TeaUpdate(m, tea.WindowSizeMsg{Width: common.MinimumWidth, Height: 10})
+
+	// Populate elements and apply target cursor
+	m.getFilePanelItems()
+
+	panel := m.getFocusedFilePanel()
+	require.Equal(t, dir1, panel.location, "Panel should open on parent directory")
+	require.Len(t, panel.element, 20, "Should have 20 elements")
+
+	// Find expected target index for file08
+	targetIdx := -1
+	for i, el := range panel.element {
+		if el.name == "file08.txt" {
+			targetIdx = i
+			break
+		}
+	}
+	require.NotEqual(t, -1, targetIdx, "file08.txt should exist in elements")
+
+	// Ensure cursor is on file08 and target is within visible window
+	assert.Equal(t, targetIdx, panel.cursor, "Cursor should be on file08")
+
+	visStart := panel.render
+	visEnd := min(panel.render+panelElementHeight(m.mainPanelHeight), len(panel.element))
+	assert.GreaterOrEqual(t, targetIdx, visStart, "Target file must be visible (>= start)")
+	assert.Less(t, targetIdx, visEnd, "Target file must be visible (< end)")
+}
+
 func TestQuit(t *testing.T) {
 	// Test
 	// 1 - Normal quit