activeadmin-xls 2.0.2 → 3.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: c3e07f07dfdc40d3f51d3bf7ed071c2f11a204fd
-  data.tar.gz: 899009df0bbc02bc522b4c9f872d036e5ecb96b7
+  metadata.gz: 62cff616f4a9a56a7a8f4bbb5a650b0e5fc0c353
+  data.tar.gz: 7df6070e702f123847217604d1a1a7b7a37c17f7
 SHA512:
-  metadata.gz: ad917c6af301c4f86bcd51b16cd49e57e57bca0c74fa9ff437509298e02b9f3e73420601f5cd2138cf9c3be11bdf4541ee5de5711b91c7c514fa033cff840065
-  data.tar.gz: 2c5f38deffdb05f528e6248daf5e4785e58a91664b397e52c148bc07c454c00e61d51b0d4f8b6dec94b0304e5c068b2d19d36c596057d5b283cc5a9ebb483f07
+  metadata.gz: 83e874292581aed3216abfa1e7a69756a7fe1b60234dc717497f6342a3b457a313cde9a8f3458a11c780dbd6d137121cb729f9bea50747dbebf91d8eae5ca193
+  data.tar.gz: 3bf5ed6974fac2f8f37b97fc2c496041d6da09be4a2e8b1e9b3fac87c4312d6c8032422ecad656b7e0b4dbd2bf53505b6fd437bda0b40c14ce5ca3a9c9c32cd5

data/CHANGELOG.md

@@ -2,11 +2,17 @@
 
 ## [Unreleased]
 
