spud_cms 0.8.17 → 0.9.0

data/spec/controllers/spud/admin/menu_items_controller_spec.rb

@@ -0,0 +1,148 @@
+require 'spec_helper'
+
+describe Spud::Admin::MenuItemsController do
+  before(:each) do
+    ActiveRecord::Base.observers.disable(:page_sweeper)
+
+    activate_authlogic
+    u = SpudUser.new(:login => "testuser",:email => "test@testuser.com",:password => "test",:password_confirmation => "test")
+    u.super_admin = true
+    u.save
+    @user = SpudUserSession.create(u)
+    Spud::Core.configure do |config|
+        config.site_name = "Test Site"
+        config.multisite_mode_enabled = false
+        config.multisite_config = []
+    end
+
+    @menu = FactoryGirl.create(:spud_menu)
+  end
+
+
+  describe :index do
+    it "should return an array of menus" do
+      2.times {|x|  s = FactoryGirl.create(:spud_menu_item,:spud_menu_id => @menu.id,:parent_id => @menu.id)}
+      get :index,:menu_id =>@menu.id
+
+      assigns(:menu_items).count.should be > 1
+    end
+    it "should redirect if the menu is not found" do
+      get :index,:menu_id => @menu.id + 1
+      response.should redirect_to spud_admin_menus_url
+      flash[:error].should_not be_blank
+    end
+  end
+
+  describe :new do
+    it "should assign a new menu" do
+      get :new,:menu_id =>@menu.id
+      assigns(:menu_item).should_not be_blank
+      response.should be_success
+    end
+  end
+
+  describe :create do
+    it "should create a new menu item with a valid form submission" do
+      lambda {
+        post :create,:menu_id => @menu.id, :spud_menu_item => FactoryGirl.attributes_for(:spud_menu_item,:parent_id => nil).reject{|k,v| k == 'id' || k == :spud_menu_id}
+      }.should change(SpudMenuItem,:count).by(1)
+      response.should be_redirect
+    end
+
+    it "should not create a menu item with an invalid form entry" do
+      yamldata = FactoryGirl.attributes_for(:spud_menu_item,:name => nil).reject{|k,v| k == 'id' || k == :spud_menu_id}
+
+      lambda {
+        post :create,:menu_id => @menu.id, :spud_menu_item => FactoryGirl.attributes_for(:spud_menu_item,:name => nil,:parent_id => nil).reject{|k,v| k == 'id' || k == :spud_menu_id}
+      }.should_not change(SpudMenuItem,:count)
+    end
+
+    it "should autoset order if not specified" do
+      menuitem = FactoryGirl.create(:spud_menu_item,:parent_id => @menu.id)
+
+      post :create,:menu_id => @menu.id, :spud_menu_item => FactoryGirl.attributes_for(:spud_menu_item,:menu_order => nil,:parent_id => nil).reject{|k,v| k == 'id' || k == :spud_menu_id}
+      assigns(:menu_item).menu_order.should == 1
+    end
+
+    it "should assign the menu name based on page name if left blank" do
+      page = FactoryGirl.create(:spud_page)
+      post :create,:menu_id => @menu.id, :spud_menu_item => FactoryGirl.attributes_for(:spud_menu_item,:name => nil,:spud_page_id => page.id).reject{|k,v| k == 'id' || k == :spud_menu_id}
+      assigns(:menu_item).name.should == page.name
+    end
+
+    it "should set the parent to a submenu if specified" do
+      menuitem = FactoryGirl.create(:spud_menu_item,:parent_id => @menu.id)
+      post :create,:menu_id => @menu.id, :spud_menu_item => FactoryGirl.attributes_for(:spud_menu_item,:parent_id => menuitem.id).reject{|k,v| k == 'id' || k == :spud_menu_id}
+      assigns(:menu_item).parent.should == menuitem
+    end
+  end
+
+  describe :edit do
+    it "should respond with menu item by id" do
+      menu1 = FactoryGirl.create(:spud_menu_item)
+      menu2 = FactoryGirl.create(:spud_menu_item)
+      get :edit, :menu_id => @menu.id,:id => menu2.id
+      assigns(:menu_item).should == menu2
+    end
+
+    it "should redirect_to index if menu item not found" do
+      get :edit,:menu_id => @menu.id,:id => "345"
+      response.should redirect_to spud_admin_menu_menu_items_url(:menu_id => @menu.id)
+    end
+  end
+
+  describe :update do
+    it "should update the name when the name attribute is changed" do
+      menu_item = FactoryGirl.create(:spud_menu_item)
+      new_name = 'MyMenu'
+      lambda {
+        put :update,:menu_id => @menu.id,:id => menu_item.id, :spud_menu_item => menu_item.attributes.merge!(:name => new_name,:parent_id => nil).reject{|k,v| k.to_sym == :spud_menu_id || k == 'id' || k.to_sym == :updated_at || k.to_sym == :created_at}
+        menu_item.reload
+      }.should change(menu_item,:name).to(new_name)
+    end
+    it "should assign a menu to a submenu if parentid is defined" do
+      menu_item1 = FactoryGirl.create(:spud_menu_item)
+      menu_item = FactoryGirl.create(:spud_menu_item)
+
+      lambda {
+        put :update,:menu_id => @menu.id,:id => menu_item.id, :spud_menu_item => menu_item.attributes.merge!(:parent_id => menu_item1.id).reject{|k,v| k.to_sym == :spud_menu_id || k == 'id' || k.to_sym == :updated_at || k.to_sym == :created_at}
+        menu_item.reload
+      }.should change(menu_item,:parent).to(menu_item1)
+    end
+  end
+
+
+  describe :destroy do
+    it "should destroy the menu item" do
+      menu_item = FactoryGirl.create(:spud_menu_item,:parent_id => @menu.id)
+      lambda {
+        delete :destroy,:menu_id => @menu.id,:id => menu_item.id
+      }.should change(SpudMenuItem,:count).by(-1)
+    end
+  end
+
+  describe :multisite do
+    before(:each) do
+      Spud::Core.configure do |config|
+        config.site_name = "Test Site"
+        config.multisite_mode_enabled = true
+        config.multisite_config = [{:hosts => ["test.host"], :site_name =>"Site B", :site_id => 1,:short_name => "siteb"}]
+      end
+      Spud::Cms.configure do |config|
+        config.multisite_config = [{:short_name => "siteb",:default_page_parts => ["Body","Sidebar"]}]
+      end
+      session[:admin_site] = 1
+    end
+
+
+      describe :edit do
+        it "should not allow editing of a menu_item in a menu that is different from the current admin site" do
+          menu_item = FactoryGirl.create(:spud_menu_item)
+          get :edit,:id => menu_item.id,:menu_id => @menu.id
+          response.should redirect_to spud_admin_menus_url
+          flash[:warning].should_not be_blank
+        end
+      end
+
+  end
+end

