activeadmin-rb 1.4.0

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

@@ -0,0 +1,58 @@
+require 'rails_helper'
+
+RSpec.describe ActiveAdmin::Views::Panel do
+  let(:arbre_panel) do
+    render_arbre_component do
+      panel "My Title" do
+        header_action link_to("My Link", "https://www.github.com/activeadmin/activeadmin")
+        span("Hello World")
+      end
+    end
+  end
+
+  let(:panel_html) { Capybara.string(arbre_panel.to_s) }
+
+  it "should have a title h3" do
+    expect(panel_html).to have_css 'h3', text: "My Title"
+  end
+
+  it "should add panel actions to the panel header" do
+    link = panel_html.find('h3 > div.header_action a')
+    expect(link.text).to eq('My Link')
+    expect(link[:href]).to eq("https://www.github.com/activeadmin/activeadmin")
+  end
+
+  it "should have a contents div" do
+    expect(panel_html).to have_css 'div.panel_contents'
+  end
+
+  it "should add children to the contents div" do
+    expect(panel_html).to have_css 'div.panel_contents > span', text: "Hello World"
+  end
+
+  context "with html-safe title" do
+    let(:arbre_panel) do
+      title_with_html = %q[Title with <abbr>HTML</abbr>].html_safe
+      render_arbre_component do
+        panel(title_with_html)
+      end
+    end
+
+    it "should allow a html_safe title" do
+      expect(panel_html).to have_css "h3", text: "Title with HTML"
+      expect(panel_html).to have_css "h3 > abbr", text: "HTML"
+    end
+  end
+
+  describe "#children?" do
+    let(:arbre_panel) do
+      render_arbre_component do
+        panel("A Panel")
+      end
+    end
+
+    it "returns false if no children have been added to the panel" do
+      expect(arbre_panel.children?).to eq false
+    end
+  end
+end

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

@@ -0,0 +1,68 @@
+require 'rails_helper'
+
+RSpec.describe ActiveAdmin::Views::SidebarSection do
+
+  let(:options) { {} }
+
+  let(:section) do
+    ActiveAdmin::SidebarSection.new("Help Section", options) do
+      span "Help Me"
+    end
+  end
+
+  let(:html) do
+    render_arbre_component section: section do
+      sidebar_section(assigns[:section])
+    end
+  end
+
+  it "should have a title h3" do
+    expect(html.find_by_tag("h3").first.content).to eq "Help Section"
+  end
+
+  it "should have the class of 'sidebar_section'" do
+    expect(html.class_list).to include("sidebar_section")
+  end
+
+  it "should have an id based on the title" do
+    expect(html.id).to eq "help-section_sidebar_section"
+  end
+
+  it "should have a contents div" do
+    expect(html.find_by_tag("div").first.class_list).to include("panel_contents")
+  end
+
+  it "should add children to the contents div" do
+    expect(html.find_by_tag("span").first.parent).to eq html.find_by_tag("div").first
+  end
+
+  context 'with a custom class attribute' do
+    let(:options) { { class: 'custom_class' } }
+
+    it "should have 'custom_class' class" do
+      expect(html.class_list).to include("custom_class")
+    end
+  end
+
+  context "with attributes_table for resource" do
+    let(:post) { Post.create!(title: "Testing.") }
+    let(:section) do
+      ActiveAdmin::SidebarSection.new("Summary", options) do
+        attributes_table do
+          row :title
+        end
+      end
+    end
+    let(:assigns) { { resource: post, section: section } }
+    let(:html) do
+      render_arbre_component assigns do
+        sidebar_section(assigns[:section])
+      end
+    end
+
+    it "should have table" do
+      expect(html.find_by_tag("th").first.content).to eq "Title"
+      expect(html.find_by_tag("td").first.content).to eq "Testing."
+    end
+  end
+end

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

@@ -0,0 +1,24 @@
+require 'rails_helper'
+
+RSpec.describe ActiveAdmin::Views::Sidebar do
+
+  let(:section) do
+    ActiveAdmin::SidebarSection.new("Section") do
+      para 'Section content.'
+    end
+  end
+
+  let(:html) do
+    render_arbre_component sections: [section] do
+      sidebar assigns[:sections], id: 'sidebar'
+    end
+  end
+
+  it "should have an id of 'sidebar'" do
+    expect(html.id).to eq 'sidebar'
+  end
+
+  it "renders the section" do
+    expect(html.find_by_tag('p').first.content).to eq 'Section content.'
+  end
+end

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

