From 2a01ecdbfd34d688f072bbf8ecdf114107b83c50 Mon Sep 17 00:00:00 2001
From: root <root@vscode.infinity>
Date: Mon, 27 Apr 2026 04:08:51 +0000
Subject: [PATCH] test(core): add missing check_auto_complete coverage
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Three test cases were missing:
- waste_not_empty guard: stock clear, waste has a card, all tableau
  face-up — must return false even with all tableau conditions satisfied
- true case: positive path confirming all-face-up + empty stock/waste
  returns true (previously untested entirely)
- The previous face_down_cards test covered only that guard, not the
  waste guard or the positive path.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 solitaire_core/src/game_state.rs | 39 ++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/solitaire_core/src/game_state.rs b/solitaire_core/src/game_state.rs
index 8c00382..6311a2f 100644
--- a/solitaire_core/src/game_state.rs
+++ b/solitaire_core/src/game_state.rs
@@ -743,6 +743,45 @@ mod tests {
         assert!(!g.check_auto_complete());
     }
 
+    #[test]
+    fn auto_complete_false_when_waste_not_empty() {
+        let mut g = new_game();
+        g.piles.get_mut(&PileType::Stock).unwrap().cards.clear();
+        // Leave the waste pile untouched (it may be empty after clearing stock,
+        // so add a card explicitly to ensure the waste guard is exercised).
+        g.piles.get_mut(&PileType::Waste).unwrap().cards.push(Card {
+            id: 99,
+            suit: Suit::Clubs,
+            rank: Rank::Ace,
+            face_up: true,
+        });
+        // Make all tableau cards face-up so only the waste guard is the blocker.
+        for i in 0..7 {
+            for c in g.piles.get_mut(&PileType::Tableau(i)).unwrap().cards.iter_mut() {
+                c.face_up = true;
+            }
+        }
+        assert!(!g.check_auto_complete());
+    }
+
+    #[test]
+    fn auto_complete_true_when_all_prerequisites_met() {
+        let mut g = new_game();
+        g.piles.get_mut(&PileType::Stock).unwrap().cards.clear();
+        g.piles.get_mut(&PileType::Waste).unwrap().cards.clear();
+        // Clear all tableau and put a single face-up card — all face-up guard passes.
+        for i in 0..7 {
+            g.piles.get_mut(&PileType::Tableau(i)).unwrap().cards.clear();
+        }
+        g.piles.get_mut(&PileType::Tableau(0)).unwrap().cards.push(Card {
+            id: 1,
+            suit: Suit::Clubs,
+            rank: Rank::Ace,
+            face_up: true,
+        });
+        assert!(g.check_auto_complete());
+    }
+
     // --- Time bonus ---
 
     #[test]