rails: request specの書き方について

Question

requst specのupdateの書き方を知りたい。

gem deviseを使用しています。

実際のコード

  def update
    current_user.update(update_params)
    redirect_to root_path
  rescue StandardError
    redirect_to root_path
  end

 private
 def update_params
   params.require(:user).permit(:lastname, :firstname, :tell, :sex, :birth_date, :user_type_id, :image)
 end

FactoryBot.define do
  factory :user do
    email { '[e-mail]'}
    password { 'password' }
    password_confirmation { 'password' }
    accepted { true }
    promotion_code { '12345' }
    confirmed_at { Date.today }
  end
end

requests/users_spec.rb
RSpec.describe "Users", type: :request do
  let(:user) { create(:user, id: 1) }


  describe "PATCH / user_path" do
    context 'パラメータで正の時' do
      let(:params_success) { { user: { lastname: 'last_name', firstname: 'first_name', tell: '09099998888', sex: '男性', birth_date: '2000-02-21', user_type_id: 1, image: '' } } }

      it 'リクエストが成功すること' do
        patch '/users/' + user.id.to_s, params: params_success
        expect(User.find_by(id: user.id).firstname).to eq 'first_name'
      end
    end
  end
end

結果>

• expected: first_name
• got: nil

困っていること

patchでレコードを更新しているはずなのに、中身が空のままでテストが落ちてしまう。

   (0.4ms)  SET NAMES utf8 COLLATE utf8_general_ci,  @@SESSION.sql_mode = CONCAT(CONCAT(@@sql_mode, ',STRICT_ALL_TABLES'), ',NO_AUTO_VALUE_ON_ZERO'),  @@SESSION.sql_auto_is_null = 0, @@SESSION.wait_timeout = 2147483
   (0.4ms)  SELECT `ar_internal_metadata`.`value` FROM `ar_internal_metadata` WHERE `ar_internal_metadata`.`key` = 'schema_sha1'
   (0.3ms)  SET NAMES utf8 COLLATE utf8_general_ci,  @@SESSION.sql_mode = CONCAT(CONCAT(@@sql_mode, ',STRICT_ALL_TABLES'), ',NO_AUTO_VALUE_ON_ZERO'),  @@SESSION.sql_auto_is_null = 0, @@SESSION.wait_timeout = 2147483
   (0.3ms)  SELECT `schema_migrations`.`version` FROM `schema_migrations` ORDER BY `schema_migrations`.`version` ASC
   (0.4ms)  BEGIN
   (0.4ms)  SAVEPOINT active_record_1
  User Exists? (0.4ms)  SELECT 1 AS one FROM `users` WHERE `users`.`email` = BINARY '[e-mail]' LIMIT 1
  User Create (0.7ms)  INSERT INTO `users` (`id`, `email`, `encrypted_password`, `confirmed_at`, `created_at`, `updated_at`, `accepted`, `promotion_code`) VALUES (1, '[e-mail]', '$2a$04$4bYXcGJzKQDHBEvXzzzbSulv3URSCxygq6dJNtTshhAY199MNvFG6', '2019-10-13 00:00:00', '2019-10-13 10:57:06.721754', '2019-10-13 10:57:06.721754', TRUE, '12345')
   (0.4ms)  RELEASE SAVEPOINT active_record_1
Started PATCH "/users/1" for 127.0.0.1 at 2019-10-13 10:57:06 +0000
   (0.5ms)  SAVEPOINT active_record_1
  User Update (0.5ms)  UPDATE `users` SET `users`.`sign_in_count` = 1, `users`.`current_sign_in_at` = '2019-10-13 10:57:06', `users`.`last_sign_in_at` = '2019-10-13 10:57:06', `users`.`current_sign_in_ip` = '127.0.0.1', `users`.`last_sign_in_ip` = '127.0.0.1', `users`.`updated_at` = '2019-10-13 10:57:06.738645' WHERE `users`.`id` = 1
   (0.3ms)  RELEASE SAVEPOINT active_record_1
