activeadmin 0.4.4 → 0.5.0.pre

data/spec/unit/views/components/site_title_spec.rb

@@ -0,0 +1,78 @@
+require 'spec_helper'
+
+describe ActiveAdmin::Views::SiteTitle do
+
+  let(:helpers){ mock_action_view }
+
+  def build_title(namespace)
+    render_arbre_component({:namespace => namespace}, helpers) do
+      insert_tag ActiveAdmin::Views::SiteTitle, assigns[:namespace]
+    end
+  end
+
+  context "when a value" do
+
+    it "renders the string when a string is passed in" do
+      namespace = mock :site_title => "Hello World", 
+                       :site_title_image => nil,
+                       :site_title_link => nil
+
+      site_title = build_title(namespace)
+      site_title.content.should == "Hello World"
+    end
+
+    it "renders the return value of a method when a symbol" do
+      helpers.should_receive(:hello_world).and_return("Hello World")
+
+      namespace = mock :site_title => :hello_world,
+                       :site_title_image => nil,
+                       :site_title_link => nil
+
+      site_title = build_title(namespace)
+      site_title.content.should == "Hello World"
+    end
+
+    it "renders the return value of a proc" do
+      namespace = mock :site_title => proc{ "Hello World" },
+                       :site_title_image => nil,
+                       :site_title_link => nil
+
+      site_title = build_title(namespace)
+      site_title.content.should == "Hello World"
+    end
+
+  end
+
+  context "when an image" do
+
+    it "renders the string when a string is passed in" do
+      helpers.should_receive(:image_tag).
+        with("an/image.png", {:alt => nil, :id => "site_title_image"}).
+        and_return("<img src=\"/assets/an/image.png\" />".html_safe)
+
+      namespace = mock :site_title => nil,
+                       :site_title_image => "an/image.png",
+                       :site_title_link => nil
+
+      site_title = build_title(namespace)
+      site_title.content.strip.should == "<img src=\"/assets/an/image.png\" />"
+    end
+
+  end
+
+  context "when a link is present" do
+
+    it "renders the string when a string is passed in" do
+      namespace = mock :site_title => "Hello World", 
+                       :site_title_image => nil,
+                       :site_title_link => "/"
+
+      site_title = build_title(namespace)
+      site_title.content.should == "<a href=\"/\">Hello World</a>"
+    end
+
+  end
+
+
+
+end

data/spec/unit/views/components/status_tag_spec.rb

@@ -2,19 +2,26 @@ require 'spec_helper'
 
 describe ActiveAdmin::Views::StatusTag do
 
-  setup_arbre_context!
-
   describe "#status_tag" do
+
+    # Helper method to build StatusTag objects in an Arbre context
+    def status_tag(*args)
+      render_arbre_component(:status_tag_args => args) do
+        status_tag(*assigns[:status_tag_args])
+      end
+    end
+
     subject { status_tag(nil) }
 
+
     its(:tag_name)    { should == 'span' }
-    its(:class_list)  { should include('status') }
+    its(:class_list)  { should include('status_tag') }
 
     context "when status is 'completed'" do
       subject { status_tag('completed') }
 
       its(:tag_name)    { should == 'span' }
-      its(:class_list)  { should include('status') }
+      its(:class_list)  { should include('status_tag') }
       its(:class_list)  { should include('completed') }
       its(:content)     { should == 'Completed' }
     end
@@ -36,21 +43,21 @@ describe ActiveAdmin::Views::StatusTag do
     context "when status is an empty string" do
       subject { status_tag('') }
 
-      its(:class_list)  { should include('status') }
+      its(:class_list)  { should include('status_tag') }
       its(:content)     { should == '' }
     end
 
     context "when status is nil" do
       subject { status_tag(nil) }
 
-      its(:class_list)  { should include('status') }
+      its(:class_list)  { should include('status_tag') }
       its(:content)     { should == '' }
     end
 
     context "when status is 'Active' and type is :ok" do
       subject { status_tag('Active', :ok) }
 
-      its(:class_list)  { should include('status') }
+      its(:class_list)  { should include('status_tag') }
       its(:class_list)  { should include('active') }
       its(:class_list)  { should include('ok') }
     end
