http-cookie 1.0.0.pre7 → 1.0.0.pre8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 36d302ade786727a1d6a34617d6f5fd5a0849ad5
-  data.tar.gz: 9f0f607f3f78427ec0f1737ce770e58aeb831110
+  metadata.gz: 2ab53e3207db1e30dadbe6f623adece56c2dd9a7
+  data.tar.gz: cc1b6d68f82a1847275e3d16ab122e86d399186d
 SHA512:
-  metadata.gz: 2288ba0ccc9dda414610bb4b87765492c480c1aa886c2b7b716c3d57a4373baa2de020cffb4cb93cfbb00f5dd7283a814c98573a3bbcc752f6f213a0dd8f290c
-  data.tar.gz: b08eb434043d9601cf743ba30a4c279356da12b4da8c243d37f9f8974ebbfe8d8b55fbeb28d2020e9f346aebcd545f5abedd8b8bb9ecb0eb76dc890576491688
+  metadata.gz: 308162bbb92376e5140b863141c55501c454fdf676617f018faa4d1098819a4fe9c9ef4b0b2f63423cee7bb862b8a79edde7d914ce940b095544251f49b9018c
+  data.tar.gz: ba8f6855bc6255f2d5729407a299fabba71ae0c1488eefc084cd823a424fbc93c6a34c80ccfb36e1b1830a18fcc1d31b4108f4e89d136dd84a443ca967162498

data/README.md

@@ -74,8 +74,8 @@ equivalent using `HTTP::Cookie`:
         cookies2 = Mechanize::Cookie.parse(uri, set_cookie2, log)
 
         # after
-        cookies1 = HTTP::Cookie.parse(set_cookie1, :origin => uri)
-        cookies2 = HTTP::Cookie.parse(set_cookie2, :origin => uri, :logger => log)
+        cookies1 = HTTP::Cookie.parse(set_cookie1, uri_or_url)
+        cookies2 = HTTP::Cookie.parse(set_cookie2, uri_or_url, :logger => log)
 
 - `Mechanize::Cookie#set_domain`
 

data/lib/http/cookie.rb

@@ -232,18 +232,15 @@ class HTTP::Cookie
     end
 
     # Parses a Set-Cookie header value `set_cookie` into an array of
-    # Cookie objects.  Parts (separated by commas) that are malformed
-    # or invalid are silently ignored.  For example, a cookie that a
-    # given origin is not allowed to issue is not included in the
-    # resulted array.
+    # Cookie objects taking `origin` as the source URI/URL.  Parts
+    # (separated by commas) that are malformed or invalid are silently
+    # ignored.  For example, cookies that a given origin is not
+    # allowed to issue are excluded from the resulted array.
     #
     # If a block is given, each cookie object is passed to the block.
     #
     # Available option keywords are below:
     #
-    # :origin
-    # : The cookie's origin URI/URL
-    #
     # :created_at
     # : The creation time of the cookies parsed.
     #
@@ -252,12 +249,14 @@ class HTTP::Cookie
     #
     # ### Compatibility Note for Mechanize::Cookie users
     #
-    # * Order of parameters is a slightly different in
+    # * Order of parameters changed in
     #   `HTTP::Cookie.parse`.  Compare these:
     #
     #         Mechanize::Cookie.parse(uri, set_cookie[, log])
     #
-    #         HTTP::Cookie.parse(set_cookie, :origin => uri[, :logger => # log])
+    #         HTTP::Cookie.parse(set_cookie, uri[, :logger => # log])
+    #
+    # * `HTTP::Cookie.parse` does not accept nil for `set_cookie`.
     #
     # * `HTTP::Cookie.parse` does not yield nil nor include nil in an
     #   returned array.  It simply ignores unparsable parts.
@@ -267,12 +266,12 @@ class HTTP::Cookie
     #   implementations.  In particular, it is capable of parsing
     #   cookie definitions containing double-quotes just as
     #   naturally expected.
