alchemy_cms 2.2.4 → 2.3.rc5

data/spec/controllers/base_controller_spec.rb

@@ -1,61 +1,63 @@
 require 'spec_helper'
 
-describe Alchemy::BaseController do
+module Alchemy
+  describe BaseController do
 
-  describe "#set_language_from" do
+    let(:default_language) { Language.get_default }
+    let(:klingonian) { FactoryGirl.create(:klingonian) }
 
-    it "should set the language from id" do
-      @language = Alchemy::Language.get_default
-      controller.send :set_language_from, @language.id
-      controller.session[:language_id].should == @language.id
-    end
+    describe "#set_language_from" do
 
-    it "should set the language from code" do
-      @language = FactoryGirl.create(:language, :code => 'kl')
-      controller.send :set_language_from, "kl"
-      controller.session[:language_id].should == @language.id
-      controller.session[:language_code].should == @language.code
-    end
+      it "should set the language from id" do
+        controller.send :set_language_from, default_language.id
+        controller.session[:language_id].should == default_language.id
+      end
+
+      it "should set the language from code" do
+        controller.send :set_language_from, klingonian.code
+        controller.session[:language_id].should == klingonian.id
+        controller.session[:language_code].should == klingonian.code
+      end
+
+      it "should set the language from id as string" do
+        controller.send :set_language_from, default_language.id.to_s
+        controller.session[:language_id].should == default_language.id
+        controller.session[:language_code].should == default_language.code
+      end
 
-    it "should set the language from id as string" do
-      @language = FactoryGirl.create(:language)
-      controller.send :set_language_from, @language.id.to_s
-      controller.session[:language_id].should == @language.id
-      controller.session[:language_code].should == @language.code
     end
 
-  end
+    describe "#set_language" do
 
-  describe "#set_language" do
+      context "with no lang param" do
 
-    context "with no lang param" do
+        it "should set the default language" do
+          controller.stub!(:params).and_return({})
+          controller.send :set_language
+          controller.session[:language_id].should == default_language.id
+          controller.session[:language_code].should == default_language.code
+        end
 
-      it "should set the default language" do
-        controller.stub!(:params).and_return({})
-        controller.send :set_language
-        controller.session[:language_id].should == Alchemy::Language.get_default.id
-        controller.session[:language_code].should == Alchemy::Language.get_default.code
       end
 
-    end
+      context "with lang param" do
 
-    context "with lang param" do
+        it "should set the language" do
+          controller.stub!(:params).and_return({:lang => klingonian.code})
+          controller.send :set_language
+          controller.session[:language_id].should == klingonian.id
+          controller.session[:language_code].should == klingonian.code
+        end
 
-      it "should set the language" do
-        @language = FactoryGirl.create(:language)
-        controller.stub!(:params).and_return({:lang => 'kl'})
-        controller.send :set_language
-        controller.session[:language_id].should == @language.id
-        controller.session[:language_code].should == @language.code
-      end
+        context "for language that does not exist" do
 
-      context "for language that does not exist" do
+          it "should set the language to default" do
+            controller.stub!(:params).and_return({:lang => 'fo'})
+            controller.send :set_language
+            controller.session[:language_id].should == default_language.id
+            controller.session[:language_code].should == default_language.code
+          end
 
-        it "should set the language to default" do
-          controller.stub!(:params).and_return({:lang => 'fo'})
-          controller.send :set_language
-          controller.session[:language_id].should == Alchemy::Language.get_default.id
-          controller.session[:language_code].should == Alchemy::Language.get_default.code
         end
 
       end
@@ -63,5 +65,4 @@ describe Alchemy::BaseController do
     end
 
   end
-
 end

data/spec/controllers/elements_controller_spec.rb

@@ -0,0 +1,30 @@
+require 'spec_helper'
+
+module Alchemy
+  describe ElementsController do
+
+    let(:page)       { FactoryGirl.create(:public_page, :restricted => true) }
+    let(:element)    { FactoryGirl.create(:element, :page => page, :name => 'download') }
+
+    it "should not be possible to see restricted elements" do
+      get :show, :id => element.id
+      response.status.should == 302
+      response.should redirect_to(login_path)
+    end
+
+    context "as a registered user" do
+
+      before do
+        activate_authlogic
+        UserSession.create(FactoryGirl.create(:registered_user))
+      end
+
+      it "should be possible to see restricted elements" do
+        get :show, :id => element.id
+        response.status.should == 200
+      end
+
+    end
+
+  end
+end

