- Fix table of contents slugs

Author: DannyBen

lib/madness/refinements/string_refinements.rb

@@ -12,7 +12,7 @@ def to_href
       end
 
       def to_slug
-        downcase.strip.tr(' ', '-').remove(/[^\w-]/)
+        downcase.strip.gsub(/[^\w]/, '-').gsub(/-+/, '-').remove(/(^-|-$)/)
       end
 
       # This is here so we can have one place that defines how to convert

spec/approvals/document-toc

@@ -6,8 +6,8 @@
 <li><a href="#subject-1">Subject 1</a>
 
 <ul>
-<li><a href="#subject-11">Subject 1.1</a></li>
-<li><a href="#subject-12">Subject 1.2</a></li>
+<li><a href="#subject-1-1">Subject 1.1</a></li>
+<li><a href="#subject-1-2">Subject 1.2</a></li>
 </ul></li>
 <li><a href="#subject-2">Subject 2</a></li>
 </ul>

spec/madness/refinements/string_refinements_spec.rb

@@ -10,10 +10,10 @@
   end
 
   describe '#to_slug' do
-    subject { 'String with !23@  ' }
+    subject { '! String -- with 23 !symbols !' }
 
     it 'converts string to slug' do
-      expect(subject.to_slug).to eq 'string-with-23'
+      expect(subject.to_slug).to eq 'string-with-23-symbols'
     end
   end