fix/cookie jar delete

History.md

@@ -1,3 +1,9 @@
+## 2.2.1 / 2025-10-1
+
+* Bug fixes:
+  * `Rack::Test::CookieJar#delete` and `Rack::Test::CookieJar#get_cookie`
+    now accept a cookie name as Symbol.
+
 ## 2.2.0 / 2024-12-23
 
 * Bug fixes:

lib/rack/test/cookie_jar.rb

@@ -165,15 +165,15 @@ def []=(name, value)
       # no such cookie exists.
       def get_cookie(name)
         @cookies.each do |cookie|
-          return cookie if cookie.name == name
+          return cookie if cookie.name == name.to_s
         end
         nil
       end
 
       # Delete all cookies with the given name from the cookie jar.
       def delete(name)
         @cookies.reject! do |cookie|
-          cookie.name == name
+          cookie.name == name.to_s
         end
         nil
       end

spec/rack/test/cookie_jar_spec.rb

@@ -39,6 +39,14 @@
     jar.get_cookie(cookie_name+'a').must_be_nil
   end
 
+  it '#get_cookie with a populated jar returns full cookie objects when passed a symbol' do
+    jar = Rack::Test::CookieJar.new
+    jar.get_cookie(cookie_name.to_sym).must_be_nil
+    jar[cookie_name] = cookie_value
+    jar.get_cookie(cookie_name.to_sym).must_be_kind_of Rack::Test::Cookie
+    jar.get_cookie(cookie_name+'a').must_be_nil
+  end
+
   it '#for returns the cookie header string delimited by semicolon and a space' do
     jar = Rack::Test::CookieJar.new
     jar['a'] = 'b'
@@ -80,4 +88,18 @@
     jar.merge(['', 'c=d'], URI.parse('/'))
     jar.to_hash.must_equal 'c' => 'd'
   end
+
+  it '#delete deletes the cookie from the jar when passed a string' do
+    jar = Rack::Test::CookieJar.new
+    jar[cookie_name] = cookie_value
+    jar.delete(cookie_name)
+    jar.to_hash.must_be_empty
+  end
+
+  it '#delete deletes the cookie from the jar when passed a symbol' do
+    jar = Rack::Test::CookieJar.new
+    jar[cookie_name] = cookie_value
+    jar.delete(cookie_name.to_sym)
+    jar.to_hash.must_be_empty
+  end
 end