Simplify maximum limit of direct messages specs

Now the tests are easier to understand: when the limit is 3, if you
create 3, the fourth one is invalid. If you create 2, the third one is
valid.
This commit is contained in:
Javi Martín
2019-09-25 23:23:34 +02:00
parent d09be11a08
commit 802be29773

View File

@@ -36,9 +36,7 @@ describe DirectMessage do
it "is not valid if above maximum" do
sender = create(:user)
direct_message1 = create(:direct_message, sender: sender)
direct_message2 = create(:direct_message, sender: sender)
direct_message3 = create(:direct_message, sender: sender)
3.times { create(:direct_message, sender: sender) }
direct_message4 = build(:direct_message, sender: sender)
expect(direct_message4).not_to be_valid
@@ -46,8 +44,7 @@ describe DirectMessage do
it "is valid if below maximum" do
sender = create(:user)
direct_message1 = create(:direct_message, sender: sender)
direct_message2 = create(:direct_message, sender: sender)
2.times { create(:direct_message, sender: sender) }
direct_message3 = build(:direct_message, sender: sender)
expect(direct_message3).to be_valid