Rspecを使ってコントローラのdeleteのテストをしていますが、エラーが出るのを解消したい

Question

初学者です。
オリジナルアプリで、初めてコントローラーのテストをしています。
いろんな方の記事を見てテストを記載していますが、解決方法が分からない部分が出てきましたので質問させて頂きました。
記載方法が間違っている部分が多いと思いますが、何卒宜しくお願い致します。

解決したいこと
Rspec を使ってコントローラのdeleteのテストをしていますが、ActiveRecord::InvalidForeignKey:のエラーと
No route matchesのエラーが出るのを解消したいです。

環境
ruby 2.6.5
rails 6.0.0
gem 'rspec-rails', '~> 4.0.0.beta3'
gem 'rails-controller-testing'
gem "devise"

エラーコード
ターミナル

 Failure/Error: @event = FactoryBot.create(:event)
     
     ActiveRecord::InvalidForeignKey:
       Mysql2::Error: Cannot add or update a child row: a foreign key constraint fails (`rc_app2_test`.`events`, CONSTRAINT `fk_rails_0cb5590091` FOREIGN KEY (`user_id`) REFERENCES `users` (`id`))

　Failure/Error: delete :destroy
     
     ActionController::UrlGenerationError:
       No route matches {:action=>"destroy", :controller=>"events"}

events_controller_spec.rbの該当箇所

require 'rails_helper'

describe EventsController, type: :controller do

 describe '#destroy' , type: :controller do

    context 'log in' do
      context 'can destroy' do
       
        it 'deletes a event' do
          sign_in @user
　　　　　　@event = FactoryBot.create(:event)
          expect{ delete :destroy, params: {id: @event.id}
        }.to change(@user.events, :count).by(-1)
        end

        it 'redirects to root_path' do

          sign_in @user
          expect(response).to redirect_to(root_path)
        end
      end

      context 'not log in' do
        it "redirects to new_user_session_path" do
          delete :destroy
          expect(response).to redirect_to(new_user_session_path)
        end
      end
    end
  end
end

routes

Rails.application.routes.draw do
  devise_for :users
  root to: "events#index"
  resources :events
  resources :customers
  # For details on the DSL available within this file, see https://guides.rubyonrails.org/routing.html
end

factorybot event.rb

FactoryBot.define do

  factory :event do
    title {"タイトル"}
    start_time {"2020/11/11/10:00"}
    end_time{"2020/11/11/10:00"}
    user_id{"1"}
    

  end
end

factorybot user.rb

FactoryBot.define do

  factory :user do
    email                 {"[e-mail]"}
    password              {"00000000"}
    password_confirmation {"00000000"}
    
  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"
  config.include FactoryBot::Syntax::Methods
  Dir[Rails.root.join('spec/support/**/*.rb')].each { |f| require f }
  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

  # 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")
end

event_controller.rb

  before_action :set_event, only: [:show, :edit, :update, :destroy]
  before_action :move_to_index,except:[:index]

  def destroy
    if @event.destroy
      redirect_to root_path, notice: 'イベントを削除しました'
    else
      flash.now[:alert] = 'イベントを削除できませんでした'
      render :show
    end
  end
  
  private

  def set_event
    @event = Event.find(params[:id])
  end
  def move_to_index
    unless user_signed_in?
      redirect_to action: :index
    end
  end

他のindexアクションとshowアクションはうまくいきました。
destroyアクションでつまづいています。
↓うまく行った部分。events_controller_spec.rb

