編集履歴

質問編集履歴

問題の追加

2021/03/02 05:19

投稿

paserin

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -4,6 +4,8 @@
 ３つエラーが出てしまうのですが、完全に行き詰ってしまったので、なぜこのエラーが出ているのか、解決策を教えて頂きたいです。
+新規投稿のフォームが空欄だとエラー文が出るように書いているのですが、titleとbodyの両方に入力をしたにもかかわらずエラー文が出力されるようになってしまい、新規投稿が出来ない状態になっています。
 ```

テストコードを追記しました

2021/03/02 05:19

投稿

paserin

スコア4

test CHANGED Viewed

	@@ -1 +1 @@
1	- rails」Rspecで表示されたエラー内容について
1	+ 「rails」Rspecで表示されたエラー内容について

test CHANGED Viewed

File without changes

テストコードを追記しました　文字制限があったので多少省きました。

2021/03/02 05:08

投稿

paserin

スコア4

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,142 +1,10 @@
-### 前提・実現したいこと
-railsで以下の要件に沿ったアプリケーションを作っています。
-・本の感想投稿へのCRUD機能
-・感想投稿の際、本のタイトルと感想に入力がなされていない場合、登録不可とする（validationの設定）
-・scaffoldの使用禁止。
-・localhost:3000/のURLを叩いた時にTOP画面が表示される(ルーティングの設定)
-・テーブル名は"posts",カラムは"title","body"でモデルを作成
-・ストロングパラメータを用いる
-・新規投稿または投稿の編集を行うと投稿した詳細画面(show.html.erb)に飛ぶ
-【レイアウト】
-・新規投稿フォームと投稿一覧表示を同一のページ内に置く
-【デザイン】
-・一覧部分にtableを用いる
 ### 発生している問題・エラーメッセージ
-３つエラーが出てしまうのですが、完全に行き詰ってしまったので解決策を教えて頂きたいです。
+３つエラーが出てしまうのですが、完全に行き詰ってしまったので、なぜこのエラーが出ているのか、解決策を教えて頂きたいです。
-```
+```
-Book モデルに関するテスト
-  実際に保存してみる
-    有効な投稿内容の場合は保存されるか
-  空白のバリデーションチェック
-    titleが空白の場合にバリデーションチェックされ空白のエラーメッセージが返ってきているか
-    bodyが空白の場合にバリデーションチェックされ空白のエラーメッセージが返ってきているか
-  titleを空白で投稿した場合に画面にエラーメッセージが表示されているか
-    エラーメッセージは正しく表示されるか
-  bodyを空白で投稿した場合に画面にエラーメッセージが表示されているか
-    エラーメッセージは正しく表示されるか
-投稿のテスト
-  トップ画面(root_path)のテスト
-    表示の確認
-      トップ画面(root_path)に一覧ページへのリンクが表示されているか
-      root_pathが"/"であるか
-  一覧画面のテスト
-    一覧の表示とリンクの確認
-      bookの一覧表示(tableタグ)と投稿フォームが同一画面に表示されているか
-      bookのタイトルと感想を表示し、詳細・編集・削除のリンクが表示されているか
-      Create Bookボタンが表示される
-    投稿処理に関するテスト
-      投稿に成功しサクセスメッセージが表示されるか (FAILED - 1)
-      投稿に失敗する
-      投稿後のリダイレクト先は正しいか (FAILED - 2)
-    book削除のテスト
-      bookの削除
-  詳細画面のテスト
-    表示の確認
-      本のタイトルと感想が画面に表示されていること
-      Editリンクが表示される
-      Backリンクが表示される
-    リンクの遷移先の確認
-      Editの遷移先は編集画面か
-      Backの遷移先は一覧画面か
-  編集画面のテスト
-    表示の確認
-      編集前のタイトルと感想がフォームに表示(セット)されている
-      Update Bookボタンが表示される
-      Showリンクが表示される
-      Backリンクが表示される
-    リンクの遷移先の確認
-      Showの遷移先は詳細画面か
-      Backの遷移先は一覧画面か
-    更新処理に関するテスト
-      更新に成功しサクセスメッセージが表示されるか
-      更新に失敗しエラーメッセージが表示されるか (FAILED - 3)
-      更新後のリダイレクト先は正しいか
@@ -174,24 +42,6 @@
-Finished in 0.76962 seconds (files took 0.98823 seconds to load)
-28 examples, 3 failures
-Failed examples:
-rspec ./spec/system/books_spec.rb:56 # 投稿のテスト 一覧画面のテスト 投稿処理に関するテスト 投稿に成功しサクセスメッセージが表示されるか
-rspec ./spec/system/books_spec.rb:67 # 投稿のテスト 一覧画面のテスト 投稿処理に関するテスト 投稿後のリダイレクト先は正しいか
-rspec ./spec/system/books_spec.rb:152 # 投稿のテスト 編集画面のテスト 更新処理に関するテスト 更新に失敗しエラーメッセージが表示されるか
 ```
