activeadmin 1.3.1 → 1.4.0

data/spec/unit/asset_registration_spec.rb

@@ -1,81 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe ActiveAdmin::AssetRegistration do
-  include ActiveAdmin::AssetRegistration
-
-  before do
-    clear_stylesheets!
-    clear_javascripts!
-  end
-
-  it "is deprecated" do
-    expect(ActiveAdmin::Deprecation)
-      .to receive(:warn)
-      .with(<<-MSG.strip_heredoc
-        The `register_stylesheet` config is deprecated and will be removed
-        in v2. Import your "sample_styles.css" stylesheet in the active_admin.scss.
-      MSG
-      )
-
-    register_stylesheet "sample_styles.css"
-
-    expect(ActiveAdmin::Deprecation)
-      .to receive(:warn)
-      .with(<<-MSG.strip_heredoc
-        The `register_javascript` config is deprecated and will be removed
-        in v2. Import your "sample_scripts.js" javascript in the active_admin.js.
-      MSG
-      )
-
-    register_javascript "sample_scripts.js"
-  end
-
-  it "should register a stylesheet file" do
-    expect(ActiveAdmin::Deprecation).to receive(:warn).once
-    register_stylesheet "active_admin.css"
-    expect(stylesheets.length).to eq 1
-    expect(stylesheets.keys.first).to eq "active_admin.css"
-  end
-
-  it "should clear all existing stylesheets" do
-    expect(ActiveAdmin::Deprecation).to receive(:warn).once
-    register_stylesheet "active_admin.css"
-    expect(stylesheets.length).to eq 1
-    clear_stylesheets!
-    expect(stylesheets).to be_empty
-  end
-
-  it "should allow media option when registering stylesheet" do
-    expect(ActiveAdmin::Deprecation).to receive(:warn).once
-    register_stylesheet "active_admin.css", media: :print
-    expect(stylesheets.values.first[:media]).to eq :print
-  end
-
-  it "shouldn't register a stylesheet twice" do
-    expect(ActiveAdmin::Deprecation).to receive(:warn).twice
-    register_stylesheet "active_admin.css"
-    register_stylesheet "active_admin.css"
-    expect(stylesheets.length).to eq 1
-  end
-
-  it "should register a javascript file" do
-    expect(ActiveAdmin::Deprecation).to receive(:warn).once
-    register_javascript "active_admin.js"
-    expect(javascripts).to eq ["active_admin.js"].to_set
-  end
-
-  it "should clear all existing javascripts" do
-    expect(ActiveAdmin::Deprecation).to receive(:warn).once
-    register_javascript "active_admin.js"
-    expect(javascripts).to eq ["active_admin.js"].to_set
-    clear_javascripts!
-    expect(javascripts).to be_empty
-  end
-
-  it "shouldn't register a javascript twice" do
-    expect(ActiveAdmin::Deprecation).to receive(:warn).twice
-    register_javascript "active_admin.js"
-    register_javascript "active_admin.js"
-    expect(javascripts.length).to eq 1
-  end
-end

data/spec/unit/authorization/authorization_adapter_spec.rb

@@ -1,61 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe ActiveAdmin::AuthorizationAdapter do
-
-  let(:adapter) { ActiveAdmin::AuthorizationAdapter.new(double, double) }
-
-  describe "#authorized?" do
-
-    it "should always return true" do
-      expect(adapter.authorized?(:read, "Resource")).to eq true
-    end
-
-  end
-
-  describe "#scope_collection" do
-
-    it "should return the collection unscoped" do
-      collection = double
-      expect(adapter.scope_collection(collection, ActiveAdmin::Auth::READ)).to eq collection
-    end
-
-  end
-
-  describe "using #normalized in a subclass" do
-
-    let(:auth_class) do
-      Class.new(ActiveAdmin::AuthorizationAdapter) do
-
-        def authorized?(action, subject = nil)
-          case subject
-          when normalized(String)
-            true
-          else
-            false
-          end
-        end
-
-      end
-    end
-
-    let(:adapter) { auth_class.new(double, double) }
-
-    it "should match against a class" do
-      expect(adapter.authorized?(:read, String)).to eq true
-    end
-
-    it 'should match against an instance' do
-      expect(adapter.authorized?(:read, "String")).to eq true
-    end
-
-    it 'should not match a different class' do
-      expect(adapter.authorized?(:read, Hash)).to eq false
-    end
-
-    it 'should not match a different instance' do
-      expect(adapter.authorized?(:read, {})).to eq false
-    end
-
-  end
-
-end

data/spec/unit/authorization/controller_authorization_spec.rb

