activeadmin 0.4.4 → 0.5.0.pre

data/features/index/page_title.feature

@@ -0,0 +1,12 @@
+Feature: Index - Page Title
+
+  Modifying the page title on the index screen
+
+  Scenario: Set a string as the title
+    Given an index configuration of:
+    """
+      ActiveAdmin.register Post do
+        index :title => "Awesome Title"
+      end
+    """
+    Then I should see the page title "Awesome Title"

data/features/new_page.feature

@@ -37,7 +37,7 @@ Feature: New Page
           f.inputs "Publishing" do
             f.input :published_at
           end
-          f.buttons
+          f.actions
         end
       end
     """
@@ -57,7 +57,7 @@ Feature: New Page
       <% url = @post.new_record? ? admin_posts_path : admin_post_path(@post) %>
       <%= active_admin_form_for @post, :url => url do |f|
             f.inputs :title, :body
-            f.buttons 
+            f.actions
           end %>
     """
     Given a configuration of:
@@ -89,7 +89,7 @@ Feature: New Page
           f.inputs "Publishing" do
             f.input :published_at
           end
-          f.buttons
+          f.actions
         end
       end
     """

data/features/registering_pages.feature

@@ -20,6 +20,21 @@ Feature: Registering Pages
     And I should see the Active Admin layout
     And I should see the content "I love chocolate."
 
+  Scenario: Registering a page with a complex name
+    Given a configuration of:
+    """
+    ActiveAdmin.register_page "Chocolate I lØve You!" do
+      content do
+        "I love chocolate."
+      end
+    end
+    """
+    When I go to the dashboard
+    And I follow "Chocolate I lØve You!"
+    Then I should see the page title "Chocolate I lØve You!"
+    And I should see the Active Admin layout
+    And I should see the content "I love chocolate."
+
   Scenario: Registering an empty page
     Given a configuration of:
     """
@@ -30,6 +45,32 @@ Feature: Registering Pages
     Then I should see the page title "Status"
     And I should see the Active Admin layout
 
+  Scenario: Registering a page with a custom title as a string
+    Given a configuration of:
+    """
+    ActiveAdmin.register_page "Status" do
+      content :title => "Custom Page Title" do
+        "I love chocolate."
+      end
+    end
+    """
+    When I go to the dashboard
+    And I follow "Status"
+    Then I should see the page title "Custom Page Title"
+
+  Scenario: Registering a page with a custom title as a proc
+    Given a configuration of:
+    """
+    ActiveAdmin.register_page "Status" do
+      content :title => proc{ "Custom Page Title from Proc" } do
+        "I love chocolate."
+      end
+    end
+    """
+    When I go to the dashboard
+    And I follow "Status"
+    Then I should see the page title "Custom Page Title from Proc"
+
   Scenario: Adding a sidebar section to a page
     Given a configuration of:
     """
@@ -64,3 +105,23 @@ Feature: Registering Pages
     When I go to the dashboard
     And I follow "Status"
     Then I should see an action item link to "Visit"
+
+  Scenario: Adding a page action to a page
+    Given a configuration of:
+    """
+    ActiveAdmin.register_page "Status" do
+      page_action :check do
+        redirect_to admin_status_path
+      end
+
+      content do
+        ("Chocolate I lØve You!" + link_to("Check", admin_status_check_path)).html_safe
+      end
+    end
+    """
+    When I go to the dashboard
+    And I follow "Status"
+    And I follow "Check"
+    Then I should see the content "Chocolate I lØve You!"
+
+

data/features/root_to.feature

@@ -0,0 +1,17 @@
+Feature: Namespace root
+
+  As a developer
+  In order to customize the welcome page
+  I want to set it in the configuration
+
+  Scenario: Default root is the Dashboard
+    Given I am logged in with capybara
+    Then I should be on the dashboard
+
+  Scenario: Set root to "stores#index"
+    Given a configuration of:
+    """
+      ActiveAdmin.application.root_to = 'stores#index'
+    """
+    Given I am logged in with capybara
+    Then I should see the page title "Bookstores"

data/features/show/default_content.feature

@@ -20,8 +20,8 @@ Feature: Show - Default Content
   Scenario: Attributes should link when linked resource is registered
     Given a show configuration of:
       """
