stricter content normalization for headings and whitespcae

licensee · Jun 27, 2017 · a3bd89d · a3bd89d
1 parent 00be430
commit a3bd89d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/lib/licensee/content_helper.rb b/lib/licensee/content_helper.rb
@@ -5,7 +5,7 @@ module Licensee
   module ContentHelper
     DIGEST = Digest::SHA1
     END_OF_TERMS_REGEX = /^\s*end of terms and conditions\s*$/i
-    HR_REGEX = /^\s*[=-]{4,}/
+    HR_REGEX = /^\s*[=\-\* ]{4,}/
     ALT_TITLE_REGEX = {
       'bsd-2-clause'       => /bsd 2-clause( \"simplified\")? license/i,
       'bsd-3-clause'       => /bsd 3-clause( \"new\" or \"revised\")? license/i,

diff --git a/spec/licensee/content_helper_spec.rb b/spec/licensee/content_helper_spec.rb
@@ -11,11 +11,13 @@ def initialize(content = nil)
   let(:content) do
     <<-EOS.freeze.gsub(/^\s*/, '')
   # The MIT License
-  =================
+	=================
 
-  Copyright 2016 Ben Balter
+	Copyright 2016 Ben Balter
+	*************************
 
   The made
+  * * * *
   up  license.
   -----------
 EOS
@@ -73,10 +75,14 @@ def initialize(content = nil)
 
     it 'strips HRs' do
       expect(normalized_content).to_not match '---'
+      expect(normalized_content).to_not match '==='
+      expect(normalized_content).to_not include '***'
+      expect(normalized_content).to_not include '* *'
     end
 
     it 'squeezes whitespace' do
       expect(normalized_content).to_not match '  '
+      expect(normalized_content).to_not match(/\t/)
     end
 
     it 'strips whitespace' do