middleman-s3_sync 4.0.1.rc.3 → 4.0.1.rc.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bea5e269ad1b736d2a88b7d8b8702642a74085ff
-  data.tar.gz: 27276f776111a9c38dfd8b8e6483c251267df0bc
+  metadata.gz: 3cc5e83da7bfd7639905564a61f26cc74916b1e8
+  data.tar.gz: c4f2d171cf39e6a5c6ea2776a8813b0dee89992d
 SHA512:
-  metadata.gz: 186a272a11c2f2c2a199a1ef011010045acb73af9ff7d5169988f66697cde61a8df8c0e90deaf2778d5a9e43acf94bfa6d4bd0215272cdd11c50cfad4cbb1ed7
-  data.tar.gz: bc83e3b244f390f7dc8ae86dd864c2155365a08b2ae180c8f1f789e5ec820fb32a5c0525ee426b4917d677323facd1ca9cd1331f45157773d0634daf7f625b86
+  metadata.gz: 8c140b7a8ac66998daa5ab29d263c8ac6c2669b359f305b6a6874afd766f855db5ffe67ba259f27a15cd3fcb95c7e7d2a4f9c7109a39080957de231f51be6bef
+  data.tar.gz: 7d6105e86327e6337e8ed340fa6122d5423fb309d5b6ed00b3f3ce2b6a3dd8d28a1db7c4da154d723275b6c73585cdb0ac2b25833402d7f37ff6efd2dc0a0f38

data/lib/middleman/s3_sync/caching_policy.rb

@@ -8,6 +8,7 @@ module Middleman
       end
 
       def caching_policy_for(content_type)
+        return default_caching_policy if content_type.nil? || content_type.empty?
         caching_policies.fetch(content_type.to_s.split(';').first.strip, caching_policies[:default])
       end
 

data/lib/middleman/s3_sync/version.rb

@@ -1,5 +1,5 @@
 module Middleman
   module S3Sync
-    VERSION = "4.0.1.rc.3"
+    VERSION = "4.0.1.rc.4"
   end
 end

data/spec/caching_policy_spec.rb

@@ -96,3 +96,25 @@ describe "Storing and retrieving policies" do
     expect(policy.policies.max_age).to eq(300)
   end
 end
+
+describe "Handling situations where the content type is nil" do
+  class CachingPolicy
+    include Middleman::S3Sync::CachingPolicy
+  end
+
+  let(:caching_policy) { CachingPolicy.new }
+
+  it "returns the default caching policy when the content type is nil" do
+    caching_policy.add_caching_policy(:default, max_age:(60 * 60 * 24 * 365))
+
+    expect(caching_policy.caching_policy_for(nil)).to_not be_nil
+    expect(caching_policy.caching_policy_for(nil).policies[:max_age]).to eq(60 * 60 * 24 * 365)
+  end
+
+  it "returns the default caching policy when the content type is blank" do
+    caching_policy.add_caching_policy(:default, max_age:(60 * 60 * 24 * 365))
+
+    expect(caching_policy.caching_policy_for("")).to_not be_nil
+    expect(caching_policy.caching_policy_for("").policies[:max_age]).to eq(60 * 60 * 24 * 365)
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: middleman-s3_sync
 version: !ruby/object:Gem::Version
-  version: 4.0.1.rc.3
+  version: 4.0.1.rc.4
 platform: ruby
 authors:
 - Frederic Jean
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-04 00:00:00.000000000 Z
+date: 2016-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: middleman-core