acts_as_api 0.3.6 → 0.3.7

data/History.txt

@@ -1,3 +1,7 @@
+=== 0.3.7 2011-06-13
+
+* Added before/around/after callbacks
+
 === 0.3.6 2011-05-13
 
 * Finally added Mongoid Support out of the box

data/Rakefile

@@ -1,7 +1,7 @@
 require 'bundler'
 require 'rspec/core'
 require 'rspec/core/rake_task'
-require 'rake/rdoctask'
+#require 'rake/rdoctask'
 
 Bundler::GemHelper.install_tasks
 
@@ -22,13 +22,14 @@ end
 
 gemspec = Gem::Specification.load("acts_as_api.gemspec")
 
-Rake::RDocTask.new do |rdoc|
-  rdoc.rdoc_dir = 'doc'
-  rdoc.title = "#{gemspec.name} #{gemspec.version}"
-  rdoc.options += gemspec.rdoc_options
-  rdoc.options << '--line-numbers' << '--inline-source'  
-  rdoc.rdoc_files.include(gemspec.extra_rdoc_files)
-  rdoc.rdoc_files.include('README.rdoc')
-end
+# causes crash in travis ci
+#Rake::RDocTask.new do |rdoc|
+#  rdoc.rdoc_dir = 'doc'
+#  rdoc.title = "#{gemspec.name} #{gemspec.version}"
+#  rdoc.options += gemspec.rdoc_options
+#  rdoc.options << '--line-numbers' << '--inline-source'  
+#  rdoc.rdoc_files.include(gemspec.extra_rdoc_files)
+#  rdoc.rdoc_files.include('README.rdoc')
+#end
 
 #bundle exec rocco examples/introduction/index.rb -t examples/introduction/layout.mustache

data/acts_as_api.gemspec

@@ -1,4 +1,4 @@
-# -*- encoding: utf-8 -*-
+# encoding: utf-8
 $:.push File.expand_path("../lib", __FILE__)
 require "acts_as_api/version"
 
@@ -15,11 +15,10 @@ Gem::Specification.new do |s|
   s.add_dependency('activemodel','>= 3.0.0')
   s.add_dependency('activesupport','>= 3.0.0')
   s.add_dependency('rack','>= 1.1.0')
-    
+
   s.add_development_dependency('rails', ['>= 3.0.0'])
   s.add_development_dependency('mongoid', ['>= 2.0.0'])
-  
-  s.has_rdoc      = true
+
   s.rdoc_options  = ['--main', 'README.rdoc', '--charset=UTF-8']
   s.extra_rdoc_files = ['README.rdoc']
   
@@ -27,4 +26,4 @@ Gem::Specification.new do |s|
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
-end
+end

data/lib/acts_as_api/base.rb

@@ -42,12 +42,13 @@ module ActsAsApi
           yield attributes
         end
 
-        write_inheritable_attribute("api_accessible_#{api_template}".to_sym, attributes)
+        class_attribute "api_accessible_#{api_template}".to_sym
+        send "api_accessible_#{api_template}=", attributes
       end
 
       # Returns an array of all the attributes that have been made accessible to the api response.
       def api_accessible_attributes(api_template)
-        read_inheritable_attribute("api_accessible_#{api_template}".to_sym)
+        begin send "api_accessible_#{api_template}".to_sym rescue nil end
       end
     end
 
@@ -57,10 +58,21 @@ module ActsAsApi
       # Will raise an exception if the passed api template is not defined for the model
       def as_api_response(api_template)
         api_attributes = self.class.api_accessible_attributes(api_template)
-        
         raise ActsAsApi::TemplateNotFoundError.new("acts_as_api template :#{api_template.to_s} was not found for model #{self.class}") if api_attributes.nil?
+          
+        before_api_response(api_template) if respond_to? :before_api_response
         
-        api_attributes.to_response_hash(self)
+        response_hash = if respond_to? :around_api_response
+          around_api_response api_template do
+            api_attributes.to_response_hash(self)
+          end
+        else
+          api_attributes.to_response_hash(self)
+        end
+
+        after_api_response(api_template) if respond_to? :after_api_response
+
+        response_hash
       end
 
     end

