From 27ee1a271248e9f1de8ecb981a6cabfa8e498b1b Mon Sep 17 00:00:00 2001
From: Peter Stone <thepeterstone@gmail.com>
Date: Fri, 6 Feb 2026 14:53:47 -1000
Subject: Fix missing settings button, disappeared events, and tab refresh bug

- Add settings gear icon link to dashboard header
- Fix GetTasksByDateRange/GetCardsByDateRange to include overdue items
  (changed from BETWEEN to <= end, filter completed tasks)
- Fix refresh replacing active tab with tasks tab by using
  htmx.trigger(body, 'refresh-tasks') instead of innerHTML+htmx.process
- Add refresh-tasks hx-trigger to meals, shopping, conditions tabs
- Add tests for overdue inclusion/exclusion, settings link, template data

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 internal/store/sqlite_test.go | 67 +++++++++++++++++++++++++++++++++++++++++--
 1 file changed, 64 insertions(+), 3 deletions(-)

(limited to 'internal/store/sqlite_test.go')

diff --git a/internal/store/sqlite_test.go b/internal/store/sqlite_test.go
index 69d188a..9c56252 100644
--- a/internal/store/sqlite_test.go
+++ b/internal/store/sqlite_test.go
@@ -482,17 +482,78 @@ func TestGetTasksByDateRange(t *testing.T) {
 		t.Fatalf("Failed to save tasks: %v", err)
 	}
 
-	// Test range covering today and tomorrow
+	// Test range covering today and tomorrow — should include all tasks <= end
 	start := now.Add(-1 * time.Hour)
 	end := tomorrow.Add(1 * time.Hour)
-	
+
 	results, err := store.GetTasksByDateRange(start, end)
 	if err != nil {
 		t.Fatalf("GetTasksByDateRange failed: %v", err)
 	}
 
 	if len(results) != 2 {
-		t.Errorf("Expected 2 tasks, got %d", len(results))
+		t.Errorf("Expected 2 tasks (today + tomorrow, not next week), got %d", len(results))
+	}
+}
+
+func TestGetTasksByDateRange_IncludesOverdue(t *testing.T) {
+	store := setupTestStoreWithTasks(t)
+	defer func() { _ = store.Close() }()
+
+	now := time.Now()
+	yesterday := now.Add(-24 * time.Hour)
+	tomorrow := now.Add(24 * time.Hour)
+
+	tasks := []models.Task{
+		{ID: "overdue", Content: "Overdue", DueDate: &yesterday},
+		{ID: "current", Content: "Current", DueDate: &now},
+	}
+
+	if err := store.SaveTasks(tasks); err != nil {
+		t.Fatalf("Failed to save tasks: %v", err)
+	}
+
+	// Query from "today" onward — overdue tasks (before start) should also be included
+	results, err := store.GetTasksByDateRange(now, tomorrow)
+	if err != nil {
+		t.Fatalf("GetTasksByDateRange failed: %v", err)
+	}
+
+	if len(results) != 2 {
+		t.Errorf("Expected 2 tasks (overdue + current), got %d", len(results))
+		for _, r := range results {
+			t.Logf("  task: %s due=%v", r.Content, r.DueDate)
+		}
+	}
+}
+
+func TestGetTasksByDateRange_ExcludesCompleted(t *testing.T) {
+	store := setupTestStoreWithTasks(t)
+	defer func() { _ = store.Close() }()
+
+	now := time.Now()
+	yesterday := now.Add(-24 * time.Hour)
+	tomorrow := now.Add(24 * time.Hour)
+
+	tasks := []models.Task{
+		{ID: "done", Content: "Completed overdue", DueDate: &yesterday, Completed: true},
+		{ID: "active", Content: "Active", DueDate: &now},
+	}
+
+	if err := store.SaveTasks(tasks); err != nil {
+		t.Fatalf("Failed to save tasks: %v", err)
+	}
+
+	results, err := store.GetTasksByDateRange(now, tomorrow)
+	if err != nil {
+		t.Fatalf("GetTasksByDateRange failed: %v", err)
+	}
+
+	if len(results) != 1 {
+		t.Errorf("Expected 1 task (only active), got %d", len(results))
+	}
+	if len(results) > 0 && results[0].ID != "active" {
+		t.Errorf("Expected active task, got %s", results[0].ID)
 	}
 }
 
-- 
cgit v1.2.3