Apply reviewer feedback

* https://gitlab.com/gitlab-org/gitlab/-/merge_requests/34961#note_365844987

ee/app/controllers/projects/licenses_controller.rb

@@ -100,7 +100,8 @@ module Projects
         read_license_policies_endpoint: expose_path(api_v4_projects_managed_licenses_path(id: @project.id)),
         write_license_policies_endpoint: write_license_policies_endpoint,
         documentation_path: help_page_path('user/compliance/license_compliance/index'),
-        empty_state_svg_path: helpers.image_path('illustrations/Dependency-list-empty-state.svg')
+        empty_state_svg_path: helpers.image_path('illustrations/Dependency-list-empty-state.svg'),
+        software_licenses: SoftwareLicense.unclassified_licenses_for(project).pluck_names
       }
     end
   end

ee/app/models/ee/project.rb

@@ -73,6 +73,7 @@ module EE
 
       has_many :protected_environments
       has_many :software_license_policies, inverse_of: :project, class_name: 'SoftwareLicensePolicy'
+      has_many :software_licenses, through: :software_license_policies
       accepts_nested_attributes_for :software_license_policies, allow_destroy: true
       has_many :packages, class_name: 'Packages::Package'
       has_many :package_files, through: :packages, class_name: 'Packages::PackageFile'

ee/app/models/software_license.rb

@@ -14,6 +14,15 @@ class SoftwareLicense < ApplicationRecord
   scope :spdx, -> { where.not(spdx_identifier: nil) }
   scope :unknown, -> { where(spdx_identifier: nil) }
   scope :grouped_by_name, -> { group(:name) }
+  scope :unreachable_limit, -> { limit(500) }
+
+  def self.unclassified_licenses_for(project)
+    spdx.id_not_in(project.software_licenses).ordered.unreachable_limit
+  end
+
+  def self.pluck_names
+    pluck(:name)
+  end
 
   def self.create_policy_for!(project:, name:, classification:)
     project.software_license_policies.create!(

ee/spec/controllers/projects/licenses_controller_spec.rb

@@ -24,16 +24,28 @@ RSpec.describe Projects::LicensesController do
             project.add_reporter(user)
           end
 
-          it 'responds to an HTML request' do
-            get :index, params: params
+          context "when requesting HTML" do
+            subject { get :index, params: params }
 
-            expect(response).to have_gitlab_http_status(:ok)
-            licenses_app_data = assigns(:licenses_app_data)
-            expect(licenses_app_data[:project_licenses_endpoint]).to eql(controller.helpers.project_licenses_path(project, detected: true, format: :json))
-            expect(licenses_app_data[:read_license_policies_endpoint]).to eql(controller.helpers.api_v4_projects_managed_licenses_path(id: project.id))
-            expect(licenses_app_data[:write_license_policies_endpoint]).to eql('')
-            expect(licenses_app_data[:documentation_path]).to eql(help_page_path('user/compliance/license_compliance/index'))
-            expect(licenses_app_data[:empty_state_svg_path]).to eql(controller.helpers.image_path('illustrations/Dependency-list-empty-state.svg'))
+            let_it_be(:mit_license) { create(:software_license, :mit) }
+            let_it_be(:apache_license) { create(:software_license, :apache_2_0) }
+            let_it_be(:custom_license) { create(:software_license, :user_entered) }
+
+            before do
+              subject
+            end
+
+            it 'returns the necessary licenses app data' do
+              licenses_app_data = assigns(:licenses_app_data)
+
+              expect(response).to have_gitlab_http_status(:ok)
+              expect(licenses_app_data[:project_licenses_endpoint]).to eql(controller.helpers.project_licenses_path(project, detected: true, format: :json))
+              expect(licenses_app_data[:read_license_policies_endpoint]).to eql(controller.helpers.api_v4_projects_managed_licenses_path(id: project.id))
+              expect(licenses_app_data[:write_license_policies_endpoint]).to eql('')
+              expect(licenses_app_data[:documentation_path]).to eql(help_page_path('user/compliance/license_compliance/index'))
+              expect(licenses_app_data[:empty_state_svg_path]).to eql(controller.helpers.image_path('illustrations/Dependency-list-empty-state.svg'))
+              expect(licenses_app_data[:software_licenses]).to eql([apache_license.name, mit_license.name])
+            end
           end
 
           it 'counts usage of the feature' do

ee/spec/factories/software_license.rb

@@ -13,5 +13,9 @@ FactoryBot.define do
       spdx_identifier { 'Apache-2.0' }
       name { 'Apache 2.0 License' }
     end
+
+    trait :user_entered do
+      spdx_identifier { nil }
+    end
   end
 end

ee/spec/models/software_license_spec.rb

@@ -85,4 +85,27 @@ RSpec.describe SoftwareLicense do
       it { expect(build(:software_license, name: 'MIT License', spdx_identifier: nil).canonical_id).to eq('mit license') }
     end
   end
+
+  describe '.unclassified_licenses_for' do
+    subject { described_class.unclassified_licenses_for(project) }
+
+    let_it_be(:mit_license) { create(:software_license, :mit) }
+    let_it_be(:apache_license) { create(:software_license, :apache_2_0) }
+    let_it_be(:nonstandard_license) { create(:software_license, :user_entered) }
+    let_it_be(:project) { create(:project) }
+
+    context 'when a project has not classified licenses' do
+      it 'returns each license in the SPDX catalogue ordered by name' do
+        expect(subject.to_a).to eql([apache_license, mit_license])
+      end
+    end
+
+    context 'when some of the licenses are classified' do
+      let_it_be(:mit_policy) { create(:software_license_policy, project: project, software_license: mit_license) }
+
+      it 'returns each license in the SPDX catalogue that has not been classified' do
+        expect(subject).to contain_exactly(apache_license)
+      end
+    end
+  end
 end

spec/lib/gitlab/import_export/all_models.yml

@@ -469,6 +469,7 @@ project:
 - prometheus_alert_events
 - self_managed_prometheus_alert_events
 - software_license_policies
+- software_licenses
 - project_registry
 - packages
 - package_files