rspecテスト時に投稿が2回される

Question

現在、アプリケーションを作成する中で、テストコードを実行しております。

機能としてはtipコントローラーのaction#newで投稿した内容が、ホーム画面action#index(root_path)に表示されるものとしております。

以下のテストコードにおいてテストを実行すると、同じ新規投稿が2回されてしまっております。

以降、削除機能をテストする際に同じ投稿が存在することは不都合スノですので、1回の投稿で1つの表示と修正したいと思います。

原因を究明しているのですが、解決方法をご教示いただけないでしょうか。

require 'rails_helper'

RSpec.describe '投稿する', type: :system do
  before do
    @user = FactoryBot.create(:user)
    @user[:id] = @user.id + 1
    @tip = FactoryBot.create(:tip)
  end
  context '投稿に失敗した時' do
    省略
  end
  context '投稿に成功した時' do
    it '投稿に成功し、トップページに投稿したタイトル、カテゴリー、説明が表示されていること' do
			省略
    end
    it ' 画像を含めた投稿が成功し、トップページに投稿した画像が表示されていること ' do
      # ログイン
      visit new_user_session_path
      fill_in 'user_email', with: @user.email
      fill_in 'user_password', with: @user.password
      find('input[type="submit"]').click
      binding.pry　#確認１
      # 新規投稿
      visit new_tip_path
      fill_in 'tip_title', with: @tip.title
      select Category.data[@tip.category_id - 1][:name], from: 'tip_category_id'
      image_path = Rails.root.join('public/images/test_image.png')
      attach_file 'tip-image-main-img', image_path, make_visible: true
      fill_in 'tip_description', with: @tip.description
      expect  do
        find('input[type="submit"]').click
      end.to change { Tip.count }.by(1)
      expect(current_path).to eq root_path
      expect(page).to have_content(@tip.title)
      expect(page).to have_content(Category.data[@tip.category_id - 1][:name])
      expect(page).to have_content(@tip.description)
      expect(page).to have_selector('img')
      binding.pry　#確認２
    end
    省略
  end
end

ログインした後のトップページ（root _path)の状態は　#確認１　です。

画像はこちらになります。

投稿後の状態は　#確認2　です。

Suscipit~　というタイトルが2つされております。

□調べたこと・仮説

とりあえず、新規投稿内の@tipをother_tipに変えてテストしてみました。

# 投稿
      binding.pry
      visit new_tip_path
      other_tip = FactoryBot.create(:tip)
      fill_in 'tip_title', with: other_tip.title
      select Category.data[other_tip.category_id - 1][:name], from: 'tip_category_id'
      image_path = Rails.root.join('public/images/test_image.png')
      attach_file 'tip-image-main-img', image_path, make_visible: true
      fill_in 'tip_description', with: other_tip.description
      expect  do
        find('input[type="submit"]').click
      end.to change { Tip.count }.by(1)
      binding.pry

しかし、結果は変わらず、同じ投稿が2回繰り返されます。

私の力不足もあり、有効な方法を見つけれませんでしたが、binding.pryで泊まった回数は上記の2箇所だけでしたので、it~ doのブロックが2回読み込まれたわけではないと想定されます。

以下、実装コード

spec/factories/tips.rb
FactoryBot.define do
  factory :tip do
    title            { Faker::Lorem.sentence }
    category_id      { 2 }
    description      { Faker::Lorem.sentence }
    association :user

    after(:build) do |tip|
      tip.image.attach(io: File.open('public/images/test_image.png'), filename: 'test_image.png')
    end
  end
end

spec/factories/users.rb
FactoryBot.define do
  factory :user do
    nickname              { 'test' }
    email                 { Faker::Internet.free_email }
    password              { Faker::Internet.password(min_length: 6) }
    password_confirmation { password }
    last_name             { '田中' }
    first_name            { '太郎' }
    introduction          { Faker::Lorem.sentence }
  end
end

spec/rails_helper.rb

# This file is copied to spec/ when you run 'rails generate rspec:install'
require 'spec_helper'
ENV['RAILS_ENV'] ||= 'test'
require File.expand_path('../config/environment', __dir__)
# Prevent database truncation if the environment is production
abort('The Rails environment is running in production mode!') if Rails.env.production?
require 'rspec/rails'
# Add additional requires below this line. Rails is not loaded until this point!

# Requires supporting ruby files with custom matchers and macros, etc, in
# spec/support/ and its subdirectories. Files matching `spec/**/*_spec.rb` are
# run as spec files by default. This means that files in spec/support that end
# in _spec.rb will both be required and run as specs, causing the specs to be
# run twice. It is recommended that you do not name files matching this glob to
# end with _spec.rb. You can configure this pattern with the --pattern
# option on the command line or in ~/.rspec, .rspec or `.rspec-local`.
#
# The following line is provided for convenience purposes. It has the downside
# of increasing the boot-up time by auto-requiring all files in the support
# directory. Alternatively, in the individual `*_spec.rb` files, manually
# require only the support files necessary.
#
# Dir[Rails.root.join('spec', 'support', '**', '*.rb')].sort.each { |f| require f }

# Checks for pending migrations and applies them before tests are run.
# If you are not using ActiveRecord, you can remove these lines.
begin
  ActiveRecord::Migration.maintain_test_schema!
rescue ActiveRecord::PendingMigrationError => e
  puts e.to_s.strip
  exit 1
end
RSpec.configure do |config|
  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
  config.fixture_path = "#{::Rails.root}/spec/fixtures"

  # 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

  # You can uncomment this line to turn off ActiveRecord support entirely.
  # config.use_active_record = false

  # RSpec Rails can automatically mix in different behaviours to your tests
  # based on their file location, for example enabling you to call `get` and
  # `post` in specs under `spec/controllers`.
  #
  # You can disable this behaviour by removing the line below, and instead
  # explicitly tag your specs with their type, e.g.:
  #
  #     RSpec.describe UsersController, type: :controller do
  #       # ...
  #     end
  #
  # The different available types are documented in the features, such as in
  # https://relishapp.com/rspec/rspec-rails/docs
  config.infer_spec_type_from_file_location!

  # Filter lines from Rails gems in backtraces.
  config.filter_rails_from_backtrace!
  # arbitrary gems may also be filtered via:
  # config.filter_gems_from_backtrace("gem name")
  Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f } # support directoryをrequire
  config.include RequestSpecHelper, type: :request # type: :requestのときにRequestHelperをinclude
end
I18n.locale = 'en'

長々と申し訳ございません。

Answer 1

beforeに記述している

@tip = FactoryBot.create(:tip)

FactoryBot.buildに変更するとどうでしょうか？

createはその時にデータベースに保存（1回投稿）され、
その後にテストでもう一度投稿するので
2回投稿されていないでしょうか？

Comments

[denisov]

Ryo-EAST様
create => build
2回投稿した原因は、まさにその通りでした。ありがとうございます。