From dc21181f6504b55afa68de63f170fcb0f1084a6b Mon Sep 17 00:00:00 2001
From: Roman Chvanikov <chvanikoff@gmail.com>
Date: Sun, 14 Apr 2019 22:29:05 +0700
Subject: [PATCH] Update updated_at field on notification read

---
 lib/pleroma/notification.ex |  5 ++++-
 test/notification_test.exs  | 23 +++++++++++++++++++++++
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/lib/pleroma/notification.ex b/lib/pleroma/notification.ex
index b357d5399..29845b9da 100644
--- a/lib/pleroma/notification.ex
+++ b/lib/pleroma/notification.ex
@@ -58,7 +58,10 @@ def set_read_up_to(%{id: user_id} = _user, id) do
         where: n.user_id == ^user_id,
         where: n.id <= ^id,
         update: [
-          set: [seen: true]
+          set: [
+            seen: true,
+            updated_at: ^NaiveDateTime.utc_now()
+          ]
         ]
       )
 
diff --git a/test/notification_test.exs b/test/notification_test.exs
index c3db77b6c..907b9e669 100644
--- a/test/notification_test.exs
+++ b/test/notification_test.exs
@@ -300,6 +300,29 @@ test "it sets all notifications as read up to a specified notification ID" do
       assert n2.seen == true
       assert n3.seen == false
     end
+
+    test "Updates `updated_at` field" do
+      user1 = insert(:user)
+      user2 = insert(:user)
+
+      Enum.each(0..10, fn i ->
+        {:ok, _activity} =
+          TwitterAPI.create_status(user1, %{
+            "status" => "#{i} hi @#{user2.nickname}"
+          })
+      end)
+
+      Process.sleep(1000)
+
+      [notification | _] = Notification.for_user(user2)
+
+      Notification.set_read_up_to(user2, notification.id)
+
+      Notification.for_user(user2)
+      |> Enum.each(fn notification ->
+        assert notification.updated_at > notification.inserted_at
+      end)
+    end
   end
 
   describe "notification target determination" do