browsercms 3.0.6 → 3.1.0

data/test/unit/helpers/menu_helper_test.rb

@@ -1,116 +1,128 @@
 require File.join(File.dirname(__FILE__), '/../../test_helper')
 
 class Cms::MenuHelperTest < ActionView::TestCase
-  
+
   def test_menu_items
     Page.first.update_attributes(:hidden => true, :publish_on_save => true)
     create_nfl_data
 
     expected = [
-      { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC", :children => [
-          { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
-          { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North", :children => [
-              { :id => "page_#{@bal.id}", :selected => true, :url => "/bal", :name => "Baltimore Ravens" },
-              { :id => "page_#{@cin.id}", :url => "/cin", :name => "Cincinnati Bengals" },
-              { :id => "page_#{@cle.id}", :url => "/cle", :name => "Cleveland Browns" },
-              { :id => "page_#{@pit.id}", :url => "/pit", :name => "Pittsburgh Steelers" }
+            { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC", :children => [
+                    { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
+                    { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North", :children => [
+                            { :id => "page_#{@bal.id}", :selected => true, :url => "/bal", :name => "Baltimore Ravens" },
+                            { :id => "page_#{@cin.id}", :url => "/cin", :name => "Cincinnati Bengals" },
+                            { :id => "page_#{@cle.id}", :url => "/cle", :name => "Cleveland Browns" },
+                            { :id => "page_#{@pit.id}", :url => "/pit", :name => "Pittsburgh Steelers" }
+                    ] },
+                    { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
+                    { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
             ] },
-          { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
-          { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
-        ] },
-      { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC" }
+            { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC" },
+            { :id => "section_#{@int.id}", :url => "/international", :name => "International" }
     ]
-    
+
     assert_equal expected, menu_items(:page => @bal)
 
     @page = @bal
     assert_equal expected, menu_items
-    
+
     expected = [
-      { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC", :children => [
-          { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
-          { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
-          { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
-          { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
-        ] },
-      { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC" }
+            { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC", :children => [
+                    { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
+                    { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
+                    { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
+                    { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
+            ] },
+            { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC" },
+            { :id => "section_#{@int.id}", :url => "/international", :name => "International" }
     ]
-    
+
     assert_equal expected, menu_items(:depth => 2)
-    
+
     expected = [
-      { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
-      { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North", :children => [
-          { :id => "page_#{@bal.id}", :selected => true, :url => "/bal", :name => "Baltimore Ravens" },
-          { :id => "page_#{@cin.id}", :url => "/cin", :name => "Cincinnati Bengals" },
-          { :id => "page_#{@cle.id}", :url => "/cle", :name => "Cleveland Browns" },
-          { :id => "page_#{@pit.id}", :url => "/pit", :name => "Pittsburgh Steelers" }
-        ] },
-      { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
-      { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
+            { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
+            { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North", :children => [
+                    { :id => "page_#{@bal.id}", :selected => true, :url => "/bal", :name => "Baltimore Ravens" },
+                    { :id => "page_#{@cin.id}", :url => "/cin", :name => "Cincinnati Bengals" },
+                    { :id => "page_#{@cle.id}", :url => "/cle", :name => "Cleveland Browns" },
+                    { :id => "page_#{@pit.id}", :url => "/pit", :name => "Pittsburgh Steelers" }
+            ] },
+            { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
+            { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
     ]
-    
+
     assert_equal expected, menu_items(:from_top => 1, :depth => 2)
-    
+
     expected = [
-      { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC", :children => [
-          { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
-          { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
-          { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
-          { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
-        ] },
-      { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC", :children => [
-          { :id => "section_#{@nfc_east.id}", :url => "/dal", :name => "East" },
-          { :id => "section_#{@nfc_north.id}", :url => "/chi", :name => "North" },
-          { :id => "section_#{@nfc_south.id}", :url => "/atl", :name => "South" },
-          { :id => "section_#{@nfc_west.id}", :url => "/ari", :name => "West" }
-        ] }
+            { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC", :children => [
+                    { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
+                    { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
+                    { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
+                    { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
+            ] },
+            { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC", :children => [
+                    { :id => "section_#{@nfc_east.id}", :url => "/dal", :name => "East" },
+                    { :id => "section_#{@nfc_north.id}", :url => "/chi", :name => "North" },
+                    { :id => "section_#{@nfc_south.id}", :url => "/atl", :name => "South" },
+                    { :id => "section_#{@nfc_west.id}", :url => "/ari", :name => "West" }
+            ] },
+            { :id => "section_#{@int.id}", :url => "/international", :name => "International" }
     ]
-    
+
     assert_equal expected, menu_items(:depth => 2, :show_all_siblings => true)
-    
+
     expected = [
-      { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC" },
-      { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC" }
+            { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC" },
+            { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC" },
+            { :id => "section_#{@int.id}", :url => "/international", :name => "International" }
     ]
-    
+
     assert_equal expected, menu_items(:depth => 1)
-    
+
     expected = [
-      { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
-      { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
-      { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
-      { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
+            { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
+            { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
+            { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" },
+            { :id => "section_#{@afc_west.id}", :url => "/den", :name => "West" }
     ]
-    
+
     assert_equal expected, menu_items(:from_top => 1, :depth => 1)
-    
+
     expected = [
-      { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
-      { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
-      { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" }
+            { :id => "section_#{@afc_east.id}", :url => "/buf", :name => "East" },
+            { :id => "section_#{@afc_north.id}", :url => "/bal", :name => "North" },
+            { :id => "section_#{@afc_south.id}", :url => "/hou", :name => "South" }
     ]
-    
+
     assert_equal expected, menu_items(:from_top => 1, :depth => 1, :limit => 3)
+
+    expected = [
+            { :id => "section_#{@afc.id}", :url => "/buf", :name => "AFC" },
+            { :id => "section_#{@nfc.id}", :url => "/dal", :name => "NFC" },
+            { :id => "section_#{@int.id}", :selected => true, :url => "/international", :name => "International" }
+    ]
+
+    assert_equal expected, menu_items(:page => @overview)
     
   end
-  
+
   def test_menu_with_links
     Page.first.update_attributes(:hidden => true, :publish_on_save => true)
-    
+
     @news = Factory(:section, :parent => root_section, :name => "News", :path => "/whatever")
     @press_releases = Factory(:page, :section => @news, :name => "Press Releases", :path => "/press_releases", :publish_on_save => true)
     @corporate_news = Factory(:link, :section => @news, :name => "Corporate News", :url => "/news", :new_window => false, :publish_on_save => true)
     @cnn = Factory(:link, :section => @news, :name => "CNN", :url => "http://www.cnn.com", :new_window => true, :publish_on_save => true)
-    
+
     expected = [
-      { :id => "section_#{@news.id}", :url => "/press_releases", :name => "News", :children => [
-          { :id => "page_#{@press_releases.id}", :selected => true, :url => "/press_releases", :name => "Press Releases" },
-          { :id => "link_#{@corporate_news.id}", :url => "/news", :name => "Corporate News" },
-          { :id => "link_#{@cnn.id}", :url => "http://www.cnn.com", :target => "_blank", :name => "CNN" }
-        ] }
+            { :id => "section_#{@news.id}", :url => "/press_releases", :name => "News", :children => [
+                    { :id => "page_#{@press_releases.id}", :selected => true, :url => "/press_releases", :name => "Press Releases" },
+                    { :id => "link_#{@corporate_news.id}", :url => "/news", :name => "Corporate News" },
+                    { :id => "link_#{@cnn.id}", :url => "http://www.cnn.com", :target => "_blank", :name => "CNN" }
+            ] }
     ]
-    
+
     @page = @press_releases
     assert_equal expected, menu_items
     assert_equal [], menu_items(:from_top => 42)
@@ -121,97 +133,107 @@ class Cms::MenuHelperTest < ActionView::TestCase
     @page = Factory(:page, :section => @section, :name => "Overview", :path => "/test", :publish_on_save => true)
 
     @draft_page = Factory(:page, :section => @section, :name => "Draft v1", :path => "/draft", :publish_on_save => true)
-    @draft_page.update_attributes(:name => "Draft v2")    
+    @draft_page.update_attributes(:name => "Draft v2")
     @never_published = Factory(:page, :section => @section, :name => "Never Published", :path => "/never_published")
-    
+
     expected = [
-      { :id => "page_#{@page.id}", :name => "Overview", :url => "/test", :selected => true },
-      { :id => "page_#{@draft_page.id}", :name => "Draft v1", :url => "/draft" }
+            { :id => "page_#{@page.id}", :name => "Overview", :url => "/test", :selected => true },
+            { :id => "page_#{@draft_page.id}", :name => "Draft v1", :url => "/draft" }
     ]
-    
+
     assert_equal expected, menu_items(:from_top => 1)
   end
-  
+
   def test_render_menu_with_path
     @test = Factory(:page, :section => root_section, :name => "Test", :path => "/test", :publish_on_save => true)
     @footer = Factory(:section, :parent => root_section, :name => "Footer", :path => "/footer")
     @about_us = Factory(:page, :section => @footer, :name => "About Us", :path => "/about_us", :publish_on_save => true)
     @contact_us = Factory(:page, :section => @footer, :name => "Contact Us", :path => "/contact_us", :publish_on_save => true)
     @privacy_policy = Factory(:page, :section => @footer, :name => "Privacy Policy", :path => "/privacy_policy", :publish_on_save => true)
-    
+
     expected = [
-      { :id => "page_#{@about_us.id}", :url => "/about_us", :name => "About Us" },
-      { :id => "page_#{@contact_us.id}", :url => "/contact_us", :name => "Contact Us" },
-      { :id => "page_#{@privacy_policy.id}", :url => "/privacy_policy", :name => "Privacy Policy" }
+            { :id => "page_#{@about_us.id}", :url => "/about_us", :name => "About Us" },
+            { :id => "page_#{@contact_us.id}", :url => "/contact_us", :name => "Contact Us" },
+            { :id => "page_#{@privacy_policy.id}", :url => "/privacy_policy", :name => "Privacy Policy" }
     ]
-    
+
     actual = menu_items(:page => @test, :path => "/footer", :from_top => 1)
     assert_equal expected, actual
 
     expected = [
-      { :id => "page_#{@about_us.id}", :url => "/about_us", :name => "About Us" },
-      { :id => "page_#{@contact_us.id}", :url => "/contact_us", :name => "Contact Us", :selected => true },
-      { :id => "page_#{@privacy_policy.id}", :url => "/privacy_policy", :name => "Privacy Policy" }
+            { :id => "page_#{@about_us.id}", :url => "/about_us", :name => "About Us" },
+            { :id => "page_#{@contact_us.id}", :url => "/contact_us", :name => "Contact Us", :selected => true },
+            { :id => "page_#{@privacy_policy.id}", :url => "/privacy_policy", :name => "Privacy Policy" }
     ]
-    
+
     actual = menu_items(:page => @contact_us, :path => "/footer", :from_top => 1)
     assert_equal expected, actual
   end
-  
+
+  test "nil current page should return empty string" do
+    @page = nil
+    self.expects(:render).never
+
+    assert_equal "", render_menu
+  end
+
   protected
-    def create_nfl_data
-      @afc = Factory(:section, :parent => root_section, :name => "AFC")
-
-      @afc_east = Factory(:section, :parent => @afc, :name => "East")
-      @buf = Factory(:page, :section => @afc_east, :name => "Buffalo Bills", :path => "/buf", :publish_on_save => true)
-      @mia = Factory(:page, :section => @afc_east, :name => "Miami Dolphins", :path => "/mia", :publish_on_save => true)
-      @ne = Factory(:page, :section => @afc_east, :name => "New England Patriots", :path => "/ne", :publish_on_save => true)
-      @nyj = Factory(:page, :section => @afc_east, :name => "New York Jets", :path => "/nyj", :publish_on_save => true)
-
-      @afc_north = Factory(:section, :parent => @afc, :name => "North")
-      @bal = Factory(:page, :section => @afc_north, :name => "Baltimore Ravens", :path => "/bal", :publish_on_save => true)
-      @cin = Factory(:page, :section => @afc_north, :name => "Cincinnati Bengals", :path => "/cin", :publish_on_save => true)
-      @cle = Factory(:page, :section => @afc_north, :name => "Cleveland Browns", :path => "/cle", :publish_on_save => true)
-      @pit = Factory(:page, :section => @afc_north, :name => "Pittsburgh Steelers", :path => "/pit", :publish_on_save => true)
-
-      @afc_south = Factory(:section, :parent => @afc, :name => "South")
-      @hou = Factory(:page, :section => @afc_south, :name => "Houston Texans", :path => "/hou", :publish_on_save => true)
-      @ind = Factory(:page, :section => @afc_south, :name => "Indianapolis Colts", :path => "/ind", :publish_on_save => true)
-      @jac = Factory(:page, :section => @afc_south, :name => "Jacksonville Jaguars", :path => "/jac", :publish_on_save => true)
-      @ten = Factory(:page, :section => @afc_south, :name => "Tennessee Titans", :path => "/ten", :publish_on_save => true)
-
-      @afc_west = Factory(:section, :parent => @afc, :name => "West")
-      @den = Factory(:page, :section => @afc_west, :name => "Denver Broncos", :path => "/den", :publish_on_save => true)
-      @kc = Factory(:page, :section => @afc_west, :name => "Kansas City Chiefs", :path => "/kc", :publish_on_save => true)
-      @oak = Factory(:page, :section => @afc_west, :name => "Oakland Raiders", :path => "/oak", :publish_on_save => true)
-      @sd = Factory(:page, :section => @afc_west, :name => "San Diego Chargers", :path => "/sd", :publish_on_save => true)
-
-      @nfc = Factory(:section, :parent => root_section, :name => "NFC")
-
-      @nfc_east = Factory(:section, :parent => @nfc, :name => "East")
-      @dal = Factory(:page, :section => @nfc_east, :name => "Dallas Cowboys", :path => "/dal", :publish_on_save => true)
-      @nyg = Factory(:page, :section => @nfc_east, :name => "New York Giants", :path => "/nyg", :publish_on_save => true)
-      @phi = Factory(:page, :section => @nfc_east, :name => "Philadelphia Eagles", :path => "/phi", :publish_on_save => true)
-      @was = Factory(:page, :section => @nfc_east, :name => "Washington Redskins", :path => "/was", :publish_on_save => true)
-
-      @nfc_north = Factory(:section, :parent => @nfc, :name => "North")
-      @chi = Factory(:page, :section => @nfc_north, :name => "Chicago Bears", :path => "/chi", :publish_on_save => true)
-      @det = Factory(:page, :section => @nfc_north, :name => "Detroit Lions", :path => "/det", :publish_on_save => true)
-      @gb = Factory(:page, :section => @nfc_north, :name => "Green Bay Packers", :path => "/gb", :publish_on_save => true)
-      @min = Factory(:page, :section => @nfc_north, :name => "Minnesota Vikings", :path => "/min", :publish_on_save => true)
-
-      @nfc_south = Factory(:section, :parent => @nfc, :name => "South")
-      @atl = Factory(:page, :section => @nfc_south, :name => "Atlanta Falcons", :path => "/atl", :publish_on_save => true)
-      @car = Factory(:page, :section => @nfc_south, :name => "Carolina Pathers", :path => "/car", :publish_on_save => true)
-      @no = Factory(:page, :section => @nfc_south, :name => "New Orleans Saints", :path => "/no", :publish_on_save => true)
-      @tb = Factory(:page, :section => @nfc_south, :name => "Tampa Bay Buccaneers", :path => "/tb", :publish_on_save => true)
-
-      @nfc_west = Factory(:section, :parent => @nfc, :name => "West")
-      @ari = Factory(:page, :section => @nfc_west, :name => "Arizona Cardinals", :path => "/ari", :publish_on_save => true)
-      @sf = Factory(:page, :section => @nfc_west, :name => "San Francisco 49ers", :path => "/sf", :publish_on_save => true)
-      @sea = Factory(:page, :section => @nfc_west, :name => "Seattle Seahawks", :path => "/sea", :publish_on_save => true)
-      @stl = Factory(:page, :section => @nfc_west, :name => "St. Louis Rams", :path => "/stl", :publish_on_save => true)
-
-    end
-  
+  def create_nfl_data
+    @afc = Factory(:section, :parent => root_section, :name => "AFC")
+
+    @afc_east = Factory(:section, :parent => @afc, :name => "East")
+    @buf = Factory(:page, :section => @afc_east, :name => "Buffalo Bills", :path => "/buf", :publish_on_save => true)
+    @mia = Factory(:page, :section => @afc_east, :name => "Miami Dolphins", :path => "/mia", :publish_on_save => true)
+    @ne = Factory(:page, :section => @afc_east, :name => "New England Patriots", :path => "/ne", :publish_on_save => true)
+    @nyj = Factory(:page, :section => @afc_east, :name => "New York Jets", :path => "/nyj", :publish_on_save => true)
+
+    @afc_north = Factory(:section, :parent => @afc, :name => "North")
+    @bal = Factory(:page, :section => @afc_north, :name => "Baltimore Ravens", :path => "/bal", :publish_on_save => true)
+    @cin = Factory(:page, :section => @afc_north, :name => "Cincinnati Bengals", :path => "/cin", :publish_on_save => true)
+    @cle = Factory(:page, :section => @afc_north, :name => "Cleveland Browns", :path => "/cle", :publish_on_save => true)
+    @pit = Factory(:page, :section => @afc_north, :name => "Pittsburgh Steelers", :path => "/pit", :publish_on_save => true)
+
+    @afc_south = Factory(:section, :parent => @afc, :name => "South")
+    @hou = Factory(:page, :section => @afc_south, :name => "Houston Texans", :path => "/hou", :publish_on_save => true)
+    @ind = Factory(:page, :section => @afc_south, :name => "Indianapolis Colts", :path => "/ind", :publish_on_save => true)
+    @jac = Factory(:page, :section => @afc_south, :name => "Jacksonville Jaguars", :path => "/jac", :publish_on_save => true)
+    @ten = Factory(:page, :section => @afc_south, :name => "Tennessee Titans", :path => "/ten", :publish_on_save => true)
+
+    @afc_west = Factory(:section, :parent => @afc, :name => "West")
+    @den = Factory(:page, :section => @afc_west, :name => "Denver Broncos", :path => "/den", :publish_on_save => true)
+    @kc = Factory(:page, :section => @afc_west, :name => "Kansas City Chiefs", :path => "/kc", :publish_on_save => true)
+    @oak = Factory(:page, :section => @afc_west, :name => "Oakland Raiders", :path => "/oak", :publish_on_save => true)
+    @sd = Factory(:page, :section => @afc_west, :name => "San Diego Chargers", :path => "/sd", :publish_on_save => true)
+
+    @nfc = Factory(:section, :parent => root_section, :name => "NFC")
+
+    @nfc_east = Factory(:section, :parent => @nfc, :name => "East")
+    @dal = Factory(:page, :section => @nfc_east, :name => "Dallas Cowboys", :path => "/dal", :publish_on_save => true)
+    @nyg = Factory(:page, :section => @nfc_east, :name => "New York Giants", :path => "/nyg", :publish_on_save => true)
+    @phi = Factory(:page, :section => @nfc_east, :name => "Philadelphia Eagles", :path => "/phi", :publish_on_save => true)
+    @was = Factory(:page, :section => @nfc_east, :name => "Washington Redskins", :path => "/was", :publish_on_save => true)
+
+    @nfc_north = Factory(:section, :parent => @nfc, :name => "North")
+    @chi = Factory(:page, :section => @nfc_north, :name => "Chicago Bears", :path => "/chi", :publish_on_save => true)
+    @det = Factory(:page, :section => @nfc_north, :name => "Detroit Lions", :path => "/det", :publish_on_save => true)
+    @gb = Factory(:page, :section => @nfc_north, :name => "Green Bay Packers", :path => "/gb", :publish_on_save => true)
+    @min = Factory(:page, :section => @nfc_north, :name => "Minnesota Vikings", :path => "/min", :publish_on_save => true)
+
+    @nfc_south = Factory(:section, :parent => @nfc, :name => "South")
+    @atl = Factory(:page, :section => @nfc_south, :name => "Atlanta Falcons", :path => "/atl", :publish_on_save => true)
+    @car = Factory(:page, :section => @nfc_south, :name => "Carolina Pathers", :path => "/car", :publish_on_save => true)
+    @no = Factory(:page, :section => @nfc_south, :name => "New Orleans Saints", :path => "/no", :publish_on_save => true)
+    @tb = Factory(:page, :section => @nfc_south, :name => "Tampa Bay Buccaneers", :path => "/tb", :publish_on_save => true)
+
+    @nfc_west = Factory(:section, :parent => @nfc, :name => "West")
+    @ari = Factory(:page, :section => @nfc_west, :name => "Arizona Cardinals", :path => "/ari", :publish_on_save => true)
+    @sf = Factory(:page, :section => @nfc_west, :name => "San Francisco 49ers", :path => "/sf", :publish_on_save => true)
+    @sea = Factory(:page, :section => @nfc_west, :name => "Seattle Seahawks", :path => "/sea", :publish_on_save => true)
+    @stl = Factory(:page, :section => @nfc_west, :name => "St. Louis Rams", :path => "/stl", :publish_on_save => true)
+
+    @int = Factory(:section, :parent => root_section, :name => "International", :path => '/international')
+    @overview = Factory(:page, :section => @int, :name => "Overview", :path => "/international", :publish_on_save => true, :hidden => true)
+  end
+
+
 end

data/test/unit/helpers/page_helper_test.rb

@@ -1,8 +1,8 @@
 require File.join(File.dirname(__FILE__), '/../../test_helper')
 
 class Cms::PageHelperTest < ActionView::TestCase
-  
-  def test_render_breadcrumbs
+
+  def setup
     @foo = Factory(:section, :name => "Foo", :parent => root_section, :path => "/foo")
     Factory(:page, :name => "Overview", :section => @foo, :path => "/foo")
     @bar = Factory(:section, :name => "Bar", :parent => @foo, :path => "/bar")
@@ -10,7 +10,9 @@ class Cms::PageHelperTest < ActionView::TestCase
     @bang = Factory(:page, :name => "Bang", :section => @bar, :path => "/bar/bang")
 
     @page = @bang
-    
+  end
+  
+  def test_render_breadcrumbs
     expected = <<HTML
 <ul class="breadcrumbs">
   <li class="first"><a href="/">My Site</a></li>
@@ -54,4 +56,10 @@ HTML
     assert_equal expected.chomp, render_breadcrumbs(:from_top => 1, :show_parent => true)
   end  
   
+  test "A missing @page should cause the breadcrumbs to render nothing, which may occur in TemplateSupport or Acts::ContentPage." do
+    @page = nil
+
+    assert_equal "", render_breadcrumbs
+  end
+  
 end

data/test/unit/lib/acts_as_content_page_test.rb

@@ -0,0 +1,72 @@
+require "test_helper"
+require "mocha"
+
+class CmsActsAsContentPageTest < ActiveSupport::TestCase
+  EXPECTED_SECTION = "/members"
+
+  class MyController < ActionController::Base
+    include Cms::Acts::ContentPage
+    requires_permission_for_section EXPECTED_SECTION
+  end
+
+  test "Arbitrary controller should have authentication methods" do
+    c = MyController.new
+
+    assert c.respond_to? :handle_access_denied_on_page
+    assert c.respond_to? :handle_not_found_on_page
+    assert c.respond_to? :handle_server_error_on_page
+    assert c.respond_to? :handle_error_with_cms_page, true
+
+    assert(c.respond_to?(:logged_in?), "Should include Cms::Authentication::Controller methods")
+    assert(c.respond_to?(:handle_server_error), "Should include Cms::ErrorHandling methods")
+
+    assert(MyController.respond_to?(:requires_permission_for_section), "Should add ClassMethods to the controller.")
+  end
+
+  test "check_access_to_section_filter" do
+    mock_user = mock()
+    mock_user.expects(:able_to_view?).with(EXPECTED_SECTION).returns(true)
+    mock_user.expects(:login).returns("mock_user_name")
+    c = MyController.new
+    c.expects(:current_user).returns(mock_user)
+
+
+    c.check_access_to_section
+
+
+  end
+
+  test "sets section in startup" do
+    class AController < ActionController::Base
+      include Cms::Acts::ContentPage
+      requires_permission_for_section EXPECTED_SECTION
+    end
+
+    assert_equal EXPECTED_SECTION, AController.in_section
+  end
+
+  class NewController < ActionController::Base
+      include Cms::Acts::ContentPage
+  end
+
+  test "placing in a section should create a before_filter for that section for all actions" do
+    NewController.expects(:before_filter).with(:check_access_to_section, {})
+
+    NewController.send :requires_permission_for_section, EXPECTED_SECTION
+
+  end
+
+  test "can put only conditions on filters" do
+    NewController.expects(:before_filter).with(:check_access_to_section, :only=>[:create])
+
+    NewController.send :requires_permission_for_section, EXPECTED_SECTION, :only=>[:create]
+  end
+
+  test "can put except conditions on filters" do
+    NewController.expects(:before_filter).with(:check_access_to_section, :except=>[:create])
+
+    NewController.send :requires_permission_for_section, EXPECTED_SECTION, :except=>[:create]
+  end
+
+
+end