snatch 1.0.3 → 1.0.4

data/VERSION

@@ -1 +1 @@
-1.0.3
+1.0.4

data/lib/snatch/clean/html.rb

@@ -1,11 +1,19 @@
 class Snatch
   class Clean
     class HTML
+      require 'uri'
+
       module HrefFixMethods
         # def remove_index_html(a)
         #   a['href'] = a['href'].sub(%r{index\.html?$}, '')
         # end
 
+        def append_index_html(a)
+          unless File.extname(a['href']).include?('.')
+            a['href'] = a['href'].sub(%r{/?$}, '') + '/index.html' 
+          end
+        end
+
         def replace_absolute(a)
           a['href'] = a['href'].sub(%r{(https?)://#{MARKETING_SITE}/}, '/')
         end

data/snatch.gemspec

@@ -5,7 +5,7 @@
 
 Gem::Specification.new do |s|
   s.name = %q{snatch}
-  s.version = "1.0.3"
+  s.version = "1.0.4"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["James Conroy-Finn"]

data/spec/snatch/clean/html_spec.rb

@@ -40,11 +40,11 @@ describe Snatch::Clean::HTML do
     # end
 
     it 'should preserve parent directories within a URL' do
-      anchor = '<a href="/folder/child/file.extension"></a>'
+      anchor = '<a href="/folder/child/index.html"></a>'
       doc = Nokogiri::XML(anchor)
       @html.doc = doc
       @html.send(:update)
-      @html.doc.to_xhtml.strip.should == '<a href="/folder/child/file.extension"></a>'
+      @html.doc.to_xhtml.strip.should == '<a href="/folder/child/index.html"></a>'
     end
 
     it 'should replace an absolute CMS URL with a domainless absolute URL' do
@@ -54,6 +54,23 @@ describe Snatch::Clean::HTML do
       end
     end
 
+    describe 'appending index.html' do
+      it 'without an extension' do
+        node = fix_node(:append_index_html, '<a href="/folder"></a>')
+        node.should have_href('/folder/index.html')
+      end
+
+      it 'with an extension' do
+        node = fix_node(:append_index_html, '<a href="folder/file.txt"></a>"')
+        node.should have_href('folder/file.txt')
+      end
+
+      it 'with an absolute URL including a domain name' do
+        node = fix_node(:append_index_html, '<a href="http://domain.com/"></a>')
+        node.should have_href('http://domain.com/index.html')
+      end
+    end
+
     it 'should encode email addresses' do
       anchor = %Q{<a href="mailto:blah@exåmplé.cøm">blah@exåmplé.cøm</a>}
       @html.class.should_receive(:url_encode).and_return('url_encode')

data/spec/spec_helper.rb

@@ -13,5 +13,5 @@ end
 def fix_node(method, xhtml)
   node = Nokogiri::XML(xhtml).children.first
   subject.send method, node
-  yield node
+  block_given? ? yield(node) : node
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: snatch
 version: !ruby/object:Gem::Version 
-  version: 1.0.3
+  version: 1.0.4
 platform: ruby
 authors: 
 - James Conroy-Finn