card:tests: cleanliness

This commit is contained in:
2026-04-12 06:37:35 +01:00
committed by oreodave
parent 6e94f05443
commit b6d22a03ef

View File

@@ -7,12 +7,12 @@ mod test_numerics {
#[test] #[test]
fn rank() { fn rank() {
// TEST: Negative numbers cannot be ranks // TEST: Negative numbers cannot be ranks
assert!(matches!(Rank::try_from(-1), Err(_))); assert!(Rank::try_from(-1).is_err());
// TEST: Numbers >= 13 cannot be a rank. // TEST: Numbers >= 13 cannot be a rank.
for i in 13..1000 { for i in 13..1000 {
assert!( assert!(
matches!(Rank::try_from(i), Err(_)), Rank::try_from(i).is_err(),
"Expected {i} not to match a rank" "Expected {i} not to match a rank"
); );
} }
@@ -36,12 +36,12 @@ mod test_numerics {
#[test] #[test]
fn suit() { fn suit() {
// TEST: Negative numbers cannot be suits // TEST: Negative numbers cannot be suits
assert!(matches!(Suit::try_from(-1), Err(_))); assert!(Suit::try_from(-1).is_err());
// TEST: Numbers >= 4 cannot be a suit. // TEST: Numbers >= 4 cannot be a suit.
for i in 4..1000 { for i in 4..1000 {
assert!( assert!(
matches!(Suit::try_from(i), Err(_)), Suit::try_from(i).is_err(),
"Expected {i} not to match a suit" "Expected {i} not to match a suit"
); );
} }
@@ -283,7 +283,7 @@ mod test_impls {
for c in cards { for c in cards {
// TEST: rank.cards() generates Playing Cards of the same rank // TEST: rank.cards() generates Playing Cards of the same rank
// as the input in deck 0. // as the input in deck 0.
assert!(matches!(c, Card::PlayingCard(_))); assert!(!c.is_joker());
let Card::PlayingCard(c) = c else { let Card::PlayingCard(c) = c else {
unreachable!("See above"); unreachable!("See above");
}; };
@@ -314,7 +314,7 @@ mod test_impls {
for c in cards { for c in cards {
// TEST: suit.cards() generates Playing Cards of the same suit // TEST: suit.cards() generates Playing Cards of the same suit
// as the input in deck 0. // as the input in deck 0.
assert!(matches!(c, Card::PlayingCard(_))); assert!(!c.is_joker());
let Card::PlayingCard(c) = c else { let Card::PlayingCard(c) = c else {
unreachable!("See above"); unreachable!("See above");
}; };