kuhsaft 0.0.6 → 0.0.7

data/spec/dummy/config/database.yml

@@ -12,8 +12,8 @@ development:
 test:
   adapter: sqlite3
   database: db/test.sqlite3
-  pool: 5
-  timeout: 5000
+  pool: 50
+  timeout: 25000
 
 production:
   adapter: sqlite3

data/spec/dummy/config/locales/en.yml

@@ -2,4 +2,6 @@
 # See http://github.com/svenfuchs/rails-i18n/tree/master/rails%2Flocale for starting points.
 
 en:
-  hello: "Hello world"
+  unpublished: "unpublished"
+  published: "published"
+  published_at: "publish on date"

data/spec/factories.rb

@@ -1,9 +1,3 @@
-Factory.define :page_part, :class => Kuhsaft::PagePart::Content do |p|
-  p.position 1
-  p.content Kuhsaft::PagePart::Markdown.new(:text => 'h1. Hello world!')
-  p.association :localized_page
-end
-
 Factory.define :localized_page, :class => Kuhsaft::LocalizedPage do |p|
   p.locale 'en'
   p.title 'English title'
@@ -11,9 +5,6 @@ Factory.define :localized_page, :class => Kuhsaft::LocalizedPage do |p|
   p.body 'hi'
   p.url ''
   p.association :page
-  p.after_create do |page|
-    page.page_parts << Factory.build(:page_part, :localized_page => page)
-  end
 end
 
 Factory.define :page, :class => Kuhsaft::Page do |p|

data/spec/models/localized_page_spec.rb

@@ -1,9 +1,8 @@
 require 'spec_helper'
 
 describe Kuhsaft::LocalizedPage do
-  
   before :each do
-    @localized_page = Factory.build :localized_page
+    @localized_page = Factory.create(:localized_page)
   end
   
   it 'should have a symbolized locale' do
@@ -11,7 +10,11 @@ describe Kuhsaft::LocalizedPage do
   end
   
   it 'should be published' do
-    Factory.create(:localized_page).published?.should be_true
+    @localized_page.published?.should be_true
+  end
+  
+  it 'should have a fulltext field' do
+    @localized_page.should respond_to(:fulltext)
   end
   
   it 'should not be published when set to false' do
@@ -34,35 +37,120 @@ describe Kuhsaft::LocalizedPage do
   end
   
   it 'should generate the slug from the title' do
-    Factory.create(:localized_page).slug.should == @localized_page.title.parameterize
+    @localized_page.slug.should eq(@localized_page.title.parameterize)
+  end
+  
+  it 'should have page_parts' do
+    @localized_page.should respond_to(:page_parts)
   end
   
   it 'should not generate the slug if the user has set it' do
     Factory.create(:localized_page, :slug => 'my-slug').slug.should == 'my-slug'
   end
   
-  it 'should have pageparts' do
-    Factory.create(:localized_page).page_parts.count.should be(1)
+  after :each do
+    @localized_page.destroy
   end
   
-  it 'should delegate childs to it\'s page' do
-    @localized_page.childs.should == @localized_page.page.childs
+  describe 'page_type' do
+    it 'should have a page_type' do
+      @localized_page.should respond_to(:page_type)
+    end
+    
+    it 'should generate the url with an empty page_type' do
+      page = Factory.create :page
+      page.translation.url.should eq('en/english-title')
+    end
+    
+    it 'should take the users url with a "redirect" page_type' do
+      page = Factory.create :page
+      page.translation.page_type = 'redirect'
+      page.translation.url = '/en/news'
+      page.save
+      page.translation.url.should eq('/en/news')
+    end
   end
   
   describe 'validations' do
-    
-    before :each do
-      @page = Factory.create :page
-      @page.title = nil
-      @page.slug = nil
+    before do
+      @localized_page = Factory.create :localized_page
+      @localized_page.title = nil
+      @localized_page.slug = nil
     end
     
     it 'should have a title' do
-      @page.translation.should have(1).error_on(:title)
+      @localized_page.should have(1).error_on(:title)
     end
     
     it 'should have a slug' do
