From b3934c2c27fa4ef3ac2c3d8c4c425100c57df4fa Mon Sep 17 00:00:00 2001
From: John O'Keefe <jrokeefe@protonmail.com>
Date: Fri, 13 Feb 2026 21:50:52 -0500
Subject: [PATCH] chore: remove old phase1_example_test.go file

Remove the old phase1_example_test.go file that was renamed to
device_test_patterns_test.go. This file should have been removed
in the previous commit but was missed.
---
 cmd/server/tests/phase1_example_test.go | 88 -------------------------
 1 file changed, 88 deletions(-)
 delete mode 100644 cmd/server/tests/phase1_example_test.go

diff --git a/cmd/server/tests/phase1_example_test.go b/cmd/server/tests/phase1_example_test.go
deleted file mode 100644
index 07952c3..0000000
--- a/cmd/server/tests/phase1_example_test.go
+++ /dev/null
@@ -1,88 +0,0 @@
-package main
-
-import (
-	"bookhoard/internal/database"
-	"bookhoard/internal/handlers"
-	"bytes"
-	"context"
-	"encoding/json"
-	"fmt"
-	"net/http"
-	"net/http/httptest"
-	"testing"
-
-	"github.com/google/uuid"
-	"github.com/jackc/pgx/v5/pgtype"
-	"github.com/stretchr/testify/assert"
-	"github.com/stretchr/testify/require"
-)
-
-// TestWithStructs demonstrates Phase 1 improvements
-// BEFORE: map[string]interface{}  ->  AFTER: handlers.* structs
-// BEFORE: No DB verification    ->  AFTER: Database verification
-
-func TestWithStructs(t *testing.T) {
-	deviceSetup := setupDeviceTest(t)
-	token := deviceSetup.UserToken
-
-	t.Run("ListDevices with struct", func(t *testing.T) {
-		req := httptest.NewRequest("GET", "/api/devices", nil)
-		req.Header.Set("Authorization", "Bearer "+token)
-		rr := httptest.NewRecorder()
-		deviceSetup.Server.Config.Handler.ServeHTTP(rr, req)
-
-		assert.Equal(t, http.StatusOK, rr.Code, "Should list devices")
-
-		var response handlers.DeviceListResponse
-		err := json.Unmarshal(rr.Body.Bytes(), &response)
-		require.NoError(t, err, "Response should match DeviceListResponse schema")
-		assert.GreaterOrEqual(t, len(response.Devices), 1, "Should have at least one device")
-
-		firstDevice := response.Devices[0]
-		assert.Equal(t, deviceSetup.Device.Name, firstDevice.DeviceName, "Should match device name")
-		assert.Equal(t, deviceSetup.Device.Type, firstDevice.DeviceType, "Should match device type")
-	})
-
-	t.Run("UpdateDevice with struct and DB verification", func(t *testing.T) {
-		syncEnabled := false
-		syncFreq := int32(15)
-
-		updateRequest := map[string]interface{}{
-			"sync_enabled":           &syncEnabled,
-			"sync_frequency_minutes": &syncFreq,
-		}
-		updateBody, _ := json.Marshal(updateRequest)
-
-		req := httptest.NewRequest("PUT", fmt.Sprintf("/api/devices/%s", deviceSetup.Device.ID), bytes.NewBuffer(updateBody))
-		req.Header.Set("Content-Type", "application/json")
-		req.Header.Set("Authorization", "Bearer "+token)
-		rr := httptest.NewRecorder()
-		deviceSetup.Server.Config.Handler.ServeHTTP(rr, req)
-
-		assert.Equal(t, http.StatusOK, rr.Code, "Should update device")
-
-		var response map[string]interface{}
-		err := json.Unmarshal(rr.Body.Bytes(), &response)
-		require.NoError(t, err, "Response should unmarshal")
-
-		assert.True(t, response["device_updated"].(bool), "Device should be updated")
-
-		// NEW: Database verification
-		pgDeviceID := pgtype.UUID{Bytes: [16]byte(deviceSetup.Device.ID), Valid: true}
-		device, err := deviceSetup.DB.GetDevice(context.Background(), pgDeviceID)
-		require.NoError(t, err, "Device should exist in database after update")
-
-		assert.Equal(t, syncEnabled, device.SyncEnabled.Bool, "DB: Sync should be disabled")
-		assert.Equal(t, syncFreq, device.SyncFrequencyMinutes.Int32, "DB: Sync frequency should be updated")
-	})
-}
-
-// verifyDeviceUpdated is a helper function for Phase 1 database verification
-func verifyDeviceUpdated(t *testing.T, db *database.Queries, deviceID uuid.UUID, syncEnabled bool, syncFreq int32) {
-	pgDeviceID := pgtype.UUID{Bytes: [16]byte(deviceID), Valid: true}
-	device, err := db.GetDevice(context.Background(), pgDeviceID)
-	require.NoError(t, err, "Device should exist in database")
-
-	assert.Equal(t, syncEnabled, device.SyncEnabled.Bool, "DB: Sync enabled should match")
-	assert.Equal(t, syncFreq, device.SyncFrequencyMinutes.Int32, "DB: Sync frequency should match")
-}