data/spec/controllers/pages_controller_spec.rb

@@ -12,7 +12,7 @@ describe Alchemy::PagesController do
   context "requested for a page containing a feed" do
 
     before(:each) do
-      @page = FactoryGirl.create(:public_page, :parent_id => @default_language_root.id, :page_layout => 'news', :name => 'News', :language => @default_language)
+      @page = FactoryGirl.create(:public_page, :parent_id => @default_language_root.id, :page_layout => 'news', :name => 'News', :language => @default_language, :do_not_autogenerate => false)
     end
 
     it "should render a rss feed" do
@@ -42,8 +42,25 @@ describe Alchemy::PagesController do
     context "with param layout set to none" do
 
       it "should not render a layout" do
-        get :show, :urlname => :home, :layout => false
-        response.body.should_not have_content('<head>')
+        get :show, :urlname => :home, :layout => 'none'
+        response.body.should_not match /<head>/
+      end
+
+    end
+
+    context "with param layout set to false" do
+
+      it "should not render a layout" do
+        get :show, :urlname => :home, :layout => 'false'
+        response.body.should_not match /<head>/
+      end
+
+    end
+
+    context "with params layout set to not existing layout" do
+
+      it "should raise ActionController::RoutingError" do
+        expect { get :show, :urlname => :home, :layout => 'lkuiuk' }.to raise_error(ActionController::RoutingError)
       end
 
     end
@@ -104,8 +121,8 @@ describe Alchemy::PagesController do
     before(:each) do
       @catalog = FactoryGirl.create(:public_page, :name => "Catalog", :parent_id => @default_language_root.id, :language => @default_language)
       @products = FactoryGirl.create(:public_page, :name => "Products", :parent_id => @catalog.id, :language => @default_language)
-      @product = FactoryGirl.create(:public_page, :name => "Screwdriver", :parent_id => @products.id, :language => @default_language)
-      @product.elements.find_by_name('article').contents.essence_texts.first.essence.update_attribute(:body, 'screwdriver')
+      @product = FactoryGirl.create(:public_page, :name => "Screwdriver", :parent_id => @products.id, :language => @default_language, :do_not_autogenerate => false)
+      @product.elements.find_by_name('article').contents.essence_texts.first.essence.update_column(:body, 'screwdriver')
       controller.stub!(:configuration) { |arg| arg == :url_nesting ? true : false }
     end
 

data/spec/controllers/pictures_controller_spec.rb

@@ -0,0 +1,82 @@
+require 'spec_helper'
+
+# Fixes missing method tempfile error
+class Rack::Test::UploadedFile
+  attr_reader :tempfile
+end
+
+module Alchemy
+  describe PicturesController do
+
+    let(:page)               { FactoryGirl.create(:public_page, :restricted => false) }
+    let(:restricted_page)    { FactoryGirl.create(:public_page, :restricted => true) }
+    let(:element)            { FactoryGirl.create(:element, :page => page, :name => 'bild', :create_contents_after_create => true) }
+    let(:restricted_element) { FactoryGirl.create(:element, :page => restricted_page, :name => 'bild', :create_contents_after_create => true) }
+    let(:picture)            { Picture.create(:image_file => fixture_file_upload(File.expand_path('../../support/image.png', __FILE__), 'image/png')) }
+
+    context "Requesting a picture that is not assigned with any page" do
+      it "should render the picture" do
+        get :show, :id => picture.id, :format => :png
+        response.status.should == 200
+      end
+    end
+
+    context "Requesting a picture that is assigned on restricted and non-restricted pages" do
+
+      before do
+        essence = element.contents.where(:name => 'image').first.essence
+        essence.picture_id = picture.id
+        essence.save
+
+        essence = restricted_element.contents.where(:name => 'image').first.essence
+        essence.picture_id = picture.id
+        essence.save
+      end
+
+      context "as guest user" do
+        it "should render the picture" do
+          get :show, :id => picture.id, :format => :png
+          response.status.should == 200
+        end
+      end
+
+    end
+
+    context "Requesting a picture that is assigned with restricted pages only" do
+
+      before do
+        essence = restricted_element.contents.where(:name => 'image').first.essence
+        essence.picture_id = picture.id
+        essence.save
+      end
+
+      context "as guest user" do
+
+        it "should not render the picture, but redirect to login path" do
+          get :show, :id => picture.id
+          response.status.should == 302
+          response.should redirect_to(login_path)
+        end
+
+      end
+
+      context "as registered user" do
+
+        before do
+          activate_authlogic
+          UserSession.create(FactoryGirl.create(:registered_user))
+        end
+
+        it "should render the picture" do
+          get :show, :id => picture.id, :format => :png
+          response.status.should == 200
+        end
+
+      end
+
+
+    end
+
+
+  end
+end

