zen 0.3 → 0.4

data/spec/zen/package/menus/helper/menu.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../../../helper', __FILE__)
 
-describe('Ramaze::Helper::Menu') do
+describe 'Ramaze::Helper::Menu' do
   behaves_like :capybara
   extend       Ramaze::Helper::Menu
 
@@ -26,7 +26,7 @@ describe('Ramaze::Helper::Menu') do
     :parent_id => child.id
   )
 
-  it('Validate a valid menu') do
+  it 'Validate a valid menu' do
     url = Menus::Controller::Menus.r(:edit, menu.id).to_s
 
     visit(url)
@@ -34,7 +34,7 @@ describe('Ramaze::Helper::Menu') do
     current_path.should == url
   end
 
-  it('Validate an invalid menu') do
+  it 'Validate an invalid menu' do
     url   = Menus::Controller::Menus.r(:edit, menu.id + 100).to_s
     index = Menus::Controller::Menus.r(:index).to_s
 
@@ -43,7 +43,7 @@ describe('Ramaze::Helper::Menu') do
     current_path.should == index
   end
 
-  it('Validate a valid menu item') do
+  it 'Validate a valid menu item' do
     url = Menus::Controller::MenuItems.r(:edit, menu.id, item.id).to_s
 
     visit(url)
@@ -51,7 +51,7 @@ describe('Ramaze::Helper::Menu') do
     current_path.should == url
   end
 
-  it('Validate an invalid menu item') do
+  it 'Validate an invalid menu item' do
     url   = Menus::Controller::MenuItems.r(:edit, menu.id, item.id + 100).to_s
     index = Menus::Controller::MenuItems.r(:index, menu.id).to_s
 
@@ -60,16 +60,8 @@ describe('Ramaze::Helper::Menu') do
     current_path.should == index
   end
 
-  it('Generate a navigation tree') do
-    tree = menu_item_tree(menu.id)
-
-    tree[nil].should          == '--'
-    tree[child.id].should     == "  #{child.name}"
-    tree[sub_child.id].should == "    #{sub_child.name}"
-  end
-
   sub_child.destroy
   child.destroy
   item.destroy
   menu.destroy
-end # describe
+end

data/spec/zen/package/menus/helper/menu_frontend.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../../../helper', __FILE__)
 
-describe('Menus::Plugin::Menus') do
+describe 'Menus::Plugin::Menus' do
   extend Ramaze::Helper::MenuFrontend
 
   nav_menu = Menus::Model::Menu.create(
@@ -33,7 +33,7 @@ describe('Menus::Plugin::Menus') do
     :sort_order => 3
   )
 
-  it("Retrieve a menu with all items") do
+  it "Retrieve a menu with all items" do
     menu = render_menu(nav_menu.slug).strip
 
     menu.include?('Spec').should                  == true
@@ -42,7 +42,7 @@ describe('Menus::Plugin::Menus') do
     menu.include?('<ul class="children">').should == true
   end
 
-  it('Retrieve a menu with all items by the menu id') do
+  it 'Retrieve a menu with all items by the menu id' do
     menu = render_menu(nav_menu.id).strip
 
     menu.include?('Spec').should                  == true
@@ -51,7 +51,7 @@ describe('Menus::Plugin::Menus') do
     menu.include?('<ul class="children">').should == true
   end
 
-  it("Retrieve a menu with only 1 item") do
+  it "Retrieve a menu with only 1 item" do
     menu = render_menu(nav_menu.slug, :limit => 1).strip
 
     menu.include?('Spec').should                  == true
@@ -60,7 +60,7 @@ describe('Menus::Plugin::Menus') do
     menu.include?('<ul class="children">').should == false
   end
 
-  it("Retrieve a menu with only the root elemements") do
+  it "Retrieve a menu with only the root elemements" do
     menu = render_menu(nav_menu.slug, :sub => false).strip
 
     menu.include?('Spec').should                  == true
@@ -69,7 +69,7 @@ describe('Menus::Plugin::Menus') do
     menu.include?('<ul class="children">').should == false
   end
 
-  it('Retrieve a set of items and sort them') do
+  it 'Retrieve a set of items and sort them' do
     menu     = render_menu(nav_menu.slug, :order => :desc, :sub => false)
     menu_asc = render_menu(nav_menu.slug, :order => :asc, :sub => false)
     menu     = Nokogiri::HTML.fragment(menu)
@@ -92,7 +92,7 @@ describe('Menus::Plugin::Menus') do
       .should == item_2.html_class
   end
 
-  it('Not set empty attributes') do
+  it 'Do not set empty attributes' do
     menu = render_menu(nav_menu.slug)
 
     menu.include?('id=""').should == false