@@ -1,43 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe "Controller Authorization", type: :controller do
-
-  let(:authorization){ controller.send(:active_admin_authorization) }
-
-  before do
-    load_resources { ActiveAdmin.register Post }
-    @controller = Admin::PostsController.new
-    allow(authorization).to receive(:authorized?)
-  end
-
-  it "should authorize the index action" do
-    expect(authorization).to receive(:authorized?).with(auth::READ, Post).and_return true
-    get :index
-    expect(response).to be_successful
-  end
-
-  it "should authorize the new action" do
-    expect(authorization).to receive(:authorized?).with(auth::CREATE, an_instance_of(Post)).and_return true
-    get :new
-    expect(response).to be_successful
-  end
-
-  it "should authorize the create action with the new resource" do
-    expect(authorization).to receive(:authorized?).with(auth::CREATE, an_instance_of(Post)).and_return true
-    post :create
-    expect(response).to redirect_to action: 'show', id: Post.last.id
-  end
-
-  it "should redirect when the user isn't authorized" do
-    expect(authorization).to receive(:authorized?).with(auth::READ, Post).and_return false
-    get :index
-    expect(response.body).to eq '<html><body>You are being <a href="http://test.host/admin">redirected</a>.</body></html>'
-    expect(response).to redirect_to '/admin'
-  end
-
-  private
-
-  def auth
-    ActiveAdmin::Authorization
-  end
-end

data/spec/unit/authorization/index_overriding_spec.rb

@@ -1,23 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe 'Index overriding', type: :controller do
-  before do
-    load_resources { ActiveAdmin.register Post }
-    @controller = Admin::PostsController.new
-
-    @controller.instance_eval do
-      def index
-        super do
-          render ActiveAdmin::Dependency.rails.render_key => 'Rendered from passed block'
-          return
-        end
-      end
-    end
-  end
-
-  it 'should call block passed to overridden index' do
-    get :index
-    expect(response.body).to eq 'Rendered from passed block'
-  end
-
-end

data/spec/unit/auto_link_spec.rb

@@ -1,93 +0,0 @@
-require 'rails_helper'
-require 'active_admin/view_helpers/active_admin_application_helper'
-require 'active_admin/view_helpers/auto_link_helper'
-require 'active_admin/view_helpers/display_helper'
-require 'active_admin/view_helpers/method_or_proc_helper'
-
-RSpec.describe "auto linking resources", type: :view do
-  include ActiveAdmin::ViewHelpers::ActiveAdminApplicationHelper
-  include ActiveAdmin::ViewHelpers::AutoLinkHelper
-  include ActiveAdmin::ViewHelpers::DisplayHelper
-  include MethodOrProcHelper
-
-  let(:active_admin_namespace){ ActiveAdmin.application.namespace(:admin) }
-  let(:post){ Post.create! title: "Hello World" }
-
-  before do
-    allow(self).to receive(:authorized?).and_return(true)
-  end
-
-  context "when the resource is not registered" do
-    before do
-      load_resources {}
-    end
-
-    it "should return the display name of the object" do
-      expect(auto_link(post)).to eq "Hello World"
-    end
-  end
-
-  context "when the resource is registered" do
-    before do
-      load_resources do
-        active_admin_namespace.register Post
-      end
-    end
-
-    it "should return a link with the display name of the object" do
-      expect(auto_link(post)).to \
-          match(%r{<a href="/admin/posts/\d+">Hello World</a>})
-    end
-
-    it "should keep locale in the url if present" do
-      expect(self).to receive(:url_options).and_return(locale: 'en')
-
-      expect(auto_link(post)).to \
-          match(%r{<a href="/admin/posts/\d+\?locale=en">Hello World</a>})
-    end
-
-    context "but the user doesn't have access" do
-      before do
-        allow(self).to receive(:authorized?).and_return(false)
-      end
-
-      it "should return the display name of the object" do
-        expect(auto_link(post)).to eq "Hello World"
-      end
-    end
-  end
-
-  context "when the resource is registered with the show action disabled" do
-    before do
-      load_resources do
-        active_admin_namespace.register(Post) { actions :all, except: :show }
-      end
-    end
-
-    it "should fallback to edit" do
-      expect(auto_link(post)).to \
-        match(%r{<a href="/admin/posts/\d+/edit">Hello World</a>})
-    end
-
-    it "should keep locale in the url if present" do
-      expect(self).to receive(:url_options).and_return(locale: 'en')
-
-      expect(auto_link(post)).to \
-        match(%r{<a href="/admin/posts/\d+/edit\?locale=en">Hello World</a>})
-    end
-  end
-
-  context "when the resource is registered with the show & edit actions disabled" do
-    before do
-      load_resources do
-        active_admin_namespace.register(Post) do
-          actions :all, except: [:show, :edit]
-        end
-      end
-
-      it "should return the display name of the object" do
-        expect(auto_link(post)).to eq "Hello World"
-      end
-    end
-  end
-end

