email links now visit the url, rather than just the path, this allows…

… for features that

    make use of the domain to function correctly

History.txt

@@ -1,7 +1,9 @@
 == master
 
-* 1 minor improvement
+* 2 minor improvements
   * Document how to use machinist named blueprints [Christopher Darroch]
+  * Email links now visit the url, rather than just the path, this allows for features that
+    make use of the domain to function correctly
 
 
 == 0.3.1

lib/pickle/email.rb

@@ -25,8 +25,7 @@ def visit_in_email(email, link_text)
 
     def click_first_link_in_email(email)
       link = links_in_email(email).first
-      request_uri = URI::parse(link).request_uri
-      visit request_uri
+      visit link
     end
 
   protected
@@ -60,15 +59,14 @@ def parse_email_for_link(email, text_or_regex)
     # e.g. confirm in http://confirm
     def parse_email_for_explicit_link(email, regex)
       regex = /#{Regexp.escape(regex)}/ unless regex.is_a?(Regexp)
-      url = links_in_email(email).detect { |link| link =~ regex }
-      URI::parse(url).request_uri if url
+      links_in_email(email).detect { |link| link =~ regex }
     end
 
     # e.g. Click here in  <a href="http://confirm">Click here</a>
     def parse_email_for_anchor_text_link(email, link_text)
-      email.body =~ %r{<a[^>]*href=['"]?([^'"]*)['"]?[^>]*?>[^<]*?#{link_text}[^<]*?</a>}
-      URI.split($~[1])[5..-1].compact!.join("?").gsub("&amp;", "&")
-      # sub correct ampersand after rails switches it (http://dev.rubyonrails.org/ticket/4002)
+      if match_data = email.body.match(%r{<a[^>]*href=['"]?([^'"]*)['"]?[^>]*?>[^<]*?#{link_text}[^<]*?</a>})
+        match_data[1]
+      end
     end
 
     def links_in_email(email, protos=['http', 'https'])

spec/pickle/email_spec.rb

@@ -143,17 +143,17 @@
     end
 
     it "should find a link for http://example.com/page" do
-      should_receive(:visit).with('/page')
+      should_receive(:visit).with('http://example.com/page')
       visit_in_email(@email1, 'http://example.com/page')
     end
 
     it "should find a link for \"example page\"" do
-      should_receive(:visit).with('/page')
+      should_receive(:visit).with('http://example.com/page')
       visit_in_email(@email1, 'example page')
     end
 
     it "should follow the first link in an email" do
-      should_receive(:visit).with('/page')
+      should_receive(:visit).with('http://example.com/page')
       click_first_link_in_email(@email1)
     end
   end