refactor: remove dead code, consolidate save/error handling, add lint gate

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)
 	}