data/spec/zen/package/menus/model/menu.rb

@@ -0,0 +1,121 @@
+require File.expand_path('../../../../../helper', __FILE__)
+
+describe 'Menus::Model::Menu' do
+  menu = Menus::Model::Menu.create(:name => 'Spec')
+
+  after do
+    Menus::Model::MenuItem.destroy
+  end
+
+  # The structure in this specification should look like the following:
+  #
+  # item 0
+  # |_ item 1
+  #   |_ item 2
+  #     |_ item 3
+  #       |_ item 4
+  #
+  it 'Build the hierarchy for 5 nested menu items' do
+    parent_id = nil
+    items     = []
+
+    5.times do |i|
+      menu_item = Menus::Model::MenuItem.create(
+        :name       => "Menu item #{i}",
+        :url        => "/#{i}",
+        :menu_id    => menu.id,
+        :sort_order => i,
+        :parent_id  => parent_id
+      )
+
+      parent_id = menu_item.id
+      items    << menu_item
+    end
+
+    tree = menu.menu_items_tree
+
+    tree.length.should == 1
+
+    tree[0][:node].name.should == 'Menu item 0'
+
+    children = tree[0][:children][0]
+
+    children[:node].name.should == 'Menu item 1'
+
+    children = children[:children][0]
+
+    children[:node].name.should == 'Menu item 2'
+
+    children = children[:children][0]
+
+    children[:node].name.should == 'Menu item 3'
+
+    children = children[:children][0]
+
+    children[:node].name.should == 'Menu item 4'
+  end
+
+  # The structure in this test should look like the following:
+  #
+  # item 0
+  # |_ item 1
+  # | |_ item 3
+  # |
+  # |_ item 4
+  #   |_ item 2
+  #
+  it 'Build the tree with non sequential parent IDs' do
+    item = Menus::Model::MenuItem.create(
+      :name       => 'Menu item 0',
+      :url        => '/0',
+      :sort_order => 0,
+      :menu_id    => menu.id
+    )
+
+    item1 = Menus::Model::MenuItem.create(
+      :name       => 'Menu item 1',
+      :url        => '/1',
+      :sort_order => 1,
+      :parent_id  => item.id,
+      :menu_id    => menu.id
+    )
+
+    # Parent ID will be set to the last item.
+    item2 = Menus::Model::MenuItem.create(
+      :name       => 'Menu item 2',
+      :url        => '/2',
+      :sort_order => 4,
+      :menu_id    => menu.id
+    )
+
+    item3 = Menus::Model::MenuItem.create(
+      :name       => 'Menu item 3',
+      :url        => '/3',
+      :sort_order => 2,
+      :parent_id  => item1.id,
+      :menu_id    => menu.id
+    )
+
+    item4 = Menus::Model::MenuItem.create(
+      :name       => 'Menu item 4',
+      :url        => '/4',
+      :sort_order => 3,
+      :menu_id    => menu.id
+    )
+
+    item2.update(:parent_id => item4.id)
+
+    tree = menu.menu_items_tree
+
+    tree.length.should == 2
+
+    tree[0][:node].name.should                             == 'Menu item 0'
+    tree[0][:children][0][:node].name.should               == 'Menu item 1'
+    tree[0][:children][0][:children][0][:node].name.should == 'Menu item 3'
+
+    tree[1][:node].name.should               == 'Menu item 4'
+    tree[1][:children][0][:node].name.should == 'Menu item 2'
+  end
+
+  menu.destroy
+end

data/spec/zen/package/menus/model/menu_item.rb

@@ -0,0 +1,34 @@
+require File.expand_path('../../../../../helper', __FILE__)
+
+describe 'Menus::Model::MenuItem' do
+  menu = Menus::Model::Menu.create(:name => 'Spec')
+
+  after do
+    Menus::Model::MenuItem.destroy
+  end
+
+  it 'Sort orders of menu items should auto increment by default' do
+    item  = menu.add_menu_item(:name => 'Menu item', :url => '/')
+    item1 = menu.add_menu_item(:name => 'Menu item 1', :url => '/1')
+    item2 = menu.add_menu_item(
+      :name       => 'Menu item 2',
+      :url        => '/2',
+      :sort_order => 5
+    )
+
+    item3 = menu.add_menu_item(:name => 'Menu item 3', :url => '/3')
+    item4 = menu.add_menu_item(
+      :name       => 'Menu item 4',
+      :url        => '/4',
+      :sort_order => 10
+    )
+
+    item.sort_order.should  == 0
+    item1.sort_order.should == 1
+    item2.sort_order.should == 5
+    item3.sort_order.should == 6
+    item4.sort_order.should == 10
+  end
+
+  menu.destroy
+end

data/spec/zen/package/sections/controller/section_entries.rb

