Adds package sorting to api

- Use Package#sort_by_attribute to sort packages in api
- Change Package#order_version scope to sort versions as expected

ee/app/models/packages/package.rb

@@ -84,6 +84,7 @@ class Packages::Package < ApplicationRecord
   def self.sort_by_attribute(method)
     case method.to_s
     when 'created_asc' then order_created
+    when 'created_at_asc' then order_created
     when 'name_asc' then order_name
     when 'name_desc' then order_name_desc
     when 'version_asc' then order_version

ee/lib/api/project_packages.rb

@@ -20,10 +20,18 @@ module API
       end
       params do
         use :pagination
+        optional :order_by, type: String, values: %w[created_at name version type], default: 'created_at',
+                            desc: 'Return packages ordered by `created_at`, `name`, `version` or `type` fields.'
+        optional :sort, type: String, values: %w[asc desc], default: 'asc',
+                        desc: 'Return packages sorted in `asc` or `desc` order.'
       end
       get ':id/packages' do
         packages = user_project.packages
 
+        if params[:order_by] && params[:sort]
+          packages = packages.sort_by_attribute("#{params[:order_by]}_#{params[:sort]}")
+        end
+
         present paginate(packages), with: EE::API::Entities::Package, user: current_user
       end
 

ee/spec/requests/api/project_packages_spec.rb

@@ -5,9 +5,9 @@ require 'spec_helper'
 describe API::ProjectPackages do
   let(:user) { create(:user) }
   let(:project) { create(:project, :public) }
-  let!(:package1) { create(:npm_package, project: project) }
+  let!(:package1) { create(:npm_package, project: project, version: '3.1.0', name: "@#{project.root_namespace.path}/foo1") }
   let(:package_url) { "/projects/#{project.id}/packages/#{package1.id}" }
-  let!(:package2) { create(:npm_package, project: project) }
+  let!(:package2) { create(:npm_package, project: project, version: '2.0.4', name: "@#{project.root_namespace.path}/foo2") }
   let!(:another_package) { create(:npm_package) }
   let(:no_package_url) { "/projects/#{project.id}/packages/0" }
   let(:wrong_package_url) { "/projects/#{project.id}/packages/#{another_package.id}" }
@@ -68,6 +68,58 @@ describe API::ProjectPackages do
           it_behaves_like 'returns paginated packages'
         end
       end
+
+      context 'with sorting' do
+        shared_examples 'package sorting' do |order_by, package_names|
+          subject { get api(url), params: { sort: sort, order_by: order_by } }
+
+          let(:packages) { package_names.map { |name| send(name) } }
+
+          context "sorting by #{order_by}" do
+            context 'ascending order' do
+              let(:sort) { 'asc' }
+
+              it 'returns the sorted packages' do
+                subject
+
+                expect(json_response.map { |package| package['id'] }).to eq(packages.map(&:id))
+              end
+            end
+
+            context 'descending order' do
+              let(:sort) { 'desc' }
+
+              it 'returns the sorted packages' do
+                subject
+
+                expect(json_response.map { |package| package['id'] }).to eq(packages.reverse.map(&:id))
+              end
+            end
+          end
+        end
+
+        let(:package3) { create(:maven_package, project: project, version: '1.1.1', name: 'zzz') }
+
+        before do
+          travel_to(1.day.ago) do
+            package3
+          end
+        end
+
+        it_behaves_like 'package sorting', 'name', [:package1, :package2, :package3]
+        it_behaves_like 'package sorting', 'created_at', [:package3, :package1, :package2]
+        it_behaves_like 'package sorting', 'version', [:package3, :package2, :package1]
+
+        context 'by packages type' do
+          before do
+            # get rid of extra npm package as it messes up
+            # the package type sorting test
+            package2.destroy
+          end
+
+          it_behaves_like 'package sorting', 'type', [:package3, :package1]
+        end
+      end
     end
 
     context 'packages feature disabled' do