data/lib/acts_as_api/responder.rb

@@ -31,10 +31,15 @@ module ActsAsApi
       super(controller, resources, options)
     end
 
-    # Overrides the base implementation of respond, replacing it with
-    # the render_for_api method.
-    def respond
-      controller.render_for_api api_template, options.merge!(format => resource)
+    # Overrides the base implementation of display, replacing it with
+    # the render_for_api method whenever api_template is specified.
+    def display(resource, given_options={})
+      if api_template.nil? || !resource.respond_to?(:as_api_response)
+        controller.render given_options.merge!(options).merge!(format => resource)
+      else
+        controller.render_for_api api_template, given_options.merge!(options).merge!(format => resource)
+      end
     end
+
   end
-end
+end

data/lib/acts_as_api/version.rb

@@ -1,3 +1,3 @@
 module ActsAsApi
-  VERSION = "0.3.6"
+  VERSION = "0.3.7"
 end

data/spec/controllers/respond_with_users_controller_spec.rb

@@ -3,17 +3,18 @@ require 'spec_helper'
 describe RespondWithUsersController do
 
   context "using active record", :orm => :active_record do
-  
+
     before(:each) do
       setup_active_record_models
     end
-  
+
     after(:each) do
      clean_up_active_record_models
     end
-  
+
     # see spec/support/controller_examples.rb
     it_behaves_like "a controller with ActsAsApi responses"
+
   end
 
   context "using mongoid", :orm => :mongoid do
@@ -30,4 +31,92 @@ describe RespondWithUsersController do
     it_behaves_like "a controller with ActsAsApi responses"
   end
 
+
+  describe "default ActionController::Responder behavior" do
+
+    context 'json responses' do
+
+      context "creating valid models" do
+
+        before(:each) do
+          post :create, :user => { :first_name => "Luke", :last_name => "Skywalker" }, :api_template => :name_only, :format => 'json', :orm => :active_record
+        end
+
+        it "should return HTTP 201 status" do
+          response.code.should == "201"
+        end
+
+        it "should contain the specified attributes" do
+          response_body_json["user"].should have_key("first_name")
+          response_body_json["user"].should have_key("last_name")
+        end
+
+        it "should contain the specified values" do
+          response_body_json["user"]["first_name"].should eql("Luke")
+          response_body_json["user"]["last_name"].should eql("Skywalker")
+        end
+      end
+
+      context "creating invalid models" do
+
+        before(:each) do
+          post :create, :user => {}, :api_template => :name_only, :format => 'json', :orm => :active_record
+        end
+
+        it "should return HTTP 422 status" do
+          response.code.should == "422"
+        end
+
+        it "should return errors as json" do
+          response_body_json['first_name'].should include("can't be blank")
+          response_body_json['last_name'].should include("can't be blank")
+        end
+
+      end
+
+    end
+
+    context 'xml responses' do
+
+      context "creating valid models" do
+
+        before(:each) do
+          post :create, :user => { :first_name => "Luke", :last_name => "Skywalker" }, :api_template => :name_only, :format => 'xml', :orm => :active_record
+        end
+
+        it "should return HTTP 201 status" do
+          response.code.should == "201"
+        end
+
+        it "should include HTTP Location header" do
+          response.headers["Location"].should == user_url(User.last)
+        end
+
+        it "should contain the specified attributes" do
+          response_body.should have_selector("user > first-name")
+          response_body.should have_selector("user > last-name")
+        end
+
+      end
+
+      context "creating invalid models" do
+
+        before(:each) do
+          post :create, :user => {}, :api_template => :name_only, :format => 'xml', :orm => :active_record
+        end
+
+        it "should return HTTP 422 status" do
+          response.code.should == "422"
+        end
+
+        it "should return errors as json" do
+          response_body.should have_selector("errors > error")
+        end
+
+      end
+    end
+
+  end
+
+
 end

