tb_cms 1.2.3 → 1.3.beta1

data/spec/dummy/db/schema.rb

@@ -1,4 +1,3 @@
-# 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.
@@ -11,185 +10,175 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20160215180157) do
+ActiveRecord::Schema.define(version: 20_160_215_180_157) do
 
-  create_table "spud_liquid_tags", force: :cascade do |t|
-    t.integer  "attachment_id",   limit: 4
-    t.string   "attachment_type", limit: 255
-    t.string   "tag_name",        limit: 255
-    t.string   "value",           limit: 255
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_liquid_tags', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.integer  'attachment_id'
+    t.string   'attachment_type'
+    t.string   'tag_name'
+    t.string   'value'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.index ['tag_name', 'value'], name: 'index_spud_liquid_tags_on_tag_name_and_value', using: :btree
   end
 
-  add_index "spud_liquid_tags", ["tag_name", "value"], name: "index_spud_liquid_tags_on_tag_name_and_value", using: :btree
-
-  create_table "spud_menu_items", force: :cascade do |t|
-    t.string   "parent_type",  limit: 255
-    t.integer  "parent_id",    limit: 4
-    t.integer  "item_type",    limit: 4
-    t.integer  "spud_page_id", limit: 4
-    t.integer  "menu_order",   limit: 4,   default: 0
-    t.string   "url",          limit: 255
-    t.datetime "created_at"
-    t.datetime "updated_at"
-    t.string   "name",         limit: 255
-    t.integer  "spud_menu_id", limit: 4
-    t.string   "classes",      limit: 255
+  create_table 'spud_menu_items', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' 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'
+    t.datetime 'updated_at'
+    t.string   'name'
+    t.integer  'spud_menu_id'
+    t.string   'classes'
+    t.index ['menu_order'], name: 'index_spud_menu_items_on_menu_order', using: :btree
+    t.index ['parent_type', 'parent_id'], name: 'index_spud_menu_items_on_parent_type_and_parent_id', using: :btree
+    t.index ['spud_menu_id'], name: 'index_spud_menu_items_on_spud_menu_id', using: :btree
   end
 
-  add_index "spud_menu_items", ["menu_order"], name: "index_spud_menu_items_on_menu_order", using: :btree
-  add_index "spud_menu_items", ["parent_type", "parent_id"], name: "index_spud_menu_items_on_parent_type_and_parent_id", using: :btree
-  add_index "spud_menu_items", ["spud_menu_id"], name: "index_spud_menu_items_on_spud_menu_id", using: :btree
-
-  create_table "spud_menus", force: :cascade do |t|
-    t.string   "name",        limit: 255
-    t.text     "description", limit: 65535
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_menus', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.string   'name'
+    t.text     'description', limit: 65_535
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
   end
 
-  create_table "spud_page_partial_revisions", force: :cascade do |t|
-    t.string   "name",         limit: 255
-    t.text     "content",      limit: 65535
-    t.string   "format",       limit: 255
-    t.integer  "spud_page_id", limit: 4
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_page_partial_revisions', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.string   'name'
+    t.text     'content', limit: 65_535
+    t.string   'format'
+    t.integer  'spud_page_id'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.index ['spud_page_id', 'name'], name: 'revision_idx', using: :btree
   end
 
-  add_index "spud_page_partial_revisions", ["spud_page_id", "name"], name: "revision_idx", using: :btree
-
-  create_table "spud_page_partials", force: :cascade do |t|
-    t.integer  "spud_page_id",      limit: 4
-    t.string   "name",              limit: 255
-    t.text     "content",           limit: 65535
-    t.string   "format",            limit: 255
-    t.datetime "created_at"
-    t.datetime "updated_at"
-    t.string   "symbol_name",       limit: 255
-    t.text     "content_processed", limit: 65535
+  create_table 'spud_page_partials', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.integer  'spud_page_id'
+    t.string   'name'
+    t.text     'content', limit: 65_535
+    t.string   'format'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.string   'symbol_name'
+    t.text     'content_processed', limit: 65_535
+    t.index ['spud_page_id'], name: 'index_spud_page_partials_on_spud_page_id', using: :btree
   end
 
