From 6e4e1050d9dba2b7b2523fdd1768823ab85feef4 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Thu, 20 Aug 2020 18:42:06 +0000 Subject: Add latest changes from gitlab-org/gitlab@13-3-stable-ee --- spec/models/commit_status_spec.rb | 56 ++++++++++++++------------------------- 1 file changed, 20 insertions(+), 36 deletions(-) (limited to 'spec/models/commit_status_spec.rb') diff --git a/spec/models/commit_status_spec.rb b/spec/models/commit_status_spec.rb index cd0110a787b..7f893d6a100 100644 --- a/spec/models/commit_status_spec.rb +++ b/spec/models/commit_status_spec.rb @@ -66,51 +66,35 @@ RSpec.describe CommitStatus do describe '#processed' do subject { commit_status.processed } - context 'when ci_atomic_processing is disabled' do + context 'status is latest' do before do - stub_feature_flags(ci_atomic_processing: false) - - commit_status.save! + commit_status.update!(retried: false, status: :pending) end - it { is_expected.to be_nil } + it { is_expected.to be_falsey } end - context 'when ci_atomic_processing is enabled' do + context 'status is retried' do before do - stub_feature_flags(ci_atomic_processing: true) - end - - context 'status is latest' do - before do - commit_status.update!(retried: false, status: :pending) - end - - it { is_expected.to be_falsey } + commit_status.update!(retried: true, status: :pending) end - context 'status is retried' do - before do - commit_status.update!(retried: true, status: :pending) - end - - it { is_expected.to be_truthy } - end + it { is_expected.to be_truthy } + end - it "processed state is always persisted" do - commit_status.update!(retried: false, status: :pending) + it "processed state is always persisted" do + commit_status.update!(retried: false, status: :pending) - # another process does mark object as processed - CommitStatus.find(commit_status.id).update_column(:processed, true) + # another process does mark object as processed + CommitStatus.find(commit_status.id).update_column(:processed, true) - # subsequent status transitions on the same instance - # always saves processed=false to DB even though - # the current value did not change - commit_status.update!(retried: false, status: :running) + # subsequent status transitions on the same instance + # always saves processed=false to DB even though + # the current value did not change + commit_status.update!(retried: false, status: :running) - # we look at a persisted state in DB - expect(CommitStatus.find(commit_status.id).processed).to eq(false) - end + # we look at a persisted state in DB + expect(CommitStatus.find(commit_status.id).processed).to eq(false) end end @@ -438,7 +422,7 @@ RSpec.describe CommitStatus do end it 'returns a correct compound status' do - expect(described_class.all.slow_composite_status(project: project)).to eq 'running' + expect(described_class.all.composite_status).to eq 'running' end end @@ -448,7 +432,7 @@ RSpec.describe CommitStatus do end it 'returns status that indicates success' do - expect(described_class.all.slow_composite_status(project: project)).to eq 'success' + expect(described_class.all.composite_status).to eq 'success' end end @@ -459,7 +443,7 @@ RSpec.describe CommitStatus do end it 'returns status according to the scope' do - expect(described_class.latest.slow_composite_status(project: project)).to eq 'success' + expect(described_class.latest.composite_status).to eq 'success' end end end -- cgit v1.2.3