Commit a196d949 authored by Mayra Cabrera's avatar Mayra Cabrera Committed by Robert Speicher

Fixes broken specs

- Remove unwanted chat was failing as a consequence of moving external support from Premium to starter. We have a dependency on config_processor a project with a repository must be used

Closes #5171
parent 58f0bd67
require 'spec_helper' require 'spec_helper'
describe EE::Gitlab::Ci::Pipeline::Chain::RemoveUnwantedChatJobs do describe EE::Gitlab::Ci::Pipeline::Chain::RemoveUnwantedChatJobs do
let(:project) { create(:project) } let(:project) { create(:project, :repository) }
let(:pipeline) do let(:pipeline) do
build(:ci_pipeline_with_one_job, project: project, ref: 'master') build(:ci_pipeline_with_one_job, project: project, ref: 'master')
......
...@@ -3,7 +3,7 @@ require 'spec_helper' ...@@ -3,7 +3,7 @@ require 'spec_helper'
describe BuildFinishedWorker do describe BuildFinishedWorker do
describe '#perform' do describe '#perform' do
it 'schedules a ChatNotification job for a chat build' do it 'schedules a ChatNotification job for a chat build' do
build = create(:ci_build, pipeline: create(:ci_pipeline, source: :chat)) build = create(:ci_build, :success, pipeline: create(:ci_pipeline, source: :chat))
expect(ChatNotificationWorker) expect(ChatNotificationWorker)
.to receive(:perform_async) .to receive(:perform_async)
...@@ -13,7 +13,7 @@ describe BuildFinishedWorker do ...@@ -13,7 +13,7 @@ describe BuildFinishedWorker do
end end
it 'does not schedule a ChatNotification job for a regular build' do it 'does not schedule a ChatNotification job for a regular build' do
build = create(:ci_build, pipeline: create(:ci_pipeline)) build = create(:ci_build, :success, pipeline: create(:ci_pipeline))
expect(ChatNotificationWorker) expect(ChatNotificationWorker)
.not_to receive(:perform_async) .not_to receive(:perform_async)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment