From bdc7b5b1b6be54796105937d2eea9f6b5d76695f Mon Sep 17 00:00:00 2001
From: Micheal Wilkinson <micheal.wilkinson@xc.betfanatics.com>
Date: Tue, 17 Mar 2026 23:14:09 +0000
Subject: [PATCH] Adding tests for getting notes by id

---
 internal/store/sqlite/sqlite.go      |  6 ++-
 internal/store/sqlite/sqlite_test.go | 61 ++++++++++++++++++++++++++++
 2 files changed, 66 insertions(+), 1 deletion(-)

diff --git a/internal/store/sqlite/sqlite.go b/internal/store/sqlite/sqlite.go
index eeeae42..abf9fd0 100644
--- a/internal/store/sqlite/sqlite.go
+++ b/internal/store/sqlite/sqlite.go
@@ -113,6 +113,10 @@ func (s *SQLiteStore) SaveNote(ctx context.Context, note models.Note) (models.No
 	return note, nil
 }
 
+func (s *SQLiteStore) GetNoteByID(ctx context.Context, id int) (models.Note, error) {
+	return models.Note{}, fmt.Errorf("not implemented")
+}
+
 func (s *SQLiteStore) validateSchema(ctx context.Context) error {
 	_, err := s.write.ExecContext(ctx, `
 		CREATE TABLE IF NOT EXISTS notes (
@@ -139,4 +143,4 @@ func (s *SQLiteStore) scanNote(results *sql.Row) (models.Note, error) {
 		return models.Note{}, fmt.Errorf("failed to scan note: %w", err)
 	}
 	return note, nil
-}
+}
\ No newline at end of file
diff --git a/internal/store/sqlite/sqlite_test.go b/internal/store/sqlite/sqlite_test.go
index a2b0c07..8fd1d55 100644
--- a/internal/store/sqlite/sqlite_test.go
+++ b/internal/store/sqlite/sqlite_test.go
@@ -151,3 +151,64 @@ func TestSaveNote(t *testing.T) {
 		})
 	}
 }
+
+func TestGetNoteByID(t *testing.T) {
+	dbPath := filepath.Join(t.TempDir(), "test.db")
+	store, err := sqlite.NewSQLiteStore(context.Background(), dbPath)
+	if err != nil {
+		t.Fatalf("failed to create SQLite store: %v", err)
+	}
+	defer func() {
+		store.Close()
+		if _, err := os.Stat(dbPath); !errors.Is(err, os.ErrNotExist) {
+			if err := os.Remove(dbPath); err != nil {
+				t.Errorf("failed to clean up database file: %v", err)
+			}
+		}
+	}()
+	store.SaveNote(context.Background(), models.Note{
+		Content: "Test note",
+		LastUpdate: time.Date(2000, 1, 1, 0, 0, 0, 0, time.UTC),
+	})
+
+	testcases := []struct {
+		name          string
+		id            int
+		expectedNote  models.Note
+		expectedError bool
+	}{
+		{
+			name: "get existing note",
+			id:   1,
+			expectedNote: models.Note{
+				ID:         1,
+				Content:    "Test note",
+				LastUpdate: time.Date(2000, 1, 1, 0, 0, 0, 0, time.UTC),
+			},
+			expectedError: false,
+		},
+		{
+			name:          "get non-existent note",
+			id:            999,
+			expectedNote:  models.Note{},
+			expectedError: true,
+		},
+	}
+	for _, tc := range testcases {
+		t.Run(tc.name, func(t *testing.T) {
+			note, err := store.GetNoteByID(context.Background(), tc.id)
+			if tc.expectedError {
+				if err == nil {
+					t.Errorf("expected an error but got none")
+				}
+			} else {
+				if err != nil {
+					t.Errorf("unexpected error: %v", err)
+				}
+				if note.ID != tc.expectedNote.ID || note.Content != tc.expectedNote.Content || !note.LastUpdate.Equal(tc.expectedNote.LastUpdate) {
+					t.Errorf("expected note %+v but got %+v", tc.expectedNote, note)
+				}
+			}
+		})
+	}
+}