data/spec/unit/batch_actions/resource_spec.rb

@@ -1,84 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe ActiveAdmin::BatchActions::ResourceExtension do
-
-  let(:resource) do
-    namespace = ActiveAdmin::Namespace.new(ActiveAdmin::Application.new, :admin)
-    namespace.batch_actions = true
-    namespace.register(Post)
-  end
-
-  describe "default action" do
-
-    it "should have the default action by default" do
-      expect(resource.batch_actions.size).to eq 1
-      expect(resource.batch_actions.first.sym == :destroy).to eq true
-    end
-
-  end
-
-  describe "adding a new batch action" do
-
-    before do
-      resource.clear_batch_actions!
-      resource.add_batch_action :flag, "Flag" do
-        # Empty
-      end
-    end
-
-    it "should add an batch action" do
-      expect(resource.batch_actions.size).to eq 1
-    end
-
-    it "should store an instance of BatchAction" do
-      expect(resource.batch_actions.first).to be_an_instance_of(ActiveAdmin::BatchAction)
-    end
-
-    it "should store the block in the batch action" do
-      expect(resource.batch_actions.first.block).to_not eq nil
-    end
-
-  end
-
-  describe "removing batch action" do
-
-    before do
-      resource.remove_batch_action :destroy
-    end
-
-    it "should allow for batch action removal" do
-      expect(resource.batch_actions.size).to eq 0
-    end
-
-  end
-
-  describe "#display_if_block" do
-
-    it "should return true by default" do
-      action = ActiveAdmin::BatchAction.new :default, "Default"
-      expect(action.display_if_block.call).to eq true
-    end
-
-    it "should return the :if block if set" do
-      action = ActiveAdmin::BatchAction.new :with_block, "With Block", if: proc { false }
-      expect(action.display_if_block.call).to eq false
-    end
-
-  end
-
-  describe "batch action priority" do
-
-    it "should have a default priority" do
-      action = ActiveAdmin::BatchAction.new :default, "Default"
-      expect(action.priority).to eq 10
-    end
-
-    it "should correctly order two actions" do
-      priority_one = ActiveAdmin::BatchAction.new :one, "One", priority: 1
-      priority_ten = ActiveAdmin::BatchAction.new :ten, "Ten", priority: 10
-      expect(priority_one).to be < priority_ten
-    end
-
-  end
-
-end

data/spec/unit/batch_actions/settings_spec.rb

@@ -1,61 +0,0 @@
-require 'rails_helper'
-
-RSpec.describe "Batch Actions Settings" do
-  let(:app) { ActiveAdmin::Application.new }
-  let(:ns) { ActiveAdmin::Namespace.new(app, "Admin") }
-  let(:post_resource) { ns.register Post }
-
-  it "should be disabled globally by default" do
-    # Note: the default initializer would set it to true
-
-    expect(app.batch_actions).to eq false
-    expect(ns.batch_actions).to eq false
-    expect(post_resource.batch_actions_enabled?).to eq false
-  end
-
-  it "should be settable to true" do
-    app.batch_actions = true
-    expect(app.batch_actions).to eq true
-  end
-
-  it "should be an inheritable_setting" do
-    app.batch_actions = true
-    expect(ns.batch_actions).to eq true
-  end
-
-  it "should be settable at the namespace level" do
-    app.batch_actions = true
-    ns.batch_actions = false
-
-    expect(app.batch_actions).to eq true
-    expect(ns.batch_actions).to eq false
-  end
-
-  it "should be settable at the resource level" do
-    expect(post_resource.batch_actions_enabled?).to eq false
-    post_resource.batch_actions = true
-    expect(post_resource.batch_actions_enabled?).to eq true
-  end
-
-  it "should inherit the setting on the resource from the namespace" do
-    ns.batch_actions = false
-    expect(post_resource.batch_actions_enabled?).to eq false
-    expect(post_resource.batch_actions).to be_empty
-
-    post_resource.batch_actions = true
-    expect(post_resource.batch_actions_enabled?).to eq true
-    expect(post_resource.batch_actions).to_not be_empty
-  end
-
-  it "should inherit the setting from the namespace when set to nil" do
-    ns.batch_actions = true
-
-    post_resource.batch_actions = true
-    expect(post_resource.batch_actions_enabled?).to eq true
-    expect(post_resource.batch_actions).to_not be_empty
-
-    post_resource.batch_actions = nil
-    expect(post_resource.batch_actions_enabled?).to eq true # inherited from namespace
-    expect(post_resource.batch_actions).to_not be_empty
-  end
-end