data/spec/models/active_record_spec.rb

@@ -23,6 +23,7 @@ describe ActiveRecord, :orm => "active_record" do
     it_supports "creating a sub hash in the api template"
     it_supports "trying to render an api template that is not defined"
     it_supports "untouched models" 
+    it_supports "defining a model callback"
   end
   
 end

data/spec/models/mongoid_spec.rb

@@ -22,7 +22,8 @@ describe Mongoid, :orm => "mongoid" do
     it_supports "listing attributes in the api template"
     it_supports "creating a sub hash in the api template"
     it_supports "trying to render an api template that is not defined"
-    it_supports "untouched models" 
+    it_supports "untouched models"
+    it_supports "defining a model callback"
   end
   
 end

data/spec/models/vanilla_ruby_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe Class, :orm => "vanilla" do
+  
+  before(:each) do
+    setup_roflscale_models
+  end
+
+  after(:each) do
+    clean_up_roflscale_models
+  end
+  
+  describe :act_as_api do
+    # deactivated as I won't implement a fake association model just for specs ;)
+    # it_supports "including an association in the api template"
+    it_supports "calling a closure in the api template"
+    it_supports "conditional if statements"
+    it_supports "conditional unless statements" 
+    it_supports "acts_as_api is enabled"
+    it_supports "extending a given api template"
+    it_supports "calling a method in the api template"
+    it_supports "renaming"
+    it_supports "listing attributes in the api template"
+    it_supports "creating a sub hash in the api template"
+    it_supports "trying to render an api template that is not defined"
+    # deactivated as acts_as_api won't get mixed into any class
+    # it_supports "untouched models"
+    it_supports "defining a model callback"
+  end
+  
+end

data/spec/rails_app/app/controllers/application_controller.rb

@@ -1,3 +1,11 @@
 class ApplicationController < ActionController::Base
   protect_from_forgery
+
+  before_filter do
+    if params[:orm] == :active_record
+      @user_model = User
+    elsif params[:orm] == :mongoid
+      @user_model = MongoUser
+    end
+  end
 end

data/spec/rails_app/app/controllers/respond_with_users_controller.rb

@@ -3,14 +3,6 @@ class RespondWithUsersController < ApplicationController
   respond_to :json, :xml
 
   self.responder = ActsAsApi::Responder
-  
-  before_filter do
-    if params[:orm] == :active_record
-      @user_model = User
-    elsif params[:orm] == :mongoid
-      @user_model = MongoUser
-    end
-  end
 
   def index
     @users = @user_model.all
@@ -23,4 +15,19 @@ class RespondWithUsersController < ApplicationController
     respond_with @user, :api_template => params[:api_template].to_sym, :root => :user
   end
 
+  def show_default
+    @user = @user_model.find(params[:id])
+    respond_with @user
+  end
+
+  def create
+    @user = @user_model.new(params[:user])
+
+    if @user.save
+      respond_with @user, :api_template => params[:api_template]
+    else
+      respond_with @user
+    end
+  end
+
 end

data/spec/rails_app/app/controllers/users_controller.rb

@@ -1,13 +1,5 @@
 class UsersController < ApplicationController
-  
-  before_filter do
-    if params[:orm] == :active_record
-      @user_model = User
-    elsif params[:orm] == :mongoid
-      @user_model = MongoUser
-    end
-  end
-  
+
   def index
     @users = @user_model.all
 
@@ -16,7 +8,7 @@ class UsersController < ApplicationController
       format.json { render_for_api params[:api_template].to_sym, :json => @users, :root => :users }
     end
   end
-  
+
   def show
     @user = @user_model.find(params[:id])
 
@@ -25,6 +17,15 @@ class UsersController < ApplicationController
       format.xml  { render_for_api params[:api_template].to_sym, :xml => @user, :root => :user }
       format.json { render_for_api params[:api_template].to_sym, :json => @user, :root => :user }
     end
-  end  
-  
+  end
+
+  def show_default
+    @user = @user_model.find(params[:id])
+    respond_to do |format|
+      format.xml { render :xml => @user }
+      format.json { render :json => @user }
+    end
+  end
+
+
 end

