puffer 0.0.29 → 0.0.30

data/spec/fabricators/mongoid_orm/primal.rb

@@ -0,0 +1,12 @@
+Fabricator :mongoid_orm_primal, :class_name => 'mongoid_orm/primal' do
+  string_field { Forgery::LoremIpsum.sentence }
+  symbol_filed { Forgery::LoremIpsum.word.to_sym }
+  select_field { sequence(:select_field) {|i| "option #{i}" } }
+  integer_field { sequence :integer_field }
+  float_field { sequence :float_field }
+  decimal_field { sequence :decimal_field }
+  datetime_field { sequence(:datetime_field) {|i| DateTime.now + i.days } }
+  time_field { sequence(:time_field) {|i| DateTime.now + i.hours } }
+  date_field { sequence(:date_field) {|i| DateTime.now + i.days } }
+  boolean_field { rand 2 }
+end

data/spec/generators/puffer/component/component_generator_spec.rb

@@ -0,0 +1,32 @@
+require 'spec_helper'
+
+# Generators are not automatically loaded by Rails
+require 'generators/puffer/component/component_generator'
+
+describe Puffer::ComponentGenerator do
+  before { run_generator %w(password)  }
+
+  describe 'app/components/password_component.rb' do
+    subject { file('app/components/password_component.rb') }
+    it { should exist }
+    it { should contain "class PasswordComponent < Puffer::Component::Base" }
+  end
+
+  describe 'app/components/password/index.html.erb' do
+    subject { file('app/components/password/index.html.erb') }
+    it { should exist }
+    it { should contain "# I'm index" }
+  end
+
+  describe 'app/components/password/form.html.erb' do
+    subject { file('app/components/password/form.html.erb') }
+    it { should exist }
+    it { should contain "# I'm form" }
+  end
+
+  describe 'app/components/password/filter.html.erb' do
+    subject { file('app/components/password/filter.html.erb') }
+    it { should exist }
+    it { should contain "# I'm filter" }
+  end
+end

data/spec/generators/puffer/controller/controller_generator_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+# Generators are not automatically loaded by Rails
+require 'generators/puffer/controller/controller_generator'
+
+describe Puffer::ControllerGenerator do
+  describe 'controller namespaced in admin module added by default' do
+    before { run_generator %w(User name:string email:string)  }
+
+    describe 'app/controllers/admin/users_controller.rb' do
+      subject { file('app/controllers/admin/users_controller.rb') }
+      it { should exist }
+      it { should contain "class Admin::UsersController < Puffer::Base" }
+      it { should contain "# field :id" }
+      it { should contain "field :email" }
+      it { should contain "field :password" }
+      it { should contain "# field :created_at" }
+      it { should contain "# field :updated_at" }
+    end
+  end
+
+  describe 'controller namespace can be specified in admin module added by default' do
+    before { run_generator %w(Moderator::User name:string email:string)  }
+
+    describe 'app/controllers/moderator/users_controller.rb' do
+      subject { file('app/controllers/moderator/users_controller.rb') }
+      it { should exist }
+      it { should contain "class Moderator::UsersController < Puffer::Base" }
+    end
+  end
+end

data/spec/helpers/puffer_helper_spec.rb

