api-resource 0.7.1 → 0.7.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 12920e4c95319de6f1ad5bc8319ba6782ef2d1bd
-  data.tar.gz: d1e236843cf77d8ceb0e10c513352070a0a7b434
+  metadata.gz: 5d337a40cfc027bdc85680315d1b0c87557a7e1b
+  data.tar.gz: cff63a9be56dd173e1a4df49be49d6397efc10e6
 SHA512:
-  metadata.gz: 6570f8461faab3595bd5c7362ace4a92799da969c46bc5fe73d5e1936a7f93b0af26527d9fd0749a66aa57c6db22f08735235be2d559972e7deccdf85fbdde2c
-  data.tar.gz: f4e2bbc66ef4aaaf70ea71f00ce82cc79995a48d4e0f09fe78799accb4b10db60b164c21c4ec8722ac76836ce7717bd2054a809f8e24c4895cbf29146e68966f
+  metadata.gz: dcf0645d2a1e55149238a85dae91fcc374bc6406e0034dfc0ab3258a62568012f8871500e28b8c30fbeb6acbbfe020c678efdc630d757c5eece7b3255972e505
+  data.tar.gz: d27781a96ca4e611aef5bcab3ae42747564210648b7708d1e8262a0acb204f9b91f3c2daa89558b22719ebab5467ab7f9b716c5ce4dd1528c78238823fa9a882

data/lib/api-resource/resource.rb

@@ -162,7 +162,7 @@ module ApiResource
     def self.method_missing(m, *args, &_)
       case (m)
         when /^by_(.+)/
-          resource_path = build_url($1.pluralize, args[0], self.resource_path)
+          resource_path = build_path($1.pluralize, args[0], self.resource_path)
           Class.new(self) do
             self.resource_path = resource_path
           end
@@ -174,7 +174,7 @@ module ApiResource
     def method_missing(m, *args, &_)
       case m
         when /^by_(.+)/
-          self.resource_path = self.class.build_url($1.pluralize, args[0], resource_path)
+          self.resource_path = self.class.build_path($1.pluralize, args[0], resource_path)
           self
         else
           super
@@ -240,7 +240,11 @@ module ApiResource
 
     def self.build_url(*paths)
       raise RuntimeError, 'Empty base_url' if base_url.blank?
-      URI::parse(base_url).merge(paths.map { |p| "#{URI.encode p.to_s}" }.join('/')).to_s
+      URI::parse(base_url).merge(build_path(*paths)).to_s
+    end
+
+    def self.build_path(*paths)
+      paths.map { |p| URI.encode p.to_s.gsub(%r{\A/*(.*?)/*\z}, '\1') }.join('/')
     end
 
     def self.log=(param)

data/lib/api-resource/version.rb

@@ -1,3 +1,3 @@
 module ApiResource
-  VERSION = '0.7.1'
+  VERSION = '0.7.2'
 end

data/spec/blog_api.rb

@@ -0,0 +1,20 @@
+module BlogApi
+  class Resource < ApiResource::Resource
+    self.base_url = 'http://api.example.com'
+  end
+
+  class Category < Resource
+    attr_accessor :id, :name
+  end
+
+  class Blog < Resource
+    attr_accessor :id, :title
+
+    def do_stuff_method
+    end
+  end
+
+  class Tag < Resource
+    attr_accessor :id, :name
+  end
+end

data/spec/resource_path_spec.rb

@@ -0,0 +1,34 @@
+require_relative 'blog_api'
+
+RSpec.describe ApiResource::Resource do
+
+  class MyResource < ApiResource::Resource; end
+
+  let(:resource_class) { MyResource }
+
+  let(:resource_name) { MyResource.to_s.tableize }
+
+  it 'return tabeleized class name' do
+    expect(resource_class.resource_path).to eq(resource_name)
+  end
+
+  context '::by_resource' do
+    it 'prepends parent resource path' do
+      expect(resource_class.by_foo_bar('44').resource_path).to eq("foo_bars/44/#{ resource_name }")
+    end
+
+    it 'prepends multiple parent resource paths' do
+      expect(resource_class.by_bar(15).by_foo(88).resource_path).to eq("foos/88/bars/15/#{ resource_name }")
+    end
+  end
+
+  context '#by_resource' do
+    it 'prepends parent resource path' do
+      expect(resource_class.new.by_foo_bar('44').resource_path).to eq("foo_bars/44/#{ resource_name }")
+    end
+
+    it 'prepends multiple parent resource paths' do
+      expect(resource_class.new.by_bar(15).by_foo(88).resource_path).to eq("foos/88/bars/15/#{ resource_name }")
+    end
+  end
+end

data/spec/resource_spec.rb

@@ -1,25 +1,6 @@
-RSpec.describe ApiResource::Resource do
-  module BlogApi
-    class Resource < ApiResource::Resource
-      self.base_url = 'http://api.example.com'
-    end
-
-    class Category < Resource
-      attr_accessor :id, :name
-    end
-
-    class Blog < Resource
-      attr_accessor :id, :title
-
-      def do_stuff_method
-      end
-    end
-
-    class Tag < Resource
-      attr_accessor :id, :name
-    end
-  end
+require_relative 'blog_api'
 
+RSpec.describe ApiResource::Resource do
   before do
     @expected_resources = { data: [{ id: 1257, title: 'Hello' },
                                    { id: 33, title: 'World' },

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: api-resource
 version: !ruby/object:Gem::Version
-  version: 0.7.1
+  version: 0.7.2
 platform: ruby
 authors:
 - Chaker Nakhli
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-04-13 00:00:00.000000000 Z
+date: 2015-04-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: simple-hmac
@@ -135,6 +135,8 @@ files:
 - lib/api-resource.rb
 - lib/api-resource/resource.rb
 - lib/api-resource/version.rb
+- spec/blog_api.rb
+- spec/resource_path_spec.rb
 - spec/resource_spec.rb
 - spec/spec_helper.rb
 homepage: http://www.wizypay.com
@@ -163,5 +165,7 @@ signing_key:
 specification_version: 4
 summary: A lightweight REST API access library.
 test_files:
+- spec/blog_api.rb
+- spec/resource_path_spec.rb
 - spec/resource_spec.rb
 - spec/spec_helper.rb