-    def parse(set_cookie, options = nil, &block)
+    def parse(set_cookie, origin, options = nil, &block)
       if options
         logger = options[:logger]
-        origin = options[:origin] and origin = URI(origin)
         created_at = options[:created_at]
       end
+      origin = URI(origin)
 
       [].tap { |cookies|
         s = Scanner.new(set_cookie, logger)
@@ -416,6 +415,7 @@ class HTTP::Cookie
 
   # See #origin.
   def origin=(origin)
+    return origin if origin == @origin
     @origin.nil? or
       raise ArgumentError, "origin cannot be changed once it is set"
     origin = URI(origin)
@@ -545,7 +545,7 @@ class HTTP::Cookie
   # Returns a string for use in a Cookie header value,
   # i.e. "name=value".
   def cookie_value
-    "#{@name}=#{@value}"
+    "#{@name}=#{Scanner.quote(@value)}"
   end
   alias to_s cookie_value
 
@@ -559,7 +559,7 @@ class HTTP::Cookie
     origin = origin ? URI(origin) : @origin or
       raise "origin must be specified to produce a value for Set-Cookie"
 
-    string = "#{@name}=#{Scanner.quote(@value)}"
+    string = cookie_value
     if @for_domain
       string << "; Domain=#{@domain}"
     end

data/lib/http/cookie/scanner.rb

@@ -23,7 +23,7 @@ class HTTP::Cookie::Scanner < StringScanner
   class << self
     def quote(s)
       return s unless s.match(RE_BAD_CHAR)
-      '"' << s.gsub(RE_BAD_CHAR, "\\\\\\1") << '"'
+      '"' << s.gsub(/([\\"])/, "\\\\\\1") << '"'
     end
   end
 

data/lib/http/cookie/version.rb

@@ -1,5 +1,5 @@
 module HTTP
   class Cookie
-    VERSION = "1.0.0.pre7"
+    VERSION = "1.0.0.pre8"
   end
 end

data/lib/http/cookie_jar.rb

@@ -117,6 +117,29 @@ class HTTP::CookieJar
   end
   include Enumerable
 
+  # Parses a Set-Cookie field value `set_cookie` sent from a URI
+  # `origin` and adds the cookies parsed as valid to the jar.  Returns
+  # an array of cookies that have been added.  If a block is given, it
+  # is called after each cookie is added.
+  #
+  # `jar.parse(set_cookie, origin)` is a shorthand for this:
+  #
+  #         HTTP::Cookie.parse(set_cookie, origin) { |cookie|
+  #           jar.add(cookie)
+  #         }
+  #
+  # See HTTP::Cookie.parse for available options.
+  def parse(set_cookie, origin, options = nil) # :yield: cookie
+    if block_given?
+      HTTP::Cookie.parse(set_cookie, origin, options) { |cookie|
+        add(cookie)
+        yield cookie
+      }
+    else
+      HTTP::Cookie.parse(set_cookie, origin, options, &method(:add))
+    end
+  end
+
   # call-seq:
   #   jar.save(filename_or_io, **options)
   #   jar.save(filename_or_io, format = :yaml, **options)

data/test/test_http_cookie.rb

@@ -48,7 +48,7 @@ class TestHTTPCookie < Test::Unit::TestCase
     dates.each do |date|
       cookie = "PREF=1; expires=#{date}"
       silently do
-        assert_equal 1, HTTP::Cookie.parse(cookie, :origin => url) { |c|
+        assert_equal 1, HTTP::Cookie.parse(cookie, url) { |c|
           assert c.expires, "Tried parsing: #{date}"
           assert_equal(true, c.expires < yesterday)
         }.size
@@ -61,7 +61,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     uri = URI.parse 'http://example'
 
-    assert_equal 1, HTTP::Cookie.parse(cookie_str, :origin => uri) { |cookie|
+    assert_equal 1, HTTP::Cookie.parse(cookie_str, uri) { |cookie|
       assert_equal 'a', cookie.name
       assert_equal 'b', cookie.value
     }.size
@@ -72,7 +72,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     uri = URI.parse 'http://example'
 
-    assert_equal 1, HTTP::Cookie.parse(cookie_str, :origin => uri) { |cookie|
+    assert_equal 1, HTTP::Cookie.parse(cookie_str, uri) { |cookie|
       assert_equal 'foo',               cookie.name
       assert_equal 'bar',               cookie.value
       assert_equal '/',                 cookie.path
@@ -86,11 +86,11 @@ class TestHTTPCookie < Test::Unit::TestCase
     cookie_str = "foo=#{'Cookie' * 680}; path=/ab/"
     assert_equal(HTTP::Cookie::MAX_LENGTH - 1, cookie_str.bytesize)
 
-    assert_equal 1, HTTP::Cookie.parse(cookie_str, :origin => uri).size
+    assert_equal 1, HTTP::Cookie.parse(cookie_str, uri).size
 
-    assert_equal 1, HTTP::Cookie.parse(cookie_str.sub(';', 'x;'), :origin => uri).size
+    assert_equal 1, HTTP::Cookie.parse(cookie_str.sub(';', 'x;'), uri).size
 
-    assert_equal 0, HTTP::Cookie.parse(cookie_str.sub(';', 'xx;'), :origin => uri).size
+    assert_equal 0, HTTP::Cookie.parse(cookie_str.sub(';', 'xx;'), uri).size
   end
 
   def test_parse_quoted
@@ -99,7 +99,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     uri = URI.parse 'http://example'
 
-    assert_equal 1, HTTP::Cookie.parse(cookie_str, :origin => uri) { |cookie|
+    assert_equal 1, HTTP::Cookie.parse(cookie_str, uri) { |cookie|
       assert_equal 'quoted', cookie.name
       assert_equal 'value', cookie.value
       assert_equal 'comment is "comment"', cookie.comment
@@ -109,7 +109,7 @@ class TestHTTPCookie < Test::Unit::TestCase
   def test_parse_weird_cookie
     cookie = 'n/a, ASPSESSIONIDCSRRQDQR=FBLDGHPBNDJCPCGNCPAENELB; path=/'
     url = URI.parse('http://www.searchinnovation.com/')
-    assert_equal 1, HTTP::Cookie.parse(cookie, :origin => url) { |c|
+    assert_equal 1, HTTP::Cookie.parse(cookie, url) { |c|
       assert_equal('ASPSESSIONIDCSRRQDQR', c.name)
       assert_equal('FBLDGHPBNDJCPCGNCPAENELB', c.value)
     }.size
@@ -118,7 +118,7 @@ class TestHTTPCookie < Test::Unit::TestCase
   def test_double_semicolon
     double_semi = 'WSIDC=WEST;; domain=.williams-sonoma.com; path=/'
     url = URI.parse('http://williams-sonoma.com/')
-    assert_equal 1, HTTP::Cookie.parse(double_semi, :origin => url) { |cookie|
+    assert_equal 1, HTTP::Cookie.parse(double_semi, url) { |cookie|
       assert_equal('WSIDC', cookie.name)
       assert_equal('WEST', cookie.value)
     }.size
@@ -127,13 +127,13 @@ class TestHTTPCookie < Test::Unit::TestCase
   def test_parse_bad_version
     bad_cookie = 'PRETANET=TGIAqbFXtt; Name=/PRETANET; Path=/; Version=1.2; Content-type=text/html; Domain=192.168.6.196; expires=Friday, 13-November-2026  23:01:46 GMT;'
     url = URI.parse('http://localhost/')
-    assert_equal 0, HTTP::Cookie.parse(bad_cookie, :origin => url).size
+    assert_equal 0, HTTP::Cookie.parse(bad_cookie, url).size
   end
 
   def test_parse_bad_max_age
     bad_cookie = 'PRETANET=TGIAqbFXtt; Name=/PRETANET; Path=/; Max-Age=1.2; Content-type=text/html; Domain=192.168.6.196; expires=Friday, 13-November-2026  23:01:46 GMT;'
     url = URI.parse('http://localhost/')
-    assert_equal 0, HTTP::Cookie.parse(bad_cookie, :origin => url).size
+    assert_equal 0, HTTP::Cookie.parse(bad_cookie, url).size
   end
 
   def test_parse_date_fail
@@ -146,7 +146,7 @@ class TestHTTPCookie < Test::Unit::TestCase
     silently do
       dates.each do |date|
         cookie = "PREF=1; expires=#{date}"
-        assert_equal 1, HTTP::Cookie.parse(cookie, :origin => url) { |c|
+        assert_equal 1, HTTP::Cookie.parse(cookie, url) { |c|
           assert_equal(true, c.expires.nil?)
         }.size
       end
@@ -158,7 +158,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     cookie_str = 'a=b; domain=.example.com'
 
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => url).first
+    cookie = HTTP::Cookie.parse(cookie_str, url).first
 
     assert_equal 'example.com', cookie.domain
     assert cookie.for_domain?
@@ -170,7 +170,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     cookie_str = 'a=b; domain=example.com'
 
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => url).first
+    cookie = HTTP::Cookie.parse(cookie_str, url).first
 
     assert_equal 'example.com', cookie.domain
     assert cookie.for_domain?
@@ -178,9 +178,7 @@ class TestHTTPCookie < Test::Unit::TestCase
   end
 
   def test_parse_public_suffix
-    cookie_str = 'a=b; domain=com'
-
-    cookie = HTTP::Cookie.parse(cookie_str).first
+    cookie = HTTP::Cookie.new('a', 'b', :domain => 'com')
     assert_equal('com', cookie.domain)
     assert_equal(false, cookie.for_domain?)
 
@@ -198,7 +196,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     cookie_str = 'a=b;'
 
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => url).first
+    cookie = HTTP::Cookie.parse(cookie_str, url).first
 
     assert_equal 'example.com', cookie.domain
     assert !cookie.for_domain?
@@ -211,23 +209,23 @@ class TestHTTPCookie < Test::Unit::TestCase
     epoch, date = 4485353164, 'Fri, 19 Feb 2112 19:26:04 GMT'
     base = Time.at(1363014000)
 
-    cookie = HTTP::Cookie.parse("name=Akinori; expires=#{date}", :origin => url).first
+    cookie = HTTP::Cookie.parse("name=Akinori; expires=#{date}", url).first
     assert_equal Time.at(epoch), cookie.expires
 
-    cookie = HTTP::Cookie.parse('name=Akinori; max-age=3600', :origin => url).first
+    cookie = HTTP::Cookie.parse('name=Akinori; max-age=3600', url).first
     assert_in_delta Time.now + 3600, cookie.expires, 1
-    cookie = HTTP::Cookie.parse('name=Akinori; max-age=3600', :origin => url, :created_at => base).first
+    cookie = HTTP::Cookie.parse('name=Akinori; max-age=3600', url, :created_at => base).first
     assert_equal base + 3600, cookie.expires
 
     # Max-Age has precedence over Expires
-    cookie = HTTP::Cookie.parse("name=Akinori; max-age=3600; expires=#{date}", :origin => url).first
+    cookie = HTTP::Cookie.parse("name=Akinori; max-age=3600; expires=#{date}", url).first
     assert_in_delta Time.now + 3600, cookie.expires, 1
-    cookie = HTTP::Cookie.parse("name=Akinori; max-age=3600; expires=#{date}", :origin => url, :created_at => base).first
+    cookie = HTTP::Cookie.parse("name=Akinori; max-age=3600; expires=#{date}", url, :created_at => base).first
     assert_equal base + 3600, cookie.expires
 
-    cookie = HTTP::Cookie.parse("name=Akinori; expires=#{date}; max-age=3600", :origin => url).first
+    cookie = HTTP::Cookie.parse("name=Akinori; expires=#{date}; max-age=3600", url).first
     assert_in_delta Time.now + 3600, cookie.expires, 1
-    cookie = HTTP::Cookie.parse("name=Akinori; expires=#{date}; max-age=3600", :origin => url, :created_at => base).first
+    cookie = HTTP::Cookie.parse("name=Akinori; expires=#{date}; max-age=3600", url, :created_at => base).first
     assert_equal base + 3600, cookie.expires
   end
 
@@ -241,7 +239,7 @@ class TestHTTPCookie < Test::Unit::TestCase
       'name=Akinori; expires=',
       'name=Akinori; max-age=',
     ].each { |str|
-      cookie = HTTP::Cookie.parse(str, :origin => url).first
+      cookie = HTTP::Cookie.parse(str, url).first
       assert cookie.session?, str
     }
 
@@ -249,7 +247,7 @@ class TestHTTPCookie < Test::Unit::TestCase
       'name=Akinori; expires=Mon, 19 Feb 2012 19:26:04 GMT',
       'name=Akinori; max-age=3600',
     ].each { |str|
-      cookie = HTTP::Cookie.parse(str, :origin => url).first
+      cookie = HTTP::Cookie.parse(str, url).first
       assert !cookie.session?, str
     }
   end
@@ -273,7 +271,7 @@ class TestHTTPCookie < Test::Unit::TestCase
       "no_domain2=no_domain; Expires=Sun, 06 Nov 2011 00:29:53 GMT; no_expires=nope; Domain, " \
       "no_domain3=no_domain; Expires=Sun, 06 Nov 2011 00:29:53 GMT; no_expires=nope; Domain="
 
-    cookies = HTTP::Cookie.parse cookie_str, :origin => url
+    cookies = HTTP::Cookie.parse cookie_str, url
     assert_equal 15, cookies.length
 
     name = cookies.find { |c| c.name == 'name' }
@@ -323,7 +321,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     cookie_params.keys.combine.each do |keys|
       cookie_text = [cookie_value, *keys.map { |key| cookie_params[key] }].join('; ')
-      cookie, = HTTP::Cookie.parse(cookie_text, :origin => url)
+      cookie, = HTTP::Cookie.parse(cookie_text, url)
 
       assert_equal('12345%7D=ASDFWEE345%3DASda', cookie.to_s)
       assert_equal('/', cookie.path)
@@ -340,7 +338,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     cookie_params.keys.combine.each do |keys|
       cookie_text = [cookie_value, *keys.map { |key| cookie_params[key] }].join('; ')
-      cookie, = HTTP::Cookie.parse(cookie_text, :origin => url)
+      cookie, = HTTP::Cookie.parse(cookie_text, url)
 
       assert_equal('12345%7D=', cookie.to_s)
       assert_equal('', cookie.value)
@@ -360,7 +358,7 @@ class TestHTTPCookie < Test::Unit::TestCase
     cookie_params.keys.combine.each do |keys|
       next if keys.include?('path')
       cookie_text = [cookie_value, *keys.map { |key| cookie_params[key] }].join('; ')
-      cookie, = HTTP::Cookie.parse(cookie_text, :origin => url)
+      cookie, = HTTP::Cookie.parse(cookie_text, url)
 
       assert_equal('12345%7D=ASDFWEE345%3DASda', cookie.to_s)
       assert_equal('/', cookie.path)
@@ -379,7 +377,7 @@ class TestHTTPCookie < Test::Unit::TestCase
     cookie_params.keys.combine.each do |keys|
       next unless keys.include?('secure')
       cookie_text = [cookie_value, *keys.map { |key| cookie_params[key] }].join('; ')
-      cookie, = HTTP::Cookie.parse(cookie_text, :origin => url)
+      cookie, = HTTP::Cookie.parse(cookie_text, url)
 
       assert_equal('12345%7D=ASDFWEE345%3DASda', cookie.to_s)
       assert_equal('/', cookie.path)
@@ -390,6 +388,16 @@ class TestHTTPCookie < Test::Unit::TestCase
     end
   end
 
+  def test_cookie_value
+    [
+      ['foo="bar  baz"', 'bar  baz'],
+      ['foo="bar\"; \"baz"', 'bar"; "baz'],
+    ].each { |cookie_value, value|
+      cookie = HTTP::Cookie.new('foo', value)
+      assert_equal(cookie_value, cookie.cookie_value)
+    }
+  end
+
   def test_set_cookie_value
     url = URI.parse('http://rubyforge.org/')
 
@@ -398,8 +406,8 @@ class TestHTTPCookie < Test::Unit::TestCase
       date = Time.at(Time.now.to_i)
       cookie_params.keys.combine.each do |keys|
         cookie_text = [cookie_value, *keys.map { |key| cookie_params[key] }].join('; ')
-        cookie, = HTTP::Cookie.parse(cookie_text, :origin => url, :created_at => date)
-        cookie2, = HTTP::Cookie.parse(cookie.set_cookie_value, :origin => url, :created_at => date)
+        cookie, = HTTP::Cookie.parse(cookie_text, url, :created_at => date)
+        cookie2, = HTTP::Cookie.parse(cookie.set_cookie_value, url, :created_at => date)
 
         assert_equal(cookie.name, cookie2.name)
         assert_equal(cookie.value, cookie2.value)
@@ -427,7 +435,7 @@ class TestHTTPCookie < Test::Unit::TestCase
 
     cookie_params.keys.combine.each do |keys|
       cookie_text = [cookie_value, *keys.map { |key| cookie_params[key] }].join(';')
-      cookie, = HTTP::Cookie.parse(cookie_text, :origin => url)
+      cookie, = HTTP::Cookie.parse(cookie_text, url)
 
       assert_equal('12345%7D=ASDFWEE345%3DASda', cookie.to_s)
       assert_equal('/', cookie.path)
@@ -615,11 +623,11 @@ class TestHTTPCookie < Test::Unit::TestCase
     assert_equal '/foo/bar', uri.path
 
     cookie_str = 'a=b'
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => uri).first
+    cookie = HTTP::Cookie.parse(cookie_str, uri).first
     assert '/foo/', cookie.path
 
     cookie_str = 'a=b; path=/foo'
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => uri).first
+    cookie = HTTP::Cookie.parse(cookie_str, uri).first
     assert '/foo', cookie.path
 
     uri = URI.parse('http://example.com')
@@ -627,16 +635,16 @@ class TestHTTPCookie < Test::Unit::TestCase
     assert_equal '', uri.path
 
     cookie_str = 'a=b'
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => uri).first
+    cookie = HTTP::Cookie.parse(cookie_str, uri).first
     assert '/', cookie.path
 
     cookie_str = 'a=b; path=/foo'
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => uri).first
+    cookie = HTTP::Cookie.parse(cookie_str, uri).first
     assert '/foo', cookie.path
   end
 
   def test_domain_nil
-    cookie = HTTP::Cookie.parse('a=b').first
+    cookie = HTTP::Cookie.new('a', 'b')
     assert_raises(RuntimeError) {
       cookie.valid_for_uri?('http://example.com/')
     }
@@ -646,7 +654,7 @@ class TestHTTPCookie < Test::Unit::TestCase
     url = URI.parse('http://host.dom.example.com:8080/')
 
     cookie_str = 'a=b; domain=Example.Com'
-    cookie = HTTP::Cookie.parse(cookie_str, :origin => url).first
+    cookie = HTTP::Cookie.parse(cookie_str, url).first
     assert 'example.com', cookie.domain
 
     cookie.domain = DomainName(url.host)
@@ -666,8 +674,7 @@ class TestHTTPCookie < Test::Unit::TestCase
   def test_origin=
     url = URI.parse('http://example.com/path/')
 
-    cookie_str = 'a=b'
-    cookie = HTTP::Cookie.parse(cookie_str).first
+    cookie = HTTP::Cookie.new('a', 'b')
     cookie.origin = url
     assert_equal '/path/', cookie.path
     assert_equal 'example.com', cookie.domain
@@ -676,8 +683,7 @@ class TestHTTPCookie < Test::Unit::TestCase
       cookie.origin = URI.parse('http://www.example.com/')
     }
 
-    cookie_str = 'a=b; domain=.example.com; path=/'
-    cookie = HTTP::Cookie.parse(cookie_str).first
+    cookie = HTTP::Cookie.new('a', 'b', :domain => '.example.com', :path => '/')
     cookie.origin = url
     assert_equal '/', cookie.path
     assert_equal 'example.com', cookie.domain
@@ -686,8 +692,7 @@ class TestHTTPCookie < Test::Unit::TestCase
       cookie.origin = URI.parse('http://www.example.com/')
     }
 
-    cookie_str = 'a=b; domain=example.com'
-    cookie = HTTP::Cookie.parse(cookie_str).first
+    cookie = HTTP::Cookie.new('a', 'b', :domain => '.example.com')
     assert_raises(ArgumentError) {
       cookie.origin = URI.parse('http://example.org/')
     }
@@ -706,7 +711,7 @@ class TestHTTPCookie < Test::Unit::TestCase
   def test_valid_for_uri?
     {
       HTTP::Cookie.parse('a1=b',
-        :origin => 'http://example.com/dir/file.html').first => {
+        'http://example.com/dir/file.html').first => {
         true => [
           'http://example.com/dir/',
           'http://example.com/dir/test.html',
@@ -726,7 +731,7 @@ class TestHTTPCookie < Test::Unit::TestCase
         ]
       },
       HTTP::Cookie.parse('a2=b; path=/dir2/',
-        :origin => 'http://example.com/dir/file.html').first => {
+        'http://example.com/dir/file.html').first => {
         true => [
           'http://example.com/dir2/',
           'http://example.com/dir2/test.html',
@@ -746,7 +751,7 @@ class TestHTTPCookie < Test::Unit::TestCase
         ]
       },
       HTTP::Cookie.parse('a4=b; domain=example.com; path=/dir2/',
-        :origin => URI('http://example.com/dir/file.html')).first => {
+        URI('http://example.com/dir/file.html')).first => {
         true => [
           'https://example.com/dir2/test.html',
           'http://example.com/dir2/test.html',
@@ -762,7 +767,7 @@ class TestHTTPCookie < Test::Unit::TestCase
         ]
       },
       HTTP::Cookie.parse('a4=b; secure',
-        :origin => URI('https://example.com/dir/file.html')).first => {
+        URI('https://example.com/dir/file.html')).first => {
         true => [
           'https://example.com/dir/test.html',
         ],
@@ -774,7 +779,7 @@ class TestHTTPCookie < Test::Unit::TestCase
         ]
       },
       HTTP::Cookie.parse('a5=b',
-        :origin => URI('https://example.com/')).first => {
+        URI('https://example.com/')).first => {
         true => [
           'https://example.com',
         ],
@@ -783,7 +788,7 @@ class TestHTTPCookie < Test::Unit::TestCase
         ]
       },
       HTTP::Cookie.parse('a6=b; path=/dir',
-        :origin => 'http://example.com/dir/file.html').first => {
+        'http://example.com/dir/file.html').first => {
         true => [
           'http://example.com/dir',
           'http://example.com/dir/',