data/spec/rails_app/app/models/mongo_user.rb

@@ -119,6 +119,30 @@ class MongoUser
     t.add :last_name, :unless => lambda{|u| nil }
   end  
 
+  def before_api_response(api_response)
+    @before_api_response_called = true
+  end
+  
+  def before_api_response_called?
+    !!@before_api_response_called
+  end
+  
+  def after_api_response(api_response)
+    @after_api_response_called = true
+  end
+  
+  def after_api_response_called?
+    !!@after_api_response_called
+  end
+  
+  def skip_api_response=(should_skip)
+    @skip_api_response = should_skip
+  end
+  
+  def around_api_response(api_response)
+    @skip_api_response ? { :skipped => true } : yield
+  end
+  
   def over_thirty?
     age > 30
   end

data/spec/rails_app/app/models/user.rb

@@ -1,123 +1,148 @@
 class User < ActiveRecord::Base
+  validates :first_name, :last_name, :presence => true
 
   has_many :tasks
-  
+
   has_one :profile
-  
+
   acts_as_api
-  
+
   api_accessible :name_only do |t|
     t.add :first_name
     t.add :last_name
   end
-  
+
   api_accessible :only_full_name do |t|
     t.add :full_name
   end
-  
+
   api_accessible :rename_last_name do |t|
     t.add :last_name, :as => :family_name
   end
-  
+
   api_accessible :rename_full_name do |t|
     t.add :full_name, :as => :other_full_name
-  end  
-    
+  end
+
   api_accessible :with_former_value do |t|
     t.add :first_name
-    t.add :last_name          
+    t.add :last_name
   end
-  
+
   api_accessible :age_and_first_name, :extend => :with_former_value do |t|
     t.add :age
     t.remove :last_name
   end
-  
+
   api_accessible :calling_a_proc do |t|
     t.add Proc.new{|model| model.full_name.upcase  }, :as => :all_caps_name
     t.add Proc.new{|model| Time.now.class.to_s  }, :as => :without_param
-  end  
-  
+  end
+
   api_accessible :calling_a_lambda do |t|
     t.add lambda{|model| model.full_name.upcase  }, :as => :all_caps_name
     t.add lambda{|model| Time.now.class.to_s  }, :as => :without_param
   end
-  api_accessible :include_tasks do |t| 
+  api_accessible :include_tasks do |t|
     t.add :tasks
   end
-     
-  api_accessible :include_profile do |t| 
+
+  api_accessible :include_profile do |t|
     t.add :profile
-  end     
-        
+  end
+
   api_accessible :other_sub_template do |t|
     t.add :first_name
     t.add :tasks, :template => :other_template
   end
-   
-  api_accessible :include_completed_tasks do |t| 
+
+  api_accessible :include_completed_tasks do |t|
     t.add "tasks.completed.all", :as => :completed_tasks
   end
-  
-  api_accessible :sub_node do |t| 
+
+  api_accessible :sub_node do |t|
     t.add Hash[:foo => :say_something], :as => :sub_nodes
   end
-  
-  api_accessible :nested_sub_node do |t| 
+
+  api_accessible :nested_sub_node do |t|
     t.add Hash[:foo, Hash[:bar, :last_name]], :as => :sub_nodes
   end
-    
-  api_accessible :nested_sub_hash do |t| 
+
+  api_accessible :nested_sub_hash do |t|
     t.add :sub_hash
   end
-  
-  api_accessible :if_over_thirty do |t| 
+
+  api_accessible :if_over_thirty do |t|
     t.add :first_name
-    t.add :last_name, :if => :over_thirty? 
-  end  
+    t.add :last_name, :if => :over_thirty?
+  end
 
-  api_accessible :if_returns_nil do |t| 
+  api_accessible :if_returns_nil do |t|
     t.add :first_name
     t.add :last_name, :if => :return_nil
   end