@@ -0,0 +1,62 @@
+require 'spec_helper'
+
+# Specs in this file have access to a helper object that includes
+# the PufferHelper. For example:
+#
+# describe PufferHelper do
+#   describe "string concat" do
+#     it "concats two strings with spaces" do
+#       helper.concat_strings("this","that").should == "this that"
+#     end
+#   end
+# end
+describe PufferHelper do
+
+  it "generates scopes navigation" do
+    helper.stub(:puffer_namespace).and_return(:admin)
+    
+    navigation = []
+    helper.puffer_scopes_navigation do |*args|
+      navigation.push args
+    end
+
+    navigation.should == [
+      [:admin, "/admin", true],
+      [:orms, "/orms", false]
+    ]
+  end
+
+  it "generates groups navigation" do
+    helper.stub(:puffer_namespace).and_return(:admin)
+    helper.stub(:resource).and_return mock(:resource_node => mock, :root => mock(:resource_node => mock(:group => :posting)))
+
+    navigation = []
+    helper.puffer_groups_navigation do |*args|
+      navigation.push args
+    end
+
+    navigation.should == [
+      [:users, "/admin/users", false],
+      [:posting, "/admin/posts", true],
+      [:news, "/admin/news", false]
+    ]
+  end
+
+  it "generates resources navigation" do
+    helper.stub(:puffer_namespace).and_return(:admin)
+    helper.stub(:configuration).and_return mock(:group => :posting)
+    helper.stub(:resource).and_return mock(:resource_node => mock, :root => mock(:resource_node => Rails.application.routes.resources_tree[15]))
+
+    navigation = []
+    helper.puffer_resources_navigation do |*args|
+      navigation.push args
+    end
+
+    navigation.should == [
+      ["Post", "/admin/posts", false],
+      ["Tagged post", "/admin/tagged_posts", false],
+      ["Category", "/admin/categories", true]
+    ]
+  end
+
+end

data/spec/lib/filters_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe Puffer::Filters do
+
+  describe 'controller filters class generation' do
+
+    it 'should generate class with proper name' do
+      Puffer::Filters.controller_filters(Admin::UsersController).should == Admin::UsersControllerFilters
+    end
+
+    it 'should return Puffer::Filters subclass' do
+      Puffer::Filters.controller_filters(Admin::UsersController).superclass.should == Puffer::Filters
+    end
+
+    it 'should set proper attributes' do
+      Puffer::Filters.controller_filters(Admin::PostsController).new.attributes.keys.should == %w(user.email status title body puffer_search puffer_order)
+    end
+
+  end
+
+end

data/spec/lib/orm_adapter/active_record_spec.rb

@@ -0,0 +1,33 @@
+require 'spec_helper'
+require 'lib/orm_adapter/base_shared'
+
+describe "ActiveRecordOrm" do
+
+  describe 'columns_hash' do
+    it 'should be proper' do
+      ActiveRecordOrm::Primal.to_adapter.columns_hash.should == {
+        "id" => {:type => :integer},
+        "string_field" => {:type => :string},
+        "text_field" => {:type => :text},
+        "select_field" => {:type => :string},
+        "integer_field" => {:type => :integer},
+        "float_field" => {:type => :float},
+        "decimal_field" => {:type => :decimal},
+        "datetime_field" => {:type => :datetime},
+        "timestamp_field" => {:type => :datetime},
+        "time_field" => {:type => :time},
+        "date_field" => {:type => :date},
+        "boolean_field" => {:type => :boolean}
+      }
+    end
+  end
+
+  describe 'filter' do
+    it_should_behave_like "an adapter" do
+      let(:model) {ActiveRecordOrm::Primal}
+      let(:fabric) {:active_record_orm_primal}
+      let(:controller) {Orms::ActiveRecordOrmPrimalsController}
+    end
+  end
+
+end

data/spec/lib/orm_adapter/base_shared.rb