describe EventsController, type: :controller do
  before do
    @user = FactoryBot.create(:user)
  end

  describe 'GET #index' do
    it "renders the :index template" do
      get :index
      expect(response).to render_template :index
    end
    it "returns a 200 response" do
      get :index
      expect(response).to have_http_status "200"
    end
  end

  describe 'GET #new' , type: :controller do

    context 'log in' do
      it "renders the :new template" do
        sign_in @user
        get :new
        expect(response).to render_template :new
      end
    end
    context 'not log in' do
      it "redirects to new_user_session_path" do
        get :new
        expect(response).to redirect_to(new_user_session_path)
      end
    end
  end

  describe '#create' , type: :controller do
    let(:params) { { title: "タイトル" , event: attributes_for(:event) } }

    context 'log in' do
      context 'can save' do
        subject {
          post :create,
          params: params
        }
        it 'count up event' do
          sign_in @user
          expect{ subject }.to change(Event, :count).by(1)
        end

        it 'redirects to root_path' do
          sign_in @user
          subject
          expect(response).to redirect_to(root_path)
        end
      end
      context 'can not save' do
        let(:invalid_params) { { event: attributes_for(:event, title: nil) } }

        subject {
          post :create,
          params: invalid_params
        }

        it 'does not count up' do
          sign_in @user
          expect{ subject }.not_to change(Event, :count)
        end

        it 'renders index' do
          sign_in @user
          subject
          expect(response).to redirect_to(new_event_path)
        end
      end
      context 'not log in' do
        it "redirects to new_user_session_path" do
          post :create
          expect(response).to redirect_to(new_user_session_path)
        end
      end
    end
  end

ご回答宜しくお願い致します。

Answer 1

botでemail {"[e-mail]"}と固定のemailを使っています。
testは通常は終わるたびにDBを戻すことになっていますが、何かのタイミングで削除されないうちに次のテストのbeforeが走ったのだろうと思われます。
emailを乱数的につくるか、Faker 使うかしてみてください

Comments

[a-ichi]

winterboum様

ありがとうございます。
Faker使ってみます。

Answer 2

問題となっているのは

user_id{"1"}

ここです。
userがid="1"で作られていない場合、これでは無効になってしまいます。

というわけで、associationを使います。

今回は、factory nameと association nameが同一になるので

FactoryBot.define do

  factory :event do
    title {"タイトル"}
    start_time {"2020/11/11/10:00"}
    end_time {"2020/11/11/10:00"}
    user
  end
end

で大丈夫なはずです。

他の書き方としては

user factory: :user
association :user, factory: :user
user {association :user}

など様々な書き方があります。

Comments

[a-ichi]

asm様
ご回答ありがとうございます。
上記の件、試すとidが無いエラーは解消されました。
ありがとうございます。
ただ、変わりに
1) EventsController#destroy log in can destroy deletes a event
Failure/Error: @event = FactoryBot.create(:event)

ActiveRecord::RecordInvalid:
バリデーションに失敗しました: Eメールはすでに存在します
のエラーが出てしまいました。

before do
@user = FactoryBot.create(:user)
end

でログインしているからeメールのバリデーション がかかってしまったのでしょうか？

また、
2) EventsController#destroy log in can destroy redirects to root_path
Failure/Error: delete :destroy

ActionController::UrlGenerationError:
No route matches {:action=>"destroy", :controller=>"events"}

こちらのエラーの解消方法は分かりますでしょうか？

[asm]

> 1) バリデーションに失敗しました: Eメールはすでに存在します

なんらかの理由ですでにtest用データベースに同emailを利用しているuserがあるようです
$ rails db:test:prepare
コマンドでtest用データベースを再構築できるかもしれません。

> 2) EventsController#destroy log in can destroy redirects to root_path

Deviseのsign_inヘルパーはログイン"状態"にするのであってログインリクエストを発行するのとは違います。
よってログインした時のリダイレクト先をこの方法でテストする事はできません。
また、EventsControllerのテストでやることなのかも疑問です。

[a-ichi]

asm様

> 1) バリデーションに失敗しました: Eメールはすでに存在します
$ rails db:test:prepareコマンドを入力してみましたが、
warning ../package.json: No license field のエラーでした。

package.jsonに"license": "UNLICENSED"を入力し、yarn installをしましたが、warning ../package.json: No license fieldのエラーは変わらずでした。

シークエルプロを確認しましたが、testの中には何も記載がありませんでした。
何か他に理由があるのでしょうか？

> 2) EventsController#destroy log in can destroy redirects to root_path
こちらはEventsControllerのテストでやるべき事では無いのですね。こちらは諦めます。ありがとうございます。