fix: group tasks by section

Author: antoniolg

internal/cli/list.go

@@ -73,16 +73,19 @@ func groupTasksBySection(items []*tasks.Task, filter string) (map[string][]taskR
 	sectionIDs := map[string]string{}
 	sectionNames := map[string]bool{}
 
-	for i, item := range items {
+	for _, item := range items {
 		if _, ok := metadata.Extract(item.Notes, "justdoit_section"); ok {
 			sectionIDs[item.Id] = item.Title
 			if !sectionNames[item.Title] {
 				order = append(order, item.Title)
 				sectionNames[item.Title] = true
 			}
+		}
+	}
+	for i, item := range items {
+		if _, ok := metadata.Extract(item.Notes, "justdoit_section"); ok {
 			continue
 		}
-
 		sectionName := "General"
 		if parent, ok := sectionIDs[item.Parent]; ok {
 			sectionName = parent

internal/cli/list_test.go

@@ -0,0 +1,27 @@
+package cli
+
+import (
+	"testing"
+
+	"google.golang.org/api/tasks/v1"
+)
+
+func TestGroupTasksBySectionBuildsSectionIndexFirst(t *testing.T) {
+	sectionID := "section-1"
+	items := []*tasks.Task{
+		{Id: "task-1", Title: "Follow up", Parent: sectionID},
+		{Id: sectionID, Title: "Recurrentes", Notes: "justdoit_section=1"},
+	}
+
+	sections, order := groupTasksBySection(items, "")
+	if len(sections) == 0 {
+		t.Fatalf("expected sections, got none")
+	}
+	tasks := sections["Recurrentes"]
+	if len(tasks) != 1 || tasks[0].Title != "Follow up" {
+		t.Fatalf("expected task under section, got %#v", sections)
+	}
+	if len(order) == 0 || order[0] != "Recurrentes" {
+		t.Fatalf("expected section order to include Recurrentes, got %#v", order)
+	}
+}