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

Should use eq because we want orders

parent f3b02b9e
...@@ -245,8 +245,7 @@ describe Ci::ProcessPipelineService, services: true do ...@@ -245,8 +245,7 @@ describe Ci::ProcessPipelineService, services: true do
end end
it 'starts from the second stage' do it 'starts from the second stage' do
expect(builds.map(&:status)).to contain_exactly( expect(builds.map(&:status)).to eq(%w[skipped pending created])
'skipped', 'pending', 'created')
end end
end end
...@@ -258,14 +257,12 @@ describe Ci::ProcessPipelineService, services: true do ...@@ -258,14 +257,12 @@ describe Ci::ProcessPipelineService, services: true do
end end
it 'skips second stage and continues on third stage' do it 'skips second stage and continues on third stage' do
expect(builds.map(&:status)).to contain_exactly( expect(builds.map(&:status)).to eq(%w[pending created created])
'pending', 'created', 'created')
builds.first.success builds.first.success
builds.each(&:reload) builds.each(&:reload)
expect(builds.map(&:status)).to contain_exactly( expect(builds.map(&:status)).to eq(%w[success skipped pending])
'success', 'skipped', 'pending')
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