diff --git a/spec/finders/pipelines_finder_spec.rb b/spec/finders/pipelines_finder_spec.rb index a218f315f792cfe2a305c7e23e4409e58a104b31..b5cb7d75c5e7083b84b6917101ccfee2e3ac8a30 100644 --- a/spec/finders/pipelines_finder_spec.rb +++ b/spec/finders/pipelines_finder_spec.rb @@ -206,26 +206,26 @@ describe PipelinesFinder do context 'when order_by and sort are passed' do context 'when order_by and sort are valid' do - let(:params) { { order_by: 'created_at', sort: 'asc' } } + let(:params) { { order_by: 'user_id', sort: 'asc' } } it 'sorts pipelines by default' do - expect(subject).to eq(Ci::Pipeline.order(created_at: :asc)) + expect(subject).to eq(Ci::Pipeline.order(user_id: :asc)) end end context 'when order_by is invalid' do - let(:params) { { order_by: 'invalid_column', sort: 'desc' } } + let(:params) { { order_by: 'invalid_column', sort: 'asc' } } 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: :asc)) end end context 'when sort is invalid' do - let(:params) { { order_by: 'created_at', sort: 'invalid_sort' } } + let(:params) { { order_by: 'user_id', sort: 'invalid_sort' } } 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(user_id: :desc)) end end