-      @page.translation.should have(1).error_on(:slug)
+      @localized_page.should have(1).error_on(:slug)
+    end
+    
+    after do
+      @localized_page.destroy
+    end
+  end
+  
+  describe 'fulltext' do
+    before do
+      @page =  Factory.create :page
+      @page.translation.keywords = 'key words'
+      @page.translation.description = 'descrip tion'
+      @page.translation.title = 'my title'
+      @page.translation.page_parts << Kuhsaft::PagePart::Markdown.new(:text => 'oh la la1')
+      @page.translation.page_parts << Kuhsaft::PagePart::Markdown.new(:text => 'oh la la2')
+      @page.translation.page_parts << Kuhsaft::PagePart::Markdown.new(:text => 'oh la la3')
+      @page.translation.page_parts << Kuhsaft::PagePart::Markdown.new(:text => nil)
+      @page.save
+    end
+    
+    it 'should collect the fulltext when saved' do
+      @page.translation.should_receive(:collect_fulltext)
+      @page.save
+    end
+    
+    it 'should contain the title' do
+      @page.fulltext.should include('my title')
+    end
+    
+    it 'should contain the keywords' do
+      @page.fulltext.should include('key words')
+    end
+    
+    it 'should contain the description' do
+      @page.fulltext.should include('descrip tion')
+    end
+    
+    it 'should contain the page part content' do
+      @page.fulltext.should include('oh la la')
+    end
+    
+    it 'should convert all data to strings' do
+      expect { @page.translation.collect_fulltext }.to_not raise_error
+    end
+  end
+  
+  describe 'search' do
+    before do
+      Factory.create :page
+      Factory.create :page
+      Factory.create :page
+    end
+    
+    it 'should find any containing the search term' do
+      Kuhsaft::LocalizedPage.search('hi').should have_at_least(0).items
+    end
+    
+    it 'should find with "English Title"' do
+      Kuhsaft::LocalizedPage.search('English Title').should have_at_least(1).item
+    end
+    
+    it 'should only find results with the current locale' do
+      Kuhsaft::LocalizedPage.search('English Title').should be_all { |p| p.locale == Kuhsaft::Page.current_translation_locale }
+    end
+    
+    it 'should only find published results' do
+      Kuhsaft::LocalizedPage.search('English Title').should be_all { |p| p.published? == true }
     end
   end
 end

data/spec/models/page_part_content_spec.rb

@@ -1,22 +1,64 @@
 require 'spec_helper'
 
-describe Kuhsaft::PagePart::Content do
-  
-  before :each do 
-    @page_part = Factory.build :page_part
-  end
-  
-  it 'should serialize it\'s content when saved' do
-    @page_part.content.should_receive(:to_yaml)
-    @page_part.save
-  end
-  
-  it 'should deserialize it\'s content when loaded' do
-    @page_part.save
-    @page_part.content.should be_a(Kuhsaft::PagePart::Base)
+describe 'PagePart' do
+  describe 'Content' do
+    before do
+      @content = Kuhsaft::PagePart::Content.new
+    end
+    
+    it 'should belong to a LocalizedPage' do
+      @content.should respond_to(:localized_page)
+    end
+    
+    it 'should have a content to store serialized data' do
+      @content.should respond_to(:content)
+    end
+    
+    context 'class' do
+      it 'should keep a list of the serializeable attributes' do
+        Kuhsaft::PagePart::Content.serializeable_attributes.should be_a(Array)
+      end
+      
+      it 'should keep a list of searchable attributes' do
+        Kuhsaft::PagePart::Content.searchable_attributes.should be_a(Array)
+      end
+      
+      it 'should have a list of page_part_types' do
+        Kuhsaft::PagePart::Content.page_part_types.should be_all { |p| p.superclass.should eq Kuhsaft::PagePart::Content }
+      end
+      
+      it 'should have the Markdown PagePart by default' do
+        Kuhsaft::PagePart::Content.descendants.should include(Kuhsaft::PagePart::Markdown)
+      end
+      
+      it 'should convert to_name' do
+        Kuhsaft::PagePart::Markdown.to_name.should eq('Markdown')
+      end
+    end
   end
   
