scrub old unsent messages

fixes CNVS-25039 test plan - run specs Change-Id: Id7b48c83d674d3fbe932f7e4f22630a702fedbde Reviewed-on: https://gerrit.instructure.com/68127 Reviewed-by: Andrew Butterfield <[email protected]> Tested-by: Jenkins QA-Review: Heath Hales <[email protected]> Product-Review: Joel Hough <[email protected]>
Timvandervoord · Dec 11, 2015 · 2b703a1 · 2b703a1
1 parent ca344bc
commit 2b703a1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/lib/message_scrubber.rb b/lib/message_scrubber.rb
@@ -89,7 +89,7 @@ def delete_messages(scope)
   #
   # Returns a column name string.
   def filter_attribute
-    'sent_at'
+    'updated_at'
   end
 
   # Internal: The class object to delete records from (e.g. 'Message').

diff --git a/spec/lib/message_scrubber_spec.rb b/spec/lib/message_scrubber_spec.rb
@@ -21,12 +21,13 @@
 describe MessageScrubber do
 
   # Helpers
-  def message(sent_at)
-    message = Message.new(notification: @notification, context: @context,
+  def message(updated_at)
+    Timecop.travel(updated_at) do
+      message = Message.new(notification: @notification, context: @context,
             communication_channel: @recipient.communication_channel)
-    message.sent_at = sent_at
-    message.save!
-    message
+      message.save!
+      message
+    end
   end
 
   def old_messages(count = 2)