diff options
author | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-10-21 15:12:11 +0200 |
---|---|---|
committer | Kamil Trzcinski <ayufan@ayufan.eu> | 2016-10-26 11:37:23 +0200 |
commit | 47b2add4f62d64d1a46109910c86805728e548c4 (patch) | |
tree | 58bee6cd9dbd1f4b7be7761578b9a18486998b36 /spec/lib/gitlab/optimistic_locking_spec.rb | |
parent | 5d7ee7a1b6c818dd0ccba6a393875072dabd7eba (diff) | |
download | gitlab-ce-47b2add4f62d64d1a46109910c86805728e548c4.tar.gz |
Add tests for optimistic locking
Diffstat (limited to 'spec/lib/gitlab/optimistic_locking_spec.rb')
-rw-r--r-- | spec/lib/gitlab/optimistic_locking_spec.rb | 28 |
1 files changed, 28 insertions, 0 deletions
diff --git a/spec/lib/gitlab/optimistic_locking_spec.rb b/spec/lib/gitlab/optimistic_locking_spec.rb new file mode 100644 index 00000000000..75c78cf077a --- /dev/null +++ b/spec/lib/gitlab/optimistic_locking_spec.rb @@ -0,0 +1,28 @@ +require 'spec_helper' + +describe Gitlab::OptimisticLocking, lib: true do + describe '#retry_lock' do + let!(:pipeline) { create(:ci_pipeline) } + let!(:pipeline2) { Ci::Pipeline.find(pipeline.id) } + + it 'does not reload object if state changes' do + expect(pipeline).not_to receive(:reload) + expect(pipeline).to receive(:succeed).and_call_original + + described_class.retry_lock(pipeline) do |subject| + subject.succeed + end + end + + it 'retries action if exception is raised' do + pipeline.succeed + + expect(pipeline2).to receive(:reload).and_call_original + expect(pipeline2).to receive(:drop).twice.and_call_original + + described_class.retry_lock(pipeline2) do |subject| + subject.drop + end + end + end +end |