data/spec/dummy/app/models/event.rb

@@ -1,4 +1,5 @@
 class Event < ActiveRecord::Base
-  attr_accessible :name, :hidden_name, :starts_at, :ends_at, :description, :published, :entrance_fee, :location_id
+  attr_accessible :name, :hidden_name, :starts_at, :ends_at, :description, :published, :entrance_fee, :location_id, :organizer_id
   validates_presence_of :name
+
 end

data/spec/dummy/config/database.yml

@@ -5,13 +5,14 @@ sqlite: &sqlite
 mysql: &mysql
   adapter: mysql2
   username: root
-  password: 
+  password:
   database: alchemy_cms_dummy_<%= Rails.env %>
 
 postgresql: &postgresql
   adapter: postgresql
+  <% if ENV['CI'] %>
   username: postgres
-  password: 
+  <% end %>
   database: alchemy_cms_dummy_<%= Rails.env %>
   min_messages: ERROR
 

data/spec/dummy/db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended to check this file into your version control system.
 
-ActiveRecord::Schema.define(:version => 20120611221734) do
+ActiveRecord::Schema.define(:version => 20120831135441) do
 
   create_table "alchemy_attachments", :force => true do |t|
     t.string   "name"
@@ -24,14 +24,6 @@ ActiveRecord::Schema.define(:version => 20120611221734) do
     t.datetime "updated_at",   :null => false
   end
 
-  create_table "alchemy_essence_booleans", :force => true do |t|
-    t.boolean "value"
-    t.timestamps
-    t.userstamps
-  end
-
-  add_index "alchemy_essence_booleans", ["value"], :name => "index_alchemy_essence_booleans_on_value"
-
   create_table "alchemy_cells", :force => true do |t|
     t.integer  "page_id"
     t.string   "name"
@@ -86,6 +78,16 @@ ActiveRecord::Schema.define(:version => 20120611221734) do
     t.datetime "updated_at",                        :null => false
   end
 
+  create_table "alchemy_essence_booleans", :force => true do |t|
+    t.boolean  "value"
+    t.datetime "created_at", :null => false
+    t.datetime "updated_at", :null => false
+    t.integer  "creator_id"
+    t.integer  "updater_id"
+  end
+
+  add_index "alchemy_essence_booleans", ["value"], :name => "index_alchemy_essence_booleans_on_value"
+
   create_table "alchemy_essence_dates", :force => true do |t|
     t.datetime "date"
     t.integer  "creator_id"
@@ -154,9 +156,11 @@ ActiveRecord::Schema.define(:version => 20120611221734) do
   end
 
   create_table "alchemy_essence_selects", :force => true do |t|
-    t.string "value"
-    t.timestamps
-    t.userstamps
+    t.string   "value"
+    t.datetime "created_at", :null => false
+    t.datetime "updated_at", :null => false
+    t.integer  "creator_id"
+    t.integer  "updater_id"
   end
 
   add_index "alchemy_essence_selects", ["value"], :name => "index_alchemy_essence_selects_on_value"
@@ -205,7 +209,7 @@ ActiveRecord::Schema.define(:version => 20120611221734) do
     t.integer  "creator_id"
     t.integer  "updater_id"
     t.boolean  "default",        :default => false
-    t.string   "country_code"
+    t.string   "country_code",   :default => "",      :null => false
   end
 
   add_index "alchemy_languages", ["language_code", "country_code"], :name => "index_alchemy_languages_on_language_code_and_country_code"
@@ -224,17 +228,17 @@ ActiveRecord::Schema.define(:version => 20120611221734) do
     t.integer  "rgt"
     t.integer  "parent_id"
     t.integer  "depth"
-    t.boolean  "visible",                         :default => false
-    t.boolean  "public",                          :default => false
-    t.boolean  "locked",                          :default => false
+    t.boolean  "visible",          :default => false
+    t.boolean  "public",           :default => false
+    t.boolean  "locked",           :default => false
     t.integer  "locked_by"