-        ActiveAdmin.register User
         ActiveAdmin.register Post
+        ActiveAdmin.register User
       """
     Then I should see the attribute "Author" with "jane_doe"
     And I should see a link to "jane_doe"

data/features/show/page_title.feature

@@ -31,3 +31,17 @@ Feature: Show - Page Title
       end
     """
     Then I should see the page title "Title: Hello World"
+
+  Scenario: Default title
+    Given a show configuration of:
+    """
+      ActiveAdmin.register Post
+    """
+    Then I should see the page title "Hello World"
+
+  Scenario: Default title with no display name method candidate
+    Given a show configuration of:
+    """
+      ActiveAdmin.register Tag
+    """
+    Then I should see the page title "Tag #"

data/features/sidebar_sections.feature

@@ -98,6 +98,32 @@ Feature: Sidebar Sections
     When I follow "New Post"
     Then I should not see a sidebar titled "Help"
 
+  Scenario: Create a sidebar for only one action with if clause with method symbol
+    Given a configuration of:
+    """
+    module SidebarHelper
+      def can_sidebar?; false; end
+    end
+    ActiveAdmin.register Post do
+      controller { helper SidebarHelper }
+      sidebar :help, :only => :index, :if => :can_sidebar? do
+        "Need help? Email us at help@example.com"
+      end
+    end
+    """
+    When I am on the index page for posts
+    Then I should not see a sidebar titled "Help"
+
+    When I follow "View"
+    Then I should not see a sidebar titled "Help"
+
+    When I follow "Edit Post"
+    Then I should not see a sidebar titled "Help"
+
+    When I am on the index page for posts
+    When I follow "New Post"
+    Then I should not see a sidebar titled "Help"
+
   Scenario: Create a sidebar for only one action with if clause that returns true
     Given a configuration of:
     """

data/features/site_title.feature

@@ -41,7 +41,7 @@ Feature: Site title
     Given a configuration of:
     """
       ActiveAdmin.application.site_title_image = nil # Configuration is not reset between scenarios
-      ActiveAdmin.application.site_title = proc { "Hello #{controller.current_admin_user.email}" }
+      ActiveAdmin.application.site_title = proc { "Hello #{controller.current_admin_user.try(:email) || 'you!'}" }
     """
     When I am on the dashboard
     And I should see the site title "Hello admin@example.com"

data/features/specifying_actions.feature

@@ -9,15 +9,15 @@ Feature: Specifying Actions
           actions :index
           index do
             column do |post|
-              link_to "View", admin_post_path(post)
+              link_to "View", "/admin/posts/1"
             end
           end
         end
       """
-	And I am logged in
+    And I am logged in
     And a post with the title "Hello World" exists
     When I am on the index page for posts
-    Then an "AbstractController::ActionNotFound" exception should be raised when I follow "View"
+    Then an "ActionController::RoutingError" exception should be raised when I follow "View"
 
   Scenario: Specify a custom collection action with template
     Given a configuration of:
@@ -30,9 +30,9 @@ Feature: Specifying Actions
           collection_action :import
         end
       """
-    Given "app/views/admin/posts/import.html.erb" contains:
+    Given "app/views/admin/posts/import.html.arb" contains:
       """