-  
-  api_accessible :if_over_thirty_proc do |t| 
+
+  api_accessible :if_over_thirty_proc do |t|
     t.add :first_name
     t.add :last_name, :if => lambda{|u| u.over_thirty? }
-  end  
+  end
 
-  api_accessible :if_returns_nil_proc do |t| 
+  api_accessible :if_returns_nil_proc do |t|
     t.add :first_name
     t.add :last_name, :if => lambda{|u| nil }
   end
-  
-  api_accessible :unless_under_thirty do |t| 
+
+  api_accessible :unless_under_thirty do |t|
     t.add :first_name
-    t.add :last_name, :unless => :under_thirty? 
-  end  
+    t.add :last_name, :unless => :under_thirty?
+  end
 
-  api_accessible :unless_returns_nil do |t| 
+  api_accessible :unless_returns_nil do |t|
     t.add :first_name
     t.add :last_name, :unless => :return_nil
   end
-  
-  api_accessible :unless_under_thirty_proc do |t| 
+
+  api_accessible :unless_under_thirty_proc do |t|
     t.add :first_name
     t.add :last_name, :unless => lambda{|u| u.under_thirty? }
-  end  
+  end
 
-  api_accessible :unless_returns_nil_proc do |t| 
+  api_accessible :unless_returns_nil_proc do |t|
     t.add :first_name
     t.add :last_name, :unless => lambda{|u| nil }
-  end  
+  end
+  
+  def before_api_response(api_response)
+    @before_api_response_called = true
+  end
+  
+  def before_api_response_called?
+    !!@before_api_response_called
+  end
+  
+  def after_api_response(api_response)
+    @after_api_response_called = true
+  end
+  
+  def after_api_response_called?
+    !!@after_api_response_called
+  end
+  
+  def skip_api_response=(should_skip)
+    @skip_api_response = should_skip
+  end
+  
+  def around_api_response(api_response)
+    @skip_api_response ? { :skipped => true } : yield
+  end
 
   def over_thirty?
     age > 30
   end
-  
+
   def under_thirty?
     age < 30
-  end  
+  end
 
   def return_nil
     nil
@@ -130,7 +155,7 @@ class User < ActiveRecord::Base
   def say_something
     "something"
   end
