From 237191c5faacbbf41a1c8287bc5cefc55c65b206 Mon Sep 17 00:00:00 2001 From: James Waller Date: Tue, 11 Aug 2026 21:09:53 +0100 Subject: [PATCH 1/8] test --- README.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/README.md b/README.md index c2bec0368b7..22892c752e7 100644 --- a/README.md +++ b/README.md @@ -21,3 +21,5 @@ 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! + +James's version of Boot.dev's Notely app. \ No newline at end of file From e15f0aa1c8771d17d7106faf79a58fb584e49d55 Mon Sep 17 00:00:00 2001 From: James Waller Date: Wed, 12 Aug 2026 19:58:17 +0100 Subject: [PATCH 2/8] test --- .github/workflows/ci.yml | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) create mode 100644 .github/workflows/ci.yml diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml new file mode 100644 index 00000000000..664032071d1 --- /dev/null +++ b/.github/workflows/ci.yml @@ -0,0 +1,22 @@ +name: ci + +on: + pull_request: + branches: [main] + +jobs: + tests: + name: Tests + runs-on: ubuntu-latest + + steps: + - name: Check out code + uses: actions/checkout@v6 + + - name: Set up Go + uses: actions/setup-go@v6 + with: + go-version: "1.26.0" + + - name: Force Failure + run: (exit 1) \ No newline at end of file From 5b7c4c208ec6a5c4c46dc9e24d45d3c5f99659bd Mon Sep 17 00:00:00 2001 From: James Waller Date: Wed, 12 Aug 2026 20:03:43 +0100 Subject: [PATCH 3/8] test --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 664032071d1..40bdca78318 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -19,4 +19,4 @@ jobs: go-version: "1.26.0" - name: Force Failure - run: (exit 1) \ No newline at end of file + run: go version \ No newline at end of file From 39c3b8aa3e7a6adb5ff8582e88533c2aac69d672 Mon Sep 17 00:00:00 2001 From: James Waller Date: Wed, 12 Aug 2026 20:08:17 +0100 Subject: [PATCH 4/8] Add unit tests for GetAPIKey --- internal/auth/get_api_key_test.go | 63 +++++++++++++++++++++++++++++++ 1 file changed, 63 insertions(+) create mode 100644 internal/auth/get_api_key_test.go diff --git a/internal/auth/get_api_key_test.go b/internal/auth/get_api_key_test.go new file mode 100644 index 00000000000..663478c15f8 --- /dev/null +++ b/internal/auth/get_api_key_test.go @@ -0,0 +1,63 @@ +package auth + +import ( + "net/http" + "testing" +) + +func TestGetAPIKey(t *testing.T) { + tests := []struct { + name string + headers http.Header + wantKey string + wantErr string + }{ + { + name: "no authorization header", + headers: http.Header{}, + wantErr: ErrNoAuthHeaderIncluded.Error(), + }, + { + name: "malformed header - missing ApiKey prefix", + headers: http.Header{ + "Authorization": []string{"Bearer some-token"}, + }, + wantErr: "malformed authorization header", + }, + { + name: "malformed header - no value", + headers: http.Header{ + "Authorization": []string{"ApiKey"}, + }, + wantErr: "malformed authorization header", + }, + { + name: "valid api key", + headers: http.Header{ + "Authorization": []string{"ApiKey my-secret-key"}, + }, + wantKey: "my-secret-key", + }, + } + + for _, tt := range tests { + t.Run(tt.name, func(t *testing.T) { + gotKey, err := GetAPIKey(tt.headers) + if tt.wantErr != "" { + if err == nil { + t.Fatalf("expected error %q, got nil", tt.wantErr) + } + if err.Error() != tt.wantErr { + t.Fatalf("expected error %q, got %q", tt.wantErr, err.Error()) + } + return + } + if err != nil { + t.Fatalf("unexpected error: %v", err) + } + if gotKey != tt.wantKey { + t.Fatalf("expected key %q, got %q", tt.wantKey, gotKey) + } + }) + } +} From cbe0566e82abc428fb2dfdce3511d6456fc5b2ba Mon Sep 17 00:00:00 2001 From: James Waller Date: Wed, 12 Aug 2026 20:42:53 +0100 Subject: [PATCH 5/8] Add test step to CI, break GetAPIKey to verify CI catches failures --- .github/workflows/ci.yml | 4 ++-- internal/auth/auth.go | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 40bdca78318..43d841f43fd 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -18,5 +18,5 @@ jobs: with: go-version: "1.26.0" - - name: Force Failure - run: go version \ No newline at end of file + - name: Run tests + run: go test ./... diff --git a/internal/auth/auth.go b/internal/auth/auth.go index f969aacf638..c18bb42942e 100644 --- a/internal/auth/auth.go +++ b/internal/auth/auth.go @@ -19,5 +19,5 @@ func GetAPIKey(headers http.Header) (string, error) { return "", errors.New("malformed authorization header") } - return splitAuth[1], nil + return "", nil } From 6bd5c7cbcd4a44ad98d2a2983bf1b4ed9385f7a1 Mon Sep 17 00:00:00 2001 From: James Waller Date: Wed, 12 Aug 2026 20:43:04 +0100 Subject: [PATCH 6/8] Fix GetAPIKey to return the API key --- internal/auth/auth.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/auth/auth.go b/internal/auth/auth.go index c18bb42942e..f969aacf638 100644 --- a/internal/auth/auth.go +++ b/internal/auth/auth.go @@ -19,5 +19,5 @@ func GetAPIKey(headers http.Header) (string, error) { return "", errors.New("malformed authorization header") } - return "", nil + return splitAuth[1], nil } From de54fee92a79ebc8585ebe22e1b3be4000d8084a Mon Sep 17 00:00:00 2001 From: James Waller Date: Wed, 12 Aug 2026 20:49:01 +0100 Subject: [PATCH 7/8] Add test coverage to CI test step --- .github/workflows/ci.yml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 43d841f43fd..d989666eef8 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -19,4 +19,4 @@ jobs: go-version: "1.26.0" - name: Run tests - run: go test ./... + run: go test ./... -cover From b5a3db296babdb428958f4dcb9066b31abfe434a Mon Sep 17 00:00:00 2001 From: James Waller Date: Mon, 17 Aug 2026 20:00:34 +0100 Subject: [PATCH 8/8] Add CI test status badge to README --- README.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/README.md b/README.md index 22892c752e7..0c983c34f02 100644 --- a/README.md +++ b/README.md @@ -1,3 +1,6 @@ +![CI Badge](https://github.com/wallyflops/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).