@@ -0,0 +1,97 @@
+require 'spec_helper'
+
+shared_examples "an adapter" do
+
+  def filter options = {}
+    model.to_adapter.filter(model, controller.index_fields, options).all.to_a
+  end
+
+
+  def nth *indexes
+    model.all.to_a.values_at *indexes
+  end
+
+  context 'without params' do
+
+    before do
+      5.times { Fabricate fabric }
+    end
+
+    it 'should return all records' do
+      filter.should == model.all.to_a
+    end
+
+  end
+
+  context 'search' do
+
+    before do
+      string_fields = %w(hello hello world hello world)
+      select_fields = %w(hello world helloworld hello world)
+      5.times { |i|
+        Fabricate fabric, :string_field => string_fields[i], :select_field => select_fields[i]
+      }
+    end
+
+    it 'should return all searched records' do
+      filter(:search => 'hel').should == nth(0, 1, 2, 3)
+    end
+
+    it 'should return searched records without conditioned' do
+      filter(:search => 'hel', :conditions => {'select_field' => 'hello'}).should == nth(0, 3)
+    end
+
+    it 'should return searched records without conditioned' do
+      filter(:search => 'hel', :conditions => {'select_field' => 'world'}).should == nth(1)
+    end
+
+  end
+
+  context 'select' do
+
+    before do
+      select_fields = %w(hello world hello hello world)
+      boolean_fields = %w(true false false true false)
+      5.times { |i|
+        Fabricate fabric, :select_field => select_fields[i], :boolean_field => boolean_fields[i]
+      }
+    end
+
+    it 'should return all conditioned records' do
+      filter(:conditions => {'select_field' => 'hello'}).should == nth(0, 2, 3)
+    end
+
+    it 'should return all conditioned records' do
+      filter(:conditions => {'boolean_field' => false}).should == nth(1, 2, 4)
+    end
+
+    it 'should stack conditions with end' do
+      filter(:conditions => {'select_field' => 'hello', 'boolean_field' => true}).should == nth(0, 3)
+    end
+
+  end
+
+  context 'order' do
+
+    before do
+      integer_fields = %w(2, 3, 5, 4, 1)
+      5.times { |i|
+        Fabricate fabric, :integer_field => integer_fields[i]
+      }
+    end
+
+    it 'should order records asc' do
+      filter(:order => 'integer_field').should == nth(4, 0, 1, 3, 2)
+    end
+
+    it 'should order records asc' do
+      filter(:order => ['integer_field', :asc]).should == nth(4, 0, 1, 3, 2)
+    end
+
+    it 'should order records desc' do
+      filter(:order => ['integer_field', :desc]).should == nth(2, 3, 1, 0, 4)
+    end
+
+  end
+
+end

data/spec/lib/orm_adapter/mongoid_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+require 'lib/orm_adapter/base_shared'
+
+describe "MongoidOrm" do
+
+  describe 'columns_hash' do
+    it 'should be proper' do
+      MongoidOrm::Primal.to_adapter.columns_hash.should == {
+        "_type" => {:type => :string},
+        "_id" => {:type => :"bson/object_id"},
+        "string_field" => {:type => :string},
+        "symbol_field" => {:type => :symbol},
+        "select_field" => {:type => :string},
+        "integer_field" => {:type => :integer},
+        "float_field" => {:type => :float},
+        "decimal_field" => {:type => :big_decimal},
+        "datetime_field" => {:type => :date_time},
+        "time_field" => {:type => :time},
+        "date_field" => {:type => :date},
+        "boolean_field" => {:type => :boolean},
+        "array_field" => {:type => :array},
+        "hash_field" => {:type => :hash},
+        "set_field" => {:type => :set},
+        "range_field" => {:type => :range}
+      }
+    end
+  end
+
+  describe 'filter' do
+    it_should_behave_like "an adapter" do
+      def filter options = {}
+        model.to_adapter.filter(model, controller.index_fields, options).order([:_id, :asc]).all.to_a
+      end
+
+
+      def nth *indexes
+        model.where.order([:_id, :asc]).all.to_a.values_at *indexes
+      end
+
+      let(:model) {MongoidOrm::Primal}
+      let(:fabric) {:mongoid_orm_primal}
+      let(:controller) {Orms::MongoidOrmPrimalsController}
+    end
+  end
+
+end

data/spec/lib/resource/routing_spec.rb

@@ -9,7 +9,7 @@ describe Puffer::Resource do
     @mock_category = mock_model Category, :id => 42
 
     get admin_category_path(@category)
-    resource = Puffer::Resource.new request.params.merge(:controller => 'admin/categories', :action => 'show'), controller
+    resource = controller.resource
 
     resource.collection_path.should == admin_categories_path
     resource.member_path.should == admin_category_path(@category)
