summaryrefslogtreecommitdiff
path: root/spec/services
diff options
context:
space:
mode:
authorYorick Peterse <yorickpeterse@gmail.com>2016-10-14 10:15:39 +0000
committerYorick Peterse <yorickpeterse@gmail.com>2016-10-14 10:15:39 +0000
commit35cd09d6ae596fb252f9a6e6965ac154ef1e90ca (patch)
treecc9aee621cb2b159667b106b1d61fd3045d855d6 /spec/services
parent456149345ba441b00d4de96b8eb211b9e1aab7e9 (diff)
parentbba47886264d0ca7650d2b4b202d69984650b0ae (diff)
downloadgitlab-ce-35cd09d6ae596fb252f9a6e6965ac154ef1e90ca.tar.gz
Merge branch '15463-extract-update-merge-requests' into 'master'
Extract project#update_merge_requests to its own worker from GitPushService See merge request !6767
Diffstat (limited to 'spec/services')
-rw-r--r--spec/services/git_push_service_spec.rb4
-rw-r--r--spec/services/merge_requests/refresh_service_spec.rb3
2 files changed, 4 insertions, 3 deletions
diff --git a/spec/services/git_push_service_spec.rb b/spec/services/git_push_service_spec.rb
index 8e3e12114f2..dd2a9e9903a 100644
--- a/spec/services/git_push_service_spec.rb
+++ b/spec/services/git_push_service_spec.rb
@@ -184,8 +184,8 @@ describe GitPushService, services: true do
context "Updates merge requests" do
it "when pushing a new branch for the first time" do
- expect(project).to receive(:update_merge_requests).
- with(@blankrev, 'newrev', 'refs/heads/master', user)
+ expect(UpdateMergeRequestsWorker).to receive(:perform_async).
+ with(project.id, user.id, @blankrev, 'newrev', 'refs/heads/master')
execute_service(project, user, @blankrev, 'newrev', 'refs/heads/master' )
end
end
diff --git a/spec/services/merge_requests/refresh_service_spec.rb b/spec/services/merge_requests/refresh_service_spec.rb
index 5b4e4908add..e515bc9f89c 100644
--- a/spec/services/merge_requests/refresh_service_spec.rb
+++ b/spec/services/merge_requests/refresh_service_spec.rb
@@ -62,7 +62,8 @@ describe MergeRequests::RefreshService, services: true do
it { expect(@merge_request.notes).not_to be_empty }
it { expect(@merge_request).to be_open }
- it { expect(@merge_request.merge_when_build_succeeds).to be_falsey}
+ it { expect(@merge_request.merge_when_build_succeeds).to be_falsey }
+ it { expect(@merge_request.diff_head_sha).to eq(@newrev) }
it { expect(@fork_merge_request).to be_open }
it { expect(@fork_merge_request.notes).to be_empty }
it { expect(@build_failed_todo).to be_done }