db97f9d08c
We were very inconsistent regarding these rules. Personally I prefer no empty lines around blocks, clases, etc... as recommended by the Ruby style guide [1], and they're the default values in rubocop, so those are the settings I'm applying. The exception is the `private` access modifier, since we were leaving empty lines around it most of the time. That's the default rubocop rule as well. Personally I don't have a strong preference about this one. [1] https://rubystyle.guide/#empty-lines-around-bodies
54 lines
1.5 KiB
Ruby
54 lines
1.5 KiB
Ruby
require "rails_helper"
|
|
|
|
describe Flag do
|
|
let(:user) { create(:user) }
|
|
let(:comment) { create(:comment) }
|
|
|
|
describe ".flag" do
|
|
it "creates a flag when there is none" do
|
|
expect { Flag.flag(user, comment) }.to change { Flag.count }.by(1)
|
|
expect(Flag.last.user).to eq(user)
|
|
expect(Flag.last.flaggable).to eq(comment)
|
|
end
|
|
|
|
it "does nothing if the flag already exists" do
|
|
Flag.flag(user, comment)
|
|
expect(Flag.flag(user, comment)).to eq(false)
|
|
expect(Flag.by_user_and_flaggable(user, comment).count).to eq(1)
|
|
end
|
|
|
|
it "increases the flag count" do
|
|
expect { Flag.flag(user, comment) }.to change { comment.reload.flags_count }.by(1)
|
|
end
|
|
end
|
|
|
|
describe ".unflag" do
|
|
it "raises an error if the flag does not exist" do
|
|
expect(Flag.unflag(user, comment)).to eq(false)
|
|
end
|
|
|
|
describe "when the flag already exists" do
|
|
before { Flag.flag(user, comment) }
|
|
|
|
it "removes an existing flag" do
|
|
expect { Flag.unflag(user, comment) }.to change { Flag.count }.by(-1)
|
|
end
|
|
|
|
it "decreases the flag count" do
|
|
expect { Flag.unflag(user, comment) }.to change { comment.reload.flags_count }.by(-1)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe ".flagged?" do
|
|
it "returns false when the user has not flagged the comment" do
|
|
expect(Flag.flagged?(user, comment)).not_to be
|
|
end
|
|
|
|
it "returns true when the user has flagged the comment" do
|
|
Flag.flag(user, comment)
|
|
expect(Flag.flagged?(user, comment)).to be
|
|
end
|
|
end
|
|
end
|