@@ -24,7 +24,7 @@ describe Puffer::Resource do
     @mock_news = mock_model News, :id => 42
 
     get admin_news_path(@news)
-    resource = Puffer::Resource.new request.params.merge(:controller => 'admin/news', :action => 'show'), controller
+    resource = controller.resource
 
     resource.collection_path.should == admin_news_index_path
     resource.member_path.should == admin_news_path(@news)
@@ -40,7 +40,7 @@ describe Puffer::Resource do
     @mock_category = mock_model Category, :id => 42
 
     get admin_post_category_path(@post, @category)
-    resource = Puffer::Resource.new request.params.merge(:controller => 'admin/categories', :action => 'show'), controller
+    resource = controller.resource
 
     resource.collection_path.should == admin_post_categories_path(@post)
     resource.member_path.should == admin_post_category_path(@post, @category)
@@ -55,7 +55,7 @@ describe Puffer::Resource do
     @profile = @user.profile
 
     get admin_user_profile_path(@user)
-    resource = Puffer::Resource.new request.params.merge(:controller => 'admin/profiles', :action => 'show'), controller
+    resource = controller.resource
 
     resource.collection_path.should == admin_user_profile_path(@user)
     resource.member_path.should == admin_user_profile_path(@user)
@@ -69,7 +69,7 @@ describe Puffer::Resource do
     @mock_tag = mock_model Tag, :id => 42
 
     get admin_user_profile_tag_path(@user, @tag)
-    resource = Puffer::Resource.new request.params.merge(:controller => 'admin/tags', :action => 'show'), controller
+    resource = controller.resource
 
     resource.collection_path.should == admin_user_profile_tags_path(@user)
     resource.member_path.should == admin_user_profile_tag_path(@user, @tag)

data/spec/lib/resource/tree_spec.rb

@@ -0,0 +1,31 @@
+require 'spec_helper'
+
+describe "Tree" do
+
+  it "should generate proper tree" do
+    Rails.application.routes.resources_tree.map(&:to_struct).should == [
+      {:scope => :admin, :current => :users, :children => [:profile, :posts], :ancestors => []},
+      {:scope => :admin, :current => :profile, :children => [:tags], :ancestors => [:users]},
+      {:scope => :admin, :current => :tags, :children => [], :ancestors => [:users, :profile]},
+      {:scope => :admin, :current => :posts, :children => [:categories, :tags], :ancestors => [:users]},
+      {:scope => :admin, :current => :categories, :children => [], :ancestors => [:users, :posts]},
+      {:scope => :admin, :current => :tags, :children => [], :ancestors => [:users, :posts]},
+      {:scope => :admin, :current => :profiles, :children => [:tags], :ancestors => []},
+      {:scope => :admin, :current => :tags, :children => [], :ancestors => [:profiles]},
+      {:scope => :admin, :current => :posts, :children => [:user, :categories], :ancestors => []},
+      {:scope => :admin, :current => :user, :children => [], :ancestors => [:posts]},
+      {:scope => :admin, :current => :categories, :children => [], :ancestors => [:posts]},
+      {:scope => :admin, :current => :tagged_posts, :children => [:user, :categories, :tags], :ancestors => []},
+      {:scope => :admin, :current => :user, :children => [], :ancestors => [:tagged_posts]},
+      {:scope => :admin, :current => :categories, :children => [], :ancestors => [:tagged_posts]},
+      {:scope => :admin, :current => :tags, :children => [], :ancestors => [:tagged_posts]},
+      {:scope => :admin, :current => :categories, :children => [:posts], :ancestors => []},
+      {:scope => :admin, :current => :posts, :children => [], :ancestors => [:categories]},
+      {:scope => :admin, :current => :news, :children => [], :ancestors => []},
+      {:scope => :admin, :current => :puffer_users, :children => [], :ancestors => []},
+      {:scope => :orms, :current => :active_record_orm_primals, :children => [], :ancestors => []},
+      {:scope => :orms, :current => :mongoid_orm_primals, :children => [], :ancestors => []},
+      {:scope => :admin, :current => :session, :children => [], :ancestors => []}
+    ]
+  end
+end

