lerko/nib-v1
1
1
Fork 0
Code Issues Pull Requests 2 Actions Packages Projects Releases Wiki Activity

fix(api): check http errors before using response in tests

Browse Source
This commit is contained in:
Tyler Koenig
2026-05-17 14:10:40 -04:00
parent 6d8170d219
commit 4980714583
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
internal/api/api_test.go
+12 -3
View File
@@ -630,7 +630,10 @@ func TestUpdateEntity_Title(t *testing.T) {
req, _ := http.NewRequest("PUT", srv.URL+"/api/entities/"+created.ID, bytes.NewReader(
mustJSON(map[string]any{"title": "new title"})))
req.Header.Set("Content-Type", "application/json")
resp, _ := http.DefaultClient.Do(req)
resp, err := http.DefaultClient.Do(req)
if err != nil {
t.Fatalf("request failed: %v", err)
}
defer resp.Body.Close()
if resp.StatusCode != http.StatusOK {
@@ -651,7 +654,10 @@ func TestUpdateEntity_Description(t *testing.T) {
req, _ := http.NewRequest("PUT", srv.URL+"/api/entities/"+created.ID, bytes.NewReader(
mustJSON(map[string]any{"description": "new desc"})))
req.Header.Set("Content-Type", "application/json")
resp, _ := http.DefaultClient.Do(req)
resp, err := http.DefaultClient.Do(req)
if err != nil {
t.Fatalf("request failed: %v", err)
}
defer resp.Body.Close()
if resp.StatusCode != http.StatusOK {
@@ -674,7 +680,10 @@ func TestListEntities_TitleInResponse(t *testing.T) {
"title": title,
}).Body.Close()
resp, _ := http.Get(srv.URL + "/api/entities")
resp, err := http.Get(srv.URL + "/api/entities")
if err != nil {
t.Fatalf("request failed: %v", err)
}
defer resp.Body.Close()
var entities []EntityResponse