@@ -0,0 +1,82 @@
+require 'rails_helper'
+
+RSpec.describe ActiveAdmin::Views::SiteTitle do
+
+  let(:helpers){ mock_action_view }
+  let(:settings) { ActiveAdmin::SettingsNode.build(ActiveAdmin::NamespaceSettings) }
+
+  def build_title(namespace)
+    render_arbre_component({namespace: namespace}, helpers) do
+      insert_tag ActiveAdmin::Views::SiteTitle, assigns[:namespace]
+    end
+  end
+
+  def double(params)
+    params.each { |key, value| settings.send "#{key}=", value }
+    settings
+  end
+
+  context "when a value" do
+
+    it "renders the string when a string is passed in" do
+      namespace = double site_title: "Hello World",
+                         site_title_image: nil,
+                         site_title_link: nil
+
+      site_title = build_title(namespace)
+      expect(site_title.content).to eq "Hello World"
+    end
+
+    it "renders the return value of a method when a symbol" do
+      expect(helpers).to receive(:hello_world).and_return("Hello World")
+
+      namespace = double site_title: :hello_world,
+                         site_title_image: nil,
+                         site_title_link: nil
+
+      site_title = build_title(namespace)
+      expect(site_title.content).to eq "Hello World"
+    end
+
+    it "renders the return value of a proc" do
+      namespace = double site_title: proc{ "Hello World" },
+                         site_title_image: nil,
+                         site_title_link: nil
+
+      site_title = build_title(namespace)
+      expect(site_title.content).to eq "Hello World"
+    end
+
+  end
+
+  context "when an image" do
+
+    it "renders the string when a string is passed in" do
+      expect(helpers).to receive(:image_tag).
+        with("an/image.png", alt: nil, id: "site_title_image").
+        and_return '<img src="/assets/an/image.png" />'.html_safe
+
+      namespace = double site_title: nil,
+                         site_title_image: "an/image.png",
+                         site_title_link: nil
+
+      site_title = build_title(namespace)
+      expect(site_title.content.strip).to eq '<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 = double site_title: "Hello World",
+                         site_title_image: nil,
+                         site_title_link: "/"
+
+      site_title = build_title(namespace)
+      expect(site_title.content).to eq '<a href="/">Hello World</a>'
+    end
+
+  end
+
+end

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

@@ -0,0 +1,265 @@
+require 'rails_helper'
+
+RSpec.describe ActiveAdmin::Views::StatusTag do
+
+  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) }
+
+    describe '#tag_name' do
+      subject { super().tag_name }
+      it      { is_expected.to eq 'span' }
+    end
+
+    describe '#class_list' do
+      subject { super().class_list }
+      it      { is_expected.to include('status_tag') }
+    end
+
+    context "when status is 'completed'" do
+      subject { status_tag('completed') }
+
+      describe '#tag_name' do
+        subject { super().tag_name }
+        it      { is_expected.to eq 'span' }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('completed') }
+      end
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq 'Completed' }
+      end
+    end
+
+    context "when status is 'in_progress'" do
+      subject { status_tag('in_progress') }
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('in_progress') }
+      end
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq 'In Progress' }
+      end
+    end
+
+    context "when status is 'In progress'" do
+      subject { status_tag('In progress') }
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('in_progress') }
+      end
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq 'In Progress' }
+      end
+    end
+
+    context "when status is an empty string" do
+      subject { status_tag('') }
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq '' }
+      end
+    end
+
+    context "when status is 'false'" do
+      subject { status_tag('false') }
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq('No') }
+      end
+    end
+
+    context "when status is false" do
+      subject { status_tag(false) }
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq('No') }
+      end
+    end
+
+    context "when status is nil" do
+      subject { status_tag(nil) }
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq('No') }
+      end
+    end
+
+    context "when status is 'Active' and type is :ok" do
+      subject { status_tag('Active', :ok) }
+
+      describe 'is deprecated' do
+        subject { super().class_list }
+        it do
+          expect(ActiveAdmin::Deprecation)
+            .to receive(:warn)
+            .with(<<-MSG.strip_heredoc
+              The `type` parameter has been deprecated. Provide the "ok" type as
+              a class instead. For example, `status_tag(status, :ok, class: "abc")`
+              would change to `status_tag(status, class: "ok abc")`. Also note that
+              the "ok" CSS rule will be removed too, so you'll have to provide
+              the styles yourself. See https://github.com/activeadmin/activeadmin/blob/master/CHANGELOG.md#110-
+              for more information.
+            MSG
+            )
+          is_expected.to include('ok')
+        end
+      end
+
+      describe '#class_list' do
+        before  { expect(ActiveAdmin::Deprecation).to receive(:warn).once }
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#class_list' do
+        before  { expect(ActiveAdmin::Deprecation).to receive(:warn).once }
+        subject { super().class_list }
+        it      { is_expected.to include('active') }
+      end
+
+      describe '#class_list' do
+        before  { expect(ActiveAdmin::Deprecation).to receive(:warn).once }
+        subject { super().class_list }
+        it      { is_expected.to include('ok') }
+      end
+    end
+
+    context "when status is 'Active' and class is 'ok'" do
+      subject { status_tag('Active', class: 'ok') }
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('active') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('ok') }
+      end
+    end
+
+    context "when status is 'Active' and label is 'on'" do
+      subject { status_tag('Active', label: 'on') }
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq 'on' }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('active') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.not_to include('on') }
+      end
+    end
+
+    context "when status is 'So useless', class is 'woot awesome' and id is 'useless'" do
+      subject { status_tag('So useless', class: 'woot awesome', id: 'useless') }
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq 'So Useless' }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('status_tag') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('so_useless') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('woot') }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.to include('awesome') }
+      end
+
+      describe '#id' do
+        subject { super().id }
+        it      { is_expected.to eq 'useless' }
+      end
+    end
+
+    context "when status is set to a Fixnum" do
+      subject { status_tag(42) }
+
+      describe '#content' do
+        subject { super().content }
+        it      { is_expected.to eq '42' }
+      end
+
+      describe '#class_list' do
+        subject { super().class_list }
+        it      { is_expected.not_to include('42') }
+      end
+    end
+  end
+end