Tokenized emails

app/mailers/broadcast_mailer.rb

@@ -1,14 +1,14 @@
 class BroadcastMailer < ActionMailer::Base
   def self.recipients
-    User.where(:notify_updates => true).to_notify.map(&:contact_email)
+    User.where(:notify_updates => true).to_notify
   end
 
-  def broadcast(message, email)
-    article_finder = ->(e) { article_url(Article[e]) }
+  def broadcast(message, subscriber)
+    article_finder = ->(e) { ArticleLink.new(Article[e]).url(subscriber.share_token) }
 
     @body = Mustache.render(message[:body], :article => article_finder)
 
-    mail(:to      => email,
+    mail(:to      => subscriber.contact_email,
          :subject => message[:subject])
   end
 end

app/models/article_link.rb

@@ -1,7 +1,7 @@
 class ArticleLink
   include Rails.application.routes.url_helpers
 
-  def initialize(article, params)
+  def initialize(article, params={})
     self.article = article
     self.params  = params
   end

app/models/broadcaster.rb

@@ -1,11 +1,14 @@
 class Broadcaster
   def self.notify_subscribers(params)
-    BroadcastMailer.recipients.each do |email|
-      BroadcastMailer.broadcast(params, email).deliver
+    BroadcastMailer.recipients.each do |subscriber|
+      BroadcastMailer.broadcast(params, subscriber).deliver
     end
   end
 
   def self.notify_testers(params)
-    BroadcastMailer.broadcast(params, params[:to]).deliver
+    subscriber = Struct.new(:contact_email, :share_token)  
+                       .new(params[:to], "testtoken")
+
+    BroadcastMailer.broadcast(params, subscriber).deliver
   end
 end

test/unit/mailers/broadcast_mailer_test.rb

@@ -16,14 +16,14 @@ class BroadcastMailerTest < ActionMailer::TestCase
     assert_equal 5, messages.count
 
     do_not_mail.each do |user|
-     refute messages.any? { |m| m.to.include?(user.contact_email) },
-           "User with status '#{user.status}' was sent a broadcast"
+      refute messages.any? { |m| m.to.include?(user.contact_email) },
+            "User with status '#{user.status}' was sent a broadcast"
     end
   end
 
 
   test "article links can be generated from template" do
-    FactoryGirl.create(:user) 
+    user = FactoryGirl.create(:user) 
 
     article = FactoryGirl.create(:article, :slug => "a-fancy-article")
 
@@ -33,7 +33,7 @@ class BroadcastMailerTest < ActionMailer::TestCase
 
     message = ActionMailer::Base.deliveries.first
 
-    url  = Rails.application.routes.url_helpers.article_url(article)
+    url  = ArticleLink.new(article).url(user.share_token)
 
     expected_body = "Here's an amazing article\n#{url}"