-        <p>We are currently working on this feature...</p>
+        para "We are currently working on this feature..."
       """
     And I am logged in
     When I am on the index page for posts
@@ -46,7 +46,7 @@ Feature: Specifying Actions
           action_item(:only => :show) do
             link_to('Review', review_admin_post_path)
           end
-
+  
           member_action :review do
             @post = Post.find(params[:id])
           end
@@ -90,3 +90,4 @@ Feature: Specifying Actions
     Then I should see "Review: Hello World"
     And I should see the page title "Review"
     And I should see the Active Admin layout
+

data/features/step_definitions/action_link_steps.rb

@@ -3,5 +3,5 @@ Then /^I should see a member link to "([^"]*)"$/ do |name|
 end
 
 Then /^I should not see a member link to "([^"]*)"$/ do |name|
-  page.should_not have_css("a.member_link", :text => name)
+  %{Then I should not see "#{name}" within "a.member_link"}
 end

data/features/step_definitions/additional_web_steps.rb

@@ -10,6 +10,10 @@ Then /^I should see a sortable table header with "([^"]*)"$/ do |content|
   page.should have_css('th.sortable', :text => content)
 end
 
+Then /^I should not see a sortable table header with "([^"]*)"$/ do |content|
+  page.should_not have_css('th.sortable', :text => content)
+end
+
 Then /^I should not see a sortable table header$/ do
   step "I should not see \"th.sortable\""
 end
@@ -64,7 +68,9 @@ Then /^I should wait and see "([^"]*)"(?: within "([^"]*)")?$/ do |text, selecto
 end
 
 Then /^I should see the page title "([^"]*)"$/ do |title|
-  page.should have_css('h2#page_title', :text => title)
+  within('h2#page_title') do
+    page.should have_content(title)
+  end
 end
 
 Then /^I should see a fieldset titled "([^"]*)"$/ do |title|

data/features/step_definitions/batch_action_steps.rb

@@ -0,0 +1,94 @@
+Then /^I (should|should not) see the batch action "([^"]*)"$/ do |maybe, title|
+  %{Then I #{maybe} see the batch action :#{title.gsub(' ','').gsub(" Selected", "").underscore} "#{title}"}
+end
+
+Then /^I (should|should not) be asked to confirm "([^"]*)" for "([^"]*)"$/ do |maybe, confirmation, title|
+  within "#batch_actions_popover" do
+    unless maybe == "should not"
+      link = page.find "a.batch_action", :text => title
+      link["data-confirm"].should match( confirmation )
+    else
+      page.should_not have_css("a.batch_action", :text => title)
+    end
+  end
+end
+
+Then /^I (should|should not) see the batch action :([^\s]*) "([^"]*)"$/ do |maybe, sym, title|
+  within "#batch_actions_selector" do
+    unless maybe == "should not"
+      link = page.find "a.batch_action", :text => title
+      link["data-action"].should match( sym )
+      link[:href].should match( "#" )
+    else
+      page.should_not have_css("a.batch_action", :text => title)
+    end
+  end
+end
+
+Then /^the (\d+)(?:st|nd|rd|th) batch action should be "([^"]*)"$/ do |index, title|
+  within "#batch_actions_selector" do
+    page.all( "a.batch_action" )[index.to_i - 1].text.should match( title )
+  end
+end 
+
+When /^I check the (\d+)(?:st|nd|rd|th) record$/ do |index|
+  page.all( "table.index_table input[type=checkbox]" )[index.to_i].set( true )
+end
+
+When /^I uncheck the (\d+)(?:st|nd|rd|th) record$/ do |index|
+  page.all( "table.index_table input[type='checkbox']" )[index.to_i].set( false )
+end
+
+When /^I toggle the collection selection$/ do 
+  toggle_box = page.find( "#collection_selection_toggle_all" )
+  toggle_box.click
+end
+
+Then /^I should see (\d+) record(?:s)? selected$/ do |count|
+  within "table.index_table" do
+    unless count.to_i == 0
+      page.should have_xpath(".//input[@type='checkbox' and @checked='checked']", :count => count)
+    else 
+      page.should have_no_xpath(".//input[@type='checkbox' and @checked='checked']")
+    end
+  end
+end
+
+Then /^I should see that the batch action button is disabled$/ do
+  page.should have_css("#batch_actions_selector .dropdown_menu_button.disabled")
+end
+
+Then /^I (should|should not) see the batch action button$/ do |maybe|
+  if maybe == "should not"
+    page.should_not have_css("div.table_tools #batch_actions_selector .dropdown_menu_button")    
+  else
+    page.should have_css("div.table_tools #batch_actions_selector .dropdown_menu_button")
+  end
+end
+
+Then "I should not see the batch actions selector" do
+  page.should_not have_css("div.table_tools #batch_actions_selector")
+end
+
+Then /^I should see the batch action popover exists$/ do
+  page.should have_css("#batch_actions_selector")
+end
+
+Given /^I submit the batch action form with "([^"]*)"$/ do |action|
+  page.find(:css, "#batch_action").set(action) 
+
+  within("#main_content") do
+    @params = page.all("input").reduce({}) do |acc, input|
+      acc.store(input['name'], input['value'])
+      acc
+    end
+  end
+
+  form = page.find("#collection_selection")
+
+  page.driver.submit(form['method'].to_sym, form['action'], @params)
+end
+
+Then /^I should not see checkboxes in the table$/ do
+  page.should_not have_css( ".paginated_collection table input[type=checkbox]" )
+end

data/features/step_definitions/configuration_steps.rb

@@ -1,10 +1,11 @@
 module ActiveAdminReloading
 
   def load_active_admin_configuration(configuration_content)
+    ActiveAdmin::Event.dispatch ActiveAdmin::Application::BeforeLoadEvent, ActiveAdmin.application
     eval(configuration_content)
-    ActiveAdmin::Event.dispatch ActiveAdmin::Application::LoadEvent, ActiveAdmin.application
+    ActiveAdmin::Event.dispatch ActiveAdmin::Application::AfterLoadEvent, ActiveAdmin.application
     Rails.application.reload_routes!
-    ActiveAdmin.application.namespaces.values.each{|n| n.load_menu! }
+    ActiveAdmin.application.namespaces.values.each{|n| n.reset_menu! }
   end
 
 end
@@ -26,10 +27,22 @@ module ActiveAdminContentsRollback
   def self.rollback!
     recorded_files.each do |filename, contents|
       # contents will be nil if the file didin't exist
-      if contents
-        File.open(filename, "w+") {|f| f << contents }
+      if contents.present?
+        File.open(filename, "w") {|f| f << contents }
       else
         File.delete(filename)
+
+        # Delete parent directories
+        begin
+          dir = File.dirname(filename)
+          until dir == Rails.root
+            Dir.rmdir(dir)
+            dir = dir.split('/')[0..-2].join('/')
+          end
+        rescue Errno::ENOTEMPTY
+          # Directory not empty
+        end
+
       end
     end
 
@@ -46,7 +59,6 @@ end
 
 Given /^a configuration of:$/ do |configuration_content|
   load_active_admin_configuration(configuration_content)
-  ActiveAdmin.application.namespaces.values.each{|n| n.load_menu! }
 end
 
 Given /^an index configuration of:$/ do |configuration_content|
@@ -57,11 +69,21 @@ Given /^an index configuration of:$/ do |configuration_content|
 end
 
 Given /^a show configuration of:$/ do |configuration_content|
+  resource = configuration_content.match(/ActiveAdmin\.register (\w+)/)[1]
   load_active_admin_configuration(configuration_content)
 
-  step 'I am logged in'
-  step "I am on the index page for posts"
-  step 'I follow "View"'
+  case resource
+  when "Post"
+    step 'I am logged in'
+    step "I am on the index page for posts"
+    step 'I follow "View"'
+  when "Tag"
+    step 'I am logged in'
+    Tag.create!
+    visit admin_tag_path(Tag.last)
+  else
+    raise "#{resource} is not supported"
+  end
 end
 
 Given /^"([^"]*)" contains:$/ do |filename, contents|