Merge branch 'bvl-fix-lic-check' into 'master'

Check global feature directly on `License` class

Closes #8484

See merge request gitlab-org/gitlab-ee!8505

ee/lib/ee/api/entities.rb

@@ -24,7 +24,7 @@ module EE
           expose :only_mirror_protected_branches, if: ->(project, _) { project.mirror? }
           expose :mirror_overwrites_diverged_branches, if: ->(project, _) { project.mirror? }
           expose :external_authorization_classification_label,
-                 if: ->(project, _) { project.feature_available?(:external_authorization_service) }
+                 if: ->(_, _) { License.feature_available?(:external_authorization_service) }
         end
       end
 

ee/spec/requests/api/projects_spec.rb

@@ -6,6 +6,20 @@ describe API::Projects do
   let(:user) { create(:user) }
   let(:project) { create(:project, namespace: user.namespace) }
 
+  describe 'GET /projects' do
+    it 'does not break on license checks' do
+      stub_licensed_features(external_authorization_service: true)
+      enable_namespace_license_check!
+
+      create(:project, :private, namespace: user.namespace)
+      create(:project, :public, namespace: user.namespace)
+
+      get api('/projects', user)
+
+      expect(response).to have_gitlab_http_status(200)
+    end
+  end
+
   describe 'POST /projects' do
     context 'when importing with mirror attributes' do
       let(:import_url) { generate(:url) }