@@ -1,6 +1,6 @@
 require File.expand_path('../../../../../helper', __FILE__)
 
-describe("Sections::Controller::SectionEntries") do
+describe "Sections::Controller::SectionEntries" do
   behaves_like :capybara
 
   textbox_id  = CustomFields::Model::CustomFieldType[:name => 'textbox'].id
@@ -13,9 +13,7 @@ describe("Sections::Controller::SectionEntries") do
     :comment_format          => 'plain'
   )
 
-  group = CustomFields::Model::CustomFieldGroup.create(
-    :name => 'Spec fields'
-  )
+  group = CustomFields::Model::CustomFieldGroup.create(:name => 'Spec fields')
 
   field = CustomFields::Model::CustomField.create(
     :name                  => 'Spec field',
@@ -50,16 +48,7 @@ describe("Sections::Controller::SectionEntries") do
   title_field   = lang('section_entries.labels.title')
   status_field  = lang('section_entries.special.status_hash.published')
 
-  it('Submit a form without a CSRF token') do
-    response = page.driver.post(
-      Sections::Controller::SectionEntries.r(:save).to_s
-    )
-
-    response.body.include?(lang('zen_general.errors.csrf')).should == true
-    response.status.should                                         == 403
-  end
-
-  it('Find no existing entries') do
+  it 'Find no existing entries' do
     message = lang('section_entries.messages.no_entries')
 
     visit(index_url)
@@ -68,14 +57,26 @@ describe("Sections::Controller::SectionEntries") do
     page.has_content?(message).should           == true
   end
 
-  it("Create a new section entry") do
+  it 'Try to create a new section entry with a missing CSRF token' do
+    visit(index_url)
+    click_link(new_button)
+
+    within '#section_entry_form' do
+      find('input[name="csrf_token"]').set('')
+      click_on(save_button)
+    end
+
+    page.has_content?(lang('zen_general.errors.csrf')).should == true
+  end
+
+  it "Create a new section entry" do
     visit(index_url)
     click_link(new_button)
 
     current_path.should                  == new_url
     page.has_field?('Spec field').should == true
 
-    within('#section_entry_form') do
+    within '#section_entry_form' do
       fill_in(title_field , :with => 'Spec entry')
       select(status_field , :from => 'form_section_entry_status_id')
       fill_in('Spec field', :with => 'Spec field value')
@@ -94,13 +95,38 @@ describe("Sections::Controller::SectionEntries") do
     page.find_field('Spec field').value.should    == 'Spec field value'
   end
 
-  it('Search for a section entry') do
+  it 'Try to create a new entry with invalid data' do
+    visit(index_url)
+    click_link(new_button)
+
+    within '#section_entry_form' do
+      fill_in(title_field, :with => 'Title')
+      click_on(save_button)
+    end
+
+    page.find('input[name="title"]').value.should == 'Title'
+
+    page.has_selector?(
+      "label[for=\"form_custom_field_value_#{field.id}\"] span.error"
+    ).should == true
+
+    within '#section_entry_form' do
+      fill_in(title_field, :with => '')
+      fill_in(field.name, :with => 'Custom value')
+      click_on(save_button)
+    end
+
+    page.has_selector?('label[for="form_title"] span.error').should == true
+    page.find_field(field.name).value.should == 'Custom value'
+  end
+
+  it 'Search for a section entry' do
     search_button = lang('zen_general.buttons.search')
     error         = lang('zen_general.errors.invalid_search')
 
     visit(index_url)
 
-    within('#search_form') do
+    within '#search_form' do
       fill_in('query', :with => 'Spec entry')
       click_on(search_button)
     end
@@ -108,7 +134,7 @@ describe("Sections::Controller::SectionEntries") do
     page.has_content?(error).should        == false
     page.has_content?('Spec entry').should == true
 
-    within('#search_form') do
+    within '#search_form' do
       fill_in('query', :with => 'does not exist')
       click_on(search_button)
     end
@@ -117,14 +143,14 @@ describe("Sections::Controller::SectionEntries") do
     page.has_content?('Spec entry').should == false
   end
 
-  it("Edit an existing section entry") do
+  it "Edit an existing section entry" do
     visit(index_url)
     click_link('Spec entry')
 
     current_path.should =~ /#{edit_url}\/[0-9]+/
 
     # Update the entry
-    within('#section_entry_form') do
+    within '#section_entry_form' do
       fill_in(title_field , :with => 'Spec entry modified')
       fill_in('Spec field', :with => 'Spec field value modified')
       check("form_custom_field_value_#{field_1.id}_1")
@@ -138,14 +164,58 @@ describe("Sections::Controller::SectionEntries") do
       .value.should == 'chuck'
   end
 
-  it("Edit an existing section entry with invalid data") do
+  enable_javascript
+
+  it 'Automatically save a section entry' do
+    visit(index_url)
+    click_link('Spec entry modified')
+
+    within '#section_entry_form' do
+      fill_in('title', :with => 'Spec entry autosave')
+    end
+
+    click_link('Spec fields')
+
+    within '#section_entry_form' do
+      fill_in('Spec field', :with => 'Spec field value autosave')
+    end
+
+    autosave_form('section_entry_form')
+
+    visit(index_url)
+
+    page.has_content?('Spec entry autosave').should == true
+
+    click_link('Spec entry autosave')
+
+    page.find_field('Spec field').value.should == 'Spec field value autosave'
+
+    within '#section_entry_form' do
+      fill_in('title', :with => 'Spec entry modified')
+    end
+
+    click_link('Spec fields')
+
+    within '#section_entry_form' do
+      fill_in('Spec field', :with => 'Spec field value modified')
+      click_on(save_button)
+    end
+
+    page.has_selector?('span.error').should       == false
+    page.find('input[name="title"]').value.should == 'Spec entry modified'
+    page.find_field('Spec field').value.should    == 'Spec field value modified'
+  end
+
+  disable_javascript
+
+  it "Edit an existing section entry with invalid data" do
     visit(index_url)
     click_link('Spec entry')
 
     current_path.should =~ /#{edit_url}\/[0-9]+/
 
     # Update the entry
-    within('#section_entry_form') do
+    within '#section_entry_form' do
       fill_in(field.name , :with => '')
       click_on(save_button)
     end
@@ -155,24 +225,24 @@ describe("Sections::Controller::SectionEntries") do
     ).should == true
   end
 
