オリジナルのバリデーションをテストする

Question

前提条件

Rails5で簡単なモデルを作成しています。
Eventモデルに二つの属性を追加しました。
マイグレーションファイルを作成したとき二つの属性の型はdatetime型に設定し,
値の大きさに制限をかけるオリジナルのバリデーションを作成しました。また、存在性をチェックするバリデーションも追加しています。

#event.rb

#start_date: datetime
#end_date: datetime 

class Event < ActiveRecord::Base
  belongs_to :user
  default_scope -> { order(created_at: :desc) }
  validates :user_id, presence: true
  validates :title, presence: true
  validates :text, presence: true
  validates :number_limit, presence: true, numericality: { greater_than: 2 }
  validates :start_date, presence: true
  validates :end_date, presence: true
  validate :end_date_limit

  private
     def end_date_limit 
       if end_date < start_date
          errors.add(:end_date, "が開始日時より前になっています")
       end
     end
end

フォームで受け取ったstart_dateの値がend_dateより大きくなっていたらエラーを表示させるというシンプルなバリデーションです。
開発環境ではこのバリデーションは正しく動作しますが、自作のバリデーションのため、テストを書こうと思ったところつまずいてしまいました。

エラーの出たテスト

実行したテスト

# events_interface_test.rb
require 'test_helper'

class EventsInterfaceTest < ActionDispatch::IntegrationTest
  # test "the truth" do
  #   assert true
  # end
  def setup
    @user=users(:michael)
    @event=events(:red)
  end

  test "event interface" do
    log_in_as(@user)
    get root_path
    
    assert_select 'div.pagination', count: 2
    #eventを追加する無効な送信
    assert_no_difference 'Event.count' do
        post events_path, params: { event: { title: "", text: "", number_limit: "", start_date: "", end_date: ""}}
    end
    assert_select 'div#error_explanation'

    #eventを追加する有効な送信
    title="event"
    text="event"
    number_limit=2
    start_date=Time.current
    end_date=Time.current+10.year
    assert_difference 'Event.count', 1 do
      post events_path, params: { event: { title: title, text: text, number_limit: number_limit,
                                           start_date: start_date, end_date: end_date}}
    end
    assert_redirected_to root_url
    follow_redirect!
    assert_match title, response.body

出力されたエラー

Error:
EventsInterfaceTest#test_event_interface:
NoMethodError: undefined method `<' for nil:NilClass
    app/models/event.rb:14:in `end_date_limit'
    app/controllers/events_controller.rb:18:in `create'
    test/integration/events_interface_test.rb:19:in `block (2 levels) in <class:EventsInterfaceTest>'
    test/integration/events_interface_test.rb:18:in `block in <class:EventsInterfaceTest>'


bin/rails test test/integration/events_interface_test.rb:12

#events_controller.rb

class EventsController < ApplicationController
  before_action :logged_in_user, except: [:index, :show]
  before_action :correct_user, only: [:edit, :update, :destroy]
  def index
    @events=Event.paginate(page: params[:page])
  end

  def show
    @event=Event.find(params[:id])
  end

  def new
    @event=Event.new
  end

  def create
    @event=current_user.events.build(event_params)
    if @event.save
      flash[:success]="イベントを投稿しました"
      redirect_to root_url
    else
      render 'events/new'
    end
   end
  private
    #アクセスできる属性を制限

    def event_params
      params.require(:event).permit(:title, :text, :number_limit, :start_date, :end_date)
    end

    def correct_user
      @event=current_user.events.find_by(id: params[:id])
      redirect_to root_url if @event.nil?
    end
  end

events_interface_test.rbのeventを追加する無効な送信でエラーが出ていますが、ほかにも出ていないか確認しました。
eventを追加する有効な送信のテストではFailureが出ていました。正しい値で送信したはずですがEventの数は変わっていないようです。

Failure:
EventsInterfaceTest#test_event_interface [C:/Users/Sano Mashio/Desktop/play_with/test/integration/events_interface_test.rb:25]:
"Event.count" didn't change by 1.
Expected: 35
  Actual: 34


bin/rails test test/integration/events_interface_test.rb:12

なぜこのようなエラーが出るのか自分でも考えてみました。
まずtest時にstart_date,end_dateを表現するために使ったTime.currentですがTime.currentの親クラスと実際にActive Recordから取得できる
event.start_dateの親クラスはどちらもActiveSupport::TimeWithZoneでしたのでtestで扱っているd型は間違っていないと考えています。
rails console では Time.current < Time.current+10.year　のように大きさを比べることができます。この要領でオリジナルのバリデーション
を追加し、テストも書こうとしています。
どのような考えのもとで書けばテストが通るようになるか教えていただけないでしょうか。

Comments

[gouf]

解決した場合、ベストアンサーを設定することで、この質問を解決済みにすることができます

Answer 1

events_interface_test.rb:19　って
post events_path, params: { event: { title: "", text: "", number_limit: "", start_date: "", end_date: ""}}
ここでしょうか。
start,end 共に値がないため、
if end_date < start_date
が
if nil < nil
になっています

Comments

[Sano]

コメントアウトありがとうございます。
def end_date_limit
if end_date < start_date
errors.add(:end_date, "が開始日時より前になっています")
end
end

を
def end_date_limit
if self.end_date && self.start_date
if self.end_date<self.start_date
errors.add(:end_date, "が開始日時より前になっています")
end
end
end
として値が与えられていたときだけ比較するように書き直しました。
これで無効な送信の start_date: "", end_date: "" が動くようになりました。
ありがとうございました。