Merge pull request #92 from theantichris/feat/issue-87/sort-daily-tasks

Feat/issue 87/sort daily tasks

Author: NucleoFusion

internal/models/daily/items.go

@@ -4,6 +4,7 @@ import (
 	"fmt"
 	"os"
 	"path/filepath"
+	"slices"
 	"time"
 
 	"github.com/SourcewareLab/Toney/v2/internal/config"
@@ -76,6 +77,10 @@ func GetItems() []Task {
 		fmt.Println(err.Error())
 	}
 
+	slices.SortStableFunc(tasks, func(a, b Task) int {
+		return getStatusOrder(a.Status) - getStatusOrder(b.Status)
+	})
+
 	return tasks
 }
 
@@ -129,3 +134,18 @@ func filterRolloverTasks(tasks []Task) []Task {
 	}
 	return result
 }
+
+func getStatusOrder(s enums.TaskStatus) int {
+	switch s {
+	case enums.Started:
+		return 0
+	case enums.Pending:
+		return 1
+	case enums.Abandoned:
+		return 2
+	case enums.Complete:
+		return 3
+	default:
+		return 4
+	}
+}

internal/models/daily/items_test.go

@@ -106,6 +106,40 @@ func TestFilterRolloverTasks(t *testing.T) {
 	}
 }
 
+func TestGetStatusOrder(t *testing.T) {
+	tests := []struct {
+		name   string
+		status enums.TaskStatus
+		want   int
+	}{
+		{"started is first", enums.Started, 0},
+		{"pending is second", enums.Pending, 1},
+		{"abandoned is third", enums.Abandoned, 2},
+		{"complete is last", enums.Complete, 3},
+		{"unknown status returns 4", enums.TaskStatus(99), 4},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			got := getStatusOrder(tt.status)
+			if got != tt.want {
+				t.Errorf("getStatusOrder(%d) = %d, want %d", tt.status, got, tt.want)
+			}
+		})
+	}
+
+	// Verify relative ordering: Started < Pending < Abandoned < Complete
+	if getStatusOrder(enums.Started) >= getStatusOrder(enums.Pending) {
+		t.Error("Started should sort before Pending")
+	}
+	if getStatusOrder(enums.Pending) >= getStatusOrder(enums.Abandoned) {
+		t.Error("Pending should sort before Abandoned")
+	}
+	if getStatusOrder(enums.Abandoned) >= getStatusOrder(enums.Complete) {
+		t.Error("Abandoned should sort before Complete")
+	}
+}
+
 func TestFindMostRecentFile(t *testing.T) {
 	tests := []struct {
 		name     string