-  it 'should have a text in the default markdown PagePart' do
-    @page_part.content.text.should == 'h1. Hello world!'
+  describe 'Markdown' do
+    before do
+      @m = Kuhsaft::PagePart::Markdown.new
+    end
+    
+    it 'should have text' do
+      @m.should respond_to(:text)
+    end
+    
+    it 'should have searchable text' do
+      Kuhsaft::PagePart::Markdown.searchable_attributes.should include(:text)
+    end
+    
+    it 'should store text' do
+      @m.text = 'hi'
+      @m.text.should eq('hi')
+    end
+    
+    it 'should restore the serialized attributes when loaded' do
+      m = Kuhsaft::PagePart::Markdown.create(:text => 'hi')
+      m2 = Kuhsaft::PagePart::Markdown.find(m.id)
+      m2.text.should eq('hi')
+    end
   end
-end
+end

data/spec/models/page_part_spec.rb

@@ -14,4 +14,4 @@ describe Kuhsaft::PagePart do
       lambda { Kuhsaft::PagePart.all.each { |p| p.to_s.camelize.constantize } }.should_not raise_error NameError
     end
   end
-end
+end

data/spec/models/page_spec.rb

@@ -2,6 +2,14 @@ require 'spec_helper'
 
 describe Kuhsaft::Page do
   
+  before do
+    set_lang :en
+  end
+  
+  after do
+    reset_lang
+  end
+  
   before :each do 
     destroy_all_pages
     @page = Factory.create :page
@@ -111,11 +119,19 @@ describe Kuhsaft::Page do
     page = Factory.create :page
     child = Factory.create :page
     page.childs << child
-    page.body = nil
+    page.translation.body = nil
     page.save
     page.link.should == child.link
   end
   
+  it 'should not modify the url when it\'s a redirect' do
+    page = Factory.create :page
+    page.translation.page_type = 'redirect'
+    page.translation.url = '/en/news'
+    page.save
+    page.link.should eq('/en/news')
+  end
+  
   it 'should find its translated content by url' do
     destroy_all_pages
     page = Factory.create(:page)
@@ -140,38 +156,27 @@ describe Kuhsaft::Page do
     Kuhsaft::Page.translation_locales.first.should be(:de)
   end
   
-  describe 'should delegate property' do
-    it 'should have a localized_page to delegate to' do
-      @page.translation.should be_a(Kuhsaft::LocalizedPage)
-    end
-    
-    it 'should delegate the title to the translation' do
-      @page.title = 'Hello'
-      @page.translation.title.should == 'Hello'
-    end
-    
-    it 'should delegate the slug to the translation' do
-      @page.slug = 'my-slug'
-      @page.translation.slug.should == 'my-slug'
-    end
-    
-    it 'should delegate the url to the translation' do
-      @page.url.should == @page.translation.url
-    end
-    
-    it 'should delegate the keywords to the translation' do
-      @page.keywords = 'my keywords are superb'
-      @page.translation.keywords.should == 'my keywords are superb'
+  it 'should have a translation' do
+    @page.translation.should be_a(Kuhsaft::LocalizedPage)
+  end
+  
+  it 'accepts a page_part_type to determine which page_part needs to be added' do
+    @page.should respond_to(:page_part_type)
+  end
+  
+  describe 'should delegate' do
+    it 'slug, title, keywords and description to the translation' do
+      [:slug, :title, :keywords, :description].each do |attr|
+        @page.send(attr).should eq(@page.translation.send(attr))
+      end
     end
     
-    it 'should delegate the description to the translation' do
-      @page.description = 'my description'
-      @page.translation.description.should == 'my description'
+    it 'url to the translation' do
+      @page.url.should eq(@page.translation.url)
     end
     
-    it 'should delegate the locale to the translation' do
-      @page.locale = 'de'
-      @page.translation.locale.should == :de
+    it 'locale to the translation' do
+      @page.locale.should be(@page.translation.locale)
     end
   end
 end

data/spec/models/publish_state_spec.rb

