fix: не считать голоса выбывших и не падать на удалённом блоке

Разбор правки плана вскрыл два случая, когда голосование расходится с
реальностью. Голос участника оставался в подсчёте после того, как он ушёл
из игры: знаменатель пересчитывался по живому составу, а бюллетень
ушедшего решал за оставшихся. И блок, удалённый из библиотеки, пока
правка ждала своей очереди, ронял её применение на внешнем ключе —
правка навсегда застревала в голосовании.

Теперь считаем только бюллетени тех, кто ещё в игре, а в план ставим
уцелевшие блоки; если не уцелело ни одного — применять нечего.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_014tEZQY5PDtiqnKibQkGtRk

Author: denis1011101

app/models/training_plan_proposal.rb

@@ -60,8 +60,13 @@ def reject!
   end
 
   def apply!
+    # Блок могли удалить из библиотеки, пока правка ждала своей очереди: в план
+    # ставим то, что уцелело, а если не уцелело ничего — применять нечего.
+    ids = blocks.map(&:id)
+    return update!(status: "rejected") if ids.empty?
+
     transaction do
-      game.replace_training_plan!(training_block_ids)
+      game.replace_training_plan!(ids)
       update!(status: "applied")
     end
   end
@@ -79,16 +84,17 @@ def vote!(voter, in_favor)
     return false unless voting? && voter_ids.include?(voter&.id)
 
     training_plan_votes.find_or_initialize_by(user_id: voter.id).update!(in_favor: in_favor)
+    @ballots = nil
     settle!
     true
   end
 
   def votes_in_favor
-    training_plan_votes.where(in_favor: true).count + 1
+    ballots.count(&:in_favor?) + 1
   end
 
   def votes_against
-    training_plan_votes.where(in_favor: false).count
+    ballots.count { |ballot| !ballot.in_favor? }
   end
 
   def votes_expected
@@ -97,6 +103,12 @@ def votes_expected
 
   private
 
+  # Голос того, кто уже вышел из игры, не считаем: иначе бюллетень ушедшего
+  # решает за тех, кто на корт всё-таки выйдет.
+  def ballots
+    @ballots ||= training_plan_votes.where(user_id: voter_ids).to_a
+  end
+
   # Ждать последний голос незачем: как только одна сторона взяла большинство,
   # остальные бюллетени ничего не меняют.
   def settle!

test/models/training_plan_proposal_test.rb

@@ -28,6 +28,45 @@ class TrainingPlanProposalTest < ActiveSupport::TestCase
     assert_equal [ @warmup.id ], proposal.game.reload.training_block_ids
   end
 
+  # Ушедший из игры на корт не выйдет, и его голос не должен решать за тех, кто выйдет.
+  test "the ballot of someone who left the game stops counting" do
+    proposal = proposal_for(players: 3)
+    voters = proposal.voter_ids.map { |id| User.find(id) }
+
+    proposal.approve!
+    proposal.vote!(voters.first, true)
+    assert_predicate proposal, :voting?
+
+    proposal.game.participations.find_by(user_id: voters.first.id).destroy
+    proposal = TrainingPlanProposal.find(proposal.id)
+
+    proposal.vote!(voters.second, false)
+    assert_predicate proposal, :voting?
+
+    proposal.vote!(voters.third, false)
+    assert_predicate proposal, :rejected?
+  end
+
+  test "a block deleted from the library drops out of the change" do
+    proposal = proposal_for(players: 0)
+    TrainingBlock.find(proposal.training_block_ids.first).destroy
+
+    proposal.approve!
+
+    assert_predicate proposal, :applied?
+    assert_equal [ @warmup.id ], proposal.game.reload.training_block_ids
+  end
+
+  test "a change loses its meaning together with the last block" do
+    proposal = proposal_for(players: 0)
+    TrainingBlock.where(id: proposal.training_block_ids).find_each(&:destroy)
+
+    proposal.approve!
+
+    assert_predicate proposal, :rejected?
+    assert_empty proposal.game.reload.training_block_ids
+  end
+
   test "a change with nobody else to ask applies right after approval" do
     proposal = proposal_for(players: 0)