Merge pull request #176 from denis1011101/feature/game-chat-fixes-and-weekly-reset

Feature/game chat fixes and weekly reset

Author: denis1011101

app/models/game.rb

@@ -61,15 +61,27 @@ def coaches
     [ coach, second_coach ].compact
   end
 
-  # Чат игры живёт до конца дня игры. У повторяющейся игры одной даты нет,
-  # поэтому там режим просто протухает через сутки, и его включают заново —
-  # угадывать, какое занятие человек имел в виду, мы не беремся.
-  CHAT_MAX_TTL = 24.hours
-
+  # Чат живёт ровно столько же, сколько состав: до еженедельного сброса участий —
+  # ночь с пятницы на субботу, 4 утра (ResetParticipationsJob в recurring.yml,
+  # там же чистка прошедших разовых игр). Раньше у серии режим протухал через
+  # сутки, и человек оставался без чата посреди недели, хотя состав, которому он
+  # писал, никуда не делся.
+  CHAT_RESET_WDAY = 6
+  CHAT_RESET_HOUR = 4
+
+  def self.next_weekly_reset_at(from = Time.current)
+    from = from.in_time_zone
+    reset = from.beginning_of_day.change(hour: CHAT_RESET_HOUR)
+    reset += 1.day while reset <= from || reset.wday != CHAT_RESET_WDAY
+    reset
+  end
+
+  # Считаем не «ближайшую субботу вообще», а ту, что действительно разберёт этот
+  # состав: сброс сносит участия уже отыгранного вхождения, а чистка — только
+  # прошедшую разовую игру. Игру, назначенную после ближайшей субботы, эта ночь
+  # не касается, и гасить её чат в 4 утра не за что.
   def chat_open_until
-    return CHAT_MAX_TTL.from_now if recurring? || date.blank?
-
-    closes_at = date.end_of_day
+    closes_at = Game.next_weekly_reset_at(chat_window_anchor)
     closes_at > Time.current ? closes_at : nil
   end
 
@@ -472,6 +484,13 @@ def next_time
     time
   end
 
+  # День, после которого состав этой игры пойдёт под нож: у серии — ближайшее
+  # вхождение, у разовой — её дата.
+  def chat_window_anchor
+    occurrence = recurring? ? (next_date || date) : date
+    occurrence ? occurrence.end_of_day : Time.current
+  end
+
   def previous_occurrence_before_next_date
     return nil unless recurring? && next_date.present?
 

config/recurring.yml

@@ -29,6 +29,15 @@ production:
   reset_participations:
     class: ResetParticipationsJob
     schedule: every saturday at 4am
+  # В ту же ночь, что и сброс: до неё состав и чат прошедшей игры живы, после —
+  # нет. Раньше обе задачи ходили из /etc/cron.d/getcourt по пятницам; кроновые
+  # строки с 2026-09-02 закомментированы.
+  cleanup_past_one_off_games:
+    class: CleanupPastOneOffGamesJob
+    schedule: every saturday at 4:15am
+  cleanup_past_tournaments:
+    class: CleanupPastTournamentsJob
+    schedule: every saturday at 4:30am
   reset_inactive_player_statistics:
     class: ResetInactivePlayerStatisticsJob
     schedule: at 4:30am every day

test/models/game_test.rb

@@ -404,4 +404,55 @@ class GameTest < ActiveSupport::TestCase
   ensure
     game&.destroy
   end
+
+  test "chat stays open until the weekly reset, not for a fixed day" do
+    game = Game.create!(court: courts(:one), user: users(:one), date: Date.new(2026, 9, 3), kind: "game")
+
+    # Среда: игра в четверг, разберёт её состав ближайшая суббота.
+    travel_to Time.zone.local(2026, 9, 2, 21, 0) do
+      assert_equal Time.zone.local(2026, 9, 5, 4, 0), Game.next_weekly_reset_at
+      assert_equal Time.zone.local(2026, 9, 5, 4, 0), game.chat_open_until
+      assert game.chat_open?
+    end
+  ensure
+    game&.destroy
+  end
+
+  # Ближайшая суббота ничего не делает с составом игры, назначенной после неё:
+  # чистка сносит только прошедшие разовые игры, сброс — только отыгранные серии.
+  test "a game scheduled beyond the coming reset keeps its chat until its own" do
+    one_off = Game.create!(court: courts(:one), user: users(:one), date: Date.new(2026, 9, 20), kind: "game")
+    # Серия с ближайшим вхождением в воскресенье — уже за субботней границей.
+    series = Game.create!(court: courts(:one), user: users(:one), date: Date.new(2026, 9, 6), recurring: true, kind: "game")
+
+    travel_to Time.zone.local(2026, 9, 2, 21, 0) do
+      assert_equal Time.zone.local(2026, 9, 26, 4, 0), one_off.chat_open_until
+      assert_equal Time.zone.local(2026, 9, 12, 4, 0), series.chat_open_until
+      assert one_off.chat_open?
+      assert series.chat_open?
+    end
+  ensure
+    [ one_off, series ].compact.each(&:destroy)
+  end
+
+  test "chat closes once the cleanup that removes the game has passed" do
+    game = Game.create!(court: courts(:one), user: users(:one), date: Date.new(2026, 8, 25), kind: "game")
+
+    travel_to Time.zone.local(2026, 9, 2, 21, 0) do
+      assert_nil game.chat_open_until
+      assert_not game.chat_open?
+    end
+  ensure
+    game&.destroy
+  end
+
+  test "the chat window rolls over to the next week right at the reset" do
+    travel_to Time.zone.local(2026, 9, 5, 3, 59) do
+      assert_equal Time.zone.local(2026, 9, 5, 4, 0), Game.next_weekly_reset_at
+    end
+
+    travel_to Time.zone.local(2026, 9, 5, 4, 0) do
+      assert_equal Time.zone.local(2026, 9, 12, 4, 0), Game.next_weekly_reset_at
+    end
+  end
 end

test/telegram/chat/relay_test.rb

@@ -63,11 +63,29 @@ class Telegram::Chat::RelayTest < ActiveSupport::TestCase
     end
   end
 
-  test "a finished game stops accepting messages" do
+  # Отыгранная игра живёт в базе до еженедельной чистки, и состав вместе с ней:
+  # обсудить её после матча — обычное дело, поэтому чат закрывается не концом
+  # дня игры, а сбросом.
+  test "a played game keeps its chat until the weekly reset" do
+    enqueued = nil
+
     in_chat_mode do
       @game.update_columns(date: Date.current - 2, recurring: false)
 
-      stub_singleton(Telegram::RelayChatMessageJob, :perform_later, ->(*) { flunk "игра прошла" }) do
+      stub_singleton(Telegram::RelayChatMessageJob, :perform_later, ->(*args) { enqueued = args }) do
+        assert Telegram::Chat::Relay.handle_message(tg_message("привет"))
+      end
+      assert_equal @game.id, Telegram::Chat::Session.game_id(@player.telegram_chat_id)
+    end
+
+    assert_equal [ @game.id, @player.id, "привет" ], enqueued
+  end
+
+  test "a deleted game closes the chat mode" do
+    in_chat_mode do
+      @game.destroy
+
+      stub_singleton(Telegram::RelayChatMessageJob, :perform_later, ->(*) { flunk "игры больше нет" }) do
         assert_not Telegram::Chat::Relay.handle_message(tg_message("привет"))
       end
       assert_nil Telegram::Chat::Session.game_id(@player.telegram_chat_id)