data/spec/lib/resource_spec.rb

@@ -2,22 +2,20 @@ require 'spec_helper'
 
 describe Puffer::Resource do
 
-  def default_params
-    ActiveSupport::HashWithIndifferentAccess.new(:controller => 'admin/categories', :action => 'index', :plural => true, :ancestors => [], :children => [])
-  end
-
-  def plain_params
-    ActiveSupport::HashWithIndifferentAccess.new(:plural => true, :ancestors => [], :children => [])
+  def tree_node id, params = {}
+    ActiveSupport::HashWithIndifferentAccess.new(:puffer => Rails.application.routes.resources_tree[id]).merge(params)
   end
 
   it "no tree" do
-    resource = Puffer::Resource.new default_params
+    resource = Puffer::Resource.new tree_node(17)
+
     resource.parent.should be_nil
     resource.children.should be_empty
   end
 
   it "full tree" do
-    resource = Puffer::Resource.new default_params.merge(:ancestors => [:posts], :children => [:users, :posts])
+    resource = Puffer::Resource.new tree_node(3)
+
     resource.parent.should_not be_nil
     resource.ancestors.count.should == 1
     resource.children.count.should == 2
@@ -26,11 +24,10 @@ describe Puffer::Resource do
   describe "#parent" do
 
     it "common params" do
-      resource = Puffer::Resource.new default_params.merge(:ancestors => [:users, :posts], :user_id => 37, :post_id => 42)
-      parent = resource.parent
+      resource = Puffer::Resource.new tree_node(4, :user_id => 37, :post_id => 42)
 
-      parent.params.should == plain_params.merge(:ancestors => [:users], :user_id => 37, :id => 42)
-      parent.controller_name.should == 'posts'
+      resource.parent.params.should == tree_node(3, :user_id => 37, :id => 42)
+      resource.parent.name.should == :posts
     end
 
   end
@@ -38,31 +35,32 @@ describe Puffer::Resource do
   describe "#children" do
 
     it "common params" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/posts', :ancestors => [:users], :children => [:categories], :user_id => 37)
+      resource = Puffer::Resource.new tree_node(3, :user_id => 37)
       child = resource.children.first
 
-      child.params.should == plain_params.merge(:ancestors => [:users, :posts], :user_id => 37)
-      child.controller_name.should == 'categories'
+      child.params.should == tree_node(4, :user_id => 37)
+      child.name.should == :categories
     end
 
     it "plural params" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/users', :children => [:profile])
+      resource = Puffer::Resource.new tree_node(0)
       child = resource.children.first
 
-      child.params.should == plain_params.merge(:ancestors => [:users], :plural => false)
-      child.controller_name.should == 'profiles'
+      child.params.should == tree_node(1)
+      child.name.should == :profile
     end
 
     it "params with :id" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/posts', :ancestors => [:users], :children => [:categories], :user_id => 37, :id => 42)
+      resource = Puffer::Resource.new tree_node(3, :user_id => 37, :id => 42)
       child = resource.children.first
 
-      child.params.should == plain_params.merge(:ancestors => [:users, :posts], :user_id => 37, :post_id => 42)
-      child.controller_name.should == 'categories'
+      child.params.should == tree_node(4, :user_id => 37, :post_id => 42)
+      child.name.should == :categories
     end
 
   end
-
+  
+=begin
   describe "#collection" do
 
     before do
@@ -71,21 +69,22 @@ describe Puffer::Resource do
     end
 
     it "no parent" do
-      resource = Puffer::Resource.new default_params
+      resource = Puffer::Resource.new tree_node(15)
       resource.collection.should == Category.limit(25).all
     end
 
     it "plural parent" do
