cache_rules 0.1.7 → 0.1.8

data/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Changelog
 
+## 0.1.8 (2014-02-09)
+
+  * Add tests to ensure URLs with query parameters are maintained
+
 ## 0.1.7 (2014-02-02)
 
   * Refactor and simplify `revalidate_response()` method

data/cache_rules.gemspec

@@ -6,7 +6,7 @@ require 'date'
 
 Gem::Specification.new do |s|
   s.name        = 'cache_rules'
-  s.version     = '0.1.7'
+  s.version     = '0.1.8'
 
   s.date        = Date.today.to_s
 

data/test/test_cache_rules.rb

@@ -20,15 +20,26 @@ class TestCacheRules < MiniTest::Test
     assert_equal got_money, "Dolla Dolla Bill Y'all"
   end
 
+  def test_validate_column0_query_params_fail
+    request = {"Host"=>"test.url"}
+
+    result = CacheRules.validate('http://test.url/test1?test=string', request)
+
+    assert_equal result[:code], 307
+    assert_nil   result[:body]
+    assert_equal result[:headers]['Cache-Lookup'], 'MISS'
+    refute_equal result[:headers]['Location'], "http://test.url/test1"
+  end
+
   def test_validate_column0
     request = {"Host"=>"test.url"}
 
-    result = CacheRules.validate('http://test.url/test1', request)
+    result = CacheRules.validate('http://test.url/test1?test=string', request)
 
     assert_equal result[:code], 307
     assert_nil   result[:body]
     assert_equal result[:headers]['Cache-Lookup'], 'MISS'
-    assert_equal result[:headers]['Location'], "http://test.url/test1"
+    assert_equal result[:headers]['Location'], "http://test.url/test1?test=string"
   end
 
   def test_validate_column1
@@ -128,8 +139,8 @@ class TestCacheRules < MiniTest::Test
     request = {"Host"=>"test.url", "If-None-Match"=>["*"], "Cache-Control"=>{"max-age"=>{"token"=>"100000000", "quoted_string"=>nil}}}
     cached  = {"Date"=>{"httpdate"=>"Sat, 03 Jan 2015 07:03:45 GMT", "timestamp"=>1420268625}, "Cache-Control"=>{"s-maxage"=>{"token"=>"100000000", "quoted_string"=>nil}}, "X-Cache-Req-Date"=>{"httpdate"=>"Sat, 03 Jan 2015 07:03:45 GMT", "timestamp"=>1420268625}, "X-Cache-Res-Date"=>{"httpdate"=>"Sat, 03 Jan 2015 07:03:45 GMT", "timestamp"=>1420268625}, "Last-Modified" => {"httpdate"=>"Sat, 03 Jan 2015 07:03:45 GMT", "timestamp"=>1420268625}} 
 
-    FakeWeb.register_uri(:head, "http://test.url/test1", :status => ["304", "Not Modified"], :date => "Sat, 03 Jan 2015 07:15:45 GMT", :Warning => "299 - \"Hello World\"")
-    result = CacheRules.revalidate_response('http://test.url/test1', request, cached)
+    FakeWeb.register_uri(:head, "http://test.url/test1?test=string", :status => ["304", "Not Modified"], :date => "Sat, 03 Jan 2015 07:15:45 GMT", :Warning => "299 - \"Hello World\"")
+    result = CacheRules.revalidate_response('http://test.url/test1?test=string', request, cached)
 
     assert_equal result[:code], 200
     assert_equal result[:body], 'cached'

data/test/test_helpers.rb

@@ -65,7 +65,7 @@ class TestHelpers < MiniTest::Test
   end
 
   def test_response_request
-    url = 'http://test.url'
+    url = 'http://test.url?test=string'
     act = CacheRules::REQUEST_TABLE[:actions]
 
     column0 = CacheRules.helper_response url, act, 0, @cached_headers
@@ -78,7 +78,7 @@ class TestHelpers < MiniTest::Test
     column7 = CacheRules.helper_response url, act, 7, @cached_headers
     column8 = CacheRules.helper_response url, act, 8, @cached_headers
 
-    assert_equal column0, {:body=>nil, :code=>307, :headers=>{"Cache-Lookup"=>"MISS", "Location"=>"http://test.url"}}
+    assert_equal column0, {:body=>nil, :code=>307, :headers=>{"Cache-Lookup"=>"MISS", "Location"=>"http://test.url?test=string"}}
     assert_equal column1, {:body=>"Gateway Timeout", :code=>504, :headers=>{"Cache-Lookup"=>"MISS"}}
 
     assert_equal column2[:body], 'cached'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: cache_rules
 version: !ruby/object:Gem::Version
-  version: 0.1.7
+  version: 0.1.8
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-02 00:00:00.000000000 Z
+date: 2015-02-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: fakeweb