seojs 0.0.7 → 0.0.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: db1d2bc5a6f425688e975276a74290e9b7555532
-  data.tar.gz: e556f95c06025db18757aef6e912456b3180d65b
+  metadata.gz: 1ad9b187fe3b62bee1d5100414497a191a87208b
+  data.tar.gz: 85ee43ec276a28e5f45565ac327de2aff1003c27
 SHA512:
-  metadata.gz: 8a2c254ec97c5404b66657212a533809a766c3d0b51590becb2cc69b624f5157b2f2099b173371893606f4036ba165bb27be25413161e648efb167c4f8590c1e
-  data.tar.gz: b442cb4eef3ae8ed973b8292e75801a38a2f0a40b2a1569757bb20560a015ff6590160d21a284fd97483ba9a2c0a7a46bd8ea0daf19b543dd56a3a51c7e1e830
+  metadata.gz: cd35276ad55de901d5c39bbdc8e5f282bc3f835edde09b9e995da82f3d6d6679cccb67014065a490123ce154f2b9eca2ac4d8558929f74d1a2df2c063dcf47f0
+  data.tar.gz: 7cd786888c498b7f1abd96e6288177aa95cab8b2872d55292cf96841748490363166b691edd569cc65a68568506aad5a5a88539d2ac85e4da2cf160b1aa174d5

data/VERSION

@@ -1 +1 @@
-0.0.7
+0.0.8

data/lib/seojs/middleware.rb

@@ -37,7 +37,12 @@ module Seojs
           if [200, 301, 302, 304].include? snapshot_response.code.to_i
             body = snapshot_response.body || ""
             body = [body] unless body.respond_to?(:each)
-            [snapshot_response.code, snapshot_response.to_hash, body]
+            headers = snapshot_response.to_hash
+            headers.each_key do |k|
+              v = headers[k]
+              headers[k] = v.join(', ') if v.kind_of? Array
+            end
+            [snapshot_response.code, headers, body]
           else
             nil
           end

data/seojs.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = "seojs"
-  s.version = "0.0.7"
+  s.version = "0.0.8"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["getseojs.com"]

data/test/test_middleware.rb

@@ -40,12 +40,21 @@ class TestMiddleware < Test::Unit::TestCase
     assert last_response.ok?
   end
   
+  def build_response(body, code, headers)
+     r = Net::HTTPResponse.new(1.0, code, "body")
+     headers.each_pair do |k,v|
+       r.add_field(k,v)
+     end
+     r.stubs(:body => body)
+     r
+  end
+  
   def test_forward_with_token
     Seojs.token = 'token'
     Net::HTTP::Get.expects(:new)
                   .with(equals("http://cdn.getseojs.com/snapshots/token/v3/http://example.org:80/hi?_escaped_fragment_="))
     Net::HTTP.expects(:start)
-             .returns(stub(:body => 'Snapshot!', :code => 200, :to_hash => {}))
+             .returns(build_response('Snapshot!', 200, {}))
     get '/hi', :_escaped_fragment_ => ''
     assert last_response.ok?
     assert_equal 'Snapshot!', last_response.body
@@ -65,9 +74,9 @@ class TestMiddleware < Test::Unit::TestCase
                   .with(equals("http://cdn.getseojs.com/snapshots/token/v3/http://example.org:80/hi?_escaped_fragment_="))
                   .returns({})
     Net::HTTP.expects(:start)
-             .returns(stub(:body => 'Snapshot!', :code => 200, :to_hash => {'ETAG'=>'abc'}))
+             .returns(build_response('Snapshot!', 200, {'ETAG'=>'abc'}))
     get '/hi', {:_escaped_fragment_ => ''}, {'HTTP_IF_NONE_MATCH' => '123'}
-    assert_include last_response.headers, 'ETAG'
+    assert_equal 'abc', last_response.headers['ETAG']
     assert last_response.ok?
     assert_equal 'Snapshot!', last_response.body
   end
@@ -77,7 +86,7 @@ class TestMiddleware < Test::Unit::TestCase
     Net::HTTP::Get.expects(:new)
                   .with(equals("http://cdn.getseojs.com/snapshots/token/v3/http://example.org:80/hi?_escaped_fragment_="))
     Net::HTTP.expects(:start)
-             .returns(stub(:body => 'not found...', :code => 404, :to_hash => {}))
+             .returns(build_response('not found...', 404, {}))
     get '/hi', :_escaped_fragment_ => ''
     assert last_response.ok?
     assert_equal 'Hello!', last_response.body
@@ -88,7 +97,7 @@ class TestMiddleware < Test::Unit::TestCase
     Net::HTTP::Get.expects(:new)
                   .with(equals("http://cdn.getseojs.com/snapshots/token/v3/http://example.org:80/hi?_escaped_fragment_="))
     Net::HTTP.expects(:start)
-             .returns(stub(:body => 'somewhere else', :code => 302, :to_hash => {'Location' => 'http://google.com/'}))
+             .returns(build_response('somewhere else', 302, {'Location' => 'http://google.com/'}))
     get '/hi', :_escaped_fragment_ => ''
     assert last_response.redirect?
     follow_redirect!

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: seojs
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
 platform: ruby
 authors:
 - getseojs.com