DatabaseCleaner, похоже, не очищает между мастями

#ruby-on-rails #ruby #ruby-on-rails-3 #rspec #database-cleaner

#ruby-on-rails #ruby #ruby-on-rails-3 #rspec #database-cleaner

Вопрос:

спасители.

У меня возникли проблемы с очисткой базы данных после каждого примера RSpec. Дело в том, что когда я запускаю rspec command, users_controller_spec.rb жалуется, что записей больше, чем ожидает пример. Действительно, записи создаются, как указано, если я проверю rails c . когда я запускаю только этот пакет, он будет успешным, поэтому я предполагаю, что это связано с тем, что DatabaseCleaner не очищает пользовательские записи, которые создают другие спецификации (количество пользовательских записей соответствует дополнительным записям, которые, как утверждает пример users_controller_spec). Они создаются в before :all блоке (если это имеет значение).

Вот мой rails_helper.rb

 # This file is copied to spec/ when you run 'rails generate rspec:install'
ENV['RAILS_ENV'] ||= 'test'
require 'spec_helper'
require File.expand_path('../../config/environment', __FILE__)
require 'rspec/rails'

# Add additional requires below this line. Rails is not loaded until this point!
require 'devise'
require 'admin/v1/dashboard_controller'
# Requires supporting ruby files with custom matchers and macros, etc, in
Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f }

# Checks for pending migrations before tests are run.
# If you are not using ActiveRecord, you can remove this line.
ActiveRecord::Migration.maintain_test_schema!

RSpec.configure do |config|
  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
  config.fixture_path = "#{::Rails.root}/spec/fixtures"

  config.include Devise::Test::ControllerHelpers, type: :controller
  config.include ControllerMacros, type: :controller
  # If you're not using ActiveRecord, or you'd prefer not to run each of your
  # examples within a transaction, remove the following line or assign false
  # instead of true.
  config.use_transactional_fixtures = true

  config.include FactoryGirl::Syntax::Methods

  config.infer_spec_type_from_file_location!

  config.before(:suite) do
    DatabaseCleaner.strategy = :transaction
    DatabaseCleaner.clean_with(:truncation)
  end

  config.around(:each) do |example|
    DatabaseCleaner.cleaning do
      example.run
    end
  end
end
  

users_controller.rb

 describe 'GET #index' do
  it 'populates an array of users' do
    user1 = create(:user)
    user2 = create(:user)
    get :index
    expect(assigns(:users)).to match_array([user1, user2])
  end
  it 'renders :index template' do
    get :index, {}
    expect(response).to render_template :index
  end
end
  

UPDATE1: здесь создаются дополнительные user записи

     require 'rails_helper'

describe Admin::V1::MessagesController do
  let(:admin_user) do
    admin_user = double('admin_user')
    allow(request.env['warden']).to receive(:authenticate!).and_return(admin_user)
    allow(controller).to receive(:current_admin_v1_admin_user).and_return(admin_user)
    p '==='
  end
  before { login_admin_user admin_user }

  describe 'GET #index' do
    it 'renders :index template' do
      get :index, {}
      expect(response).to render_template :index
    end
  end

  describe 'GET #get_users' do
    before :all do
      @user1 = create(:user, nickname: 'hiro')
      @user2 = create(:user, nickname: 'elise')
    end
    context 'with params' do
      it 'populates an array of users matching on nickname' do
        get :get_users, format: :json, query: 'h'
        expect(assigns(:users)).to match_array([@user1])
      end
    end
    context 'without params' do
      it 'populates an array of all users' do
        get :get_users, format: :json
        expect(assigns(:users)).to match_array([@user1, @user2])
      end
    end
  end

  describe 'GET #get_messages' do
    before :all do
      @user1 = create(:user)
      @user2 = create(:user)
      @message1 = create(:message, user_id: @user1.id)
      @message2 = create(:message, user_id: @user1.id)
      @message3 = create(:message, user_id: @user2.id)
    end
    context 'with user_id' do
      it 'populates an array of messages with the user_id' do
        get :get_messages, format: :json, user_id: @user1.id
        expect(assigns(:messages)).to match_array([@message1, @message2])
      end
    end
  end
end
  

Комментарии:

1. Ваша конфигурация выглядит нормально. В других спецификациях может быть ошибка (не в users_controller_apec.rb ). Сколько у вас спецификаций? Попробуйте выполнить спецификации в группах, чтобы узнать, где были созданы записи.

2. На данный момент у меня есть 8 спецификаций контроллера. И дополнительные записи создаются в примерах, где интерфейс javascript передает информацию о пользователе, а поскольку api rails извлекает некоторые пользовательские записи (следовательно, пользовательские записи факторизованы).

3. Не могли бы вы показать эти спецификации? (добавьте их в свой вопрос)

Ответ №1:

К сожалению, RSpec before(:all) плохо работает с транзакционными тестами. Код before(:all) запускается до открытия транзакции, что означает, что любые записи, созданные там, не будут откатываться при прерывании транзакции. Вы несете ответственность за ручную очистку этих элементов в after(:all) .

Смотрите rspec-rails #496 и Использование before(:all) в RSpec доставит вам много проблем, если вы не знаете, что делаете

   after(:all) do
    # before/after(:all) is not transactional; see https://www.relishapp.com/rspec/rspec-rails/docs/transactions
    DatabaseCleaner.clean_with(:truncation)
  end
  

Комментарии:

1. Спасибо! Это сработало! Никогда не думал, что будет такой чистый и логичный ответ. В конце концов, правила документации..

2. Приятно, рад, что это помогло!