-  add_index "spud_page_partials", ["spud_page_id"], name: "index_spud_page_partials_on_spud_page_id", using: :btree
-
-  create_table "spud_pages", force: :cascade do |t|
-    t.string   "name",                limit: 255
-    t.string   "url_name",            limit: 255
-    t.datetime "publish_at"
-    t.integer  "created_by",          limit: 4
-    t.integer  "updated_by",          limit: 4
-    t.string   "format",              limit: 255,   default: "html"
-    t.integer  "spud_page_id",        limit: 4
-    t.text     "meta_description",    limit: 65535
-    t.string   "meta_keywords",       limit: 255
-    t.integer  "page_order",          limit: 4
-    t.datetime "created_at"
-    t.datetime "updated_at"
-    t.integer  "visibility",          limit: 4,     default: 0
-    t.boolean  "published",                         default: true
-    t.boolean  "use_custom_url_name",               default: false
-    t.text     "notes",               limit: 65535
-    t.string   "layout",              limit: 255
-    t.string   "page_title",          limit: 255
+  create_table 'spud_pages', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' 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', limit: 65_535
+    t.string   'meta_keywords'
+    t.integer  'page_order'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.integer  'visibility',                        default: 0
+    t.boolean  'published',                         default: true
+    t.boolean  'use_custom_url_name',               default: false
+    t.text     'notes', limit: 65_535
+    t.string   'layout'
+    t.string   'page_title'
   end
 
-  create_table "spud_permalinks", force: :cascade do |t|
-    t.string   "url_name",        limit: 255
-    t.string   "attachment_type", limit: 255
-    t.integer  "attachment_id",   limit: 4
-    t.datetime "created_at"
-    t.datetime "updated_at"
-    t.integer  "site_id",         limit: 4,   default: 0, null: false
+  create_table 'spud_permalinks', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.string   'url_name'
+    t.string   'attachment_type'
+    t.integer  'attachment_id'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.integer  'site_id', default: 0, null: false
+    t.index ['attachment_type', 'attachment_id'], name: 'idx_permalink_attachment', using: :btree
+    t.index ['site_id'], name: 'idx_permalinks_site_id', using: :btree
   end
 
-  add_index "spud_permalinks", ["attachment_type", "attachment_id"], name: "idx_permalink_attachment", using: :btree
-  add_index "spud_permalinks", ["site_id"], name: "idx_permalinks_site_id", using: :btree
-
-  create_table "spud_permissions", force: :cascade do |t|
-    t.string   "name",       limit: 255, null: false
-    t.string   "tag",        limit: 255, null: false
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_permissions', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.string   'name',       null: false
+    t.string   'tag',        null: false
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.index ['tag'], name: 'index_spud_permissions_on_tag', unique: true, using: :btree
   end
 
-  add_index "spud_permissions", ["tag"], name: "index_spud_permissions_on_tag", unique: true, using: :btree
-
-  create_table "spud_role_permissions", force: :cascade do |t|
-    t.integer  "spud_role_id",        limit: 4,   null: false
-    t.string   "spud_permission_tag", limit: 255, null: false
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_role_permissions', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.integer  'spud_role_id',        null: false
+    t.string   'spud_permission_tag', null: false
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.index ['spud_permission_tag'], name: 'index_spud_role_permissions_on_spud_permission_tag', using: :btree
+    t.index ['spud_role_id'], name: 'index_spud_role_permissions_on_spud_role_id', using: :btree
   end
 
-  add_index "spud_role_permissions", ["spud_permission_tag"], name: "index_spud_role_permissions_on_spud_permission_tag", using: :btree
-  add_index "spud_role_permissions", ["spud_role_id"], name: "index_spud_role_permissions_on_spud_role_id", using: :btree
-
-  create_table "spud_roles", force: :cascade do |t|
-    t.string   "name",       limit: 255
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_roles', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.string   'name'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
   end
 
-  create_table "spud_snippets", force: :cascade do |t|
-    t.string   "name",              limit: 255
-    t.text     "content",           limit: 65535
-    t.string   "format",            limit: 255
-    t.text     "content_processed", limit: 65535
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_snippets', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.string   'name'
+    t.text     'content', limit: 65_535
+    t.string   'format'
+    t.text     'content_processed', limit: 65_535
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
+    t.index ['name'], name: 'index_spud_snippets_on_name', using: :btree
   end
 