@@ -58,7 +65,7 @@ describe ActiveAdmin::Views::StatusTag do
     context "when status is 'Active' and class is 'ok'" do
       subject { status_tag('Active', :class => 'ok') }
 
-      its(:class_list)  { should include('status') }
+      its(:class_list)  { should include('status_tag') }
       its(:class_list)  { should include('active') }
       its(:class_list)  { should include('ok') }
     end
@@ -67,7 +74,7 @@ describe ActiveAdmin::Views::StatusTag do
       subject { status_tag('Active', :label => 'on') }
 
       its(:content)     { should == 'on' }
-      its(:class_list)  { should include('status') }
+      its(:class_list)  { should include('status_tag') }
       its(:class_list)  { should include('active') }
       its(:class_list)  { should_not include('on') }
     end
@@ -76,7 +83,7 @@ describe ActiveAdmin::Views::StatusTag do
       subject { status_tag('So useless', :ok, :class => 'woot awesome', :id => 'useless') }
 
       its(:content)     { should == 'So Useless' }
-      its(:class_list)  { should include('status') }
+      its(:class_list)  { should include('status_tag') }
       its(:class_list)  { should include('ok') }
       its(:class_list)  { should include('so_useless') }
       its(:class_list)  { should include('woot') }

data/spec/unit/views/components/table_for_spec.rb

@@ -3,19 +3,19 @@ require 'spec_helper'
 describe ActiveAdmin::Views::TableFor do
   describe "creating with the dsl" do
 
-    setup_arbre_context!
-
-    let(:assigns){ {} }
-    let(:helpers){ mock_action_view }
-
     let(:collection) do
       [Post.new(:title => "First Post"), Post.new(:title => "Second Post"), Post.new(:title => "Third Post")]
     end
 
+    let(:assigns){ { :collection => collection } }
+    let(:helpers){ mock_action_view }
+
     context "when creating a column with a symbol" do
       let(:table) do
-        table_for(collection) do
-          column :title
+        render_arbre_component assigns, helpers do
+          table_for(collection) do
+            column :title
+          end
         end
       end
 
@@ -36,9 +36,11 @@ describe ActiveAdmin::Views::TableFor do
 
     context "when creating many columns with symbols" do
       let(:table) do
-        table_for(collection) do
-          column :title
-          column :created_at
+        render_arbre_component assigns, helpers do
+          table_for(collection) do
+            column :title
+            column :created_at
+          end
         end
       end
 
@@ -46,6 +48,11 @@ describe ActiveAdmin::Views::TableFor do
         table.find_by_tag("th").first.content.should == "Title"
         table.find_by_tag("th").last.content.should == "Created At"
       end
+      
+      it "should add a class to each table header based on the col name" do
+        table.find_by_tag("th").first.class_list.should include("title")
+        table.find_by_tag("th").last.class_list.should  include("created_at")
+      end
 
       it "should create a table row for each element in the collection" do
         table.find_by_tag("tr").size.should == 4 # 1 for head, 3 for rows
@@ -54,13 +61,20 @@ describe ActiveAdmin::Views::TableFor do
       it "should create a cell for each column" do
         table.find_by_tag("td").size.should == 6
       end
+      
+      it "should add a class for each cell based on the col name" do
+        table.find_by_tag("td").first.class_list.should include("title")
+        table.find_by_tag("td").last.class_list.should  include("created_at")
+      end
     end
 
     context "when creating a column with block content" do
       let(:table) do
-        table_for(collection) do
-          column :title do |post|
-            span(post.title)
+        render_arbre_component assigns, helpers do
+          table_for(collection) do
+            column :title do |post|
+              span(post.title)
+            end
           end
         end
       end
@@ -76,10 +90,12 @@ describe ActiveAdmin::Views::TableFor do
 
     context "when creating a column with multiple block content" do
       let(:table) do
-        table_for(collection) do
-          column :title do |post|
-            span(post.title)
-            span(post.title)
+        render_arbre_component assigns, helpers do
+          table_for(collection) do
+            column :title do |post|
+              span(post.title)
+              span(post.title)
+            end
           end
         end
       end
@@ -127,5 +143,9 @@ describe ActiveAdmin::Views::TableFor do
       it { should_not be_sortable }
     end
 
