Commit e72c50fc authored by Shinya Maeda's avatar Shinya Maeda

Change sort spec description

parent 3735b8aa
...@@ -216,7 +216,7 @@ describe PipelinesFinder do ...@@ -216,7 +216,7 @@ describe PipelinesFinder do
context 'when order_by is invalid' do context 'when order_by is invalid' do
let(:params) { { order_by: 'invalid_column', sort: 'desc' } } let(:params) { { order_by: 'invalid_column', sort: 'desc' } }
it 'sorts pipelines, but order_by is default' do it 'sorts pipelines with default order_by (id:)' do
expect(subject).to eq(Ci::Pipeline.order(id: :desc)) expect(subject).to eq(Ci::Pipeline.order(id: :desc))
end end
end end
...@@ -224,7 +224,7 @@ describe PipelinesFinder do ...@@ -224,7 +224,7 @@ describe PipelinesFinder do
context 'when sort is invalid' do context 'when sort is invalid' do
let(:params) { { order_by: 'created_at', sort: 'invalid_sort' } } let(:params) { { order_by: 'created_at', sort: 'invalid_sort' } }
it 'sorts pipelines, but sort is default' do it 'sorts pipelines with default sort (:desc)' do
expect(subject).to eq(Ci::Pipeline.order(created_at: :desc)) expect(subject).to eq(Ci::Pipeline.order(created_at: :desc))
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