Commit d09d6ad0 authored by Lin Jen-Shin's avatar Lin Jen-Shin

Test against all types and more status:

Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/7508#note_18748231
parent 9b124078
...@@ -124,12 +124,9 @@ describe HasStatus do ...@@ -124,12 +124,9 @@ describe HasStatus do
end end
end end
def self.random_type
%i[ci_build generic_commit_status].sample
end
context 'for scope with one status' do context 'for scope with one status' do
shared_examples 'having a job' do |type, status| shared_examples 'having a job' do |status|
%i[ci_build generic_commit_status].each do |type|
context "when it's #{status} #{type} job" do context "when it's #{status} #{type} job" do
let!(:job) { create(type, status) } let!(:job) { create(type, status) }
...@@ -153,15 +150,17 @@ describe HasStatus do ...@@ -153,15 +150,17 @@ describe HasStatus do
end end
end end
end end
end
%i[created running pending success %i[created running pending success
failed canceled skipped].each do |status| failed canceled skipped].each do |status|
it_behaves_like 'having a job', random_type, status it_behaves_like 'having a job', status
end end
end end
context 'for scope with more statuses' do context 'for scope with more statuses' do
shared_examples 'containing the job' do |type, status| shared_examples 'containing the job' do |status|
%i[ci_build generic_commit_status].each do |type|
context "when it's #{status} #{type} job" do context "when it's #{status} #{type} job" do
let!(:job) { create(type, status) } let!(:job) { create(type, status) }
...@@ -170,8 +169,10 @@ describe HasStatus do ...@@ -170,8 +169,10 @@ describe HasStatus do
end end
end end
end end
end
shared_examples 'not containing the job' do |type, status| shared_examples 'not containing the job' do |status|
%i[ci_build generic_commit_status].each do |type|
context "when it's #{status} #{type} job" do context "when it's #{status} #{type} job" do
let!(:job) { create(type, status) } let!(:job) { create(type, status) }
...@@ -180,35 +181,42 @@ describe HasStatus do ...@@ -180,35 +181,42 @@ describe HasStatus do
end end
end end
end end
end
describe '.running_or_pending' do describe '.running_or_pending' do
subject { CommitStatus.running_or_pending } subject { CommitStatus.running_or_pending }
%i[running pending].each do |status| %i[running pending].each do |status|
it_behaves_like 'containing the job', random_type, status it_behaves_like 'containing the job', status
end end
it_behaves_like 'not containing the job', random_type, :created %i[created failed success].each do |status|
it_behaves_like 'not containing the job', status
end
end end
describe '.finished' do describe '.finished' do
subject { CommitStatus.finished } subject { CommitStatus.finished }
%i[success failed canceled].each do |status| %i[success failed canceled].each do |status|
it_behaves_like 'containing the job', random_type, status it_behaves_like 'containing the job', status
end end
it_behaves_like 'not containing the job', random_type, :created %i[created running pending].each do |status|
it_behaves_like 'not containing the job', status
end
end end
describe '.cancelable' do describe '.cancelable' do
subject { CommitStatus.cancelable } subject { CommitStatus.cancelable }
%i[running pending created].each do |status| %i[running pending created].each do |status|
it_behaves_like 'containing the job', random_type, status it_behaves_like 'containing the job', status
end end
it_behaves_like 'not containing the job', random_type, :failed %i[failed success skipped canceled].each do |status|
it_behaves_like 'not containing the job', status
end
end end
end end
end end
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