Skip to content

Commit

Permalink
tests: reverse - separate empty and BOM early returns
Browse files Browse the repository at this point in the history
this should reduce its flakiness
  • Loading branch information
jqnatividad committed Jan 12, 2025
1 parent ada8fe9 commit 0e0485b
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions tests/test_reverse.rs
Original file line number Diff line number Diff line change
Expand Up @@ -54,8 +54,11 @@ fn prop_reverse_no_headers() {
}

fn prop_reverse_indexed(name: &str, rows: CsvData, headers: bool) -> bool {
if rows.is_empty()
|| rows[0].contains(&"\u{feff}".to_string())
if rows.is_empty() {
return true;
}

if rows[0].contains(&"\u{feff}".to_string())
|| rows.last().unwrap().contains(&"\u{feff}".to_string())
{
return true;
Expand Down

0 comments on commit 0e0485b

Please sign in to comment.