Add specs on #license

CHANGELOG

@@ -62,6 +62,7 @@ v 8.0.4
   - Fix "Assign All" button on Runner admin page
   - Fix search in Files
   - Add full project namespace to payload of system webhooks (Ricardo Band)
+  - Accept COPYING as licence file (Zeger-Jan van de Weg)
 
 v 8.0.3
   - Fix URL shown in Slack notifications
@@ -82,7 +83,6 @@ v 8.0.2
   - Fix Reply by email for non-UTF-8 messages.
   - Add option to use StartTLS with Reply by email IMAP server.
   - Allow AWS S3 Server-Side Encryption with Amazon S3-Managed Keys for backups (Paul Beattie)
-  - Accept COPYING as licence file (Zeger-Jan van de Weg)
 
 v 8.0.1
   - Remove git refs used internally by GitLab from network graph (Stan Hu)

app/models/repository.rb

@@ -216,7 +216,11 @@ class Repository
 
       # If `licence`, `copying` and `copying.lesser` are found, return in the
       # following order: licence, copying, copying.lesser
-      licenses.find { |l| l =~ /\Alicence/i } || licenses.sort.first
+      regex = [/\Alicense(\..*)?\z/i, /\Acopying(\..{0,3})?\z/i, /\Acopying.lesser/i]
+
+      regex.map do |re|
+        licenses.find { |l| l.name =~ re }
+      end.compact.first
     end
   end
 

spec/models/repository_spec.rb

@@ -91,4 +91,15 @@ describe Repository do
       it { expect(subject.data.lines[2]).to eq("  - Feature: Replace teams with group membership\n") }
     end
   end
+
+  describe "#license" do
+    it 'test selection preference' do
+      repository.send(:cache).expire(:license)
+      TestBlob = Struct.new(:name)
+      files = [TestBlob.new('file'), TestBlob.new('license'), TestBlob.new('copying')]
+      expect(repository.tree).to receive(:blobs).and_return(files)
+
+      expect(repository.license.name).to eq('license')
+    end
+  end
 end