data/spec/controllers/spud/admin/menus_controller_spec.rb

@@ -0,0 +1,121 @@
+require 'spec_helper'
+
+describe Spud::Admin::MenusController do
+  before(:each) do
+    activate_authlogic
+    u = SpudUser.new(:login => "testuser",:email => "test@testuser.com",:password => "test",:password_confirmation => "test")
+    u.super_admin = true
+    u.save
+    @user = SpudUserSession.create(u)
+    Spud::Core.configure do |config|
+        config.site_name = "Test Site"
+        config.multisite_mode_enabled = false
+        config.multisite_config = []
+    end
+  end
+
+
+  describe :index do
+    it "should return an array of menus" do
+      2.times {|x|  s = FactoryGirl.create(:spud_menu)}
+      get :index
+
+      assigns(:menus).count.should be > 1
+    end
+
+  end
+
+  describe :new do
+    it "should response with new menu" do
+      get :new
+      assigns(:menu).should_not be_blank
+      response.should be_success
+    end
+  end
+
+  describe :create do
+    it "should create a new menu with a valid form submission" do
+      lambda {
+        post :create, :spud_menu => FactoryGirl.attributes_for(:spud_menu).reject{|k,v| k == 'site_id' || k == 'id'}
+      }.should change(SpudMenu,:count).by(1)
+      response.should be_redirect
+    end
+
+    it "should not create a menu with an invalid form entry" do
+      lambda {
+        post :create, :spud_menu => FactoryGirl.attributes_for(:spud_menu,:name => nil).reject{|k,v| k == 'site_id' || k == 'id'}
+      }.should_not change(SpudMenu,:count)
+
+
+    end
+  end
+
+  describe :multisite do
+    before(:each) do
+      Spud::Core.configure do |config|
+        config.site_name = "Test Site"
+        config.multisite_mode_enabled = true
+        config.multisite_config = [{:hosts => ["test.host"], :site_name =>"Site B", :site_id => 1}]
+      end
+
+    end
+    it "should not allow editing of a menu on a different site id" do
+      menu2 = FactoryGirl.create(:spud_menu,:site_id => 1)
+      get :edit,:id => menu2.id
+
+      response.should redirect_to spud_admin_menus_url
+    end
+  end
+
+  describe :edit do
+    it "should response with menu by id" do
+      menu1 = FactoryGirl.create(:spud_menu)
+      menu2 = FactoryGirl.create(:spud_menu)
+      get :edit,:id => menu2.id
+      assigns(:menu).should == menu2
+    end
+    it "should redirect to index if menu not found" do
+      get :edit,:id => 3
+      response.should redirect_to spud_admin_menus_url
+    end
+
+  end
+
+  describe :update do
+    it "should update the name when the name attribute is changed" do
+      menu = FactoryGirl.create(:spud_menu)
+      new_name = 'MyMenu'
+      lambda {
+        put :update,:id => menu.id, :spud_menu => menu.attributes.merge!(:name => new_name).reject{|k,v| k == 'site_id' || k == 'id'}
+        menu.reload
+      }.should change(menu,:name).to(new_name)
+
+    end
+
+    it "should redirect to the admin menus after a successful update" do
+      menu = FactoryGirl.create(:spud_menu)
+      put :update,:id => menu.id,:spud_menu => menu.attributes.merge!(:name => "MyMenu").reject{|k,v| k == 'site_id' || k == 'id'}
+
+      response.should redirect_to(spud_admin_menu_menu_items_url(:menu_id => menu.id))
+    end
+  end
+
+  describe :destroy do
+    it "should destroy the menu" do
+      menu = FactoryGirl.create(:spud_menu)
+      lambda {
+        delete :destroy, :id => menu.id
+      }.should change(SpudMenu,:count).by(-1)
+      response.should be_redirect
+    end
+
+    it "should not destroy the menu with a wrong id" do
+      menu = FactoryGirl.create(:spud_menu)
+      lambda {
+        delete :destroy,:id => "23532"
+      }.should_not change(SpudMenu,:count)
+      response.should be_redirect
+    end
+
+  end
+end