-  
+
   def sub_hash
     {
       :foo => "bar",

data/spec/rails_app/app/models/vanilla_profile.rb

@@ -0,0 +1,16 @@
+class VanillaProfile
+  extend ActsAsApi::Base
+  
+  attr_accessor :user, :avatar, :homepage, :created_at, :updated_at
+  
+  def initialize(opts)
+    opts.each do |k,v|
+      self.send :"#{k}=", v
+    end
+  end
+  
+  def model_name
+    'profile'
+  end
+  
+end

data/spec/rails_app/app/models/vanilla_task.rb

@@ -0,0 +1,16 @@
+class VanillaTask
+  extend ActsAsApi::Base
+
+  attr_accessor :user, :heading, :description, :time_spent, :done, :created_at, :updated_at
+
+  def initialize(opts)
+    opts.each do |k,v|
+      self.send :"#{k}=", v
+    end
+  end  
+  
+  def model_name
+    'task'
+  end
+
+end

data/spec/rails_app/app/models/vanilla_untouched.rb

@@ -0,0 +1,2 @@
+class VanillaUntouched
+end

data/spec/rails_app/app/models/vanilla_user.rb

@@ -0,0 +1,176 @@
+class VanillaUser
+  extend ActsAsApi::Base
+  
+  attr_accessor :profile, :tasks, :first_name, :last_name, :age, :active, :created_at, :updated_at
+  
+  
+  def initialize(opts)
+    @tasks = []
+    opts.each do |k,v|
+      self.send :"#{k}=", v
+    end
+  end  
+  
+  def model_name
+    'user'
+  end
+
+  acts_as_api
+  
+  api_accessible :name_only do |t|
+    t.add :first_name
+    t.add :last_name
+  end
+  
+  api_accessible :only_full_name do |t|
+    t.add :full_name
+  end
+  
+  api_accessible :rename_last_name do |t|
+    t.add :last_name, :as => :family_name
+  end
+  
+  api_accessible :rename_full_name do |t|
+    t.add :full_name, :as => :other_full_name
+  end  
+    
+  api_accessible :with_former_value do |t|
+    t.add :first_name
+    t.add :last_name          
+  end
+  
+  api_accessible :age_and_first_name, :extend => :with_former_value do |t|
+    t.add :age
+    t.remove :last_name
+  end
+  
+  api_accessible :calling_a_proc do |t|
+    t.add Proc.new{|model| model.full_name.upcase  }, :as => :all_caps_name
+    t.add Proc.new{|model| Time.now.class.to_s  }, :as => :without_param
+  end  
+  
+  api_accessible :calling_a_lambda do |t|
+    t.add lambda{|model| model.full_name.upcase  }, :as => :all_caps_name
+    t.add lambda{|model| Time.now.class.to_s  }, :as => :without_param
+  end
+  api_accessible :include_tasks do |t| 
+    t.add :tasks
+  end
+     
+  api_accessible :include_profile do |t| 
+    t.add :profile
+  end     
+        
+  api_accessible :other_sub_template do |t|
+    t.add :first_name
+    t.add :tasks, :template => :other_template
+  end
+   
+  api_accessible :include_completed_tasks do |t| 
+    t.add "tasks.completed.all", :as => :completed_tasks
+  end
+  
+  api_accessible :sub_node do |t| 
+    t.add Hash[:foo => :say_something], :as => :sub_nodes
+  end
+  
+  api_accessible :nested_sub_node do |t| 
+    t.add Hash[:foo, Hash[:bar, :last_name]], :as => :sub_nodes
+  end
+    
+  api_accessible :nested_sub_hash do |t| 
+    t.add :sub_hash
+  end
+  
+  api_accessible :if_over_thirty do |t| 
+    t.add :first_name
+    t.add :last_name, :if => :over_thirty? 
+  end  
+
+  api_accessible :if_returns_nil do |t| 
+    t.add :first_name
+    t.add :last_name, :if => :return_nil
+  end
+  
+  api_accessible :if_over_thirty_proc do |t| 
+    t.add :first_name
+    t.add :last_name, :if => lambda{|u| u.over_thirty? }
+  end  
+
+  api_accessible :if_returns_nil_proc do |t| 
+    t.add :first_name
+    t.add :last_name, :if => lambda{|u| nil }
+  end
+  
+  api_accessible :unless_under_thirty do |t| 
+    t.add :first_name
+    t.add :last_name, :unless => :under_thirty? 
+  end  
+
+  api_accessible :unless_returns_nil do |t| 
+    t.add :first_name
+    t.add :last_name, :unless => :return_nil
+  end
+  
+  api_accessible :unless_under_thirty_proc do |t| 
+    t.add :first_name
+    t.add :last_name, :unless => lambda{|u| u.under_thirty? }
+  end  
+
+  api_accessible :unless_returns_nil_proc do |t| 
+    t.add :first_name
+    t.add :last_name, :unless => lambda{|u| nil }
+  end  
+
+  def before_api_response(api_response)
+    @before_api_response_called = true
+  end
+  
+  def before_api_response_called?
+    !!@before_api_response_called
+  end
+  
+  def after_api_response(api_response)
+    @after_api_response_called = true
+  end
+  
+  def after_api_response_called?
+    !!@after_api_response_called
+  end
+  
+  def skip_api_response=(should_skip)
+    @skip_api_response = should_skip
+  end
+  
+  def around_api_response(api_response)
+    @skip_api_response ? { :skipped => true } : yield
+  end
+  
+  def over_thirty?
+    age > 30
+  end
+  
+  def under_thirty?
+    age < 30
+  end  
+
+  def return_nil
+    nil
+  end
+
+  def full_name
+    '' << first_name.to_s << ' ' << last_name.to_s
+  end
+
+  def say_something
+    "something"
+  end
+  
+  def sub_hash
+    {
+      :foo => "bar",
+      :hello => "world"
+    }
+  end
+
+end

data/spec/rails_app/config/routes.rb

@@ -1,7 +1,15 @@
 RailsApp::Application.routes.draw do
 
-  resources :users
-  
-  resources :respond_with_users
+  resources :users do
+    member do
+      get 'show_default'
+    end
+  end
+
+  resources :respond_with_users do
+    member do
+      get 'show_default'
+    end
+  end
 
 end

data/spec/support/controller_examples.rb

@@ -44,8 +44,28 @@ shared_examples_for "a controller with ActsAsApi responses" do
 
     end
 
-  end
+    describe 'get a user without specifying an api template' do
+
+      before(:each) do
+        get :show_default, :format => 'xml', :id => @luke.id, :orm => @orm_for_testing
+      end
+
+      it "should respond with HTTP 200" do
+        response.code.should == "200"
+      end
+
+      it "should render the model with standard to_xml" do
+        # TODO: figure out how to compare xml generated by mongo models.
+
+        # This line works fine with ActiveRecord, but not Mongoid:
+        # response.body.should == @luke.to_xml
+
+        response.body.length.should == @luke.to_xml.length
+      end
+
+    end
 
+  end
 
   describe 'json responses' do
 
@@ -124,6 +144,22 @@ shared_examples_for "a controller with ActsAsApi responses" do
 
     end
 
+    describe 'get a user without specifying an api template' do
+
+      before(:each) do
+        get :show_default, :format => 'json', :id => @luke.id, :orm => @orm_for_testing
+      end
+
+      it "should respond with HTTP 200" do
+        response.code.should == "200"
+      end
+
+      it "should render the model with to_json" do
+        response.body.should == @luke.to_json
+      end
+
+    end
+
   end
 
   describe 'Rails 3 default style json responses' do
@@ -242,5 +278,4 @@ shared_examples_for "a controller with ActsAsApi responses" do
     end
   end
 
-
 end

data/spec/support/model_examples/callbacks.rb

@@ -0,0 +1,38 @@
+shared_examples_for "defining a model callback" do
+
+  describe "for a" do
+    
+    describe "around_api_response" do
+      
+      it "skips rendering if not yielded" do
+        @luke.skip_api_response = true
+        @luke.as_api_response(:name_only).keys.should include(:skipped)
+      end
+      
+      it "renders if yielded" do
+        @luke.as_api_response(:name_only).keys.should_not include(:skipped)
+      end
+    
+    end
+    
+    describe "before_api_response" do 
+      
+      it "is called properly" do
+        @luke.as_api_response(:name_only)
+        @luke.before_api_response_called?.should eql(true)
+      end
+      
+    end
+    
+    describe "after_api_response" do 
+      
+      it "is called properly" do
+        @luke.as_api_response(:name_only)
+        @luke.after_api_response_called?.should eql(true)
+      end
+      
+    end
+    
+  end
+
+end

data/spec/support/simple_fixtures.rb

@@ -21,7 +21,7 @@ module SimpleFixtures
   def clean_up_active_record_models
     @user_model.delete_all
     @task_model.delete_all
-  end  
+  end
   
   def setup_mongoid_models
     @orm_for_testing  = :mongoid    
@@ -49,6 +49,30 @@ module SimpleFixtures
     @user_model.delete_all
   end
   
+  def setup_roflscale_models
+    @orm_for_testing  = :vanilla
+    @user_model       = VanillaUser
+    @task_model       = VanillaTask
+    @profile_model    = VanillaProfile
+    @untouched_model  = VanillaUntouched
+
+    @luke = @user_model.new({ :first_name => 'Luke',      :last_name => 'Skywalker', :age => 25, :active => true  })  
+    @han  = @user_model.new({ :first_name => 'Han',       :last_name => 'Solo',      :age => 35, :active => true  })
+    @leia = @user_model.new({ :first_name => 'Princess',  :last_name => 'Leia',      :age => 25, :active => false })
+
+    @luke.profile = @profile_model.new({ :user => @luke, :avatar => 'picard.jpg', :homepage => 'lukasarts.com' })
+
+    @destroy_deathstar = @task_model.new({ :user => @luke, :heading => "Destroy Deathstar", :description => "XWing, Shoot, BlowUp",  :time_spent => 30,  :done => true })
+    @study_with_yoda   = @task_model.new({ :user => @luke, :heading => "Study with Yoda",   :description => "Jedi Stuff, ya know",   :time_spent => 60,  :done => true })
+    @win_rebellion     = @task_model.new({ :user => @luke, :heading => "Win Rebellion",     :description => "no idea yet...",        :time_spent => 180, :done => false })
+    
+    @luke.tasks << @destroy_deathstar << @study_with_yoda << @win_rebellion
+  end
+  
+  def clean_up_roflscale_models
+    # nothing to do ;)
+  end  
+  
 end
 
 

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: acts_as_api
 version: !ruby/object:Gem::Version 
-  hash: 31
+  hash: 29
   prerelease: 
   segments: 
   - 0
   - 3
-  - 6
-  version: 0.3.6
+  - 7
+  version: 0.3.7
 platform: ruby
 authors: 
 - "Christian B\xC3\xA4uerlein"
@@ -15,7 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-14 00:00:00 +02:00
+date: 2011-06-13 00:00:00 +02:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -134,6 +134,7 @@ files:
 - spec/controllers/users_controller_spec.rb
 - spec/models/active_record_spec.rb
 - spec/models/mongoid_spec.rb
+- spec/models/vanilla_ruby_spec.rb
 - spec/rails_app/.gitignore
 - spec/rails_app/Rakefile
 - spec/rails_app/app/controllers/application_controller.rb
@@ -148,6 +149,10 @@ files:
 - spec/rails_app/app/models/task.rb
 - spec/rails_app/app/models/untouched.rb
 - spec/rails_app/app/models/user.rb
+- spec/rails_app/app/models/vanilla_profile.rb
+- spec/rails_app/app/models/vanilla_task.rb
+- spec/rails_app/app/models/vanilla_untouched.rb
+- spec/rails_app/app/models/vanilla_user.rb
 - spec/rails_app/app/views/layouts/application.html.erb
 - spec/rails_app/config.ru
 - spec/rails_app/config/application.rb
@@ -191,6 +196,7 @@ files:
 - spec/support/controller_examples.rb
 - spec/support/it_supports.rb
 - spec/support/model_examples/associations.rb
+- spec/support/model_examples/callbacks.rb
 - spec/support/model_examples/closures.rb
 - spec/support/model_examples/conditional_if.rb
 - spec/support/model_examples/conditional_unless.rb
@@ -244,6 +250,7 @@ test_files:
 - spec/controllers/users_controller_spec.rb
 - spec/models/active_record_spec.rb
 - spec/models/mongoid_spec.rb
+- spec/models/vanilla_ruby_spec.rb
 - spec/rails_app/.gitignore
 - spec/rails_app/Rakefile
 - spec/rails_app/app/controllers/application_controller.rb
@@ -258,6 +265,10 @@ test_files:
 - spec/rails_app/app/models/task.rb
 - spec/rails_app/app/models/untouched.rb
 - spec/rails_app/app/models/user.rb
+- spec/rails_app/app/models/vanilla_profile.rb
+- spec/rails_app/app/models/vanilla_task.rb
+- spec/rails_app/app/models/vanilla_untouched.rb
+- spec/rails_app/app/models/vanilla_user.rb
 - spec/rails_app/app/views/layouts/application.html.erb
 - spec/rails_app/config.ru
 - spec/rails_app/config/application.rb
@@ -301,6 +312,7 @@ test_files:
 - spec/support/controller_examples.rb
 - spec/support/it_supports.rb
 - spec/support/model_examples/associations.rb
+- spec/support/model_examples/callbacks.rb
 - spec/support/model_examples/closures.rb
 - spec/support/model_examples/conditional_if.rb
 - spec/support/model_examples/conditional_unless.rb