diff options
Diffstat (limited to 'internal/admin/api_rooms_bulk_delete_test.go')
| -rw-r--r-- | internal/admin/api_rooms_bulk_delete_test.go | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/internal/admin/api_rooms_bulk_delete_test.go b/internal/admin/api_rooms_bulk_delete_test.go index 083b890..66464a5 100644 --- a/internal/admin/api_rooms_bulk_delete_test.go +++ b/internal/admin/api_rooms_bulk_delete_test.go @@ -14,20 +14,20 @@ import ( "testing" ) -func postBulkDelete(t *testing.T, s *AdminServer, body string) (map[string]any, int) { +func postBulkDelete(t *testing.T, s *AdminServer, body string) map[string]any { t.Helper() req := httptest.NewRequest(http.MethodPost, "/api/rooms/bulk-delete", strings.NewReader(body)) rec := httptest.NewRecorder() s.handleBulkDelete(rec, req) var resp map[string]any _ = json.Unmarshal(rec.Body.Bytes(), &resp) - return resp, rec.Code + return resp } func TestBulkDeleteRejectsDuplicateIDs(t *testing.T) { s := newTestAdminServer(t, map[int]string{1: "id: 1\nname: Room\n"}) path, _ := s.world.GetRoomPath(1) - resp, _ := postBulkDelete(t, s, `{"ids":[1,1]}`) + resp := postBulkDelete(t, s, `{"ids":[1,1]}`) if errMsg, _ := resp["error"].(string); !strings.Contains(errMsg, "duplicate room id 1") { t.Fatalf("duplicate IDs not rejected: %v", resp) } @@ -60,7 +60,7 @@ func TestBulkDeleteFailureRollsBackDeletions(t *testing.T) { t.Cleanup(func() { _ = os.Chmod(sub, 0o755); os.RemoveAll(sub) }) path1, _ := s.world.GetRoomPath(1) - resp, _ := postBulkDelete(t, s, `{"ids":[1,3]}`) + resp := postBulkDelete(t, s, `{"ids":[1,3]}`) if errMsg, _ := resp["error"].(string); !strings.Contains(errMsg, "delete room 3") { t.Fatalf("expected room-3 delete failure: %v", resp) } |