this repo has no description
fix: update claud.md to always run go fmt
+37
-36
+1
CLAUDE.md
+1
CLAUDE.md
···
21
21
22
22
## Code Quality
23
23
24
+
- **Run `go fmt`** on all Go code before committing
24
25
- **No trailing whitespace** - validate with `git diff --check` before committing
25
26
- **No Docker usage** without explicitly discussing it first
26
27
- **Keep changes minimal** - avoid over-engineering or adding unnecessary features
+1
-1
internal/handler/api_v1_integration_test.go
+1
-1
internal/handler/api_v1_integration_test.go
+23
-23
internal/handler/api_v1_links_test.go
+23
-23
internal/handler/api_v1_links_test.go
···
18
18
// methods we need for testing.
19
19
type mockAPIStore struct {
20
20
data.Store
21
-
links []data.IRCLink
22
-
linkByID *data.IRCLink
23
-
linkByIDFn func(id int) (*data.IRCLink, error)
24
-
linksByURL []data.IRCLink
25
-
linksByURLFn func(url string) ([]data.IRCLink, error)
26
-
insertedLinkID int
27
-
insertLinkFn func(user, title, url, contentType string) (int, error)
28
-
deleteLinkFn func(id int) error
29
-
err error
21
+
links []data.IRCLink
22
+
linkByID *data.IRCLink
23
+
linkByIDFn func(id int) (*data.IRCLink, error)
24
+
linksByURL []data.IRCLink
25
+
linksByURLFn func(url string) ([]data.IRCLink, error)
26
+
insertedLinkID int
27
+
insertLinkFn func(user, title, url, contentType string) (int, error)
28
+
deleteLinkFn func(id int) error
29
+
err error
30
30
}
31
31
32
32
func (m *mockAPIStore) GetRecentIRCLinks(ctx context.Context, days int, offsetDays int) ([]data.IRCLink, error) {
···
89
89
checkBody func(t *testing.T, body []byte)
90
90
}{
91
91
{
92
-
name: "returns empty list",
93
-
method: http.MethodGet,
94
-
path: "/api/v1/links",
95
-
links: []data.IRCLink{},
92
+
name: "returns empty list",
93
+
method: http.MethodGet,
94
+
path: "/api/v1/links",
95
+
links: []data.IRCLink{},
96
96
expectedStatus: http.StatusOK,
97
97
checkBody: func(t *testing.T, body []byte) {
98
98
var resp APILinksResponse
···
221
221
},
222
222
},
223
223
{
224
-
name: "limit capped at 1000",
225
-
method: http.MethodGet,
226
-
path: "/api/v1/links?limit=5000",
227
-
links: []data.IRCLink{},
224
+
name: "limit capped at 1000",
225
+
method: http.MethodGet,
226
+
path: "/api/v1/links?limit=5000",
227
+
links: []data.IRCLink{},
228
228
expectedStatus: http.StatusOK,
229
229
checkBody: func(t *testing.T, body []byte) {
230
230
var resp APILinksResponse
···
237
237
},
238
238
},
239
239
{
240
-
name: "invalid limit uses default",
241
-
method: http.MethodGet,
242
-
path: "/api/v1/links?limit=abc",
243
-
links: []data.IRCLink{},
240
+
name: "invalid limit uses default",
241
+
method: http.MethodGet,
242
+
path: "/api/v1/links?limit=abc",
243
+
links: []data.IRCLink{},
244
244
expectedStatus: http.StatusOK,
245
245
checkBody: func(t *testing.T, body []byte) {
246
246
var resp APILinksResponse
···
736
736
},
737
737
},
738
738
{
739
-
name: "plain text response for duplicate",
740
-
body: `{"url":"https://example.com/article","user":"testuser"}`,
739
+
name: "plain text response for duplicate",
740
+
body: `{"url":"https://example.com/article","user":"testuser"}`,
741
741
linksByURL: []data.IRCLink{
742
742
{
743
743
ID: 10,
+7
-7
internal/handler/api_v1_redirect_test.go
+7
-7
internal/handler/api_v1_redirect_test.go
···
181
181
182
182
func TestAPIv1_RedirectHandler_ClickTracking(t *testing.T) {
183
183
tests := []struct {
184
-
name string
185
-
path string
186
-
sigQueryParam string
187
-
clickSigningKey string
188
-
linkURL string
189
-
expectIncrement bool
190
-
expectedStatus int
184
+
name string
185
+
path string
186
+
sigQueryParam string
187
+
clickSigningKey string
188
+
linkURL string
189
+
expectIncrement bool
190
+
expectedStatus int
191
191
}{
192
192
{
193
193
name: "valid signature increments clicks",
+5
-5
internal/handler/api_v1_stats_test.go
+5
-5
internal/handler/api_v1_stats_test.go
···
14
14
// mockStatsStore is a mock implementation of data.Store for testing stats API handlers.
15
15
type mockStatsStore struct {
16
16
data.Store
17
-
userStats []data.UserStat
18
-
userStatsFn func(sortBy string, limit int, offset int) ([]data.UserStat, error)
19
-
links []data.IRCLink
20
-
quotes []data.Quote
21
-
err error
17
+
userStats []data.UserStat
18
+
userStatsFn func(sortBy string, limit int, offset int) ([]data.UserStat, error)
19
+
links []data.IRCLink
20
+
quotes []data.Quote
21
+
err error
22
22
}
23
23
24
24
func (m *mockStatsStore) GetUserStats(ctx context.Context, sortBy string, limit int, offset int) ([]data.UserStat, error) {