maxcdn 0.1.3 → 0.1.4

data/.gitignore

@@ -17,3 +17,4 @@ pkg
 *.gem
 *.lock
 .DS_Store
+npm-debug.log

data/BENCHMARKS.md

@@ -10,11 +10,11 @@ Running benchmarks as follows in order:
  maxcdn.purge('ZONEID', [ 'FILE1','FILE2' ])
  
        user     system      total        real
-get   :  0.050000   0.070000   0.120000 (  1.004877)
-get   :  0.020000   0.000000   0.020000 (  0.534179)
-post  :  0.020000   0.000000   0.020000 ( 18.060298)
-put   :  0.030000   0.010000   0.040000 (  0.981422)
-delete:  0.020000   0.000000   0.020000 ( 11.770821)
-purge :  0.020000   0.000000   0.020000 ( 13.584882)
-purge :  0.020000   0.000000   0.020000 ( 17.310386)
-purge :  0.030000   0.010000   0.040000 ( 16.446759)
+get   :  0.050000   0.070000   0.120000 (  1.026412)
+get   :  0.010000   0.000000   0.010000 (  0.636914)
+post  :  0.020000   0.010000   0.030000 ( 11.083888)
+put   :  0.010000   0.000000   0.010000 (  1.240014)
+delete:  0.020000   0.000000   0.020000 (  6.846295)
+purge :  0.010000   0.010000   0.020000 ( 11.578944)
+purge :  0.020000   0.000000   0.020000 (  0.956149)
+purge :  0.030000   0.000000   0.030000 (  1.870104)

data/README.md

@@ -12,6 +12,8 @@ Do you like building cool stuff?  Do APIs keep you up at night? We're looking fo
 gem install maxcdn
 ```
 
+> Requires Ruby 1.9.2+ (see: [Travis](https://travis-ci.org/MaxCDN/ruby-maxcdn) for passing Ruby versions.)
+
 #### With Bundler
 
 ```
@@ -96,6 +98,10 @@ bundle exec ruby ./test/integration.rb # requires host's IP be whitelisted
 
 # Change Log
 
+##### 0.1.4
+
+* Fixing bug where purging files purges entire zone.
+
 ##### 0.1.3
 
 * Requested changes for debugging and development support. (See issue #2).

data/VERSION

@@ -1 +1 @@
-0.0.1
+0.1.4

data/lib/maxcdn.rb

@@ -99,20 +99,40 @@ module MaxCDN
       response_json
     end
 
-    [ :post, :get, :put, :delete ].each do |method|
+    [ :post, :put ].each do |method|
       define_method(method) do |uri, data={}, options={}|
-        options[:body] ||= true if method != :get
+        options[:body] ||= true
+        self._response_as_json method.to_s, uri, options, data
+      end
+    end
+
+    [ :get, :delete ].each do |method|
+      define_method(method) do |uri, data={}, options={}|
+        options[:body] = false
         self._response_as_json method.to_s, uri, options, data
       end
     end
 
     def purge zone_id, file_or_files=nil, options={}
-      unless file_or_files.nil?
-        return self.delete("/zones/pull.json/#{zone_id}/cache",
-                 { "files" => file_or_files }, options)
+      if file_or_files.nil?
+        return self.delete("/zones/pull.json/#{zone_id}/cache", {}, options)
+      end
+
+      if file_or_files.is_a?(String)
+        return self.delete("/zones/pull.json/#{zone_id}/cache", { "files" => file_or_files }, options)
+      end
+
+      if file_or_files.is_a?(Array)
+        return file_or_files.map do |file|
+          self.delete("/zones/pull.json/#{zone_id}/cache", { "files" => file }, options)
+        end
+      end
+
+      if file_or_files.is_a?(Hash) and (file_or_files.has_key(:files) or file_or_files.has_key("files"))
+          return self.delete("/zones/pull.json/#{zone_id}/cache", { "files" => file_or_files }, options)
       end
 
-      self.delete("/zones/pull.json/#{zone_id}/cache", {}, options)
+      raise MaxCDN::APIException.new("Invalid file_or_files argument for delete method.")
     end
   end
 end

data/lib/maxcdn/version.rb

@@ -1,3 +1,3 @@
 module MaxCDN
-  VERSION = "0.1.3"
+  VERSION = "0.1.4"
 end

data/test/integration.rb

@@ -56,7 +56,8 @@ class Client < Minitest::Test
     popularfiles = @max.get("reports/popularfiles.json")["data"]["popularfiles"]
     assert_equal 200, @max.purge(zone, popularfiles[0]["uri"])["code"], "purge file"
 
-    assert_equal 200, @max.purge(zone, [ popularfiles[0]["uri"], popularfiles[1]["uri"]])["code"], "purge files"
+    files = [popularfiles[0]["uri"], popularfiles[1]["uri"]]
+    assert_equal [{"code"=>200},{"code"=>200}], @max.purge(zone, files), "purge files"
   end
 end
 

data/test/maxcdn_test.rb

@@ -28,21 +28,21 @@ stub_request(:put, host+"/account.json")
   .with(:body => "foo=bar", :headers => expected_headers)
   .to_return(:body => '{"foo":"bar"}')
 
-stub_request(:delete, host+"/zones/pull.json/12345/cache")
-  .with(:body => "files=foo.txt", :headers => expected_headers)
+# requests without :body
+expected_headers['Content-Type'] = "application/x-www-form-urlencoded"
+stub_request(:get, host+"/account.json")
+  .with(:headers => expected_headers)
   .to_return(:body => '{"foo":"bar"}')
 
-stub_request(:delete, host+"/zones/pull.json/12345/cache")
-  .with(:body => "files[0]=foo.txt&files[1]=bar.txt", :headers => expected_headers)
+stub_request(:delete, host+"/zones/pull.json/12345/cache?files=foo.txt")
+  .with(:headers => expected_headers)
   .to_return(:body => '{"foo":"bar"}')
 
-stub_request(:delete, host+"/zones/pull.json/12345/cache")
+stub_request(:delete, host+"/zones/pull.json/12345/cache?files=bar.txt")
   .with(:headers => expected_headers)
   .to_return(:body => '{"foo":"bar"}')
 
-# requests without :body
-expected_headers['Content-Type'] = "application/x-www-form-urlencoded"
-stub_request(:get, host+"/account.json")
+stub_request(:delete, host+"/zones/pull.json/12345/cache")
   .with(:headers => expected_headers)
   .to_return(:body => '{"foo":"bar"}')
 
@@ -138,7 +138,7 @@ class Client < Minitest::Test
   end
 
   def test_purge_files
-    assert_equal({ "foo" => "bar" }, @max.purge(12345, [ "foo.txt", "bar.txt" ]))
+    assert_equal([{ "foo" => "bar" },{ "foo" => "bar" }], @max.purge(12345, [ "foo.txt", "bar.txt" ]))
   end
 end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: maxcdn
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.4
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-16 00:00:00.000000000 Z
+date: 2014-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: signet
@@ -83,7 +83,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4324918256587917915
+      hash: 545417652511611192
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -92,7 +92,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -4324918256587917915
+      hash: 545417652511611192
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23