@@ -0,0 +1,45 @@
+require 'spec_helper'
+
+describe Kuhsaft::PublishState do
+  context 'unpublished' do
+    before do
+      @publish_state = Kuhsaft::PublishState.new(:name => 'unpublished', :value => Kuhsaft::PublishState::UNPUBLISHED)
+    end
+    
+    it 'should be UNPUBLISHED' do
+      @publish_state.value.should be(Kuhsaft::PublishState::UNPUBLISHED)
+    end
+    
+    it 'should have a translated name' do
+      @publish_state.human_name.should eq(I18n.translate('unpublished'))
+    end
+  end
+  
+  context 'published' do
+    before do
+      @publish_state = Kuhsaft::PublishState.new(:name => 'published', :value => Kuhsaft::PublishState::PUBLISHED)
+    end
+    
+    it 'should be PUBLISHED' do
+      @publish_state.value.should be(Kuhsaft::PublishState::PUBLISHED)
+    end
+    
+    it 'should have a translated name' do
+      @publish_state.human_name.should eq(I18n.translate('published'))
+    end
+  end
+  
+  context 'published_at' do
+    before do
+      @publish_state = Kuhsaft::PublishState.new(:name => 'published_at', :value => Kuhsaft::PublishState::PUBLISHED_AT)
+    end
+    
+    it 'should be PUBLISHED_AT' do
+      @publish_state.value.should be(Kuhsaft::PublishState::PUBLISHED_AT)
+    end
+    
+    it 'should have a translated name' do
+      @publish_state.human_name.should eq(I18n.translate('published_at'))
+    end
+  end
+end

data/spec/routing/pages_routing_spec.rb

@@ -11,56 +11,63 @@ describe 'routing to Kuhsaft::PagesController' do
 end
 
 describe 'routing to Kuhsaft::Admin::PagesController' do
-  it 'routes /admin/pages to admin/pages#index' do
-    { :get => '/admin/pages' }.should route_to(
+  it 'routes /en/admin/pages to admin/pages#index' do
+    { :get => '/en/admin/pages' }.should route_to(
       :controller => 'kuhsaft/admin/pages',
-      :action => 'index'
+      :action => 'index',
+      :locale => 'en'
     )
   end
 
-  it 'routes POST /admin/pages to admin/pages#create' do
-    { :post => '/admin/pages' }.should route_to(
+  it 'routes POST /en/admin/pages to admin/pages#create' do
+    { :post => '/en/admin/pages' }.should route_to(
       :controller => 'kuhsaft/admin/pages',
-      :action => 'create'
+      :action => 'create',
+      :locale => 'en'
     )
   end
   
-  it 'routes PUT /admin/pages/:id to admin/pages#update' do
-    { :put => '/admin/pages/1' }.should route_to(
+  it 'routes PUT /en/admin/pages/:id to admin/pages#update' do
+    { :put => '/en/admin/pages/1' }.should route_to(
       :controller => 'kuhsaft/admin/pages',
       :action => 'update',
-      :id => '1'
+      :id => '1',
+      :locale => 'en'
     )
   end
   
-  it 'routes DELETE /admin/pages/:id to admin/pages#destroy' do
-    { :delete => '/admin/pages/1' }.should route_to(
+  it 'routes DELETE /en/admin/pages/:id to admin/pages#destroy' do
+    { :delete => '/en/admin/pages/1' }.should route_to(
       :controller => 'kuhsaft/admin/pages',
       :action => 'destroy',
-      :id => '1'
+      :id => '1',
+      :locale => 'en'
     )
   end
   
-  it 'routes /admin/pages/:id to admin/pages#show' do
-    { :get => '/admin/pages/1' }.should route_to(
+  it 'routes /en/admin/pages/:id to admin/pages#show' do
+    { :get => '/en/admin/pages/1' }.should route_to(
       :controller => 'kuhsaft/admin/pages',
       :action => 'show',
-      :id => '1'
+      :id => '1',
+      :locale => 'en'
     )
   end
   
-  it 'routes /admin/pages/new to admin/pages#new' do
-    { :get => '/admin/pages/new' }.should route_to(
+  it 'routes /en/admin/pages/new to admin/pages#new' do
+    { :get => '/en/admin/pages/new' }.should route_to(
       :controller => 'kuhsaft/admin/pages',
-      :action => 'new'
+      :action => 'new',
+      :locale => 'en'
     )  
   end
   
-  it 'routes /admin/pages/:id/edit to admin/pages#edit' do
-    { :get => '/admin/pages/1/edit' }.should route_to(
+  it 'routes /en/admin/pages/:id/edit to admin/pages#edit' do
+    { :get => '/en/admin/pages/1/edit' }.should route_to(
         :controller => 'kuhsaft/admin/pages',
         :action => 'edit',
-        :id => '1'    
+        :id => '1',
+        :locale => 'en'    
     )
   end
 end