Add 'Test' section to README

.github/workflows/ci.yml

@@ -0,0 +1,47 @@
+name: ci
+
+on:
+  pull_request:
+    branches: [main]
+
+jobs:
+  tests:
+    name: Tests
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v4
+
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: "1.22"
+
+      - name: Run tests
+        run: go test ./... --cover
+
+  style:
+    name: Style
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v4
+
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          go-version: "1.22"
+
+      - name: Check formatting
+        run: test -z $(go fmt ./...)
+
+      - name: Install staticcheck
+        run: go install honnef.co/go/tools/cmd/staticcheck@latest
+
+      - name: Run staticcheck
+        run: staticcheck ./...
+
+      - name: Install gosec
+        run: go install github.com/securego/gosec/v2/cmd/gosec@latest

README.md

@@ -1,4 +1,6 @@
-# learn-cicd-starter (Notely)
+![CI](https://github.com/ivanbonifazi-cpu/learn-cicd-starter/actions/workflows/ci.yml/badge.svg)
+
+ # learn-cicd-starter (Notely)
 
 This repo contains the starter code for the "Notely" application for the "Learn CICD" course on [Boot.dev](https://boot.dev).
 
@@ -21,3 +23,4 @@ go build -o notely && ./notely
 *This starts the server in non-database mode.* It will serve a simple webpage at `http://localhost:8080`.
 
 You do *not* need to set up a database or any interactivity on the webpage yet. Instructions for that will come later in the course!
+IVAN

internal/auth/auth_test.go

@@ -0,0 +1,60 @@
+package auth
+
+import (
+	"net/http"
+	"testing"
+)
+
+func TestGetAPIKey(t *testing.T) {
+	// Test case 1: Valid API key in header
+	t.Run("Valid API key", func(t *testing.T) {
+		headers := http.Header{}
+		headers.Set("Authorization", "ApiKey my-secret-key-123")
+
+		apiKey, err := GetAPIKey(headers)
+
+		if err != nil {
+			t.Fatalf("Expected no error, got %v", err)
+		}
+
+		expected := "my-secret-key-123"
+		if apiKey != expected {
+			t.Errorf("Expected %s, got %s", expected, apiKey)
+		}
+	})
+
+	// Test case 2: No Authorization header
+	t.Run("No Authorization header", func(t *testing.T) {
+		headers := http.Header{}
+
+		_, err := GetAPIKey(headers)
+
+		if err == nil {
+			t.Fatal("Expected an error, got nil")
+		}
+	})
+
+	// Test case 3: Malformed Authorization header (missing "ApiKey")
+	t.Run("Malformed header - no ApiKey prefix", func(t *testing.T) {
+		headers := http.Header{}
+		headers.Set("Authorization", "Bearer my-token")
+
+		_, err := GetAPIKey(headers)
+
+		if err == nil {
+			t.Fatal("Expected an error, got nil")
+		}
+	})
+
+	// Test case 4: Malformed Authorization header (only "ApiKey", no key)
+	t.Run("Malformed header - no key value", func(t *testing.T) {
+		headers := http.Header{}
+		headers.Set("Authorization", "ApiKey")
+
+		_, err := GetAPIKey(headers)
+
+		if err == nil {
+			t.Fatal("Expected an error, got nil")
+		}
+	})
+}

json.go

@@ -30,5 +30,7 @@ func respondWithJSON(w http.ResponseWriter, code int, payload interface{}) {
 		return
 	}
 	w.WriteHeader(code)
-	w.Write(dat)
+	if _, err := w.Write(dat); err != nil {
+		log.Printf("Error writing response: %v", err)
+	}
 }

main.go

@@ -7,6 +7,7 @@ import (
 	"log"
 	"net/http"
 	"os"
+	"time"
 
 	"github.com/go-chi/chi"
 	"github.com/go-chi/cors"
@@ -89,8 +90,9 @@ func main() {
 
 	router.Mount("/v1", v1Router)
 	srv := &http.Server{
-		Addr:    ":" + port,
-		Handler: router,
+		Addr:              ":" + port,
+		Handler:           router,
+		ReadHeaderTimeout: 10 * time.Second,
 	}
 
 	log.Printf("Serving on port: %s\n", port)