activeadmin-rails 1.7.0

data/spec/javascripts/coffeescripts/jquery.aa.table-checkbox-toggler-spec.js.coffee

@@ -0,0 +1,34 @@
+describe "ActiveAdmin.TableCheckboxToggler", ->
+  
+  beforeEach ->
+    loadFixtures('table_checkboxes.html');
+
+    @collection = $("#collection")
+    @toggle_all = @collection.find(".toggle_all")
+
+    @checkboxes = @collection.find(":checkbox").not(@toggle_all)
+    
+    new ActiveAdmin.TableCheckboxToggler({}, @collection)
+
+  describe "'selected' class for table row", ->
+    it "should add the class 'selected' to rows when their checkbox is checked ", ->
+      checkbox = $("#item_1")
+      checkbox.attr("checked", "checked")      
+      checkbox.trigger("change")
+
+      expect(checkbox.parents("tr")).toHaveAttr("class", "selected")
+
+    it "should remove the class 'selected' from rows when their checkbox is unchecked ", ->
+      checkbox = $("#item_1")
+      checkbox.trigger("change")
+
+      expect(checkbox.parents("tr")).not.toHaveAttr("class", "selected")
+
+  describe "clicking a cell", ->
+    it "should toggle the checkbox when a cell is clicked", ->
+      checkbox = $("#item_1")
+      row = checkbox.parents("td")
+      $(row).trigger("click")
+
+      expect(checkbox).toHaveAttr("checked", "checked")
+

data/spec/javascripts/fixtures/checkboxes.html

@@ -0,0 +1,9 @@
+<div id="collection">
+  <input type="checkbox" name="toggle_all" class="toggle_all" value="">
+
+  <input type="checkbox" name="item1" id="item_1" value="1">
+  <input type="checkbox" name="item2" id="item_2" value="2">
+  <input type="checkbox" name="item3" id="item_3" value="3">
+  <input type="checkbox" name="item4" id="item_4" value="4">
+</div>
+

data/spec/javascripts/fixtures/table_checkboxes.html

@@ -0,0 +1,17 @@
+<table id="collection">
+  <thead>
+    <tr>
+      <td>
+        <input type="checkbox" name="toggle_all" class="toggle_all" value="">
+      </td>
+    </tr>
+  </thead>
+  <tbody>
+    <tr><td><input type="checkbox" name="item1" id="item_1" value="1"></td></tr>
+    <tr><td><input type="checkbox" name="item2" id="item_2" value="2"></td></tr>
+    <tr><td><input type="checkbox" name="item3" id="item_3" value="3"></td></tr>
+    <tr><td><input type="checkbox" name="item4" id="item_4" value="4"></td></tr>
+  </tbody>
+</table>
+
+

data/spec/javascripts/helpers/SpecHelper.js

@@ -0,0 +1,3 @@
+beforeEach(function() {
+  window.inject = $.jasmine.inject;
+});

data/spec/javascripts/support/jasmine.yml