Processing by UsersController#update as HTML
  Parameters: {"user"=>{"id"=>"1", "lastname"=>"last_name", "firstname"=>"first_name", "tell"=>"09088997766", "sex"=>"男性", "birth_date"=>"1996-04-21", "user_type_id"=>"1", "image"=>""}, "id"=>"1"}
  User Load (0.6ms)  SELECT `users`.* FROM `users` WHERE `users`.`id` = 1 LIMIT 1
Unpermitted parameter: :id
   (0.3ms)  SAVEPOINT active_record_1
  User Update (0.4ms)  UPDATE `users` SET `users`.`updated_at` = '2019-10-13 10:57:06.770118', `users`.`firstname` = 'first_name', `users`.`lastname` = 'last_name', `users`.`tell` = '09088997766', `users`.`sex` = '男性', `users`.`birth_date` = '1996-04-21', `users`.`user_type_id` = 1 WHERE `users`.`id` = 1
  ActiveStorage::Blob Load (0.5ms)  SELECT `active_storage_blobs`.* FROM `active_storage_blobs` INNER JOIN `active_storage_attachments` ON `active_storage_blobs`.`id` = `active_storage_attachments`.`blob_id` WHERE `active_storage_attachments`.`record_id` = 1 AND `active_storage_attachments`.`record_type` = 'User' AND `active_storage_attachments`.`name` = 'image' LIMIT 1
   (0.4ms)  ROLLBACK TO SAVEPOINT active_record_1
Redirected to http://www.example.com/users/1/profile
Completed 302 Found in 31ms (ActiveRecord: 4.1ms | Allocations: 15462)
   (1.1ms)  ROLLBACK

Answer 1

こんにちは。

test.log 側に、PATCHリクエストを送信した時のログが出ていると思います。

• テストの際に送信されているリクエストパラメータ
• developmentモード＆ブラウザで想定操作をした際のリクエストパラメータ

を比べてみてはいかがでしょうか？
うまくリクエストが通っていれば、developmentモードと同じように、PATCH操作によってUPDATEのSQLが走ります。
バリデーションエラーや権限エラーであれば、たぶん同様にログが出ていると思います。

また、PATCHのリクエストにあたっては、認証が必要でしょうか？
deviceを使っていらっしゃるので、もし認証が必要な操作なら、サインイン状態になっていないといけないかな？と思います。

expect(User.find_by(id: user.id).firstname).to eq 'first_name'

のところを、いったん

expect(response).to have_http_status(302)

としてみて、ステータスコードを確認してみてはいかがでしょう。

Comments

[nanase21]

ご教示いただきありがとうございます。
ご指摘通り、userはfactorybotで作ってもログインはしていませんでした。
なので、request spec用にsign_in helperを作成し、再度試してみたのですが、やはり上手くいきませんでした。
test.logをアップしたのでもし原因がお分かりであればご教示いただけると幸いです。

[suama]

ログを見ると、「Unpermitted parameter: :id」というエラーが出ていますね。
String Parameterの設定を見てみるとよいのでは。
developmentモードで画面からリクエストを出した時のパラメータと、比較してみてくださいね。

テストだと、
Parameters: {"user"=>{"id"=>"1", "lastname"=>"last_name", "firstname"=>"first_name", "tell"=>"09088997766", "sex"=>"男性", "birth_date"=>"1996-04-21", "user_type_id"=>"1", "image"=>""}, "id"=>"1"}
というパラメータになっています。ここに、"id" が入っていますので、その辺を確認すればいいのかなと。

認証の件は大丈夫そうですが、deviseのドキュメントも添えておきますね。
https://github.com/plataformatec/devise/wiki/How-To:-sign-in-and-out-a-user-in-Request-type-specs-(specs-tagged-with-type:-:request)