Files
bvandeusen 8600b253fd style: gofmt fixes across api package
CI's golangci-lint run flagged three files; two pre-existed this
branch but the Plan 3 seedTrackWithFile struct-literal alignment
is new. Applying gofmt across all three keeps the lint baseline clean.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-04-21 23:02:20 -04:00

228 lines
5.9 KiB
Go

package api
import (
"net/url"
"testing"
"github.com/jackc/pgx/v5/pgtype"
"git.fabledsword.com/bvandeusen/minstrel/internal/db/dbq"
)
func TestUUIDToString(t *testing.T) {
var u pgtype.UUID
if err := u.Scan("00112233-4455-6677-8899-aabbccddeeff"); err != nil {
t.Fatalf("scan: %v", err)
}
got := uuidToString(u)
want := "00112233-4455-6677-8899-aabbccddeeff"
if got != want {
t.Errorf("uuidToString = %q, want %q", got, want)
}
var zero pgtype.UUID
if s := uuidToString(zero); s != "" {
t.Errorf("uuidToString(invalid) = %q, want empty", s)
}
}
func TestParseUUID(t *testing.T) {
u, ok := parseUUID("00112233-4455-6677-8899-aabbccddeeff")
if !ok || !u.Valid {
t.Fatalf("parseUUID valid: ok=%v valid=%v", ok, u.Valid)
}
if _, ok := parseUUID("not-a-uuid"); ok {
t.Error("parseUUID accepted garbage")
}
if _, ok := parseUUID(""); ok {
t.Error("parseUUID accepted empty string")
}
}
func TestYearFromDate(t *testing.T) {
var d pgtype.Date
if y := yearFromDate(d); y != 0 {
t.Errorf("yearFromDate(invalid) = %d, want 0", y)
}
}
func TestDurationMsToSec(t *testing.T) {
cases := []struct {
ms int32
want int
}{
{0, 0},
{500, 1}, // rounds up
{499, 0}, // rounds down
{1500, 2}, // rounds up
{60000, 60},
}
for _, c := range cases {
if got := durationMsToSec(c.ms); got != c.want {
t.Errorf("durationMsToSec(%d) = %d, want %d", c.ms, got, c.want)
}
}
}
func TestCoverURLAndStreamURL(t *testing.T) {
var u pgtype.UUID
if err := u.Scan("00112233-4455-6677-8899-aabbccddeeff"); err != nil {
t.Fatalf("scan: %v", err)
}
if got := coverURL(u); got != "/api/albums/00112233-4455-6677-8899-aabbccddeeff/cover" {
t.Errorf("coverURL = %q", got)
}
if got := streamURL(u); got != "/api/tracks/00112233-4455-6677-8899-aabbccddeeff/stream" {
t.Errorf("streamURL = %q", got)
}
}
func TestArtistRefFrom(t *testing.T) {
var id pgtype.UUID
_ = id.Scan("00112233-4455-6677-8899-aabbccddeeff")
a := dbq.Artist{ID: id, Name: "Beatles", SortName: "Beatles"}
got := artistRefFrom(a, 7)
if got.ID != "00112233-4455-6677-8899-aabbccddeeff" {
t.Errorf("ID = %q", got.ID)
}
if got.Name != "Beatles" || got.AlbumCount != 7 {
t.Errorf("ref = %+v", got)
}
}
func TestAlbumRefFrom(t *testing.T) {
var aid, artID pgtype.UUID
_ = aid.Scan("00000000-0000-0000-0000-000000000001")
_ = artID.Scan("00000000-0000-0000-0000-000000000002")
var rel pgtype.Date
_ = rel.Scan("1969-09-26")
a := dbq.Album{ID: aid, Title: "Abbey Road", ArtistID: artID, ReleaseDate: rel}
got := albumRefFrom(a, "Beatles", 17, 2820)
if got.Title != "Abbey Road" || got.ArtistName != "Beatles" {
t.Errorf("ref = %+v", got)
}
if got.TrackCount != 17 || got.DurationSec != 2820 {
t.Errorf("counts = %+v", got)
}
if got.Year != 1969 {
t.Errorf("year = %d", got.Year)
}
if got.CoverURL != "/api/albums/00000000-0000-0000-0000-000000000001/cover" {
t.Errorf("cover_url = %q", got.CoverURL)
}
}
func TestTrackRefFrom(t *testing.T) {
var tid, aid, artID pgtype.UUID
_ = tid.Scan("00000000-0000-0000-0000-000000000010")
_ = aid.Scan("00000000-0000-0000-0000-000000000001")
_ = artID.Scan("00000000-0000-0000-0000-000000000002")
trackNum := int32(3)
discNum := int32(1)
t2 := dbq.Track{
ID: tid, Title: "Something", AlbumID: aid, ArtistID: artID,
TrackNumber: &trackNum, DiscNumber: &discNum,
DurationMs: 183_000,
}
got := trackRefFrom(t2, "Abbey Road", "Beatles")
if got.Title != "Something" || got.AlbumTitle != "Abbey Road" || got.ArtistName != "Beatles" {
t.Errorf("ref = %+v", got)
}
if got.TrackNumber != 3 || got.DiscNumber != 1 {
t.Errorf("positions = %+v", got)
}
if got.DurationSec != 183 {
t.Errorf("duration = %d, want 183", got.DurationSec)
}
if got.StreamURL != "/api/tracks/00000000-0000-0000-0000-000000000010/stream" {
t.Errorf("stream_url = %q", got.StreamURL)
}
}
func TestTrackRefFromNilPositions(t *testing.T) {
var tid, aid, artID pgtype.UUID
_ = tid.Scan("00000000-0000-0000-0000-000000000010")
_ = aid.Scan("00000000-0000-0000-0000-000000000001")
_ = artID.Scan("00000000-0000-0000-0000-000000000002")
t2 := dbq.Track{ID: tid, AlbumID: aid, ArtistID: artID, DurationMs: 1000}
got := trackRefFrom(t2, "A", "B")
if got.TrackNumber != 0 || got.DiscNumber != 0 {
t.Errorf("nil positions should be 0, got %+v", got)
}
}
func TestParsePaging(t *testing.T) {
tests := []struct {
name string
query url.Values
wantLimit int
wantOffset int
wantErr bool
}{
{
name: "default (empty values)",
query: url.Values{},
wantLimit: 50,
wantOffset: 0,
wantErr: false,
},
{
name: "limit=0 clamps to 1",
query: url.Values{"limit": {"0"}},
wantLimit: 1,
wantOffset: 0,
wantErr: false,
},
{
name: "limit=500 clamps to 200",
query: url.Values{"limit": {"500"}},
wantLimit: 200,
wantOffset: 0,
wantErr: false,
},
{
name: "offset=-5 clamps to 0",
query: url.Values{"offset": {"-5"}},
wantLimit: 50,
wantOffset: 0,
wantErr: false,
},
{
name: "limit=abc errors",
query: url.Values{"limit": {"abc"}},
wantLimit: 0,
wantOffset: 0,
wantErr: true,
},
{
name: "offset=xyz errors",
query: url.Values{"offset": {"xyz"}},
wantLimit: 0,
wantOffset: 0,
wantErr: true,
},
{
name: "normal values",
query: url.Values{"limit": {"25"}, "offset": {"100"}},
wantLimit: 25,
wantOffset: 100,
wantErr: false,
},
}
for _, tt := range tests {
t.Run(tt.name, func(t *testing.T) {
gotLimit, gotOffset, err := parsePaging(tt.query)
if (err != nil) != tt.wantErr {
t.Errorf("parsePaging() err = %v, wantErr %v", err, tt.wantErr)
return
}
if gotLimit != tt.wantLimit {
t.Errorf("parsePaging() limit = %d, want %d", gotLimit, tt.wantLimit)
}
if gotOffset != tt.wantOffset {
t.Errorf("parsePaging() offset = %d, want %d", gotOffset, tt.wantOffset)
}
})
}
}