@@ -0,0 +1,74 @@
+# src_files
+#
+# Return an array of filepaths relative to src_dir to include before jasmine specs.
+# Default: []
+#
+# EXAMPLE:
+#
+# src_files:
+#   - lib/source1.js
+#   - lib/source2.js
+#   - dist/**/*.js
+#
+src_files:
+  - spec/javascripts/support/jquery.min.js
+  - spec/javascripts/support/jquery-ui-1.8.16.custom.min.js
+  - app/assets/javascripts/active_admin/compiled/lib/**/*.js
+  - app/assets/javascripts/active_admin/compiled/components/**/*.js
+
+# stylesheets
+#
+# Return an array of stylesheet filepaths relative to src_dir to include before jasmine specs.
+# Default: []
+#
+# EXAMPLE:
+#
+# stylesheets:
+#   - css/style.css
+#   - stylesheets/*.css
+#
+stylesheets:
+
+# helpers
+#
+# Return an array of filepaths relative to spec_dir to include before jasmine specs.
+# Default: ["helpers/**/*.js"]
+#
+# EXAMPLE:
+#
+helpers:
+  - helpers/**/*.js
+
+
+# spec_files
+#
+#  Return an array of filepaths relative to spec_dir to include.
+# Default: ["**/*[sS]pec.js"]
+#
+# EXAMPLE:
+#
+spec_files:
+  - **/*[sS]pec.js
+
+
+# src_dir
+#
+# Source directory path. Your src_files must be returned relative to this path. Will use root if left blank.
+# Default: project root
+#
+# EXAMPLE:
+#
+# src_dir: public
+#
+src_dir:
+
+# spec_dir
+#
+# Spec directory path. Your spec_files must be returned relative to this path.
+# Default: spec/javascripts
+#
+# EXAMPLE:
+#
+# spec_dir: spec/javascripts
+#
+spec_dir: spec/javascripts/

data/spec/javascripts/support/jasmine_runner.rb

@@ -0,0 +1,19 @@
+$:.unshift(ENV['JASMINE_GEM_PATH']) if ENV['JASMINE_GEM_PATH'] # for gem testing purposes
+
+require 'jasmine'
+require 'spec'
+
+jasmine_config = Jasmine::Config.new
+spec_builder = Jasmine::SpecBuilder.new(jasmine_config)
+
+should_stop = false
+
+Spec::Runner.configure do |config|
+  config.after(:suite) do
+    spec_builder.stop if should_stop
+  end
+end
+
+spec_builder.start
+should_stop = true
+spec_builder.declare_suites

data/spec/rails_helper.rb

@@ -0,0 +1,67 @@
+require 'spec_helper'
+
+ENV['RAILS_ENV'] = 'test'
+
+require 'rails'
+ENV['RAILS_ROOT'] = File.expand_path("../rails/rails-#{Rails.version}", __FILE__)
+
+# Create the test app if it doesn't exists
+unless File.exist?(ENV['RAILS_ROOT'])
+  system 'rake setup'
+end
+
+require 'active_record'
+require 'active_admin'
+require 'devise'
+ActiveAdmin.application.load_paths = [ENV['RAILS_ROOT'] + "/app/admin"]
+
+require ENV['RAILS_ROOT'] + '/config/environment'
+
+require 'rspec/rails'
+
+# Disabling authentication in specs so that we don't have to worry about
+# it allover the place
+ActiveAdmin.application.authentication_method = false
+ActiveAdmin.application.current_user_method = false
+
+RSpec.configure do |config|
+  config.disable_monkey_patching!
+  config.use_transactional_fixtures = true
+  config.use_instantiated_fixtures = false
+  config.render_views = false
+  config.filter_run focus: true
+  config.filter_run_excluding skip: true
+  config.run_all_when_everything_filtered = true
+  config.color = true
+  config.order = :random
+  config.example_status_persistence_file_path = ".rspec_failures"
+
+  devise = ActiveAdmin::Dependency.devise >= '4.2' ? Devise::Test::ControllerHelpers : Devise::TestHelpers
+  config.include devise, type: :controller
+
+  require 'support/active_admin_integration_spec_helper'
+  config.include ActiveAdminIntegrationSpecHelper
+
+  require 'support/active_admin_request_helpers'
+  config.include ActiveAdminRequestHelpers, type: :request
+
+  # Setup Some Admin stuff for us to play with
+  config.before(:suite) do
+    ActiveAdminIntegrationSpecHelper.load_defaults!
+    ActiveAdminIntegrationSpecHelper.reload_routes!
+  end
+
+  # Clear cached ActionView template partials.
+  config.after { ActionView::LookupContext::DetailsKey.clear }
+end
+
+# Force deprecations to raise an exception.
+if Rails.gem_version >= Gem::Version.new("7.1.0")
+  Rails.application.deprecators.behavior = :raise
+else
+  ActiveSupport::Deprecation.behavior = :raise
+end
+
+# improve the performance of the specs suite by not logging anything
+# see http://blog.plataformatec.com.br/2011/12/three-tips-to-improve-the-performance-of-your-test-suite/
+Rails.logger.level = Logger::FATAL

data/spec/requests/default_namespace_spec.rb

@@ -0,0 +1,81 @@
+require 'rails_helper'
+
+RSpec.describe ActiveAdmin::Application, type: :request do
+
+  include Rails.application.routes.url_helpers
+
+  let(:resource) { ActiveAdmin.register Category }
+
+  [false, nil].each do |value|
+
+    describe "with a #{value} default namespace" do
+
+      around do |example|
+        with_custom_default_namespace(value) { example.call }
+      end
+
+      it "should generate resource paths" do
+        expect(resource.route_collection_path).to eq "/categories"
+      end
+
+      it "should generate a log out path" do
+        expect(destroy_admin_user_session_path).to eq "/logout"
+      end
+
+      it "should generate a log in path" do
+        expect(new_admin_user_session_path).to eq "/login"
+      end
+
+    end
+
+  end
+
+  describe "with a test default namespace" do
+
+    around do |example|
+      with_custom_default_namespace(:test) { example.call }
+    end
+
+    it "should generate resource paths" do
+      expect(resource.route_collection_path).to eq "/test/categories"
+    end
+
+    it "should generate a log out path" do
+      expect(destroy_admin_user_session_path).to eq "/test/logout"
+    end
+
+    it "should generate a log in path" do
+      expect(new_admin_user_session_path).to eq "/test/login"
+    end
+
+  end
+
+  describe "with a namespace with underscores in the name" do
+
+    around do |example|
+      with_custom_default_namespace(:abc_123) { example.call }
+    end
+
+    it "should generate resource paths" do
+      expect(resource.route_collection_path).to eq "/abc_123/categories"
+    end
+
+    it "should generate a log out path" do
+      expect(destroy_admin_user_session_path).to eq "/abc_123/logout"
+    end
+
+    it "should generate a log in path" do
+      expect(new_admin_user_session_path).to eq "/abc_123/login"
+    end
+
+  end
+
+  private
+
+  def with_custom_default_namespace(namespace)
+    application = ActiveAdmin::Application.new
+    application.default_namespace = namespace
+
+    with_temp_application(application) { yield }
+  end
+end

data/spec/requests/memory_spec.rb

@@ -0,0 +1,25 @@
+require 'rails_helper'
+
+RSpec.describe "Memory Leak", type: :request, if: RUBY_ENGINE == 'ruby' do
+  before do
+    load_defaults!
+  end
+
+  def count_instances_of(klass)
+    ObjectSpace.each_object(klass) { }
+  end
+
+  [ActiveAdmin::Namespace, ActiveAdmin::Resource].each do |klass|
+    it "should not leak #{klass}" do
+      previously_disabled = GC.enable
+      GC.start
+      count = count_instances_of(klass)
+
+      load_defaults!
+
+      GC.start
+      GC.disable if previously_disabled
+      expect(count_instances_of klass).to be <= count
+    end
+  end
+end

data/spec/requests/pages/form_spec.rb

@@ -0,0 +1,73 @@
+require 'rails_helper'
+
+RSpec.describe 'views/resources/edit, new', type: :request do
+  include Rails.application.routes.url_helpers
+
+  describe "#title" do
+    let!(:application) { ActiveAdmin::Application.new }
+    let(:namespace) { application.namespace(:root) { |n| n.site_title = nil } }
+    let(:page) { Capybara.string(response.body) }
+
+    around do |example|
+      with_temp_application(application) do
+        load_resources { resource }
+        example.call
+      end
+      namespace.unload!
+    end
+
+    context "when :title is set" do
+      let(:resource) do
+        namespace.register User do
+          form(title: "My Page Title")
+        end
+      end
+
+      it "should show the set page title" do
+        get new_user_path
+        expect(page.title).to eq "My Page Title"
+      end
+    end
+
+    context "when page_title is assigned" do
+      let(:resource) do
+        namespace.register User do
+          controller do
+            def new
+              @page_title = "My Page Title"
+              new!
+            end
+          end
+        end
+      end
+
+      it "should show the set page title" do
+        get new_user_path
+        expect(page.title).to eq "My Page Title"
+      end
+    end
+
+    context "when page_title is not assigned" do
+      let(:resource) do
+        namespace.register Post do
+        end
+      end
+      let(:post){ Post.create! }
+
+      it "should show the correct I18n text on the new action" do
+        get new_post_path
+        expect(page).to have_css 'head title', text: /New Post/, count: 1, visible: false
+        expect(page.title).to eq 'New Post'
+      end
+
+      it "should show the correct I18n text on the edit action" do
+        get edit_post_path(post)
+
+        expect(page).to have_css 'head title', text: /Edit Post/, count: 1, visible: false
+        expect(page).to have_css 'body.active_admin', count: 1
+        expect(page).to have_css "body.#{namespace.name}_namespace", count: 1
+        expect(page.title).to eq 'Edit Post'
+      end
+    end
+  end
+end

data/spec/requests/pages/index_spec.rb

@@ -0,0 +1,96 @@
+require 'rails_helper'
+
+RSpec.describe 'views/resource/index', type: :request do
+  include Rails.application.routes.url_helpers
+
+  let(:application){ ActiveAdmin::Application.new }
+  let(:namespace) { application.namespace(:root) { |n| n.site_title = nil } }
+  let(:resource) { namespace.register Post }
+  let(:page) { Capybara.string(response.body) }
+
+  around do |example|
+    with_temp_application(application) do
+      load_resources { resource }
+      example.call
+    end
+    namespace.unload!
+  end
+
+  describe "#title" do
+    context "when config[:title] is assigned" do
+      context "with a Proc" do
+        let(:resource) do
+          namespace.register User do
+            index(title: ->{ "My Page Title" })
+          end
+        end
+
+        it "should return the value of the assigned Proc" do
+          get resource.route_collection_path
+          expect(page.title).to eq "My Page Title"
+        end
+      end
+
+      context "with a String" do
+        let(:resource) do
+          namespace.register User do
+            index(title: "My Page Title")
+          end
+        end
+
+        it "should return the assigned String" do
+          get resource.route_collection_path
+          expect(page.title).to eq "My Page Title"
+        end
+      end
+
+      context "with a Integer" do
+        let(:resource) do
+          namespace.register User do
+            index(title: 1)
+          end
+        end
+
+        it "should return the Integer" do
+          get resource.route_collection_path
+          expect(page.title).to eq '1'
+        end
+      end
+    end
+
+    context "when page_title is assigned" do
+      let(:resource) do
+        namespace.register User do
+          controller do
+            def index
+              @page_title = "My Page Title"
+              index!
+            end
+          end
+        end
+      end
+
+      it "should return the set page title" do
+        get resource.route_collection_path
+        expect(page.title).to eq "My Page Title"
+      end
+    end
+
+    context "when page_title is not assigned" do
+      it "should return the correct I18n text" do
+        get resource.route_collection_path
+
+        expect(page).to have_css 'head title', text: /Posts/, count: 1, visible: false
+        expect(page.title).to eq 'Posts'
+      end
+    end
+  end
+
+  describe "#sidebar" do
+    it "displays sidebar by default" do
+      get resource.route_collection_path
+      expect(page).to have_css 'div.sidebar#sidebar'
+      expect(page).to have_css 'form.filter_form[method=get]'
+    end
+  end
+end

data/spec/requests/pages/layout_spec.rb

@@ -0,0 +1,59 @@
+require 'rails_helper'
+
+RSpec.describe 'layout', type: :request do
+  include Rails.application.routes.url_helpers
+
+  let(:application){ ActiveAdmin::Application.new }
+  let(:namespace) { application.namespace(:root) }
+  let(:resource) { namespace.register User }
+  let(:page) { Capybara.string(response.body) }
+  let(:user) { User.create! }
+
+  around do |example|
+    with_temp_application(application) do
+      load_resources { resource }
+      example.call
+    end
+    namespace.unload!
+  end
+
+  describe "the head" do
+
+    it "should have title tag" do
+      get edit_user_path(user)
+
+      expect(page).to have_css 'head title', text: /Edit User \|/, count: 1, visible: false
+    end
+  end
+
+  describe "the body" do
+
+    it "should have class 'active_admin'" do
+      get edit_user_path(user)
+      expect(page).to have_css 'body.active_admin', count: 1
+    end
+
+    it "should have namespace class" do
+      get edit_user_path(user)
+      expect(page).to have_css "body.#{namespace.name}_namespace", count: 1
+    end
+
+  end
+
+  context "unsupported browser" do
+    let(:user_agent) { "Mozilla/5.0 (compatible; MSIE 7.0; Windows NT 6.2; Trident/6.0)" }
+
+    it "should display appropriate text" do
+      get edit_user_path(user), headers: { 'HTTP_USER_AGENT' => user_agent }
+
+      expect(page).to have_css 'div.unsupported_browser h1', text: /no longer supports Internet Explorer versions 8/
+      expect(page).to have_css 'div.unsupported_browser p', text: /upgrade your browser/
+      expect(page).to have_css 'div.unsupported_browser p', text: /turn off "Compatibility View"/
+    end
+  end
+
+  it "should not display browser warning by default" do
+    get edit_user_path(user)
+    expect(page).not_to have_css 'div.unsupported_browser'
+  end
+end

data/spec/requests/pages/show_spec.rb

@@ -0,0 +1,38 @@
+require 'rails_helper'
+
+RSpec.describe 'views/resource/show', type: :request do
+  include Rails.application.routes.url_helpers
+
+  let!(:application) { ActiveAdmin::Application.new }
+  let(:namespace) { application.namespace(:root) { |n| n.site_title = nil } }
+  let(:page) { Capybara.string(response.body) }
+  let(:post) { Post.create! }
+
+  around do |example|
+    with_temp_application(application) do
+      load_resources { resource }
+      example.call
+    end
+    namespace.unload!
+  end
+
+  describe "the resource" do
+    let(:resource) do
+      namespace.register Post do
+        show do |post|
+          attributes_table do
+            row :field
+          end
+        end
+      end
+    end
+
+    it "renders the page" do
+      get post_path(post)
+      expect(page).to have_css 'div#main_content', count: 1
+      expect(page).to have_css 'div.attributes_table tr.row-field th', text: /Field/
+    end
+
+  end
+
+end

data/spec/requests/stylesheets_spec.rb

@@ -0,0 +1,18 @@
+require 'rails_helper'
+
+RSpec.describe "Stylesheets", type: :request do
+
+  require "sprockets"
+
+  let(:css) do
+    assets = Rails.application.assets
+    assets.find_asset("active_admin.css")
+  end
+  it "should successfully render the scss stylesheets using sprockets" do
+    expect(css).to_not eq nil
+  end
+  it "should not have any syntax errors" do
+    expect(css.to_s).to_not include("Syntax error:")
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1 @@
+require 'simplecov' if ENV["COVERAGE"] == "true"