ruby-on-railsrubyrspecrspec-railsrspec-expectations

RSpec expectation ignoring message argument


Does anybody understand why the expectation below is ignoring the argument message considering it's being passed as a string?

spec.rb

context 'flash' do
  context 'fail' do
    # Changes the admin_user so it has no permission to delete offence
    it 'flash[:alert]' do
      admin_user = FactoryGirl.create :admin_user
      sign_in admin_user
      delete :destroy,  :id => offence.id, :customer_id => offence.job.customer.id
      expect(flash[:alert]).to(be_present, eq("Deletion failed. Incident can only be deleted by user who created it!"))
    end
  end
end

the controller

 def destroy
  @offence = Offence.find(params[:id])
  @events = Event.where(sub_type: 'offence').where("parent_data->> 'id' = ?", @offence.id.to_s)
  if @offence.admin_user == current_user
  ActiveRecord::Base.transaction do
    @events.each do |event|
      event.destroy!
    end
    @offence.destroy!
    redirect_to admin_customer_offences_path(@customer), notice: 'Incident deleted successfully'
  end
  else
    redirect_to admin_customer_offences_path(@customer), alert: 'Deletion failed. Incident can only be deleted by user who created it!'
  end
 end

The warning message

.WARNING: ignoring the provided expectation message argument (#<RSpec::Matchers::BuiltIn::Eq:0x007fac42c5d4b0 @expected="Incident deleted successfully">) since it is not a string or a proc.

Solution

  • This is not a valid Rspec usage. You are assuming that to is capable of handling 2 or more expectations.

    You should split them

      expect(flash[:alert]).to be_present
      expect(flash[:alert]).to eq("Deletion failed. Incident can only be deleted by user who created it!")
    

    However, your code is also redundant. If the message is equal to a string, it's for sure present. Hence the first expectation is completely useless.

    Just add

      expect(flash[:alert]).to eq("Deletion failed. Incident can only be deleted by user who created it!")