diff --git a/spec/controllers/admin/spam_logs_controller_spec.rb b/spec/controllers/admin/spam_logs_controller_spec.rb index 6beebc362e9..b51b303a714 100644 --- a/spec/controllers/admin/spam_logs_controller_spec.rb +++ b/spec/controllers/admin/spam_logs_controller_spec.rb @@ -20,10 +20,7 @@ describe Admin::SpamLogsController do describe '#destroy' do it 'removes only the spam log when removing log' do - expect { - delete :destroy, id: first_spam.id - }.to change { SpamLog.count }.by(-1) - + expect { delete :destroy, id: first_spam.id }.to change { SpamLog.count }.by(-1) expect(User.find(user.id)).to be_truthy expect(response.status).to eq(200) end diff --git a/spec/requests/api/issues_spec.rb b/spec/requests/api/issues_spec.rb index dcf1b394a3f..4443036d98f 100644 --- a/spec/requests/api/issues_spec.rb +++ b/spec/requests/api/issues_spec.rb @@ -249,12 +249,16 @@ describe API::API, api: true do end end - it "should not create a new project issue" do - expect { - post api("/projects/#{project.id}/issues", user), - title: 'new issue', description: 'content here', labels: 'label, label2' - }.not_to change(Issue, :count) + let(:params) do + { + title: 'new issue', + description: 'content here', + labels: 'label, label2' + } + end + it "should not create a new project issue" do + expect { post api("/projects/#{project.id}/issues", user), params }.not_to change(Issue, :count) expect(response.status).to eq(400) expect(json_response['message']).to eq({ "error" => "Spam detected" })