これは良い書面によるテストですか、それとも醜いのですか？

Question

アプリケーションが拒否された場合、denied_applicationsは1つ増加し、applicationsは-1になり、membersはまったく変更されません。

# team.rb 
    has_many :memberships, dependent: :destroy 
    has_many :members, -> { where(team_memberships: {state: :confirmed}) }, through: :memberships, source: :user 
    has_many :ex_members, -> { where(team_memberships: {state: :left}) }, through: :memberships, source: :user 
    has_many :applications, -> { where(state: :pending) }, class_name: "Membership" 
    has_many :denied_applications, -> { where(state: :denied) }, class_name: "Membership" 

ので、私の質問は、より可能性が高い：は、これは良いで書かテストであるか、あなただけの一部の人々は私見差アサーションが異なるテストであるべきだと主張、しかしでしょう

team.denied_applications.reload.count.must_equal(1) 
team.applications.reload.count.must_equal(0) 
team.members.reload.count.must_equal(0) 

Answer 1

を記述しますその場合、問題はありません。 、私の心には最適ではありません何

は、あなたのテストが依存していないか、お互いに干渉することを確認するために

let(:team) { Team.last } 
let(:user) { User.last } 

を呼んでいるということです、データベースは、テスト実行の間にクリーニングする必要があり、そうこれが行われている場合、あなたは次のように記述します：

let(:team) { Team.create } 
let(:user) { User.create } 

かあなたには、いくつかの検証が起こっていて、有効なを入力したくない場合は、例えば、より簡単にレコードを作成するためにfactory_girlのようなものを使用することができますデータはどこにでもあります。