Logic for some Deck tests seems flawed about jcards HOT 3 CLOSED

@Test
public void testShuffle() {
    Deck deck = new Deck();
    List<Card> originalCards = new ArrayList<>(deck.cards);
    deck.shuffle();
    int matchCount = 0;
    for (int i = 0; i < originalCards.size(); i++) {
        if (originalCards.get(i).equals(deck.cards.get(i))) {
            matchCount++;
        }
    }
    assertFalse("The deck was randomly shuffled into the same order", matchCount == originalCards.size());
}

    /**
     * Test method to check the `findCard` method of the `Deck` class. This test verifies that the `findCard` method is
     * able to locate a card with rank SEVEN and suit SPADES in the shuffled deck.
     */
    @Test
    public void testFindCard() {
        Deck deck = new Deck();
        deck.shuffle();
        int indexOfSevenOfSpades = deck.findCard(Card.Rank.SEVEN, Card.Suit.SPADES);
        Card foundCard = deck.getCards().get(indexOfSevenOfSpades);
        assertEquals(Card.Rank.SEVEN, foundCard.getRank(), "The found card's rank should be SEVEN");
        assertEquals(Card.Suit.SPADES, foundCard.getSuit(), "The found card's suit should be SPADES");
    }

    /**
    * Shuffles the deck of cards using a cryptographically secure random number generator.
    *
    * @see java.util.Collections#shuffle(List, Random)
    * @see java.security.SecureRandom
    */
   public void shuffle() {
       SecureRandom secureRandom = new SecureRandom();
       long seed = secureRandom.nextLong();
       Collections.shuffle(cards, new Random(seed));
   }

    /**
     * Retrieves the most up-to-date list of cards stored in the object.
     *
     * @return A list of {@link Card} objects stored in the object, which reflects any changes made to the underlying list.
     *
     * <p> The returned list may be empty, but it will never be
     * {@code null}.
     *
     * @see Card
     * @author lyudaio
     * @since 0.0.3
     */
    public List<Card> getCards() {
        return cards;
    }