+    context "when :sortable column is an association" do
+      let(:table_column){ build_column("Category", :category, Post) }
+      it { should_not be_sortable }
+    end
   end
 end

data/spec/unit/views/pages/layout_spec.rb

@@ -2,19 +2,60 @@ require 'spec_helper'
 
 describe ActiveAdmin::Views::Pages::Layout do
 
-  describe "the page title" do
+  let(:assigns){ {} }
+  let(:helpers) do 
+    helpers = mock_action_view
 
-    it "should be the @page_title if assigned in the controller" do
-      assigns = {:page_title => "My Page Title"}
-      layout = ActiveAdmin::Views::Pages::Layout.new(assigns, nil)
-      layout.title.should == "My Page Title"
+    helpers.stub :active_admin_application => active_admin_application,
+                 :active_admin_config => mock('Config', :action_items? => nil, :sidebar_sections? => nil),
+                 :active_admin_namespace => active_admin_namespace,
+                 :breadcrumb_links => [],
+                 :content_for => "",
+                 :csrf_meta_tag => "",
+                 :current_active_admin_user => nil,
+                 :current_active_admin_user? => false,
+                 :current_menu => mock('Menu', :items => []),
+                 :flash => {},
+                 :javascript_path => "/dummy/",
+                 :link_to => "",
+                 :render_or_call_method_or_proc_on => "",
+                 :stylesheet_link_tag => mock(:html_safe => ""),
+                 :view_factory => view_factory,
+                 :params => {:controller => 'UsersController', :action => 'edit'}
+
+    helpers
+  end
+
+  let(:active_admin_namespace){ ActiveAdmin::Namespace.new(ActiveAdmin::Application.new, :myspace) }
+  let(:active_admin_application){ ActiveAdmin.application }
+  let(:view_factory) { ActiveAdmin::ViewFactory.new }
+
+  let(:layout) do
+    render_arbre_component assigns, helpers do
+      insert_tag ActiveAdmin::Views::Pages::Layout
+    end
+  end
+
+  it "should be the @page_title if assigned in the controller" do
+    assigns[:page_title] = "My Page Title"
+
+    layout.title.should == "My Page Title"
+  end
+
+  it "should be the default translation" do
+    helpers.params[:action] = "edit"
+
+    layout.title.should == "Edit"
+  end
+
+  describe "the body" do
+
+    it "should have class 'active_admin'" do
+      layout.build.class_list.should include 'active_admin'
     end
 
-    it "should be the default translation" do
-      assigns = {}
-      helpers = mock(:params => {:action => 'edit'})
-      layout = ActiveAdmin::Views::Pages::Layout.new(assigns, helpers)
-      layout.title.should == "Edit"
+    it "should have namespace class" do
+      layout.build.class_list.should include "#{active_admin_namespace.name}_namespace"
     end
 
   end

data/spec/unit/views/tabbed_navigation_spec.rb

@@ -1,12 +1,19 @@
 require 'spec_helper'
 
+include ActiveAdmin
 describe ActiveAdmin::Views::TabbedNavigation do
 
-  setup_arbre_context!
-  include ActiveAdmin::ViewHelpers
-
   let(:menu){ ActiveAdmin::Menu.new }
-  let(:tabbed_navigation){ insert_tag(ActiveAdmin::Views::TabbedNavigation, menu) }
+
+  let(:assigns){ { :active_admin_menu => menu } }
+  let(:helpers){ mock_action_view }
+
+  let(:tabbed_navigation) do 
+    arbre(assigns, helpers) {
+      insert_tag(ActiveAdmin::Views::TabbedNavigation, active_admin_menu)
+    }.children.first
+  end
+
   let(:html) { tabbed_navigation.to_s }
 
   before do
@@ -16,18 +23,30 @@ describe ActiveAdmin::Views::TabbedNavigation do
   describe "rendering a menu" do
 
     before do
-      menu.add "Blog Posts", :admin_posts_path
-      menu.add "Reports", "/admin/reports"
+      menu.add MenuItem.new(:label => "Blog Posts",		:url => :admin_posts_path)
+      menu.add MenuItem.new(:label => "Reports",		:url => "/admin/reports")
+      menu.add MenuItem.new(:label => "Administration", :url => "/admin/administration")
+      menu.add MenuItem.new(:label => "Management",		:url => "#")
+
       reports = menu["Reports"]
