2016-12-21

Rspec　コードメモ

・通常の書き方（factory-girlなし） user_spec.rb

require 'rails_helper'
describe User do
  describe '#create' do
    it "is invalid without a email" do #ここはテストの条件式。今回の条件はemailがないと無効になるか
     user = User.new(nickname: "abe", email: "", password: "00000000", password_confirmation: "00000000") #インスタンスの生成。emailはからの設定
     user.valid? #インスタンスを保存する際に「バリデーションにより保存ができない状態であるか」を確かめる
     expect(user.errors[:email]).to include("can't be blank") #errorsメソッドを利用すると、バリデーションにより保存ができない状態である場合なぜできないのかを確認することができる
    end
  end
end

・factory-girlあり

require 'rails_helper'
describe User do
  describe '#create' do
    it "is invalid without a email" do #ここはテストの条件式。今回の条件はemailがないと無効になるか
     user = User.build(:user, nickname: "") #factory-girlを使い、buildメソッドで記述を省略
     user.valid? #インスタンスを保存する際に「バリデーションにより保存ができない状態であるか」を確かめる
     expect(user.errors[:nickname]).to include("can't be blank") #errorsメソッドを利用すると、バリデーションにより保存ができない状態である場合なぜできないのかを確認することができる
    end
    
  end
end

2016-12-20

rspec エラーまとめ

ruby on rails

・エラー文

Web Console is activated in the test environment, which is
usually a mistake. To ensure it's only activated in development
mode, move it to the development group of your Gemfile:

    gem 'web-console', group: :development

If you still want to run it the test environment (and know
what you are doing), put this in your Rails application
configuration:

    config.web_console.development_only = false

・以下を参考に解決を試みる stonebeach-dakar.hatenablog.com

手順１
config/application.rb

config.web_console.development_only = false

以上を追加

手順２
gemfile

gem 'web-console', '~> 2.0'

しかし、以上ではうまくいかなかった。

・他の原因 DBを作っていなかったことをがわかり、作成。
すると、

Running via Spring preloader in process 3663

User
  #create
    is invalid without a nickname

Finished in 0.00178 seconds (files took 4.23 seconds to load)
1 example, 0 failures

かんせー

2016-12-20

レビュー反省 2016 12 20

レビューの反省

・permitでuser_id、group_idを許すのか、mergeでuser_id、group_idを加えるのかどちらかにしましょう。

params.require(:message).permit(:message, :user_id, :group_id).merge(user_id: current_user.id, group_id: params[:group_id])

・elseの後、改行しましょう

if @message.save!
redirect_to group_path(@group)
else

redirect_to :back, alert: 'メッセージを入力してください'

こうするといい。

・save!は保存に失敗したときに例外を発生させます。単純にfalseが返ってきて欲しい時はsaveを使いましょう

・現在はchatの画面をgroups_cpntrollerのshowで実装していると思います。しかし、chat画面はmessageの一覧が表示されていると考えてmessages_controllerのindexを使って実装する方が良いかなと思います。
ルーティングでmessageをネストしているのでmessages_controllerでもgroup_idを使うことができます。そのgroup_idを使ってグループを取得し、関連付けられたメッセージを一覧で表示してみましょう。

messageコントローラにて、投稿されたmessageの表示を行うようにする。

やり方

１、messageコントローラにindexアクションを作る。

２、中身を作る。役割は以下のとおり。

def index
@group = Group.find(params[:group_id]) #グループのページにアクセスした時に、そのグループの番号を受け取る
@messages = @group.messages #上で受け取ったグループの番号に関連したmessageテーブルの値を取得する
@message = Message.new #form_forに入力される値を受け取るための箱を用意する
end

３、viewをmessagesフォルダに作る。viewで使っている変数で変える必要がある部分は変える。

・changeについて

コミットのメッセージで迷うから。

detail.chiebukuro.yahoo.co.jp

・リダイレクトの際に、いちいち変数でgroup_idを取得して飛ばす必要はない。

def create

message = Message.new(create_params)
group_id = params[:group_id] #リダイレクトをするために、現在のページのgroup_idを取得する

if message.save
redirect_to group_messages_path(group_id)

これを・・・・・・・・・

redirect_to group_messages_path(params[:group_id])

こうする。

2016-12-19

レビューの反省　2016 12 19

レビューの反省

・@groupだとgroupのインスタンスが入っている様に見えてしまうので変数名を変えましょう

def create
@message = Message.new(create_params) #入力された値を保存する
@group = params[:group_id] #リダイレクトをするために、現在のページのgroup_idを取得する

・save!は保存に失敗したときに例外を発生させます。単純にfalseが返ってきて欲しい時はsaveを使いましょう

参考

d.hatena.ne.jp

・elseの後、改行しましょう

@group = params[:group_id] #リダイレクトをするために、現在のページのgroup_idを取得する
if @message.save!
redirect_to group_path(@group)
else redirect_to :back, alert: 'メッセージを入力してください'

・permitでuser_id、group_idを許すのか、mergeでuser_id、group_idを加えるのかどちらかにしましょう。

def create_params
params.require(:message).permit(:message, :user_id, :group_id).merge(user_id: current_user.id, group_id: params[:group_id])