data/spec/controllers/spud/admin/pages_controller_spec.rb

@@ -11,7 +11,7 @@ describe Spud::Admin::PagesController do
 
   describe :index do
     it "should return an array of parent pages" do
-      2.times {|x| FactoryGirl.create(:spud_page) }
+      2.times {|x|  s = FactoryGirl.create(:spud_page)}
       get :index
 
       assigns(:pages).count.should be > 1
@@ -34,21 +34,13 @@ describe Spud::Admin::PagesController do
       response.should be_success
     end
 
-    it "should render custom layout if template in use" do
-      t = FactoryGirl.build(:spud_template)
-      t.base_layout = 'content_left'
-      t.save
-      p = FactoryGirl.create(:spud_page,:template_id => t.id)
 
-      get :show ,:id => p.id
-      assigns(:layout).should render_template(:layout => "layouts/#{t.base_layout}")
-    end
 
-    it "should redirect if no id specified" do
-      get :show
+    # it "should redirect if no id specified" do
+    #   get :show,:id => nil
 
-      response.should redirect_to spud_admin_pages_url
-    end
+    #   response.should redirect_to spud_admin_pages_url
+    # end
 
   end
 

data/spec/controllers/spud/cms/sitemaps_controller_spec.rb

@@ -2,16 +2,47 @@ require 'spec_helper'
 
 describe Spud::Cms::SitemapsController do
   describe :show do