-  add_index "spud_snippets", ["name"], name: "index_spud_snippets_on_name", using: :btree
-
-  create_table "spud_user_settings", force: :cascade do |t|
-    t.integer  "spud_user_id", limit: 4
-    t.string   "key",          limit: 255
-    t.string   "value",        limit: 255
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table 'spud_user_settings', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.integer  'spud_user_id'
+    t.string   'key'
+    t.string   'value'
+    t.datetime 'created_at'
+    t.datetime 'updated_at'
   end
 
-  create_table "spud_users", force: :cascade do |t|
-    t.string   "first_name",               limit: 255
-    t.string   "last_name",                limit: 255
-    t.boolean  "super_admin"
-    t.string   "login",                    limit: 255,                 null: false
-    t.string   "email",                    limit: 255,                 null: false
-    t.string   "crypted_password",         limit: 255,                 null: false
-    t.string   "password_salt",            limit: 255,                 null: false
-    t.string   "persistence_token",        limit: 255,                 null: false
-    t.string   "single_access_token",      limit: 255,                 null: false
-    t.string   "perishable_token",         limit: 255,                 null: false
-    t.integer  "login_count",              limit: 4,   default: 0,     null: false
-    t.integer  "failed_login_count",       limit: 4,   default: 0,     null: false
-    t.datetime "last_request_at"
-    t.datetime "current_login_at"
-    t.datetime "last_login_at"
-    t.string   "current_login_ip",         limit: 255
-    t.string   "last_login_ip",            limit: 255
-    t.datetime "created_at"
-    t.datetime "updated_at"
-    t.string   "time_zone",                limit: 255
-    t.integer  "spud_role_id",             limit: 4
-    t.boolean  "requires_password_change",             default: false
+  create_table 'spud_users', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' 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'
+    t.datetime 'updated_at'
+    t.string   'time_zone'
+    t.integer  'spud_role_id'
+    t.boolean  'requires_password_change', default: false
+    t.index ['email'], name: 'index_spud_users_on_email', using: :btree
+    t.index ['login'], name: 'index_spud_users_on_login', using: :btree
+    t.index ['spud_role_id'], name: 'index_spud_users_on_spud_role_id', using: :btree
   end
 
-  add_index "spud_users", ["email"], name: "index_spud_users_on_email", using: :btree
-  add_index "spud_users", ["login"], name: "index_spud_users_on_login", using: :btree
-  add_index "spud_users", ["spud_role_id"], name: "index_spud_users_on_spud_role_id", using: :btree
-
-  create_table "tb_redirects", force: :cascade do |t|
-    t.string   "owner_type",  limit: 255
-    t.integer  "owner_id",    limit: 4
-    t.string   "source",      limit: 255, null: false
-    t.string   "destination", limit: 255, null: false
-    t.string   "created_by",  limit: 255
-    t.datetime "created_at",              null: false
-    t.datetime "updated_at",              null: false
+  create_table 'tb_redirects', force: :cascade, options: 'ENGINE=InnoDB DEFAULT CHARSET=utf8' do |t|
+    t.string   'owner_type'
+    t.integer  'owner_id'
+    t.string   'source',      null: false
+    t.string   'destination', null: false
+    t.string   'created_by'
+    t.datetime 'created_at',  null: false
+    t.datetime 'updated_at',  null: false
+    t.index ['owner_type', 'owner_id'], name: 'index_tb_redirects_on_owner_type_and_owner_id', using: :btree
+    t.index ['source'], name: 'index_tb_redirects_on_source', unique: true, using: :btree
   end
 
-  add_index "tb_redirects", ["owner_type", "owner_id"], name: "index_tb_redirects_on_owner_type_and_owner_id", using: :btree
-  add_index "tb_redirects", ["source"], name: "index_tb_redirects_on_source", unique: true, using: :btree
-
 end

data/spec/dummy/script/rails

@@ -1,6 +1,6 @@
 #!/usr/bin/env ruby
 # This command will automatically be run when you run "rails" with Rails 3 gems installed from the root of your application.
 
-APP_PATH = File.expand_path('../../config/application',  __FILE__)
-require File.expand_path('../../config/boot',  __FILE__)
+APP_PATH = File.expand_path('../../config/application', __FILE__)
+require File.expand_path('../../config/boot', __FILE__)
 require 'rails/commands'

data/spec/helpers/cms/application_helper_spec.rb

