質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

87.59%

Rails チュートリアル 13章 RSpecでのPost数増減のテストでDeleteが機能しない

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 517

score 3

Rails チュートリアルの13章 リスト13-55においてUIに対する統合テストを初心者なりにRSpecで書いていたのですが、#destroyの部分でエラーが発生してしまいます。13章の最初からコードを見直したり、数時間調べていましたがどうにも解決しません。どうかご教示お願いします。

エラーメッセージ

Failures:

  1) microposts interface #destroy post delete post
     Failure/Error:
       expect do
         delete micropost_path(micropost)
       end.to change(Micropost, :count).by(-1)

       expected `Micropost.count` to have changed by -1, but was changed by 0
     # ./spec/requests/microposts_interface_spec.rb:30:in `block (3 levels) in <top (required)>'

Finished in 3.79 seconds (files took 19.72 seconds to load)
3 examples, 1 failure

Failed examples:

rspec ./spec/requests/microposts_interface_spec.rb:27 # microposts interface #destroy post delete post

test.log

[1m (0.1ms)  SELECT COUNT(*) FROM "microposts"
   (0.1ms)  SAVEPOINT active_record_1
  Micropost Create (0.2ms)  INSERT INTO "microposts" ("content", "user_id", "created_at", "updated_at") VALUES (?, ?, ?, ?)  [["content", "This is test post"], ["user_id", 1046959424], ["created_at", "2020-09-26 15:20:13.351638"], ["updated_at", "2020-09-26 15:20:13.351638"]]
   (5.5ms)  RELEASE SAVEPOINT active_record_1
Started DELETE "/microposts/1033843181" for 127.0.0.1 at 2020-09-27 00:20:13 +0900
Processing by MicropostsController#destroy as HTML
  Parameters: {"id"=>"1033843181"}
  User Load (0.1ms)  SELECT  "users".* FROM "users" WHERE "users"."id" = ? LIMIT ?  [["id", 1046959424], ["LIMIT", 1]]
  Micropost Load (0.2ms)  SELECT  "microposts".* FROM "microposts" WHERE "microposts"."user_id" = ? AND "microposts"."id" = ? ORDER BY "microposts"."created_at" DESC LIMIT ?  [["user_id", 1046959424], ["id", 1033843181], ["LIMIT", 1]]
   (0.1ms)  SAVEPOINT active_record_1
  Micropost Destroy (0.1ms)  DELETE FROM "microposts" WHERE "microposts"."id" = ?  [["id", 1033843181]]
   (0.1ms)  RELEASE SAVEPOINT active_record_1
Redirected to http://www.example.com/home
Completed 302 Found in 4ms (ActiveRecord: 0.5ms)
   (0.1ms)  SELECT COUNT(*) FROM "microposts"
   (0.4ms)  rollback transaction

/spec/requests/microposts_interface_spec.rb

require "rails_helper"

RSpec.describe "microposts interface" do
  let(:user) { create(:user) }
  let(:micropost) { user.microposts.create(content: "This is test post") }

  describe "#create post" do
    it "submit invalid post" do
      sign_in_as(user)
      visit home_path
      expect do
        post microposts_path, params: { micropost: { content: "" } }
      end.to change(Micropost, :count).by(0)
      expect(response).to render_template "static_pages/home"
    end
    it "submit valid post" do
      sign_in_as(user)
      visit home_path
      expect do
        post microposts_path, params: { micropost: { content: "This is second post" } }
      end.to change(Micropost, :count).by(1)
      expect(response).to redirect_to home_path
    end
  end

  describe "#destroy post" do
    it "delete post" do
      sign_in_as(user)
      visit home_path
      expect do
        delete micropost_path(micropost)
      end.to change(Micropost, :count).by(-1)
      expect(response).to redirect_to home_path
    end
  end

end

microposts.controller.rb

class MicropostsController < ApplicationController
  before_action :logged_in_user, only: [:create, :destroy]
  before_action :correct_user,   only: :destroy

  def create
    @micropost = current_user.microposts.build(micropost_params)
    if @micropost.save
      flash[:success] = "Post has created"
      redirect_to home_path
    else
      @feed_items = current_user.feed.paginate(page: params[:page])
      render "static_pages/home"
    end
  end

  def destroy
    @micropost.destroy
    flash[:success] = "Your post has deleted"
    redirect_to request.referrer || home_path
  end

  private

    def micropost_params
      params.require(:micropost).permit(:content, :picture)
    end

    def correct_user
      @micropost = current_user.microposts.find_by(id: params[:id])
      if @micropost.nil?
        redirect_to home_path
      end
    end
end

/spec/factories/users.rb

RSpec.configure do |config|
  config.include FactoryBot::Syntax::Methods
end
FactoryBot.define do
  factory :user do
    name { "Example User" }
    sequence(:email) { |n| "user_#{n}@example.com" }
    password { "password" }
    password_confirmation { "password" }
    activated {true}
    activated_at {Time.zone.now}
  end
end

補足情報(FW/ツールのバージョンなど)

source 'https://rubygems.org'

gem 'rails',        '5.2.4'
gem 'bootstrap-sass', '3.4.1'
gem 'sass-rails', '>= 3.2'
gem 'puma',         '3.9.1'
gem 'uglifier',     '3.2.0'
gem 'coffee-rails', '4.2.2'
gem 'jquery-rails', '4.3.1'
gem 'turbolinks',   '5.0.1'
gem 'jbuilder',     '2.7.0'
gem "haml-rails", "~> 2.0"
gem 'bootsnap', require: false
gem 'bcrypt',         '3.1.12'
gem 'faker',          '1.7.3'
gem 'will_paginate',           '3.1.7'
gem 'bootstrap-will_paginate', '1.0.0'
gem 'carrierwave',             '1.2.2'
gem 'mini_magick',             '4.7.0'

group :development, :test do
  gem 'sqlite3', '1.3.13'
  gem 'byebug',  '9.0.6', platform: :mri
  gem 'rspec-rails', '~> 3.8'
  gem 'spring-commands-rspec'
  gem 'capybara'
  gem 'webdrivers'
  gem 'factory_bot_rails'
  gem 'shoulda-matchers'
  gem 'rails-controller-testing'
end

group :development do
  gem 'web-console',           '3.5.1'
  gem 'listen',                '3.1.5'
  gem 'spring',                '2.0.2'
  gem 'spring-watcher-listen', '2.0.1'
end

group :test do
  gem 'minitest',                 '5.14.2'
  gem 'minitest-reporters',       '1.1.14'
  gem 'guard',                    '2.16.2'
  gem 'guard-minitest',           '2.4.4'
end

group :production do
  gem 'pg', '0.20.0'
  gem 'fog', '1.42'
end
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • hatsu

    2020/09/27 01:14

    削除する前にmicropostが作られていないために0のまま7日と思いました。
    let(:micropost) から let!(:micropost) のように変えてmicropostを作っておくといかがでしょうか。

    キャンセル

  • grinch1252

    2020/09/27 18:15

    確かにlet!にしたところ、うまくdeleteが機能してくれました。ありがとうございます。回答の方にも書いていただくことは可能でしょうか?

    キャンセル

回答 1

checkベストアンサー

0

削除する前にmicropostが作られていないために0のままのようです。

let(:micropost) から let!(:micropost) に変えて事前にmicropostを作成すると期待通り動きそうです。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 87.59%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る