+## 2.0.3
+
+### Changed
+
+* ignore test/ folder for gem building [#21][] by [@cprodhomme][]
+
 ## 2.0.2
 
 ### Changed
 
-* Remove ActiveAdmin runtime dependency version limit '<2' [#16] by [@reaper][]
+* Remove ActiveAdmin runtime dependency version limit '<2' [#16][] by [@reaper][]
 
 ## 2.0.1
 
@@ -68,8 +74,10 @@
 [#11]: https://github.com/thambley/activeadmin-xls/pull/11
 [#13]: https://github.com/thambley/activeadmin-xls/issues/13
 [#16]: https://github.com/thambley/activeadmin-xls/pull/16
+[#21]: https://github.com/thambley/activeadmin-xls/pull/21
 
 [@rewritten]: https://github.com/rewritten
 [@ejaypcanaria]: https://github.com/ejaypcanaria
 [@cpunion]: https://github.com/cpunion
 [@reaper]: https://github.com/reaper
+[@cprodhomme]: https://github.com/cprodhomme

data/activeadmin-xls.gemspec

@@ -19,11 +19,11 @@ Gem::Specification.new do |s|
   git_tracked_files = `git ls-files`.split("\n").sort
   gem_ignored_files = `git ls-files -i -X .gemignore`.split("\n")
 
-  s.files = git_tracked_files - gem_ignored_files
+  s.files = (git_tracked_files - gem_ignored_files).reject { |f| f.match(%r{^(test|spec|features)/}) }
 
-  s.add_runtime_dependency 'activeadmin', '>= 1.0.0'
+  s.add_runtime_dependency 'activeadmin', '>= 2.0.0', '< 4.0'
   s.add_runtime_dependency 'spreadsheet', '~> 1.0'
 
-  s.required_ruby_version = '>= 2.0.0'
+  s.required_ruby_version = '>= 2.7.0'
   s.require_path = 'lib'
 end

data/gemfiles/rails_42.gemfile

@@ -9,6 +9,7 @@ eval_gemfile(File.expand_path(File.join('..', 'Gemfile'), __dir__))
 gem 'activeadmin', '1.4.3'
 gem 'devise', '~> 4.2'
 gem 'rails', '4.2.11'
+gem 'sprockets', '< 4.0'
 gem 'sqlite3', '~> 1.3.0'
 gem 'turbolinks', '~> 5.0.0'
 gem 'tzinfo-data'

data/lib/active_admin/xls/engine.rb

@@ -8,15 +8,6 @@ module ActiveAdmin
         if Mime::Type.lookup_by_extension(:xls).nil?
           Mime::Type.register 'application/vnd.ms-excel', :xls
         end
-
-        ActiveAdmin::Views::PaginatedCollection.add_format :xls
-
-        ActiveAdmin::ResourceDSL.send :include, ActiveAdmin::Xls::DSL
-        ActiveAdmin::Resource.send :include, ActiveAdmin::Xls::ResourceExtension
-        ActiveAdmin::ResourceController.send(
-          :prepend,
-          ActiveAdmin::Xls::ResourceControllerExtension
-        )
       end
     end
   end

data/lib/active_admin/xls/extensions.rb

@@ -0,0 +1,14 @@
+ActiveAdmin.before_load do
+  require 'active_admin/xls/dsl'
+  require 'active_admin/xls/resource_extension'
+  require 'active_admin/xls/resource_controller_extension'
+
+  ActiveAdmin::Views::PaginatedCollection.add_format :xls
+
+  ActiveAdmin::ResourceDSL.send :include, ActiveAdmin::Xls::DSL
+  ActiveAdmin::Resource.send :include, ActiveAdmin::Xls::ResourceExtension
+  ActiveAdmin::ResourceController.send(
+    :prepend,
+    ActiveAdmin::Xls::ResourceControllerExtension
+  )
+end

data/lib/active_admin/xls/version.rb

@@ -1,6 +1,6 @@
 module ActiveAdmin
   module Xls
     # ActiveAdmin XLS gem version
-    VERSION = '2.0.2'.freeze
+    VERSION = '3.0.0'.freeze
   end
 end

data/lib/activeadmin-xls.rb

@@ -3,7 +3,5 @@ require 'active_admin'
 
 require 'active_admin/xls/version'
 require 'active_admin/xls/builder'
-require 'active_admin/xls/dsl'
-require 'active_admin/xls/resource_extension'
-require 'active_admin/xls/resource_controller_extension'
 require 'active_admin/xls/engine'
+require 'active_admin/xls/extensions'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: activeadmin-xls
 version: !ruby/object:Gem::Version
-  version: 2.0.2
+  version: 3.0.0
 platform: ruby
 authors:
 - Todd Hambley
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-10-08 00:00:00.000000000 Z
+date: 2023-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activeadmin
@@ -16,14 +16,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 2.0.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.0.0
+        version: 2.0.0
+    - - "<"
+      - !ruby/object:Gem::Version
+        version: '4.0'
 - !ruby/object:Gem::Dependency
   name: spreadsheet
   requirement: !ruby/object:Gem::Requirement
@@ -62,23 +68,11 @@ files:
 - lib/active_admin/xls/builder.rb
 - lib/active_admin/xls/dsl.rb
 - lib/active_admin/xls/engine.rb
+- lib/active_admin/xls/extensions.rb
 - lib/active_admin/xls/resource_controller_extension.rb
 - lib/active_admin/xls/resource_extension.rb
 - lib/active_admin/xls/version.rb
 - lib/activeadmin-xls.rb
-- spec/rails_helper.rb
-- spec/spec_helper.rb
-- spec/support/rails_template.rb
-- spec/support/rails_template_with_data.rb
-- spec/support/templates/admin/stores.rb
-- spec/support/templates/cucumber.rb
-- spec/support/templates/cucumber_with_reloading.rb
-- spec/support/templates/en.yml
-- spec/xls/unit/build_download_format_links_spec.rb
-- spec/xls/unit/builder_spec.rb
-- spec/xls/unit/dsl_spec.rb
-- spec/xls/unit/resource_controller_spec.rb
-- spec/xls/unit/resource_spec.rb
 homepage: https://github.com/thambley/activeadmin-xls
 licenses:
 - MIT
@@ -91,7 +85,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.0.0
+      version: 2.7.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
@@ -99,7 +93,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.14.1
+rubygems_version: 2.6.14.4
 signing_key: 
 specification_version: 4
 summary: Adds excel (xls) downloads for resources within the Active Admin framework.

data/spec/spec_helper.rb

@@ -1,46 +0,0 @@
-require 'simplecov'
-SimpleCov.start do
-  add_filter '/rails/'
-  add_filter '/spec/'
-end
-
-if ENV['CI'] == 'true'
-  require 'codecov'
-  SimpleCov.formatters = [
-    SimpleCov::Formatter::HTMLFormatter,
-    SimpleCov::Formatter::Codecov
-  ]
-end
-
-ENV['RAILS_ENV'] = 'test'
-
-# prepare ENV for rails
-require 'rails'
-ENV['RAILS_ROOT'] = File.expand_path(
-  "../rails/rails-#{Rails::VERSION::STRING}",
-  __FILE__
-)
-
-# ensure testing application is in place
-unless File.exist?(ENV['RAILS_ROOT'])
-  puts 'Please run bundle exec rake setup before running the specs.'
-  exit
-end
-
-# load up activeadmin and activeadmin-xls
-require 'active_record'
-require 'active_admin'
-require 'devise'
-require 'activeadmin-xls'
-ActiveAdmin.application.load_paths = [ENV['RAILS_ROOT'] + '/app/admin']
-
-# start up rails
-require ENV['RAILS_ROOT'] + '/config/environment'
-
-# and finally,here's rspec
-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

data/spec/support/rails_template.rb

@@ -1,127 +0,0 @@
-# Rails template to build the sample app for specs
-
-# Create a cucumber database and environment
-copy_file File.expand_path('../templates/cucumber.rb', __FILE__),
-          'config/environments/cucumber.rb'
-copy_file File.expand_path('../templates/cucumber_with_reloading.rb', __FILE__),
-          'config/environments/cucumber_with_reloading.rb'
-
-gsub_file 'config/database.yml', /^test:.*\n/, "test: &test\n"
-gsub_file 'config/database.yml',
-          /\z/,
-          "\ncucumber:\n  <<: *test\n  database: db/cucumber.sqlite3"
-gsub_file(
-  'config/database.yml',
-  /\z/,
-  "\ncucumber_with_reloading:\n  <<: *test\n  database: db/cucumber.sqlite3"
-)
-
-if File.exist?('config/secrets.yml')
-  require 'securerandom'
-  cucumber_secret = SecureRandom.hex(64)
-  gsub_file 'config/secrets.yml',
-            /\z/,
-            "\ncucumber:\n  secret_key_base: #{cucumber_secret}"
-end
-
-base_record = if Rails::VERSION::MAJOR >= 5
-                'ApplicationRecord'
-              else
-                'ActiveRecord::Base'
-              end
-
-# Generate some test models
-generate :model, 'post title:string body:text published_at:datetime author_id:integer category_id:integer'
-post_model_setup = if Rails::VERSION::MAJOR >= 5
-                     <<-MODEL
-  belongs_to :author, class_name: 'User'
-  belongs_to :category, optional: true
-  accepts_nested_attributes_for :author
-MODEL
-                   else
-                     <<-MODEL
-  belongs_to :author, class_name: 'User'
-  belongs_to :category
-  accepts_nested_attributes_for :author
-MODEL
-                   end
-inject_into_file 'app/models/post.rb',
-                 post_model_setup,
-                 after: "class Post < #{base_record}\n"
-
-generate :model, 'user type:string first_name:string last_name:string username:string age:integer'
-inject_into_file 'app/models/user.rb',
-                 "  has_many :posts, foreign_key: 'author_id'\n",
-                 after: "class User < #{base_record}\n"
-
-generate :model, 'publisher --migration=false --parent=User'
-
-generate :model, 'category name:string description:text'
-inject_into_file 'app/models/category.rb',
-                 "  has_many :posts\n  accepts_nested_attributes_for :posts\n",
-                 after: "class Category < #{base_record}\n"
-
-generate :model, 'store name:string'
-
-# Generate a model with string ids
-generate :model, 'tag name:string'
-gsub_file(
-  Dir['db/migrate/*_create_tags.rb'][0],
-  /\:tags\sdo\s.*/,
-  ":tags, id: false, primary_key: :id do |t|\n\t\t\tt.string :id\n"
-)
-id_model_setup = <<-MODEL
-  self.primary_key = :id
-  before_create :set_id
-
-  private
-  def set_id
-    self.id = 8.times.inject('') do |s,e|
-      s << (i = Kernel.rand(62); i += ((i < 10) ? 48 : ((i < 36) ? 55 : 61 ))).chr
-    end
-  end
-MODEL
-
-inject_into_file 'app/models/tag.rb',
-                 id_model_setup,
-                 after: "class Tag < #{base_record}\n"
-
-# Configure default_url_options in test environment
-inject_into_file(
-  'config/environments/test.rb',
-  "  config.action_mailer.default_url_options = { host: 'example.com' }\n",
-  after: "config.cache_classes = true\n"
-)
-
-# Add our local Active Admin to the load path
-lib_path = File.expand_path('../../../lib/activeadmin-xls', __FILE__)
-inject_into_file 'config/environment.rb',
-                 "\nrequire '#{lib_path}'\n",
-                 after: "require File.expand_path('../application', __FILE__)"
-
-# Add some translations
-append_file 'config/locales/en.yml',
-            File.read(File.expand_path('../templates/en.yml', __FILE__))
-
-# Add predefined admin resources
-directory File.expand_path('../templates/admin', __FILE__), 'app/admin'
-
-run 'rm Gemfile'
-
-$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
-
-generate :'active_admin:install'
-
-run 'rm -r test'
-run 'rm -r spec'
-
-inject_into_file 'config/initializers/active_admin.rb',
-                 "  config.download_links = %i[csv xml json xls]\n",
-                 after: "  # == Download Links\n"
-
-# Setup a root path for devise
-route "root to: 'admin/dashboard#index'"
-
-rake 'db:migrate'
-rake 'db:test:prepare'
-run '/usr/bin/env RAILS_ENV=cucumber rake db:migrate'

data/spec/support/rails_template_with_data.rb

@@ -1,68 +0,0 @@
-# Use the default
-apply File.expand_path('../rails_template.rb', __FILE__)
-
-# Register Active Admin controllers
-%w[Post User Category].each do |type|
-  generate :'active_admin:resource', type
-end
-
-scopes = <<-SCOPES
-  scope :all, default: true
-
-  scope :drafts do |posts|
-    posts.where(['published_at IS NULL'])
-  end
-
-  scope :scheduled do |posts|
-    posts.where(['posts.published_at IS NOT NULL AND posts.published_at > ?', Time.now.utc])
-  end
-
-  scope :published do |posts|
-    posts.where(['posts.published_at IS NOT NULL AND posts.published_at < ?', Time.now.utc])
-  end
-
-  scope :my_posts do |posts|
-    posts.where(author_id: current_admin_user.id)
-  end
-SCOPES
-
-if File.exist?('app/admin/post.rb')
-  inject_into_file 'app/admin/post.rb',
-                   scopes,
-                   after: "ActiveAdmin.register Post do\n"
-else
-  inject_into_file 'app/admin/posts.rb',
-                   scopes,
-                   after: "ActiveAdmin.register Post do\n"
-end
-
-# Setup some default data
-append_file 'db/seeds.rb', <<-SEEDS
-
-  users = ['Jimi Hendrix', 'Jimmy Page', 'Yngwie Malmsteen', 'Eric Clapton', 'Kirk Hammett'].collect do |name|
-    first, last = name.split(" ")
-    User.create! first_name: first,
-                 last_name: last,
-                 username: [first, last].join('-').downcase,
-                 age: rand(80)
-  end
-
-  categories = ['Rock', 'Pop Rock', 'Alt-Country', 'Blues', 'Dub-Step'].collect do |name|
-    Category.create! name: name
-  end
-
-  published_at_values = [Time.now.utc - 5.days, Time.now.utc - 1.day, nil, Time.now.utc + 3.days]
-
-  1_000.times do |i|
-    user = users[i % users.size]
-    cat = categories[i % categories.size]
-    published_at = published_at_values[i % published_at_values.size]
-    Post.create title: "Blog Post \#{i}",
-                body: "Blog post \#{i} is written by \#{user.username} about \#{cat.name}",
-                category_id: cat.id,
-                published_at: published_at,
-                author: user
-  end
-  SEEDS
-
-rake 'db:seed'

data/spec/support/templates/admin/stores.rb

@@ -1 +0,0 @@
-ActiveAdmin.register Store

data/spec/support/templates/cucumber.rb

@@ -1,24 +0,0 @@
-require File.expand_path('config/environments/test', Rails.root)
-
-# rails/railties/lib/rails/test_help.rb aborts if the environment is not 'test'. (Rails 3.0.0.beta3)
-# We can't run Cucumber/RSpec/Test_Unit tests in different environments then.
-#
-# For now, I patch StringInquirer so that Rails.env.test? returns true when Rails.env is 'test' or 'cucumber'
-#
-# https://rails.lighthouseapp.com/projects/8994-ruby-on-rails/tickets/4458-rails-should-allow-test-to-run-in-cucumber-environment
-module ActiveSupport
-  class StringInquirer < String
-    def method_missing(method_name, *arguments)
-      if method_name.to_s[-1,1] == "?"
-        test_string = method_name.to_s[0..-2]
-        if test_string == 'test'
-          self == 'test' or self == 'cucumber'
-        else
-          self == test_string
-        end
-      else
-        super
-      end
-    end
-  end
-end

data/spec/support/templates/cucumber_with_reloading.rb

@@ -1,5 +0,0 @@
-require File.expand_path('config/environments/cucumber', Rails.root)
-
-Rails.application.class.configure do
-  config.cache_classes = false
-end

data/spec/support/templates/en.yml

@@ -1,15 +0,0 @@
-# Sample translations used to test ActiveAdmin's I18n integration.
-  xls:
-    post:
-      id: ID
-      title: Title
-      body: Content
-      published_at: Published On
-      author: Publisher
-      created_at: Created
-      updated_at: Updated
-  activerecord:
-    models:
-      store:
-        one: Bookstore
-        other: Bookstores

data/spec/xls/unit/build_download_format_links_spec.rb

@@ -1,52 +0,0 @@
-require 'spec_helper'
-describe ActiveAdmin::Views::PaginatedCollection do
-  def arbre(assigns = {}, helpers = mock_action_view, &block)
-    Arbre::Context.new(assigns, helpers, &block)
-  end
-
-  def render_arbre_component(assigns = {}, helpers = mock_action_view, &block)
-    arbre(assigns, helpers, &block).children.first
-  end
-
-  # Returns a fake action view instance to use with our renderers
-  def mock_action_view(assigns = {})
-    controller = ActionView::TestCase::TestController.new
-    ActionView::Base.send :include, ActionView::Helpers
-    ActionView::Base.send :include, ActiveAdmin::ViewHelpers
-    ActionView::Base.send :include, Rails.application.routes.url_helpers
-    ActionView::Base.new(ActionController::Base.view_paths, assigns, controller)
-  end
-
-  let(:view) do
-    view = mock_action_view
-    allow(view.request).to receive(:query_parameters) { { page: '1' } }
-    allow(view.request).to receive(:path_parameters) do
-      { controller: 'admin/posts', action: 'index' }
-    end
-    view
-  end
-
-  # Helper to render paginated collections within an arbre context
-  def paginated_collection(*args)
-    render_arbre_component({ paginated_collection_args: args }, view) do
-      paginated_collection(*paginated_collection_args)
-    end
-  end
-
-  let(:collection) do
-    posts = [Post.new(title: 'First Post')]
-    Kaminari.paginate_array(posts).page(1).per(5)
-  end
-
-  let(:pagination) { paginated_collection(collection) }
-
-  before do
-    allow(collection).to receive(:except) { collection } unless collection.respond_to? :except
-    allow(collection).to receive(:group_values) { [] } unless collection.respond_to? :group_values
-    allow(collection).to receive(:reorder) { collection }
-  end
-
-  it 'renders the xls download link' do
-    expect(pagination.children.last.content).to match(/XLS/)
-  end
-end

data/spec/xls/unit/builder_spec.rb

@@ -1,243 +0,0 @@
-require 'spec_helper'
-
-module ActiveAdmin
-  # tests for builder
-  module Xls
-    describe Builder do
-      let(:builder) { Builder.new(Post) }
-      let(:content_columns) { Post.content_columns }
-
-      context 'the default builder' do
-        it 'has no header style' do
-          expect(builder.header_style).to eq({})
-        end
-        it 'has no i18n scope' do
-          expect(builder.i18n_scope).to be_nil
-        end
-        it 'has default columns' do
-          expect(builder.columns.size).to eq(content_columns.size + 1)
-        end
-      end
-
-      context 'customizing a builder' do
-        it 'deletes columns we tell it we dont want' do
-          builder.delete_columns :id, :body
-          expect(builder.columns.size).to eq(content_columns.size - 1)
-        end
-
-        it 'lets us use specific columns in a list' do
-          builder.only_columns :title, :author
-          expect(builder.columns.size).to eq(2)
-        end
-
-        it 'lets us say we dont want the header' do
-          builder.skip_header
-          expect(builder.instance_values['skip_header']).to be_truthy
-        end
-
-        it 'lets us add custom columns' do
-          builder.column(:hoge)
-          expect(builder.columns.size).to eq(content_columns.size + 2)
-        end
-
-        it 'lets us clear all columns' do
-          builder.clear_columns
-          expect(builder.columns.size).to eq(0)
-        end
-
-        context 'Using Procs for delayed content generation' do
-          let(:post) { Post.new(title: 'Hot Dawg') }
-
-          before do
-            builder.column(:hoge) do |resource|
-              "#{resource.title} - with cheese"
-            end
-          end
-
-          it 'stores the block when defining a column for later execution.' do
-            expect(builder.columns.last.data).to be_a(Proc)
-          end
-
-          it 'evaluates custom column blocks' do
-            expect(builder.columns.last.data.call(post)).to eq(
-              'Hot Dawg - with cheese'
-            )
-          end
-        end
-      end
-
-      context 'sheet generation without headers' do
-        let!(:users) { [User.new(first_name: 'bob', last_name: 'nancy')] }
-
-        let!(:posts) do
-          [Post.new(title: 'bob', body: 'is a swell guy', author: users.first)]
-        end
-
-        let!(:builder) do
-          options = {
-            header_format: { weight: :bold },
-            i18n_scope: %i[xls post]
-          }
-          Builder.new(Post, options) do
-            skip_header
-          end
-        end
-
-        before do
-          @book = Spreadsheet.open(StringIO.new(builder.serialize(posts)))
-        end
-
-        it 'does not serialize the header' do
-          expect(@book.worksheets.first[0, 0]).not_to eq('Title')
-        end
-      end
-
-      context 'whitelisted sheet generation' do
-        let!(:users) { [User.new(first_name: 'bob', last_name: 'nancy')] }
-
-        let!(:posts) do
-          [Post.new(title: 'bob', body: 'is a swell guy', author: users.first)]
-        end
-
-        let!(:builder) do
-          Builder.new(Post, header_style: {}, i18n_scope: %i[xls post]) do
-            skip_header
-            whitelist
-            column :title
-          end
-        end
-
-        before do
-          @book = Spreadsheet.open(StringIO.new(builder.serialize(posts)))
-          @collection = builder.collection
-        end
-
-        it 'does not serialize the header' do
-          sheet = @book.worksheets.first
-          expect(sheet.column_count).to eq(1)
-          expect(sheet[0, 0]).to eq(@collection.first.title)
-        end
-      end
-
-      context 'Sheet generation with a highly customized configuration.' do
-        let!(:builder) do
-          options = {
-            header_style: { size: 10, color: 'red' },
-            i18n_scope: %i[xls post]
-          }
-          Builder.new(Post, options) do
-            delete_columns :id, :created_at, :updated_at
-            column(:author) do |resource|
-              "#{resource.author.first_name} #{resource.author.last_name}"
-            end
-            after_filter do |sheet|
-              row_number = sheet.dimensions[1]
-              sheet.update_row(row_number)
-              row_number += 1
-              sheet.update_row(row_number, 'Author Name', 'Number of Posts')
-              users = collection.map(&:author).uniq(&:id)
-              users.each do |user|
-                row_number += 1
-                sheet.update_row(row_number,
-                                 "#{user.first_name} #{user.last_name}",
-                                 user.posts.size)
-              end
-            end
-            before_filter do |sheet|
-              users = collection.map(&:author)
-              users.each do |user|
-                user.first_name = 'Set In Proc' if user.first_name == 'bob'
-              end
-              row_number = sheet.dimensions[1]
-              sheet.update_row(row_number, 'Created', Time.zone.now)
-              row_number += 1
-              sheet.update_row(row_number, '')
-            end
-          end
-        end
-
-        before do
-          Post.all.each(&:destroy)
-          User.all.each(&:destroy)
-          @user = User.create!(first_name: 'bob', last_name: 'nancy')
-          @post = Post.create!(title: 'bob',
-                               body: 'is a swell guy',
-                               author: @user)
-          @book = Spreadsheet.open(StringIO.new(builder.serialize(Post.all)))
-          @collection = builder.collection
-        end
-
-        it 'provides the collection object' do
-          expect(@collection.count).to eq(Post.all.count)
-        end
-
-        it 'merges our customizations with the default header style' do
-          expect(builder.header_style[:size]).to eq(10)
-          expect(builder.header_style[:color]).to eq('red')
-          # expect(builder.header_style[:pattern_bg_color]).to eq('00')
-        end
-
-        it 'uses the specified i18n_scope' do
-          expect(builder.i18n_scope).to eq(%i[xls post])
-        end
-
-        it 'translates the header row based on our i18n scope' do
-          header_row = @book.worksheets.first.row(2)
-          expect(header_row).to eq(
-            ['Title', 'Content', 'Published On', 'Publisher']
-          )
-        end
-
-        it 'processes the before filter' do
-          expect(@book.worksheets.first.cell(0, 0)).to eq('Created')
-        end
-
-        it 'lets us work against the collection in the before filter' do
-          expect(@book.worksheets.first.last_row[0]).to eq('Set In Proc nancy')
-        end
-      end
-
-      ################################
-      context 'Sheet generation with a exceptions.' do
-        let!(:users) { [User.new(first_name: 'bob', last_name: 'nancy')] }
-
-        let!(:posts) do
-          [Post.new(title: 'bob', body: 'is a swell guy', author: users.first)]
-        end
-
-        let!(:exposts) do
-          [Post.new(title: 'sal', body: 'is a swell guy', author: users.first),
-           Post.new(title: 'err', body: 'is a swell guy', author: users.first)]
-        end
-
-        let!(:builder) do
-          Builder.new(Post, header_style: {}, i18n_scope: %i[xls post]) do
-            delete_columns :id, :created_at, :updated_at
-            column(:author) do |resource|
-              raise 'err' if resource.title == 'err'
-              "#{resource.author.first_name} #{resource.author.last_name}"
-            end
-          end
-        end
-
-        before do
-          begin
-            @book1 = Spreadsheet.open(StringIO.new(builder.serialize(exposts)))
-          rescue StandardError => err
-            raise unless err.message == 'err'
-          end
-          @book2 = Spreadsheet.open(StringIO.new(builder.serialize(posts)))
-          @collection = builder.collection
-        end
-
-        it 'does not contain data from other collections with errors' do
-          sheet = @book2.worksheets.first
-          expect(sheet.dimensions[1]).to eq(2)
-          expect(sheet[0, 0]).to eq('Title')
-          expect(sheet[1, 0]).to eq(@collection.first.title)
-        end
-      end
-      ################################
-    end
-  end
-end

data/spec/xls/unit/dsl_spec.rb

@@ -1,61 +0,0 @@
-require 'spec_helper'
-
-module ActiveAdmin
-  # tests for dsl
-  module Xls
-    describe ::ActiveAdmin::ResourceDSL do
-      context 'in a registration block' do
-        let(:builder) do
-          config = ActiveAdmin.register(Post) do
-            xls(i18n_scope: [:rspec], header_style: { size: 20 }) do
-              delete_columns :id, :created_at
-              column(:author) { |post| post.author.first_name }
-              before_filter do |sheet|
-                row_number = sheet.dimensions[1]
-                sheet.update_row(row_number, 'before_filter')
-              end
-              after_filter do |sheet|
-                row_number = sheet.dimensions[1]
-                sheet.update_row(row_number, 'after_filter')
-              end
-              skip_header
-            end
-          end
-          config.xls_builder
-        end
-
-        it 'uses our customized i18n scope' do
-          expect(builder.i18n_scope).to eq([:rspec])
-        end
-
-        it 'removed the columns we told it to ignore' do
-          %i[id create_at].each do |removed|
-            column_index = builder.columns.index { |col| col.name == removed }
-            expect(column_index).to be_nil
-          end
-        end
-
-        it 'added the columns we declared' do
-          added_index = builder.columns.index { |col| col.name == :author }
-          expect(added_index).not_to be_nil
-        end
-
-        it 'has a before filter set' do
-          expect(builder.instance_values['before_filter']).to be_a(Proc)
-        end
-
-        it 'has an after filter set' do
-          expect(builder.instance_values['after_filter']).to be_a(Proc)
-        end
-
-        it 'indicates that the header should be excluded' do
-          expect(builder.instance_values['skip_header']).to be_truthy
-        end
-
-        it 'updates the header style' do
-          expect(builder.header_style[:size]).to eq(20)
-        end
-      end
-    end
-  end
-end

data/spec/xls/unit/resource_controller_spec.rb

@@ -1,22 +0,0 @@
-require 'spec_helper'
-describe Admin::CategoriesController, type: :controller do
-  let(:mime) { Mime::Type.lookup_by_extension(:xls) }
-
-  let(:filename) do
-    "categories-#{Time.now.strftime('%Y-%m-%d')}.xls"
-  end
-
-  it 'generates an xls filename' do
-    expect(controller.xls_filename).to eq(filename)
-  end
-
-  context 'when making requests with the xls mime type' do
-    it 'returns xls attachment when requested' do
-      request.accept = mime
-      get :index
-      disposition = "attachment; filename=\"#{filename}\""
-      expect(response.headers['Content-Disposition']).to start_with(disposition)
-      expect(response.headers['Content-Transfer-Encoding']).to eq('binary')
-    end
-  end
-end

data/spec/xls/unit/resource_spec.rb

@@ -1,26 +0,0 @@
-require 'spec_helper'
-include ActiveAdmin
-
-module ActiveAdmin
-  module Xls
-    describe Resource do
-      let(:resource) { ActiveAdmin.register(Post) }
-
-      let(:custom_builder) do
-        Builder.new(Post) do
-          column(:fake) { :fake }
-        end
-      end
-
-      context 'when registered' do
-        it 'each resource has an xls_builder' do
-          expect(resource.xls_builder).to be_a(Builder)
-        end
-
-        it 'We can specify our own configured builder' do
-          expect { resource.xls_builder = custom_builder }.not_to raise_error
-        end
-      end
-    end
-  end
-end