+    before(:each) do
+        Spud::Core.configure do |config|
+          config.site_name = "Test Site"
+          config.multisite_mode_enabled = false
+          config.multisite_config = []
+        end
+
+      end
     it "should return the sitemap urls" do
-      
+
       get :show, :format => :xml
-      
+
       assigns(:pages).should == SpudPage.published_pages.public.order(:spud_page_id)
     end
-    
+
     it "should only respond to an XML format" do
       get :show
       response.response_code.should == 406
     end
+
+    describe :multisite do
+      before(:each) do
+        Spud::Core.configure do |config|
+          config.site_name = "Test Site"
+          config.multisite_mode_enabled = true
+          config.multisite_config = [{:hosts => ["test.host"], :site_name =>"Site B", :site_id => 1}]
+        end
+      end
+
+      it "should only assign pages for current site" do
+        2.times {|x|  s = FactoryGirl.create(:spud_page)}
+        3.times {|x|  s = FactoryGirl.create(:spud_page,:site_id => 1)}
+
+        get :show, :format => :xml
+
+        assigns(:pages).should == SpudPage.published_pages.public.order(:spud_page_id).site(1)
+
+      end
+
+    end
   end
-end
+
+
+end

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

@@ -1,3 +1,3 @@
-class ApplicationController < ActionController::Base
+class ApplicationController < Spud::ApplicationController
   protect_from_forgery
 end

data/spec/dummy/db/schema.rb

