From d6f9761ab9bc094aa699c4d085eb45aaf69a023f Mon Sep 17 00:00:00 2001 From: Noam Preil Date: Thu, 23 Oct 2025 16:38:30 -0500 Subject: [PATCH] cookie test: use maps.Equal instead of reflect.DeepEquals This is a drive-by patch; the test was failing for the other optimization, and I noticed this when I went to take a look and figured I'd just send the patch. --- arrow/flight/cookie_middleware_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/arrow/flight/cookie_middleware_test.go b/arrow/flight/cookie_middleware_test.go index 86605e66a..7f0ce32af 100644 --- a/arrow/flight/cookie_middleware_test.go +++ b/arrow/flight/cookie_middleware_test.go @@ -21,9 +21,9 @@ import ( "errors" "fmt" "io" + "maps" "net/http" "net/textproto" - "reflect" "strings" "testing" "time" @@ -85,7 +85,7 @@ func (s *serverAddCookieMiddleware) StartCall(ctx context.Context) context.Conte } } - if !reflect.DeepEqual(s.expectedCookies, got) { + if !maps.Equal(s.expectedCookies, got) { panic(fmt.Sprintf("did not get expected cookies, expected %+v, got %+v", s.expectedCookies, got)) }