-  it('Fail to delete an entry without an ID') do
+  it 'Fail to delete an entry without an ID' do
     visit(index_url)
     click_on(delete_button)
 
     page.has_selector?('input[name="section_entry_ids[]"]').should == true
   end
 
-  it("Delete an existing section entry") do
+  it "Delete an existing section entry" do
     visit(index_url)
 
     # Mark the entry
     check('section_entry_ids[]')
     click_on(delete_button)
 
-    page.has_selector?('table tbody tr').should == false
+    page.has_content?('Spec entry modified').should == false
   end
 
-  it('Call the event new_section_entry (before and after)') do
+  it 'Call the event new_section_entry (before and after)' do
     event_name = nil
 
     Zen::Event.listen(:before_new_section_entry) do |entry|
@@ -186,7 +256,7 @@ describe("Sections::Controller::SectionEntries") do
     visit(index_url)
     click_on(new_button)
 
-    within('#section_entry_form') do
+    within '#section_entry_form' do
       fill_in(title_field , :with => 'Entry')
       select(status_field , :from => 'form_section_entry_status_id')
       fill_in('Spec field', :with => 'Spec field value')
@@ -201,7 +271,7 @@ describe("Sections::Controller::SectionEntries") do
     Zen::Event.delete(:before_new_section_entry, :after_new_section_entry)
   end
 
-  it('Call the event edit_section_entry (before and after)') do
+  it 'Call the event edit_section_entry (before and after)' do
     event_name = nil
 
     Zen::Event.listen(:before_edit_section_entry) do |entry|
@@ -215,7 +285,7 @@ describe("Sections::Controller::SectionEntries") do
     visit(index_url)
     click_on('Entry with event')
 
-    within('#section_entry_form') do
+    within '#section_entry_form' do
       click_on(save_button)
     end
 
@@ -226,10 +296,9 @@ describe("Sections::Controller::SectionEntries") do
     Zen::Event.delete(:before_edit_section_entry, :after_edit_section_entry)
   end
 
-  it('Call the event delete_section_entry (before and after)') do
+  it 'Call the event delete_section_entry (before and after)' do
     event_name  = nil
     event_name2 = nil
-    message     = lang('section_entries.messages.no_entries')
 
     Zen::Event.listen(:before_delete_section_entry) do |entry|
       event_name = entry.title
@@ -239,14 +308,10 @@ describe("Sections::Controller::SectionEntries") do
       event_name2 = entry.title
     end
 
-    visit(index_url)
-    check('section_entry_ids[]')
-    click_on(delete_button)
+    Sections::Model::SectionEntry[:slug => 'entry'].destroy
 
-    page.has_selector?('table tbody tr').should == false
-    page.has_content?(message).should           == true
-    event_name.should                           == 'Entry modified'
-    event_name2.should                          == event_name
+    event_name.should  == 'Entry modified'
+    event_name2.should == event_name
 
     Zen::Event.delete(:before_delete_section_entry, :after_delete_section_entry)
   end