-    t.boolean  "restricted",                      :default => false
-    t.boolean  "robot_index",                     :default => true
-    t.boolean  "robot_follow",                    :default => true
-    t.boolean  "sitemap",                         :default => true
-    t.boolean  "layoutpage",                      :default => false
-    t.datetime "created_at",                                         :null => false
-    t.datetime "updated_at",                                         :null => false
+    t.boolean  "restricted",       :default => false
+    t.boolean  "robot_index",      :default => true
+    t.boolean  "robot_follow",     :default => true
+    t.boolean  "sitemap",          :default => true
+    t.boolean  "layoutpage",       :default => false
+    t.datetime "created_at",                          :null => false
+    t.datetime "updated_at",                          :null => false
     t.integer  "creator_id"
     t.integer  "updater_id"
     t.integer  "language_id"
@@ -249,10 +253,12 @@ ActiveRecord::Schema.define(:version => 20120611221734) do
     t.string   "image_filename"
     t.integer  "image_width"
     t.integer  "image_height"
-    t.datetime "created_at",     :null => false
-    t.datetime "updated_at",     :null => false
+    t.datetime "created_at",      :null => false
+    t.datetime "updated_at",      :null => false
     t.integer  "creator_id"
     t.integer  "updater_id"
+    t.string   "upload_hash"
+    t.string   "cached_tag_list"
   end
 
   create_table "alchemy_users", :force => true do |t|
@@ -292,8 +298,26 @@ ActiveRecord::Schema.define(:version => 20120611221734) do
     t.decimal  "entrance_fee", :precision => 6, :scale => 2
     t.boolean  "published"
     t.integer  "location_id"
-    t.datetime "created_at",   :null => false
-    t.datetime "updated_at",   :null => false
+    t.integer  "organizer_id"
+    t.datetime "created_at",                                 :null => false
+    t.datetime "updated_at",                                 :null => false
+  end
+
+  create_table "taggings", :force => true do |t|
+    t.integer  "tag_id"
+    t.integer  "taggable_id"
+    t.string   "taggable_type"
+    t.integer  "tagger_id"
+    t.string   "tagger_type"
+    t.string   "context"
+    t.datetime "created_at"
+  end
+
+  add_index "taggings", ["tag_id"], :name => "index_taggings_on_tag_id"
+  add_index "taggings", ["taggable_id", "taggable_type", "context"], :name => "index_taggings_on_taggable_id_and_taggable_type_and_context"
+
+  create_table "tags", :force => true do |t|
+    t.string "name"
   end
 
 end

data/spec/factories.rb

@@ -25,27 +25,40 @@ FactoryGirl.define do
   end
 
   factory :language, :class => 'Alchemy::Language' do
-    language_code "kl"
-    name 'Klingonian'
-    default false
-    frontpage_name 'Tuq'
+    name 'Deutsch'
+    code 'de'
+    default true
+    frontpage_name 'Intro'
     page_layout 'intro'
     public true
 
-    factory :language_with_country_code do
-      country_code 'cr'
+    factory :klingonian do
+      name 'Klingonian'
+      code 'kl'
+      frontpage_name 'Tuq'
+      default false
+    end
+
+    factory :english do
+      name 'English'
+      code 'en'
+      frontpage_name 'Intro'
+      default false
     end
   end
 
   factory :page, :class => 'Alchemy::Page' do
 
-    language { Alchemy::Language.find_by_language_code('kl') || FactoryGirl.create(:language) }
+    language { Alchemy::Language.get_default || FactoryGirl.create(:language) }
     sequence(:name) { |n| "A Page #{n}" }
     parent_id { (Alchemy::Page.find_by_language_root(true) || FactoryGirl.create(:language_root_page)).id }
     page_layout "standard"
 
+    # This speeds up creating of pages dramatically. Pass :do_not_autogenerate => false to generate elements
+    do_not_autogenerate true
+
     factory :language_root_page do
-      name 'Klingonian'
+      name 'Startseite'
       page_layout 'intro'
       language_root true
       public true
@@ -74,6 +87,14 @@ FactoryGirl.define do
 
   factory :element, :class => 'Alchemy::Element' do
     name 'article'
+    create_contents_after_create false
+  end
+
+  factory :picture, :class => 'Alchemy::Picture' do
+    image_file File.new(File.expand_path('../support/image.png', __FILE__))
+    name 'image'
+    image_filename 'image.png'
+    upload_hash Time.now.hash
   end
 
 end