Skip to content

Extend test in solutions_06_05#323

Open
marroko wants to merge 7 commits into
mainmatter:solutionsfrom
marroko:solutions_06_05_test
Open

Extend test in solutions_06_05#323
marroko wants to merge 7 commits into
mainmatter:solutionsfrom
marroko:solutions_06_05_test

Conversation

@marroko

@marroko marroko commented Jan 14, 2026

Copy link
Copy Markdown

assert_eq!(tickets, tickets2); test is not enough as task could be passed with incorrect solution like

pub fn iter(&self) -> std::slice::Iter<Ticket> {
    [].iter() // must be self.tickets.iter()
}

So there's a need to also validate store.iter().len().

@hdoordt
hdoordt force-pushed the solutions branch 2 times, most recently from 8c44511 to a46a8ae Compare June 15, 2026 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants