diff options
author | Douwe Maan <douwe@gitlab.com> | 2016-12-02 04:32:23 +0000 |
---|---|---|
committer | Douwe Maan <douwe@gitlab.com> | 2016-12-02 04:32:23 +0000 |
commit | ec4e7d9a9f7f3526a15905c3fb5d851e4c173349 (patch) | |
tree | a67d8d342dc0e317a38014fb1c93de771bb137db /spec/services | |
parent | 71ba28e43967b5eaf0499c2a93c0b1d4f78a35f5 (diff) | |
parent | 6b4d33566f5f434cc86381a4a1347e42bbe348ee (diff) | |
download | gitlab-ce-ec4e7d9a9f7f3526a15905c3fb5d851e4c173349.tar.gz |
Merge branch 'process-commit-worker-improvements' into 'master'
Pass commit data to ProcessCommitWorker
This changes `ProcessCommitWorker` so that it takes a Hash containing commit data instead of a commit SHA. This means the worker doesn't have to access Git just to process a commit message (and other data it may use). This in turn should solve the problem of ending up with 15 000-something jobs in the `process_commit` queue that take forever to process.
See merge request !7744
Diffstat (limited to 'spec/services')
-rw-r--r-- | spec/services/git_push_service_spec.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/spec/services/git_push_service_spec.rb b/spec/services/git_push_service_spec.rb index 9d7702f5c96..e7624e70725 100644 --- a/spec/services/git_push_service_spec.rb +++ b/spec/services/git_push_service_spec.rb @@ -263,7 +263,7 @@ describe GitPushService, services: true do author_email: commit_author.email ) - allow_any_instance_of(ProcessCommitWorker).to receive(:find_commit). + allow_any_instance_of(ProcessCommitWorker).to receive(:build_commit). and_return(commit) allow(project.repository).to receive(:commits_between).and_return([commit]) @@ -321,7 +321,7 @@ describe GitPushService, services: true do committed_date: commit_time ) - allow_any_instance_of(ProcessCommitWorker).to receive(:find_commit). + allow_any_instance_of(ProcessCommitWorker).to receive(:build_commit). and_return(commit) allow(project.repository).to receive(:commits_between).and_return([commit]) @@ -360,7 +360,7 @@ describe GitPushService, services: true do allow(project.repository).to receive(:commits_between). and_return([closing_commit]) - allow_any_instance_of(ProcessCommitWorker).to receive(:find_commit). + allow_any_instance_of(ProcessCommitWorker).to receive(:build_commit). and_return(closing_commit) project.team << [commit_author, :master] |