api-resource 0.5.1 → 0.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7fb25fa5e1a25eec2eca37948e5c8638d2ac17b2
-  data.tar.gz: 112ad65c7f8b3d611ac2e8e1a5781adc82736efa
+  metadata.gz: 5e46ba73409d59af235f64ccaebdd554a627d284
+  data.tar.gz: 6f8b0023573cb1d2895371a77732c10b5a7e8786
 SHA512:
-  metadata.gz: f4b7f563692b808b8120f3e88d5ca01b100601b354a869b9265fcdd84cf2b7661d8f2292b9950fb21bff0f06b9b16d804978e54e7966e80886ec629ae42f7edb
-  data.tar.gz: 7ebc0f1b6768180cac7fa13465e56d7acbfa20f70dc1a1d7bbc0ec6d94ea26403ba5063fd7c1e7de261f28da1a1f4b9cba1917cdb654b2cf9489151965672652
+  metadata.gz: bda443d6b27415cfd8e53519e14ebfa3781eb30f3158a4644db23ecc0b56668a72059c36e2a2f67bfb9721e4b981c9d1982d5a57128451167c0ff6cfd1191617
+  data.tar.gz: 6e49673ae732c15b76ac12086fa83c0c174a87d77796d6140e9e7615c8a29983ce2cd9463a64c18596b8998f14752748ff62a93d80ee96ec01cc942c7594796f

data/lib/api-resource/resource.rb

@@ -23,7 +23,7 @@ module ApiResource
     class_attribute :api_secret
     class_attribute :hmac_options
     class_attribute :submitted_attributes_filter
-
+    class_attribute :log
 
     def self.with_hmac(api_id, api_secret, options={})
       self.hmac         = true
@@ -173,7 +173,7 @@ module ApiResource
         when /^by_(.+)/
           with_parent_resource(args[0], $1) unless args.empty?
         else
-          # do nothing
+          super
       end
     end
 
@@ -193,7 +193,7 @@ module ApiResource
     def self.load_class(tableized_class)
       tableized_class = name.to_s.tableize.split('/')[0...-1].push(tableized_class).join('/')
       klass = tableized_class && tableized_class.to_s.singularize.classify.constantize rescue nil
-      klass if klass && klass < ApiResource::Resource
+      klass if klass && klass < Logging::Resource
     end
 
     def self.client(verb, params, *paths)
@@ -210,7 +210,9 @@ module ApiResource
       if hmac
         req.sign!(api_id, api_secret, hmac_options)
       end
-      req.execute
+      result = req.execute
+      Logging.log << result
+      result
     end
 
     def self.build_url(*paths)
@@ -218,17 +220,57 @@ module ApiResource
       URI::parse(base_url).merge(paths.map { |p| "#{URI.encode p.to_s}" }.join('/')).to_s
     end
 
-    class ResourceCollection
-      include Enumerable
+    def self.logger=(param)
+      self.log = create_log(param)
+    end
 
-      def initialize(data, meta, klass)
-        meta.each { |k, v| self.class.class_eval { define_method(k) { v } } } if meta
-        @resources = data.map { |e| klass.new(e) }
-      end
+    def self.logger
+      self.log ||= Class.new do
+        def <<(_)
+        end
+      end.new
+    end
 
-      def each(&block)
-        @resources.each(&block)
+    # Create a log that respond to << like a logger
+    # param can be 'stdout', 'stderr', a string (then we will log to that file) or a logger (then we return it)
+    def self.create_log(param)
+      return param unless param.is_a? String
+      case param
+        when 'stdout'
+          Class.new do
+            def <<(obj)
+              STDOUT.puts obj
+            end
+          end.new
+        when 'stderr'
+          Class.new do
+            def <<(obj)
+              STDERR.puts obj
+            end
+          end.new
+        else
+          Class.new do
+            def initialize(file)
+              @file = file
+            end
+
+            def <<(obj)
+              File.open(@file, 'a') { |f| f.puts obj }
+            end
+          end.new(param)
       end
     end
   end
-end
+  class ResourceCollection
+    include Enumerable
+
+    def initialize(data, meta, klass)
+      meta.each { |k, v| self.class.class_eval { define_method(k) { v } } } if meta
+      @resources = data.map { |e| klass.new(e) }
+    end
+
+    def each(&block)
+      @resources.each(&block)
+    end
+  end
+end

data/lib/api-resource/version.rb

@@ -1,3 +1,3 @@
 module ApiResource
-  VERSION = '0.5.1'
+  VERSION = '0.5.2'
 end

data/spec/resource_spec.rb

@@ -1,6 +1,6 @@
-RSpec.describe ApiResource::Resource do
+RSpec.describe Logging::Resource do
   module BlogApi
-    class Resource < ApiResource::Resource
+    class Resource < Logging::Resource
       self.base_url = 'http://api.example.com'
     end
 
@@ -195,7 +195,7 @@ RSpec.describe ApiResource::Resource do
 
   context 'empty base_url' do
     it 'throws on requests' do
-      class Foo < ApiResource::Resource;
+      class Foo < Logging::Resource;
       end
       expect(-> { Foo.all }).to raise_exception(StandardError)
     end
@@ -271,7 +271,7 @@ RSpec.describe ApiResource::Resource do
     it 'sets the returned errors' do
       expected_value = { errors: { password_confrimation: ["can't be blank"], email: ["can't be blank", 'is invalid'] } }
       stub           = req(:post, '/sign_up', expected_value, 422, sign_up.attributes)
-      expect(proc { sign_up.submit! }).to raise_error(ApiResource::ResourceError)
+      expect(proc { sign_up.submit! }).to raise_error(Logging::ResourceError)
       expect(sign_up.errors.messages).to match(expected_value[:errors])
       expect(stub).to have_been_requested
     end
@@ -279,7 +279,7 @@ RSpec.describe ApiResource::Resource do
     it 'no request if not valid' do
       sign_up.name = nil
       stub         = req(:post, '/sign_up', nil, 422)
-      expect(proc { sign_up.submit! }).to raise_error(ApiResource::ResourceError)
+      expect(proc { sign_up.submit! }).to raise_error(Logging::ResourceError)
       expect(sign_up.errors[:name]).to be_truthy
       expect(stub).to_not have_been_requested
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: api-resource
 version: !ruby/object:Gem::Version
-  version: 0.5.1
+  version: 0.5.2
 platform: ruby
 authors:
 - Chaker Nakhli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-03-29 00:00:00.000000000 Z
+date: 2015-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simple-hmac