@@ -200,396 +50,648 @@
-homes_controller.rb
+books_controller.rb
-```
+```
-class HomesController < ApplicationController
+class BooksController < ApplicationController
-  def top
+  def index
+    @books = Book.all
+    @book = Book.new
-  end
+  end
+  def create
+    @book = Book.new(book_params)
+    if @book.save
+      flash[:notice] = "Book was successfully created."
+      redirect_to book_path(book.id)
+    else
+      @books=Book.all
+      render action :index
+    end
+  end
+  def show
+    @book = Book.find(params[:id])
+  end
+  def edit
+    @book = Book.find(params[:id])
+  end
+  def update
+    @book = Book.find(params[:id])
+    if @book.update(book_params)
+      flash[:notice] = "Book was successfully updated."
+      redirect_to book_path(@book)
+    else
+      @books=Book.all
+      render action :edit
+    end
+  end
+  def destroy
+    book = Book.find(params[:id])
+    book.destroy
+    flash[:notice] = "Book was successfully destroyed."
+    redirect_to books_path
+  end
+  private
+  def book_params
+    params.permit(:title, :body)
+  end
 end
 ```
-books_controller.rb
-```
-class BooksController < ApplicationController
-  def index
-    @books = Book.all
-    @book = Book.new
-  end
-  def create
-    @book = Book.new(book_params)
-    if @book.save
-      flash[:notice] = "Book was successfully created."
-      redirect_to book_path(book.id)
-    else
-      @books=Book.all
-      render action :index
-    end
-  end
-  def show
-    @book = Book.find(params[:id])
-  end
-  def edit
-    @book = Book.find(params[:id])
-  end
-  def update
-    @book = Book.find(params[:id])
-    if @book.update(book_params)
-      flash[:notice] = "Book was successfully updated."
-      redirect_to book_path(@book)
-    else
-      @books=Book.all
-      render action :edit
-    end
-  end
-  def destroy
-    book = Book.find(params[:id])
-    book.destroy
-    flash[:notice] = "Book was successfully destroyed."
-    redirect_to books_path
-  end
-  private
-  def book_params
-    params.permit(:title, :body)
-  end
+index.html.erb
+```
+<h1>Books</h1>
+<table>
+  <tr>
+    <th>
+      Title
+    </th>
+    <th>Body</th>
+  </tr>
+  <% @books.each do |book| %>
+    <tr>
+      <th>
+        <%= book.title %>
+      </th>
+      <th>
+        <%= book.body %>
+      </th>
+      <th>
+        <%= link_to "show", book_path(book.id) %>
+      </th>
+      <th>
+        <%= link_to "edit", edit_book_path(book.id) %>
+      </th>
+      <th>
+        <%= link_to "destroy", book_path(book.id) , method: :delete %>
+      </th>
+    </tr>
+  <%end%>
+</table>
+<h1>New Book</h1>
+<%= form_with model:@book, local:true do |f| %>
+ <% if @book.errors.any? %>
+    <div id="error_explanation">
+      <h2><%= pluralize(@book.errors.count, "error") %> prohibited this book from being saved:</h2>
+      <ul>
+        <% @book.errors.full_messages.each do |message| %>
+          <li><%= message %></li>
+        <% end %>
+      </ul>
+    </div>
+　<% end %>
+  <h4>Title</h4>
+  <%= f.text_field :title %>
+  <h4>Body</h4>
+  <%= f.text_area :body %>
+  <%= f.submit 'Create Book' %>
+<% end %>
+```
+edit.html.erb
+```
+<h1>Editing Book</h1>
+<%= form_with model:@book, local:true do |f| %>
+  <% if @book.errors.any? %>
+    <div id="error_explanation">
+      <h2><%= pluralize(@book.errors.count, "error") %> prohibited this book from being saved:</h2>
+      <ul>
+        <% @book.errors.full_messages.each do |message| %>
+          <li><%= message %></li>
+        <% end %>
+      </ul>
+    </div>
+　<% end %>
+  <h4>Title</h4>
+  <%= f.text_field :title %>
+  <h4>Body</h4>
+  <%= f.text_area :body %>
+  <%= f.submit 'Update Book' %>
+<% end %>
+<%= link_to "show", book_path(@book) %>
+<%= link_to "Back", books_path %>
+```
+show.html.erb
+```
+<p>Title: <%= @book.title %></p>
+<p>Body: <%= @book.body%></p>
+<%= link_to "Edit", edit_book_path(@book) %>
+<%= link_to "Back", books_path %>
+```
+routes.rb
+```
+Rails.application.routes.draw do
+  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
+  resources :books
+  root to: 'homes#top'
 end
 ```
