Q&A
グローバルですか、かなり嫌ですね、、、
で、
モックの中で設定ではだめなのですか？
前提・実現したいこと
元々Java開発をしていましたが、現在Rails開発に携わっています。
Rspecでプロダクトコードの登録処理をテストしています。
その際のバリデーション処理は別クラスで主な処理がされているので、モック化しています。
しかし、そのバリデーションのメソッド内部でインスタンス変数に値を代入する処理がされており、specが通りません。
そのインスタンス変数を操作して、仮の値を入れて、テストを進めたいのですが、いいやり方ありますでしょうか？
試したこと
- instance_variable_setを調べており、
obj.instance_variable_set('@ary[0].hoge_id', 1)
とやってみたが、プロダクトコードの@aryには値が設定されていなかった
モックの中で設定の書き方がイマイチわかっておらず、、????????
モック載せてください
before do
allow_any_instance_of(hoge.class).to receive(:validate) do
#ここで@ary[0].fuga_id = 1とかしたがうまくいかなかった
nil
end
end
上記のような感じです。
モックとしては機能しており、
テスト実行時、validateメソッドからはnilが返るようになりましたが、グローバル変数の値を操作できませんでした
グローバル変数は例えが悪いかもだが @ary の想定
すみません、グローバル変数は$hogeですね、@hogeなのでインスタンス変数でした????
タイトル本文も修正しました????
回答1件
0
ベストアンサー
mock は戻り値を返すだけなので、インスタンス変数はできないだろうとおもいます。グローバルなら、、
rubocopは使うな、と言ってきますが、一時的にvalidateを変えてしまう方法があります。
it 'kokode' do class Hoge < AR def validate @ary ||= [] ary[0] = 1 end end ここからtest書く end
ただ、@ary[0].fuga_id = 1 ってことは @aryは単なるArrayではなく、何らかのobjectが代入されているのですね？ すると上のではだめだ。そうとうごちゃごちゃしてる。
method validate がテストと設定の二役をやってるのが良くない。
テストと設定はわけて別のmethodにするのを薦めます。
投稿2021/10/03 22:48
総合スコア23324
ご返答いただきありがとうございます！
そうですね、オブジェクトが設定されています。
なかなか難儀そうなのですね。
ご提示頂いたテストコードの中で一時的にプロダクトコードを変更するやり方は発想になかったので勉強になります。
ただ、回答に載せた方法はとても危ういです。databaseは it 毎にリセットされますが、codeを変えたのがいつ戻るか、、、
対象となるインスタンスを作って、そのmethodを書き換える という様に。
たしかにおっしゃる通りですね。
メソッドの書き換え方法は参考程度にしておき、実際はそのプロダクトコードのメソッドの処理を通してテストするようにしたいと思います。
あなたの回答
tips
プレビュー