-      resource = Puffer::Resource.new default_params.merge(:ancestors => [:posts], :post_id => @post.id)
+      resource = Puffer::Resource.new tree_node(10, :post_id => @post.id)
       resource.collection.should == @post.categories.limit(25).all
     end
 
     it "singular parent" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/tags', :ancestors => [:users, :profile], :user_id => @user.id)
+      resource = Puffer::Resource.new tree_node(2, :user_id => @user.id)
       resource.collection.should == @user.profile.tags.limit(25).all
     end
 
   end
+=end
 
   describe "#member" do
 
@@ -96,22 +95,22 @@ describe Puffer::Resource do
     end
 
     it "no parent" do
-      resource = Puffer::Resource.new default_params.merge(:id => @category.id)
+      resource = Puffer::Resource.new tree_node(15, :id => @category.id)
       resource.member.should == @category
     end
 
     it "plural parent" do
-      resource = Puffer::Resource.new default_params.merge(:ancestors => [:posts], :post_id => @post.id, :id => @post.categories.first.id)
+      resource = Puffer::Resource.new tree_node(10, :post_id => @post.id, :id => @post.categories.first.id)
       resource.member.should == @post.categories.first
     end
 
     it "singular" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/profiles', :plural => false, :ancestors => [:users], :user_id => @user.id)
+      resource = Puffer::Resource.new tree_node(1, :user_id => @user.id)
       resource.member.should == @user.profile
     end
 
     it "singular parent" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/tags', :ancestors => [:users, :profile], :user_id => @user.id, :id => @user.profile.tags.first.id)
+      resource = Puffer::Resource.new tree_node(2, :user_id => @user.id, :id => @user.profile.tags.first.id)
       resource.member.should == @user.profile.tags.first
     end
 
@@ -125,52 +124,52 @@ describe Puffer::Resource do
     end
 
     it "no parent" do
-      resource = Puffer::Resource.new default_params
+      resource = Puffer::Resource.new tree_node(15)
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Category)
     end
 
     it "no parent with attributes" do
-      resource = Puffer::Resource.new default_params.merge(:category => {:title => 'my new title'})
+      resource = Puffer::Resource.new tree_node(15, :category => {:title => 'my new title'})
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Category)
       resource.new_member.title.should == 'my new title'
     end
 
     it "plural parent" do
-      resource = Puffer::Resource.new default_params.merge(:ancestors => [:posts], :post_id => @post.id)
+      resource = Puffer::Resource.new tree_node(10, :post_id => @post.id)
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Category)
     end
 
     it "plural parent with attributes" do
-      resource = Puffer::Resource.new default_params.merge(:ancestors => [:posts], :post_id => @post.id, :category => {:title => 'my new title'})
+      resource = Puffer::Resource.new tree_node(10, :post_id => @post.id, :category => {:title => 'my new title'})
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Category)
       resource.new_member.title.should == 'my new title'
     end
 
     it "singular" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/profiles', :plural => false, :ancestors => [:users], :user_id => @user.id)
+      resource = Puffer::Resource.new tree_node(1, :user_id => @user.id)
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Profile)
     end
 
     it "singular with attributes" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/profiles', :plural => false, :ancestors => [:users], :user_id => @user.id, :profile => {:name => 'my new name'})
+      resource = Puffer::Resource.new tree_node(1, :user_id => @user.id, :profile => {:name => 'my new name'})
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Profile)
       resource.new_member.name.should == 'my new name'
     end
 
     it "singular parent" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/tags', :ancestors => [:users, :profile], :user_id => @user.id)
+      resource = Puffer::Resource.new tree_node(2, :user_id => @user.id)
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Tag)
     end
 
     it "singular parent with attributes" do
-      resource = Puffer::Resource.new default_params.merge(:controller => 'admin/tags', :ancestors => [:users, :profile], :user_id => @user.id, :tag => {:name => 'my new name'})
+      resource = Puffer::Resource.new tree_node(2, :user_id => @user.id, :tag => {:name => 'my new name'})
       resource.new_member.should be_new_record
       resource.new_member.should be_instance_of(Tag)
       resource.new_member.name.should == 'my new name'