diff options
| -rw-r--r-- | lib/api/helpers/pagination.rb | 2 | ||||
| -rw-r--r-- | spec/lib/api/helpers/pagination_spec.rb | 8 |
2 files changed, 5 insertions, 5 deletions
diff --git a/lib/api/helpers/pagination.rb b/lib/api/helpers/pagination.rb index c046c857559..bb70370ba77 100644 --- a/lib/api/helpers/pagination.rb +++ b/lib/api/helpers/pagination.rb @@ -50,7 +50,7 @@ module API def add_default_order(relation) if relation.is_a?(ActiveRecord::Relation) && relation.order_values.empty? - relation = relation.order(id: :desc) + relation = relation.order(:id) end relation diff --git a/spec/lib/api/helpers/pagination_spec.rb b/spec/lib/api/helpers/pagination_spec.rb index 9c7a45a4324..a547988d631 100644 --- a/spec/lib/api/helpers/pagination_spec.rb +++ b/spec/lib/api/helpers/pagination_spec.rb @@ -94,22 +94,22 @@ describe API::Helpers::Pagination do end context 'if order' do - it 'is not present it adds default order(id: :desc) if no order is present' do + it 'is not present it adds default order(:id) if no order is present' do resource.order_values = [] paginated_relation = subject.paginate(resource) expect(resource.order_values).to be_empty expect(paginated_relation.order_values).to be_present - expect(paginated_relation.order_values.first).to be_descending + expect(paginated_relation.order_values.first).to be_ascending expect(paginated_relation.order_values.first.expr.name).to eq :id end it 'is present it does not add anything' do - paginated_relation = subject.paginate(resource.order(created_at: :asc)) + paginated_relation = subject.paginate(resource.order(created_at: :desc)) expect(paginated_relation.order_values).to be_present - expect(paginated_relation.order_values.first).to be_ascending + expect(paginated_relation.order_values.first).to be_descending expect(paginated_relation.order_values.first.expr.name).to eq :created_at end end |