@@ -0,0 +1,146 @@
+# encoding: UTF-8
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# Note that this schema.rb definition is the authoritative source for your
+# database schema. If you need to create the application database on another
+# system, you should be using db:schema:load, not running all the migrations
+# from scratch. The latter is a flawed and unsustainable approach (the more migrations
+# you'll amass, the slower it'll run and the greater likelihood for issues).
+#
+# It's strongly recommended to check this file into your version control system.
+
+ActiveRecord::Schema.define(:version => 20121112151110) do
+
+  create_table "spud_admin_permissions", :force => true do |t|
+    t.integer  "user_id"
+    t.string   "name"
+    t.boolean  "access"
+    t.datetime "created_at", :null => false
+    t.datetime "updated_at", :null => false
+    t.string   "scope"
+  end
+
+  create_table "spud_menu_items", :force => true do |t|
+    t.string   "parent_type"
+    t.integer  "parent_id"
+    t.integer  "item_type"
+    t.integer  "spud_page_id"
+    t.integer  "menu_order",   :default => 0
+    t.string   "url"
+    t.datetime "created_at",                  :null => false
+    t.datetime "updated_at",                  :null => false
+    t.string   "name"
+    t.integer  "spud_menu_id"
+    t.string   "classes"
+  end
+
+  add_index "spud_menu_items", ["menu_order"], :name => "index_spud_menu_items_on_menu_order"
+  add_index "spud_menu_items", ["parent_type", "parent_id"], :name => "index_spud_menu_items_on_parent_type_and_parent_id"
+  add_index "spud_menu_items", ["spud_menu_id"], :name => "index_spud_menu_items_on_spud_menu_id"
+
+  create_table "spud_menus", :force => true do |t|
+    t.string   "name"
+    t.text     "description"
+    t.datetime "created_at",                 :null => false
+    t.datetime "updated_at",                 :null => false
+    t.integer  "site_id",     :default => 0, :null => false
+  end
+
+  add_index "spud_menus", ["site_id"], :name => "index_spud_menus_on_site_id"
+
+  create_table "spud_page_partial_revisions", :force => true do |t|
+    t.string   "name"
+    t.text     "content"
+    t.string   "format"
+    t.integer  "spud_page_id"
+    t.datetime "created_at",   :null => false
+    t.datetime "updated_at",   :null => false
+  end
+
+  add_index "spud_page_partial_revisions", ["spud_page_id", "name"], :name => "revision_idx"
+
+  create_table "spud_page_partials", :force => true do |t|
+    t.integer  "spud_page_id"
+    t.string   "name"
+    t.text     "content"
+    t.string   "format"
+    t.datetime "created_at",        :null => false
+    t.datetime "updated_at",        :null => false
+    t.string   "symbol_name"
+    t.text     "content_processed"
+  end
+
+  add_index "spud_page_partials", ["spud_page_id"], :name => "index_spud_page_partials_on_spud_page_id"
+
+  create_table "spud_pages", :force => true do |t|
+    t.string   "name"
+    t.string   "url_name"
+    t.datetime "publish_at"
+    t.integer  "created_by"
+    t.integer  "updated_by"
+    t.string   "format",              :default => "html"
+    t.integer  "spud_page_id"
+    t.text     "meta_description"
+    t.string   "meta_keywords"
+    t.integer  "page_order"
+    t.datetime "created_at",                              :null => false
+    t.datetime "updated_at",                              :null => false
+    t.integer  "visibility",          :default => 0
+    t.boolean  "published",           :default => true
+    t.boolean  "use_custom_url_name", :default => false
+    t.text     "notes"
+    t.integer  "site_id",             :default => 0,      :null => false
+    t.string   "layout"
+  end
+
+  add_index "spud_pages", ["site_id"], :name => "index_spud_pages_on_site_id"
+
+  create_table "spud_permalinks", :force => true do |t|
+    t.string   "url_name"
+    t.string   "attachment_type"
+    t.integer  "attachment_id"
+    t.datetime "created_at",      :null => false
+    t.datetime "updated_at",      :null => false
+    t.integer  "site_id"
+  end
+
+  add_index "spud_permalinks", ["attachment_type", "attachment_id"], :name => "index_spud_permalinks_on_attachment_type_and_attachment_id"
+  add_index "spud_permalinks", ["site_id"], :name => "index_spud_permalinks_on_site_id"
+
+  create_table "spud_user_settings", :force => true do |t|
+    t.integer  "spud_user_id"
+    t.string   "key"
+    t.string   "value"
+    t.datetime "created_at",   :null => false
+    t.datetime "updated_at",   :null => false
+  end
+
+  create_table "spud_users", :force => true do |t|
+    t.string   "first_name"
+    t.string   "last_name"
+    t.boolean  "super_admin"
+    t.string   "login",                              :null => false
+    t.string   "email",                              :null => false
+    t.string   "crypted_password",                   :null => false
+    t.string   "password_salt",                      :null => false
+    t.string   "persistence_token",                  :null => false
+    t.string   "single_access_token",                :null => false
+    t.string   "perishable_token",                   :null => false
+    t.integer  "login_count",         :default => 0, :null => false
+    t.integer  "failed_login_count",  :default => 0, :null => false
+    t.datetime "last_request_at"
+    t.datetime "current_login_at"
+    t.datetime "last_login_at"
+    t.string   "current_login_ip"
+    t.string   "last_login_ip"
+    t.datetime "created_at",                         :null => false
+    t.datetime "updated_at",                         :null => false
+    t.string   "time_zone"
+  end
+
+  add_index "spud_users", ["email"], :name => "index_spud_users_on_email"
+  add_index "spud_users", ["login"], :name => "index_spud_users_on_login"
+
+end