bugfix: a single Ten is a burn

minaorangina · Feb 22, 2021 · 4e2a82e · 4e2a82e
1 parent 0a6a643
commit 4e2a82e
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 1 deletion.
diff --git a/gamerules.go b/gamerules.go
@@ -108,7 +108,7 @@ func getLegalMoves(pile, toPlay []deck.Card) []int {
 }
 
 func isBurn(pile []deck.Card) bool {
-	if len(pile) < burnNum {
+	if len(pile) == 0 {
 		return false
 	}
 
@@ -118,6 +118,10 @@ func isBurn(pile []deck.Card) bool {
 		return true
 	}
 
+	if len(pile) < burnNum {
+		return false
+	}
+
 	// Check for Three burn
 	numThrees := 0
 	for _, c := range pile[len(pile)-burnNum:] {

diff --git a/gamerules_test.go b/gamerules_test.go
@@ -639,6 +639,16 @@ func TestGameIsBurn(t *testing.T) {
 				deck.NewCard(deck.Ten, deck.Hearts)),
 			true,
 		},
+		{
+			"handles an empty pile",
+			[]deck.Card{},
+			false,
+		},
+		{
+			"play a 10 on its own",
+			[]deck.Card{deck.NewCard(deck.Ten, deck.Spades)},
+			true,
+		},
 		{
 			"four of the same suit (Spades)",
 			[]deck.Card{