-index.html.erb
-```
-<h1>Books</h1>
-<table>
-  <tr>
-    <th>
-      Title
-    </th>
-    <th>Body</th>
-  </tr>
-  <% @books.each do |book| %>
-    <tr>
-      <th>
-        <%= book.title %>
-      </th>
-      <th>
-        <%= book.body %>
-      </th>
-      <th>
-        <%= link_to "show", book_path(book.id) %>
-      </th>
-      <th>
-        <%= link_to "edit", edit_book_path(book.id) %>
-      </th>
-      <th>
-        <%= link_to "destroy", book_path(book.id) , method: :delete %>
-      </th>
-    </tr>
-  <%end%>
-</table>
-<h1>New Book</h1>
-<%= form_with model:@book, local:true do |f| %>
- <% if @book.errors.any? %>
-    <div id="error_explanation">
-      <h2><%= pluralize(@book.errors.count, "error") %> prohibited this book from being saved:</h2>
-      <ul>
-        <% @book.errors.full_messages.each do |message| %>
-          <li><%= message %></li>
-        <% end %>
-      </ul>
-    </div>
-　<% end %>
-  <h4>Title</h4>
-  <%= f.text_field :title %>
-  <h4>Body</h4>
-  <%= f.text_area :body %>
-  <%= f.submit 'Create Book' %>
-<% end %>
-```
-edit.html.erb
-```
-<h1>Editing Book</h1>
-<%= form_with model:@book, local:true do |f| %>
-  <% if @book.errors.any? %>
-    <div id="error_explanation">
-      <h2><%= pluralize(@book.errors.count, "error") %> prohibited this book from being saved:</h2>
-      <ul>
-        <% @book.errors.full_messages.each do |message| %>
-          <li><%= message %></li>
-        <% end %>
-      </ul>
-    </div>
-　<% end %>
-  <h4>Title</h4>
-  <%= f.text_field :title %>
-  <h4>Body</h4>
-  <%= f.text_area :body %>
-  <%= f.submit 'Update Book' %>
-<% end %>
-<%= link_to "show", book_path(@book) %>
-<%= link_to "Back", books_path %>
-```
-index.html.erb
-```
-<p>Title: <%= @book.title %></p>
-<p>Body: <%= @book.body%></p>
-<%= link_to "Edit", edit_book_path(@book) %>
-<%= link_to "Back", books_path %>
-```
-index.html.erb
-```
-<h1>ようこそBookersへ！</h1>
-<p>Bookers では、さまざまな書籍に関するあなたの意見や</p>
-<p>印象を共有し交換することができます</p>
-<p><%= link_to "start", books_path %></p>
-```
-routes.rb
-```
-Rails.application.routes.draw do
-  # For details on the DSL available within this file, see http://guides.rubyonrails.org/routing.html
-  resources :books
-  root to: 'homes#top'
+テストコード
+```
+require 'rails_helper'
+describe '投稿のテスト' do
+  let!(:book) { create(:book,title:'hoge',body:'body') }
+  describe 'トップ画面(root_path)のテスト' do
+    before do
+      visit root_path
+    end
+    context '表示の確認' do
+      it 'トップ画面(root_path)に一覧ページへのリンクが表示されているか' do
+        expect(page).to have_link "", href: books_path
+      end
+      it 'root_pathが"/"であるか' do
+        expect(current_path).to eq('/')
+      end
+    end
+  end
+  describe "一覧画面のテスト" do
+    before do
+      visit books_path
+    end
+    context '一覧の表示とリンクの確認' do
+      it "bookの一覧表示(tableタグ)と投稿フォームが同一画面に表示されているか" do
+        expect(page).to have_selector 'table'
+        expect(page).to have_field 'book[title]'
+        expect(page).to have_field 'book[body]'
+      end
+      it "bookのタイトルと感想を表示し、詳細・編集・削除のリンクが表示されているか" do
+          (1..5).each do |i|
+            Book.create(title:'hoge'+i.to_s,body:'body'+i.to_s)
+          end
+          visit books_path
+          Book.all.each_with_index do |book,i|
+            j = i * 3
+            expect(page).to have_content book.title
+            expect(page).to have_content book.body
+            # Showリンク
+            show_link = find_all('a')[j]
+            expect(show_link.native.inner_text).to match(/show/i)
+            expect(show_link[:href]).to eq book_path(book)
+            # Editリンク
+            show_link = find_all('a')[j+1]
+            expect(show_link.native.inner_text).to match(/edit/i)
+            expect(show_link[:href]).to eq edit_book_path(book)
+            # Destroyリンク
+            show_link = find_all('a')[j+2]
+            expect(show_link.native.inner_text).to match(/destroy/i)
+            expect(show_link[:href]).to eq book_path(book)
+          end
+      end
+      it 'Create Bookボタンが表示される' do
+        expect(page).to have_button 'Create Book'
+      end
+    end
+    context '投稿処理に関するテスト' do
+      it '投稿に成功しサクセスメッセージが表示されるか' do
+        fill_in 'book[title]', with: Faker::Lorem.characters(number:5)
+        fill_in 'book[body]', with: Faker::Lorem.characters(number:20)
+        click_button 'Create Book'
+        expect(page).to have_content 'successfully'
+      end
+      it '投稿に失敗する' do
+        click_button 'Create Book'
+        expect(page).to have_content 'error'
+        expect(current_path).to eq('/books')
+      end
+      it '投稿後のリダイレクト先は正しいか' do
+        fill_in 'book[title]', with: Faker::Lorem.characters(number:5)
+        fill_in 'book[body]', with: Faker::Lorem.characters(number:20)
+        click_button 'Create Book'
+        expect(page).to have_current_path book_path(Book.last)
+      end
+    end
+    context 'book削除のテスト' do
+      it 'bookの削除' do
+        expect{ book.destroy }.to change{ Book.count }.by(-1)
+        # ※本来はダイアログのテストまで行うがココではデータが削除されることだけをテスト
+      end
+    end
+  end
+  describe '詳細画面のテスト' do
+    before do
+      visit book_path(book)
+    end
+    context '表示の確認' do
+      it '本のタイトルと感想が画面に表示されていること' do
+        expect(page).to have_content book.title
+        expect(page).to have_content book.body
+      end
+      it 'Editリンクが表示される' do
+        edit_link = find_all('a')[0]
+        expect(edit_link.native.inner_text).to match(/edit/i)
+			end
+      it 'Backリンクが表示される' do
+        back_link = find_all('a')[1]
+        expect(back_link.native.inner_text).to match(/back/i)
+			end
+    end
+    context 'リンクの遷移先の確認' do
+      it 'Editの遷移先は編集画面か' do
+        edit_link = find_all('a')[0]
+        edit_link.click
+        expect(current_path).to eq('/books/' + book.id.to_s + '/edit')
+      end
+      it 'Backの遷移先は一覧画面か' do
+        back_link = find_all('a')[1]
+        back_link.click
+        expect(page).to have_current_path books_path
+      end
+    end
+  end
+  describe '編集画面のテスト' do
+    before do
+      visit edit_book_path(book)
+    end
+    context '表示の確認' do
+      it '編集前のタイトルと感想がフォームに表示(セット)されている' do
+        expect(page).to have_field 'book[title]', with: book.title
+        expect(page).to have_field 'book[body]', with: book.body
+      end
+      it 'Update Bookボタンが表示される' do
+        expect(page).to have_button 'Update Book'
+      end
+      it 'Showリンクが表示される' do
+        show_link = find_all('a')[0]
+        expect(show_link.native.inner_text).to match(/show/i)
+			end
+      it 'Backリンクが表示される' do
+        back_link = find_all('a')[1]
+        expect(back_link.native.inner_text).to match(/back/i)
+			end
+    end
+    context 'リンクの遷移先の確認' do
+      it 'Showの遷移先は詳細画面か' do
+        show_link = find_all('a')[0]
+        show_link.click
+        expect(current_path).to eq('/books/' + book.id.to_s)
+      end
+      it 'Backの遷移先は一覧画面か' do
+        back_link = find_all('a')[1]
+        back_link.click
+        expect(page).to have_current_path books_path
+      end
+    end
+    context '更新処理に関するテスト' do
+      it '更新に成功しサクセスメッセージが表示されるか' do
+        fill_in 'book[title]', with: Faker::Lorem.characters(number:5)
+        fill_in 'book[body]', with: Faker::Lorem.characters(number:20)
+        click_button 'Update Book'
+        expect(page).to have_content 'successfully'
+      end
+      it '更新に失敗しエラーメッセージが表示されるか' do
+        fill_in 'book[title]', with: ""
+        fill_in 'book[body]', with: ""
+        click_button 'Update Book'
+        expect(page).to have_content 'error'
+      end
+      it '更新後のリダイレクト先は正しいか' do
+        fill_in 'book[title]', with: Faker::Lorem.characters(number:5)
+        fill_in 'book[body]', with: Faker::Lorem.characters(number:20)
+        click_button 'Update Book'
+        expect(page).to have_current_path book_path(book)
+      end
+    end
+  end
 end
 ```
-application.html.erb
-```
-<!DOCTYPE html>
-<html>
-  <head>
-   <title>BookerApp</title>
-    <%= csrf_meta_tags %>
-    <%= csp_meta_tag %>
-    <%= stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track': 'reload' %>
-    <%= javascript_include_tag 'application', 'data-turbolinks-track': 'reload' %>
-  </head>
-  <body>
-    <% if flash[:notice] %>
-     <p id="notice">
-       <%= flash[:notice] %>
-     </p>
-    <% end %>
-    <%= yield %>
-  </body>
-</html>
-```
-book.rb
-```
-class Book < ApplicationRecord
-  validates :title, presence: true
-  validates :body, presence: true
-end
-```