-      reports.add "A Sub Reports", "/admin/a-sub-reports"
-      reports.add "B Sub Reports", "/admin/b-sub-reports"
-      menu.add "Administration", "/admin/administration"
+      reports.add MenuItem.new(:label => "A Sub Reports", :url => "/admin/a-sub-reports")
+      reports.add MenuItem.new(:label => "B Sub Reports", :url => "/admin/b-sub-reports")
+
       administration = menu["Administration"]
-      administration.add "User administration", '/admin/user-administration', 10, :if => proc { false }
-      menu.add "Management", "#"
+      administration.add MenuItem.new(:label => "User administration", 
+									  :url => '/admin/user-administration', 
+									  :priority => 10, 
+									  :if => proc { false })
+
       management = menu["Management"]
-      management.add "Order management", '/admin/order-management', 10, :if => proc { false }
-      management.add "Bill management", '/admin/bill-management', 10, :if => :admin_logged_in?
+      management.add MenuItem.new(:label => "Order management", 
+								  :url => '/admin/order-management', 
+								  :priority => 10, 
+								  :if => proc { false })
+      management.add MenuItem.new(:label => "Bill management", 
+								  :url => '/admin/bill-management', 
+								  :priority => 10, 
+								  :if => :admin_logged_in?)
     end
 
     it "should generate a ul" do
@@ -74,12 +93,12 @@ describe ActiveAdmin::Views::TabbedNavigation do
     describe "marking current item" do
 
       it "should add the 'current' class to the li" do
-        assigns[:current_tab] = "Blog Posts"
+        assigns[:current_tab] = menu["Blog Posts"]
         html.should have_tag("li", :attributes => { :class => "current" })
       end
 
       it "should add the 'current' and 'has_nested' classes to the li and 'current' to the sub li" do
-        assigns[:current_tab] = "Reports/A Sub Reports"
+        assigns[:current_tab] = menu["Reports"]["A Sub Reports"]
         html.should have_tag("li", :attributes => { :id => "reports", :class => /current/ })
         html.should have_tag("li", :attributes => { :id => "reports", :class => /has_nested/ })
         html.should have_tag("li", :attributes => { :id => "a_sub_reports", :class => "current" })
@@ -92,32 +111,34 @@ describe ActiveAdmin::Views::TabbedNavigation do
   describe "returning the menu items to display" do
 
     it "should be reture one item with no if block" do
-      menu.add "Hello World", "/"
+      menu.add MenuItem.new(:label => "Hello World", :url => "/")
       tabbed_navigation.menu_items.should == menu.items
     end
 
     it "should not include a menu items with an if block that returns false" do
-      menu.add "Don't Show", "/", 10, :if => proc{ false }
+      menu.add MenuItem.new(:label => "Don't Show", :url => "/", :priority => 10, :if => proc{ false })
       tabbed_navigation.menu_items.should == []
     end
 
     it "should not include menu items with an if block that calls a method that returns false" do
-      menu.add "Don't Show", "/", 10, :if => :admin_logged_in?
+      menu.add MenuItem.new(:label => "Don't Show", :url => "/", :priority => 10, :if => :admin_logged_in?)
       tabbed_navigation.menu_items.should == []
     end
 
     it "should not display any items that have no children to display" do
-      menu.add "Parent", "#" do |p|
-        p.add "Child", "/", 10, :if => proc{ false }
+      item = MenuItem.new(:label => "Parent", :url => "#") do |p|
+        p.add MenuItem.new(:label => "Child", :url => "/", :priority => 10, :if => proc{ false })
       end
+	  menu.add item
       tabbed_navigation.menu_items.should == []
     end
 
     it "should display a parent that has a child to display" do
-      menu.add "Parent", "#" do |p|
-        p.add "Hidden Child", "/", 10, :if => proc{ false }
-        p.add "Child", "/"
+      item = MenuItem.new(:label => "Parent", :url => "#") do |p|
+        p.add MenuItem.new(:label => "Hidden Child", :url => "/", :priority => 10, :if => proc{ false })
+        p.add MenuItem.new(:label => "Child", :url => "/")
       end
+	  menu.add item
       tabbed_navigation.should have(1).menu_items
     end