diff options
author | Patricio Cano <suprnova32@gmail.com> | 2016-07-18 18:17:43 -0500 |
---|---|---|
committer | Patricio Cano <suprnova32@gmail.com> | 2016-07-26 15:17:52 -0500 |
commit | f7807c5b68b59f6a5b984ee64a6c82a3bd993d92 (patch) | |
tree | 48e30fc7667ca0a2df6d67b9ee692aede869656d /spec | |
parent | 9c34fafb8b728358a516a25120aa5f28567eae48 (diff) | |
download | gitlab-ce-f7807c5b68b59f6a5b984ee64a6c82a3bd993d92.tar.gz |
Submit all issues on public projects to Akismet if enabled.
Diffstat (limited to 'spec')
-rw-r--r-- | spec/lib/gitlab/akismet_helper_spec.rb | 12 | ||||
-rw-r--r-- | spec/requests/api/issues_spec.rb | 6 |
2 files changed, 8 insertions, 10 deletions
diff --git a/spec/lib/gitlab/akismet_helper_spec.rb b/spec/lib/gitlab/akismet_helper_spec.rb index 88a71528867..b08396da4d2 100644 --- a/spec/lib/gitlab/akismet_helper_spec.rb +++ b/spec/lib/gitlab/akismet_helper_spec.rb @@ -1,7 +1,7 @@ require 'spec_helper' describe Gitlab::AkismetHelper, type: :helper do - let(:project) { create(:project) } + let(:project) { create(:project, :public) } let(:user) { create(:user) } before do @@ -11,13 +11,13 @@ describe Gitlab::AkismetHelper, type: :helper do end describe '#check_for_spam?' do - it 'returns true for non-member' do - expect(helper.check_for_spam?(project, user)).to eq(true) + it 'returns true for public project' do + expect(helper.check_for_spam?(project)).to eq(true) end - it 'returns false for member' do - project.team << [user, :guest] - expect(helper.check_for_spam?(project, user)).to eq(false) + it 'returns false for private project' do + project.update_attribute(:visibility_level, Gitlab::VisibilityLevel::PRIVATE) + expect(helper.check_for_spam?(project)).to eq(false) end end diff --git a/spec/requests/api/issues_spec.rb b/spec/requests/api/issues_spec.rb index 12f2cfa6942..9d3d28e0b91 100644 --- a/spec/requests/api/issues_spec.rb +++ b/spec/requests/api/issues_spec.rb @@ -531,10 +531,8 @@ describe API::API, api: true do describe 'POST /projects/:id/issues with spam filtering' do before do - Grape::Endpoint.before_each do |endpoint| - allow(endpoint).to receive(:check_for_spam?).and_return(true) - allow(endpoint).to receive(:is_spam?).and_return(true) - end + allow_any_instance_of(Gitlab::AkismetHelper).to receive(:check_for_spam?).and_return(true) + allow_any_instance_of(Gitlab::AkismetHelper).to receive(:is_spam?).and_return(true) end let(:params) do |