frenetic 0.0.7 → 0.0.8

data/README.md

@@ -1,5 +1,7 @@
-# Frenetic [![Build Status][travis_status]][travis]
+# Frenetic  [![Gem Version][version_badge]][version] [![Build Status][travis_status]][travis]
 
+[version_badge]: https://badge.fury.io/rb/frenetic.png
+[version]: http://badge.fury.io/rb/frenetic
 [travis_status]: https://secure.travis-ci.org/dlindahl/frenetic.png
 [travis]: http://travis-ci.org/dlindahl/frenetic
 
@@ -182,7 +184,7 @@ supports can be added to the Hash.
 
 
 
-# Middlware
+# Middleware
 
 Frenetic supports anything that Faraday does. You may specify additional
 middleware with the `use` method:
@@ -286,4 +288,4 @@ make your code a bit more readable:
 [spire.io]: http://api.spire.io/
 [roar]: https://github.com/apotonick/roar
 [faraday]: https://github.com/technoweenie/faraday
-[rack_cache]: https://github.com/rtomayko/rack-cache
+[rack_cache]: https://github.com/rtomayko/rack-cache

data/lib/frenetic/resource.rb

@@ -1,23 +1,22 @@
+require 'active_support/inflector'
+
 class Frenetic
   class Resource
 
-    attr_reader :links
-
     def initialize( attributes = {} )
-      if attributes.is_a? Hash
-        load attributes.keys, attributes
+      self.class.apply_schema
 
-        @links = {}
-      else
-        load self.class.schema, attributes
+      attributes.each do |key, val|
+        instance_variable_set "@#{key}", val
+      end
 
-        # Load embedded resources if present
-        if attributes.resources
-          load attributes.resources.members, attributes.resources
-        end
+      @_links ||= {}
 
-        @links = attributes.links
-      end
+      build_associations attributes
+    end
+
+    def links
+      @_links
     end
 
     # Attempts to retrieve the Resource Schema from the API based on the name
@@ -33,37 +32,40 @@ class Frenetic
 
       def schema
         if self.respond_to? :api
-          class_name = self.to_s.split('::').last.downcase
+          class_name = self.to_s.demodulize.underscore
 
-          api.description.resources.schema.send(class_name).properties
+          if class_schema = api.description.resources.schema.send(class_name)
+            class_schema.properties
+          else
+            {}
+          end
         else
           raise MissingAPIReference,
-              "This Resource needs a class accessor defined as " +
-              "`.api` that references an instance of Frenetic."
+            "This Resource needs a class accessor defined as " \
+            "`.api` that references an instance of Frenetic."
         end
       end
 
-    private
+      def apply_schema
+        schema.keys.each do |key|
+          next if key[0] == '_'
 
-      def metaclass
-        metaclass = class << self; self; end
+          class_eval { attr_reader key.to_sym } unless instance_methods.include? key
+          class_eval { attr_writer key.to_sym } unless instance_methods.include? "#{key}="
+        end
       end
-    end
-
-  private
 
-    def metaclass
-      metaclass = class << self; self; end
+      def metaclass
+        class << self; self; end
+      end
     end
 
-    def load( keys, attributes )
-      keys.each do |key|
-        instance_variable_set "@#{key}", attributes[key.to_s]
+    def build_associations( attributes )
+      return unless attributes['_embedded']
 
-        metaclass.class_eval do
-          define_method key do
-            instance_variable_get( "@#{key}")
-          end
+      attributes['_embedded'].keys.each do |key|
+        self.class.class_eval do
+          attr_reader key.to_sym
         end
       end
     end

data/lib/frenetic/version.rb

@@ -1,3 +1,3 @@
 class Frenetic
-  VERSION = '0.0.7'
+  VERSION = '0.0.8'
 end

data/spec/lib/frenetic/resource_spec.rb

@@ -2,22 +2,43 @@ describe Frenetic::Resource do
 
   let(:client) { Frenetic.new(url:'http://example.org') }
 
-  let(:resource) { described_class.new }
+  let(:resource) { MyResource.new }
+
+  let(:other_resource) { MyOtherResource.new }
 
   let(:description_stub) do
-    Frenetic::HalJson::ResponseWrapper.new('resources' => { 'schema' => { 'resource' =>
-      { 'properties' => { 'foo' => 2 } }
+    Frenetic::HalJson::ResponseWrapper.new('resources' => {
+      'schema' => {
+        'my_resource' => { 'properties' => { 'foo' => 'string' } },
+        'my_other_resource' => { 'properties' => {} }
     } } )
   end
 
+  before do
+    client.stub(:description).and_return description_stub
+
+    described_class.stub(:api).and_return client
+
+    stub_const 'MyResource', Class.new(described_class)
+    stub_const 'MyOtherResource', Class.new(described_class)
+  end
+
   subject { resource }
 
   context "created from a Hash" do
-    let(:resource) { described_class.new( foo: 'bar' ) }
+    let(:resource) { MyResource.new( foo: 'bar' ) }
 
     it { should respond_to(:foo) }
+    it { should respond_to(:foo=) }
     its(:links) { should be_a Hash }
     its(:links) { should be_empty }
+
+    context 'other subclasses' do
+      subject { other_resource }
+
+      it { should_not respond_to(:foo) }
+      it { should_not respond_to(:foo=) }
+    end
   end
 
   context "created from a HAL-JSON response" do
@@ -30,17 +51,16 @@ describe Frenetic::Resource do
         'bar' => 2
       }
     end
+
     let(:wrapped_response) do
       Frenetic::HalJson::ResponseWrapper.new(api_response)
     end
-    let(:resource_a) { described_class.new( wrapped_response ) }
-    let(:resource_b) { described_class.new }
 
-    before do
-      client.stub(:description).and_return description_stub
+    let(:resource_a) { MyResource.new( wrapped_response ) }
 
-      described_class.stub(:api).and_return client
+    let(:resource_b) { MyResource.new }
 
+    before do
       resource_a && resource_b
     end
 
@@ -55,7 +75,7 @@ describe Frenetic::Resource do
     context "intiailized in sequence without data" do
       subject { resource_b }
 
-      it { should_not respond_to(:foo) }
+      it { should respond_to(:foo) }
       it { should_not respond_to(:bar) }
       its(:links) { should be_empty }
     end
@@ -80,19 +100,29 @@ describe Frenetic::Resource do
   end
 
   describe ".api_client" do
+    before do
+      described_class.unstub! :api
+    end
+
+    subject { MyResource }
+
     context "with a block" do
-      before { resource.class.api_client { client } }
+      before { subject.api_client { client } }
 
       it "should reference the defined API client" do
-        subject.class.api.should == client
+        subject.api.should eq client
+
+        MyOtherResource.should_not respond_to :api
       end
     end
 
     context "with an argument" do
-      before { resource.class.api_client client }
+      before { MyResource.api_client client }
 
       it "should reference the defined API client" do
-        subject.class.api.should == client
+        subject.api.should eq client
+
+        MyOtherResource.should_not respond_to :api
       end
     end
   end
@@ -108,7 +138,7 @@ describe Frenetic::Resource do
       end
 
       it "should return the schema for the specific resource" do
-        subject.should == description_stub.resources.schema.resource.properties
+        subject.should == description_stub.resources.schema.my_resource.properties
       end
     end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: frenetic
 version: !ruby/object:Gem::Version
-  version: 0.0.7
+  version: 0.0.8
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-01-16 00:00:00.000000000 Z
+date: 2013-02-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday