Merge pull request #180 from denis1011101/fix/chat-close-notice-silent

fix: письмо о закрытии чата приходит без звука

Author: denis1011101

app/jobs/send_telegram_notification_job.rb

@@ -1,8 +1,8 @@
 class SendTelegramNotificationJob < ApplicationJob
   queue_as :default
 
-  def perform(chat_id, text, parse_mode: nil)
-    TelegramNotifier.send_message(chat_id, text, parse_mode: parse_mode)
+  def perform(chat_id, text, parse_mode: nil, silent: false)
+    TelegramNotifier.send_message(chat_id, text, parse_mode: parse_mode, silent: silent)
   rescue => e
     Rails.logger.warn "SendTelegramNotificationJob failed: #{e.message}"
   end

app/services/telegram/chat/closure.rb

@@ -24,15 +24,19 @@ def prepare(game, reason, users = nil)
         end
 
         # Указатель гасим здесь же: иначе человек останется в чате, которого уже
-        # нет, и следующее его сообщение уйдёт в никуда. Двумя проходами и с
+        # нет, и следующее его сообщение уйдёт в никуда. Шлём без звука: чат
+        # закрывают в четыре утра субботы, и будить этим человека незачем —
+        # прочитает, когда сам откроет телеграм. Двумя проходами и с
         # оговоркой на каждого: сброс состава и удаление игры уже случились,
         # второго захода по этим людям не будет — так что одна упавшая отправка
         # не должна ни оставить остальных в мёртвом чате, ни съесть их письма.
         def deliver(game, notices)
           notices.each { |notice| guard(notice) { Session.stop_for(notice.user, game) } }
           notices.count do |notice|
             guard(notice) do
-              SendTelegramNotificationJob.perform_later(notice.user.telegram_chat_id.to_s, notice.text)
+              SendTelegramNotificationJob.perform_later(
+                notice.user.telegram_chat_id.to_s, notice.text, silent: true
+              )
             end
           end
         end

app/services/telegram/notifier.rb

@@ -6,13 +6,16 @@ module Telegram
   class Notifier
     TOKEN = ENV["TELEGRAM_BOT_TOKEN"].to_s
 
-    def self.send_message(chat_id, text, parse_mode: "Markdown", link_preview: false)
+    # silent: письмо приходит без звука и вибрации — для того, что бот шлёт по
+    # расписанию среди ночи и что человеку достаточно прочитать утром.
+    def self.send_message(chat_id, text, parse_mode: "Markdown", link_preview: false, silent: false)
       return false if TOKEN.empty? || chat_id.blank?
 
       uri = URI("https://api.telegram.org/bot#{TOKEN}/sendMessage")
       payload = { "chat_id" => chat_id.to_s, "text" => text.to_s }
       payload["parse_mode"] = parse_mode if parse_mode.present?
       payload["link_preview_options"] = Telegram::Api::LINK_PREVIEW_DISABLED unless link_preview
+      payload["disable_notification"] = "true" if silent
       res = Net::HTTP.post_form(uri, payload)
       body = JSON.parse(res.body) rescue {}
       body["ok"] == true

test/jobs/cleanup_past_one_off_games_job_test.rb

@@ -88,14 +88,15 @@ def setup
     game.participations.create!(user: player, status: "approved", approved_at: Time.current)
 
     sent = []
-    stub_singleton(SendTelegramNotificationJob, :perform_later, ->(chat_id, text) { sent << [ chat_id, text ] }) do
+    stub_singleton(SendTelegramNotificationJob, :perform_later, ->(chat_id, text, **opts) { sent << [ chat_id, text, opts ] }) do
       CleanupPastOneOffGamesJob.perform_now
     end
 
     assert_nil Game.find_by(id: game.id)
     notice = sent.find { |chat_id, _| chat_id == player.telegram_chat_id.to_s }
     assert notice, "участнику удалённой игры должно уйти письмо о закрытии чата"
-    assert_match "чат", notice.last
-    assert_no_match(/translation missing/i, notice.last)
+    assert_match "чат", notice[1]
+    assert_no_match(/translation missing/i, notice[1])
+    assert_equal({ silent: true }, notice[2], "письмо о закрытии чата приходит без звука")
   end
 end

test/jobs/reset_participations_job_test.rb

@@ -62,7 +62,7 @@ class ResetParticipationsJobTest < ActiveJob::TestCase
     with_memory_cache do
       Telegram::Chat::Session.start(player.telegram_chat_id.to_s, game)
 
-      stub_singleton(SendTelegramNotificationJob, :perform_later, ->(chat_id, text) { sent << [ chat_id, text ] }) do
+      stub_singleton(SendTelegramNotificationJob, :perform_later, ->(chat_id, text, **opts) { sent << [ chat_id, text, opts ] }) do
         ResetParticipationsJob.perform_now
       end
 
@@ -71,8 +71,9 @@ class ResetParticipationsJobTest < ActiveJob::TestCase
     end
 
     assert_equal [ player.telegram_chat_id.to_s ], sent.map(&:first)
-    assert_match "закрыт", sent.first.last
-    assert_no_match(/translation missing/i, sent.first.last)
+    assert_match "закрыт", sent.first[1]
+    assert_no_match(/translation missing/i, sent.first[1])
+    assert_equal({ silent: true }, sent.first[2], "письмо о закрытии чата приходит без звука")
   end
 
   # Организатор из состава не выпадает — ему закрывать нечего.
@@ -84,7 +85,7 @@ class ResetParticipationsJobTest < ActiveJob::TestCase
 
     sent = []
     with_memory_cache do
-      stub_singleton(SendTelegramNotificationJob, :perform_later, ->(chat_id, text) { sent << [ chat_id, text ] }) do
+      stub_singleton(SendTelegramNotificationJob, :perform_later, ->(chat_id, text, **opts) { sent << [ chat_id, text, opts ] }) do
         ResetParticipationsJob.perform_now
       end
     end

test/telegram/chat/closure_test.rb

@@ -29,15 +29,15 @@ class Telegram::Chat::ClosureTest < ActiveSupport::TestCase
 
       sent = []
       broken = @owner.telegram_chat_id.to_s
-      stub_singleton(SendTelegramNotificationJob, :perform_later, lambda { |chat_id, _text|
+      stub_singleton(SendTelegramNotificationJob, :perform_later, lambda { |chat_id, _text, **opts|
         raise "telegram is down" if chat_id == broken
 
-        sent << chat_id
+        sent << [ chat_id, opts ]
       }) do
         assert_equal 1, Telegram::Chat::Closure.notify(@game, :chat_closed_finished)
       end
 
-      assert_equal [ @player.telegram_chat_id.to_s ], sent
+      assert_equal [ [ @player.telegram_chat_id.to_s, { silent: true } ] ], sent
       assert_nil Telegram::Chat::Session.active_game(@owner.telegram_chat_id.to_s, @owner)
       assert_nil Telegram::Chat::Session.active_game(@player.telegram_chat_id.to_s, @player)
     end