Skip to content
This repository was archived by the owner on Feb 9, 2023. It is now read-only.

Conversation

@orzup
Copy link
Owner

@orzup orzup commented Jul 1, 2015

やること

  • ホームとプロフィールのstatsに関するテストを書こう
  • ホームでフィードが表示されるか確認するテストを書こう

完了条件

  • 2人にLGTMもらう
  • テストが通る

orzup added 6 commits July 1, 2015 16:39
1ページ目に表示されるべきフィードかどうかに関するテスト>に修正した。また、正しいフィードの値を担保するテスト項目をインテグレーションテストから削除した(モデルにすでにあった)。
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

このテストだと、数字が一致しただけで通ってしまうのでよくなさそう

Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#26 (comment) と同じ意味だよね?

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes!

@orzup orzup changed the title [WIP]Chapter12のエクササイズをやる Chapter12のエクササイズをやる Jul 2, 2015
@ku00
Copy link

ku00 commented Jul 2, 2015

よさそう 👍

@hanazuki
Copy link

hanazuki commented Jul 2, 2015

👍

orzup added a commit that referenced this pull request Jul 2, 2015
Chapter12のエクササイズをやった!!!はじまりのおわりだ!!
@orzup orzup merged commit 42ce28c into sample_app Jul 2, 2015
@orzup orzup deleted the exercises12 branch July 2, 2015 06:30
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants