New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

この間のアレにspec追加しました。 #12

Closed
wants to merge 4 commits into
base: class-spy
from

Conversation

Projects
None yet
1 participant
@igrep
Owner

igrep commented Sep 20, 2014

先輩、ようやくRSpecの使い方わかってきたので、
この間作ったGreatClassのspec書きました! 😇
後々になってしまってすみません! 💦


※このPull Requestはフィクションです。実在のgem、エンジニア、アプリケーションなどとは一切関係ありません(ただし、crispy自体を除く)。
※私igrepによく似た後輩がPull Requestを送った、という頭のおかしい設定ですが、ご容赦ください。
※また、specを追加したのにビルドが落ちているのはご愛嬌です。

2014年9月21日 RubyHiroba編

igrep added some commits Sep 20, 2014

SomeModule instead of some_other_object
it's shorter

I forgot to define some_other_object, but I don't want to increase the lines, so choose Module, which doesn't need to be initialized by let
end
it 'remembers to do the important thing!' do
expect(SomeModule).to receive(:some_important_method)
subject.some_subject_method

This comment has been minimized.

@igrep

igrep Sep 20, 2014

Owner

⚠️ ああ、ここDRYじゃない。
beforeブロックのsubject.some_subject_methodと完全にダブってます。
これだけ単純なメソッド呼び出しだから気にならないかもしれませんが、
引数が多かったり、もっと複雑な場合でもこのように書きますか?

@igrep

igrep Sep 20, 2014

Owner

⚠️ ああ、ここDRYじゃない。
beforeブロックのsubject.some_subject_methodと完全にダブってます。
これだけ単純なメソッド呼び出しだから気にならないかもしれませんが、
引数が多かったり、もっと複雑な場合でもこのように書きますか?

This comment has been minimized.

@igrep

igrep Sep 20, 2014

Owner

それに、やっぱり他のexampleと違って、
expectationを先に書くのが
なんだかあまりしっくりきませんねぇ。 😕

「対象のメソッドを呼んだ後に〇〇な影響を起こす」
という観点で書くほうが自然じゃありませんか?

@igrep

igrep Sep 20, 2014

Owner

それに、やっぱり他のexampleと違って、
expectationを先に書くのが
なんだかあまりしっくりきませんねぇ。 😕

「対象のメソッドを呼んだ後に〇〇な影響を起こす」
という観点で書くほうが自然じゃありませんか?

This comment has been minimized.

@igrep

igrep Sep 20, 2014

Owner

まぁ、要するに、私が作ったcrispyを使って #13 のように書き換えてください。

@igrep

igrep Sep 20, 2014

Owner

まぁ、要するに、私が作ったcrispyを使って #13 のように書き換えてください。

@igrep

This comment has been minimized.

Show comment
Hide comment
@igrep

igrep Sep 21, 2014

Owner

業務連絡:Ruby Hiroba終わったのでcloseします。

Owner

igrep commented Sep 21, 2014

業務連絡:Ruby Hiroba終わったのでcloseします。

@igrep igrep closed this Sep 21, 2014

@igrep igrep deleted the demo-pr branch Sep 21, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment