require 'spec_helper' describe Comment do context 'validations' do it 'should require a body' do comment = Fabricate.build(:comment, :body => nil) expect(comment).to_not be_valid expect(comment.errors[:body]).to include("can't be blank") end end context 'notification_recipients' do let(:app) { Fabricate(:app) } let!(:watcher) { Fabricate(:watcher, :app => app) } let(:err) { Fabricate(:problem, :app => app) } let(:comment_user) { Fabricate(:user, :email => 'author@example.com') } let(:comment) { Fabricate.build(:comment, :err => err, :user => comment_user) } before do Fabricate(:user_watcher, :app => app, :user => comment_user) end it 'includes app notification_recipients except user email' do expect(comment.notification_recipients).to eq [watcher.address] end end context 'emailable?' do let(:app) { Fabricate(:app, :notify_on_errs => true) } let!(:watcher) { Fabricate(:watcher, :app => app) } let(:err) { Fabricate(:problem, :app => app) } let(:comment_user) { Fabricate(:user, :email => 'author@example.com') } let(:comment) { Fabricate.build(:comment, :err => err, :user => comment_user) } before do Fabricate(:user_watcher, :app => app, :user => comment_user) end it 'should be true if app is emailable? and there are notification recipients' do expect(comment.emailable?).to be_true end it 'should be false if app is not emailable?' do app.update_attribute(:notify_on_errs, false) expect(comment.notification_recipients).to be_any expect(comment.emailable?).to be_false end it 'should be false if there are no notification recipients' do watcher.destroy expect(app.emailable?).to be_true expect(comment.emailable?).to be_false end end end