@@ -1,108 +1,106 @@
 require 'rails_helper'
 
-describe Cms::ApplicationHelper, :type => :helper do
+describe Cms::ApplicationHelper, type: :helper do
   before(:each) do
     helper.output_buffer = ''
   end
 
   describe '#sp_list_menu' do
-    it "should be able to find a menu by its name" do
-      menu = FactoryGirl.create(:spud_menu, :name => "Main")
-      2.times { |x| FactoryGirl.create(:spud_menu_item, :parent_id => menu.id, :spud_menu_id => menu.id, :url => "/") }
+    it 'should be able to find a menu by its name' do
+      menu = FactoryGirl.create(:spud_menu, name: 'Main')
+      2.times { |_x| FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/') }
 
-      helper.sp_list_menu(:name => "Main", :active_class => "active")
+      helper.sp_list_menu(name: 'Main', active_class: 'active')
       expect(helper.output_buffer).to match /\<li/
     end
 
-    it "should assign id to ul block" do
+    it 'should assign id to ul block' do
       menu = FactoryGirl.create(:spud_menu)
-      2.times {|x|  s = FactoryGirl.create(:spud_menu_item,:parent_id => menu.id,:spud_menu_id => menu.id,:url => "/")}
+      2.times { |_x| s = FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/') }
 
-      helper.sp_list_menu(:name => menu.name,:id => "nav")
+      helper.sp_list_menu(name: menu.name, id: 'nav')
       expect(helper.output_buffer).to match /id=\'nav\'/
     end
 
-    it "should render nested menu items" do
-      menu = FactoryGirl.create(:spud_menu,:name => "Main2")
-      s = FactoryGirl.create(:spud_menu_item,:parent_id => menu.id,:spud_menu_id => menu.id,:url => "/")
-      s2 = FactoryGirl.create(:spud_menu_item,:parent_id => menu.id,:spud_menu_id => menu.id,:url => "/")
-      s3 = FactoryGirl.create(:spud_menu_item,:parent_type => "SpudMenuItem",:parent_id => s.id,:spud_menu_id => menu.id,:url => "/",:name => "SubItem")
+    it 'should render nested menu items' do
+      menu = FactoryGirl.create(:spud_menu, name: 'Main2')
+      s = FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/')
+      s2 = FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/')
+      s3 = FactoryGirl.create(:spud_menu_item, parent_type: 'SpudMenuItem', parent_id: s.id, spud_menu_id: menu.id, url: '/', name: 'SubItem')
 
-      helper.sp_list_menu(:name => "Main2")
+      helper.sp_list_menu(name: 'Main2')
       expect(helper.output_buffer).to match /SubItem/
     end
 
-    it "should respect max depth" do
-      menu = FactoryGirl.create(:spud_menu,:name => "Main4")
-      s = FactoryGirl.create(:spud_menu_item,:parent_id => menu.id,:spud_menu_id => menu.id,:url => "/")
-      s2 = FactoryGirl.create(:spud_menu_item,:parent_id => menu.id,:spud_menu_id => menu.id,:url => "/")
-      s3 = FactoryGirl.create(:spud_menu_item,:parent_type => "SpudMenuItem",:parent_id => s.id,:spud_menu_id => menu.id,:url => "/",:name => "SubItem")
+    it 'should respect max depth' do
+      menu = FactoryGirl.create(:spud_menu, name: 'Main4')
+      s = FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/')
+      s2 = FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/')
+      s3 = FactoryGirl.create(:spud_menu_item, parent_type: 'SpudMenuItem', parent_id: s.id, spud_menu_id: menu.id, url: '/', name: 'SubItem')
 
-      helper.sp_list_menu(:name => "Main4",:max_depth => 1)
+      helper.sp_list_menu(name: 'Main4', max_depth: 1)
       expect(helper.output_buffer).to_not match /SubItem/
     end
   end
 
   describe '#sp_menu_with_seperator' do
-    it "should render a flattened list of links" do
-      menu = FactoryGirl.create(:spud_menu,:name => "Main3")
-      s = FactoryGirl.create(:spud_menu_item,:parent_id => menu.id,:spud_menu_id => menu.id,:url => "/")
-      s2 = FactoryGirl.create(:spud_menu_item,:parent_id => menu.id,:spud_menu_id => menu.id,:url => "/")
-      s3 = FactoryGirl.create(:spud_menu_item,:parent_type => "SpudMenuItem",:parent_id => s.id,:spud_menu_id => menu.id,:url => "/",:name => "SubItem")
+    it 'should render a flattened list of links' do
+      menu = FactoryGirl.create(:spud_menu, name: 'Main3')
+      s = FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/')
+      s2 = FactoryGirl.create(:spud_menu_item, parent_id: menu.id, spud_menu_id: menu.id, url: '/')
+      s3 = FactoryGirl.create(:spud_menu_item, parent_type: 'SpudMenuItem', parent_id: s.id, spud_menu_id: menu.id, url: '/', name: 'SubItem')
 
-      content = helper.sp_menu_with_seperator(:name => "Main3")
+      content = helper.sp_menu_with_seperator(name: 'Main3')
       expect(content).to match /SubItem/
       expect(content).to_not match /\<li/
     end
   end
 
   describe '#sp_list_pages' do
-    it "should be able to list created pages" do
+    it 'should be able to list created pages' do
       page = FactoryGirl.create(:spud_page)
       page2 = FactoryGirl.create(:spud_page)
-      page3 = FactoryGirl.create(:spud_page,:spud_page_id => page.id)
+      page3 = FactoryGirl.create(:spud_page, spud_page_id: page.id)
 
-
-      content = helper.sp_list_pages(:active_class => "active")
+      content = helper.sp_list_pages(active_class: 'active')
       expect(content).to match /#{page.name}/
       expect(content).to match /#{page2.name}/
       expect(content).to match /#{page3.name}/
     end
 
-    it "should assign id" do
+    it 'should assign id' do
       page = FactoryGirl.create(:spud_page)
       page2 = FactoryGirl.create(:spud_page)
 
-      content = helper.sp_list_pages(:id => "page_nav")
+      content = helper.sp_list_pages(id: 'page_nav')
       expect(content).to match /id=\'page_nav\'/
 
     end
 
-    it "should be able to exclude certain pages" do
+    it 'should be able to exclude certain pages' do
       page = FactoryGirl.create(:spud_page)
       page2 = FactoryGirl.create(:spud_page)
 
-      content = helper.sp_list_pages(:exclude => [page2.name])
+      content = helper.sp_list_pages(exclude: [page2.name])
       expect(content).to match /#{page.name}/
       expect(content).to_not match /#{page2.name}/
     end
-    it "should respect max depth" do
+    it 'should respect max depth' do
       page = FactoryGirl.create(:spud_page)
       page2 = FactoryGirl.create(:spud_page)
-      page3 = FactoryGirl.create(:spud_page,:spud_page_id => page.id)
-
+      page3 = FactoryGirl.create(:spud_page, spud_page_id: page.id)
 
-      content = helper.sp_list_pages(:max_depth => 1)
+      content = helper.sp_list_pages(max_depth: 1)
       expect(content).to match /#{page.name}/
       expect(content).to match /#{page2.name}/
       expect(content).to_not match /#{page3.name}/
     end
 
-    it "should be able to list sub pages of a particular page" do
+    it 'should be able to list sub pages of a particular page' do
       page = FactoryGirl.create(:spud_page)
       page2 = FactoryGirl.create(:spud_page)
-      page3 = FactoryGirl.create(:spud_page,:spud_page_id => page.id)
-      content = helper.sp_list_pages(:start_page_id => page.id)
+      page3 = FactoryGirl.create(:spud_page, spud_page_id: page.id)
+      content = helper.sp_list_pages(start_page_id: page.id)
       expect(content).to_not match /#{page.name}/
       expect(content).to_not match /#{page2.name}/
       expect(content).to match /#{page3.name}/

data/spec/models/spud_menu_item_spec.rb

@@ -1,30 +1,30 @@
 require 'rails_helper'
 
-describe SpudMenuItem, :type => :model do
+describe SpudMenuItem, type: :model do
 
-  it {should have_many(:spud_menu_items)}
-  it {should belong_to(:spud_page)}
-  it {should belong_to(:spud_menu)}
-  it {should belong_to(:parent)}
+  it { should have_many(:spud_menu_items) }
+  it { should belong_to(:spud_page) }
+  it { should belong_to(:spud_menu) }
+  it { should belong_to(:parent) }
 
   describe :validations do
-    it "should require a name" do
-      p = FactoryGirl.build(:spud_menu_item,:name => nil)
+    it 'should require a name' do
+      p = FactoryGirl.build(:spud_menu_item, name: nil)
       expect(p).to_not be_valid
     end
 
-    it "should require a menu_id" do
-      p = FactoryGirl.build(:spud_menu_item,:spud_menu_id => nil)
+    it 'should require a menu_id' do
+      p = FactoryGirl.build(:spud_menu_item, spud_menu_id: nil)
       expect(p).to_not be_valid
     end
 
-    it "should require a parent_type" do
-      p = FactoryGirl.build(:spud_menu_item,:parent_type => nil)
+    it 'should require a parent_type' do
+      p = FactoryGirl.build(:spud_menu_item, parent_type: nil)
       expect(p).to_not be_valid
     end
 
-    it "should require a parent_id" do
-      p = FactoryGirl.build(:spud_menu_item,:parent_id => nil)
+    it 'should require a parent_id' do
+      p = FactoryGirl.build(:spud_menu_item, parent_id: nil)
       expect(p).to_not be_valid
     end
   end

data/spec/models/spud_page_partial_revision_spec.rb

@@ -1,5 +1,5 @@
 require 'rails_helper'
 
-describe SpudPagePartialRevision, :type => :model do
+describe SpudPagePartialRevision, type: :model do
   pending "add some examples to (or delete) #{__FILE__}"
 end

data/spec/models/spud_page_partial_spec.rb

@@ -1,44 +1,43 @@
 require 'rails_helper'
 
-describe SpudPagePartial, :type => :model do
-  describe "validations" do
+describe SpudPagePartial, type: :model do
+  describe 'validations' do
 
-    it "should require a name" do
-      p = FactoryGirl.build(:spud_page_partial,:name => nil)
+    it 'should require a name' do
+      p = FactoryGirl.build(:spud_page_partial, name: nil)
       expect(p).to_not be_valid
     end
 
-    it "should respond with a symbol_name based on name" do
-      p = FactoryGirl.build(:spud_page_partial,:name => "Test Page")
-      expect(p.symbol_name).to eq("test_page")
+    it 'should respond with a symbol_name based on name' do
+      p = FactoryGirl.build(:spud_page_partial, name: 'Test Page')
+      expect(p.symbol_name).to eq('test_page')
     end
   end
 
-  describe "save hooks" do
-    it "should save the symbol name" do
-      p = FactoryGirl.create(:spud_page_partial, :name => "Test Page")
-      expect(p.attributes["symbol_name"]).to eq("test_page")
+  describe 'save hooks' do
+    it 'should save the symbol name' do
+      p = FactoryGirl.create(:spud_page_partial, name: 'Test Page')
+      expect(p.attributes['symbol_name']).to eq('test_page')
     end
 
-    it "should create a new revision if content is changed" do
-      p = FactoryGirl.create(:spud_page_partial, :name => "Test Page", :content => 'Content')
-      expect(SpudPagePartialRevision.where(:spud_page_id => p.spud_page_id).count).to eq(1)
+    it 'should create a new revision if content is changed' do
+      p = FactoryGirl.create(:spud_page_partial, name: 'Test Page', content: 'Content')
+      expect(SpudPagePartialRevision.where(spud_page_id: p.spud_page_id).count).to eq(1)
     end
 
-    it "should delete old revisions beyond max_revision count" do
+    it 'should delete old revisions beyond max_revision count' do
       Spud::Cms.configure do |config|
         config.max_revisions = 2
       end
-      p = FactoryGirl.create(:spud_page_partial,:name => "Test Page",:content =>"Home Sweet Home",:spud_page_id => 1)
-      p.content = "Nah"
+      p = FactoryGirl.create(:spud_page_partial, name: 'Test Page', content: 'Home Sweet Home', spud_page_id: 1)
+      p.content = 'Nah'
       p.save
 
-      p.content = "Another change"
+      p.content = 'Another change'
       p.save
-      expect(SpudPagePartialRevision.where(:spud_page_id => 1,:name => "Test Page").count).to eq(2)
+      expect(SpudPagePartialRevision.where(spud_page_id: 1, name: 'Test Page').count).to eq(2)
     end
 
   end
 
-
 end