chore: fix clippy lints
This commit is contained in:
parent
4780096b00
commit
ac25490435
15 changed files with 70 additions and 73 deletions
|
|
@ -174,7 +174,7 @@ where
|
|||
/// and return the duration in seconds.
|
||||
pub fn parse_video_length(text: &str) -> Option<u32> {
|
||||
static VIDEO_LENGTH_REGEX: Lazy<Regex> =
|
||||
Lazy::new(|| Regex::new(r#"(?:(\d+)[:.])?(\d{1,2})[:.](\d{2})"#).unwrap());
|
||||
Lazy::new(|| Regex::new(r"(?:(\d+)[:.])?(\d{1,2})[:.](\d{2})").unwrap());
|
||||
VIDEO_LENGTH_REGEX.captures(text).map(|cap| {
|
||||
let hrs = cap
|
||||
.get(1)
|
||||
|
|
@ -550,7 +550,7 @@ pub(crate) mod tests {
|
|||
#[case("bla 2:02 h3llo w0rld", Some(122))]
|
||||
#[case("18:22", Some(1102))]
|
||||
#[case("1:48:18", Some(6498))]
|
||||
#[case("102:12:39", Some(367959))]
|
||||
#[case("102:12:39", Some(367_959))]
|
||||
#[case("42", None)]
|
||||
fn t_parse_video_length(#[case] text: &str, #[case] expect: Option<u32>) {
|
||||
let n = parse_video_length(text);
|
||||
|
|
@ -625,11 +625,11 @@ pub(crate) mod tests {
|
|||
let number_samples: BTreeMap<Language, BTreeMap<String, u64>> =
|
||||
serde_json::from_reader(BufReader::new(json_file)).unwrap();
|
||||
|
||||
number_samples.iter().for_each(|(lang, entry)| {
|
||||
entry.iter().for_each(|(txt, expect)| {
|
||||
for (lang, entry) in &number_samples {
|
||||
for (txt, expect) in entry {
|
||||
testcase_parse_large_numstr(txt, *lang, *expect);
|
||||
});
|
||||
});
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
fn testcase_parse_large_numstr(string: &str, lang: Language, expect: u64) {
|
||||
|
|
@ -658,7 +658,7 @@ pub(crate) mod tests {
|
|||
let res = SplitTokens::new(teststr, true).collect::<Vec<_>>();
|
||||
assert_eq!(res.len(), 10);
|
||||
let res_str = res.into_iter().collect::<String>();
|
||||
assert_eq!(res_str, teststr)
|
||||
assert_eq!(res_str, teststr);
|
||||
}
|
||||
|
||||
#[test]
|
||||
|
|
@ -667,7 +667,7 @@ pub(crate) mod tests {
|
|||
let res = SplitTokens::new(teststr, false).collect::<Vec<_>>();
|
||||
assert_eq!(res.len(), 3);
|
||||
let res_str = res.join(" ");
|
||||
assert_eq!(res_str, teststr)
|
||||
assert_eq!(res_str, teststr);
|
||||
}
|
||||
|
||||
#[rstest]
|
||||
|
|
|
|||
|
|
@ -597,7 +597,7 @@ mod tests {
|
|||
"Language: {lang}, txt: `{s}`"
|
||||
);
|
||||
});
|
||||
})
|
||||
});
|
||||
}
|
||||
|
||||
#[test]
|
||||
|
|
@ -786,7 +786,7 @@ mod tests {
|
|||
"Language: {lang}, txt: `{s}`"
|
||||
);
|
||||
});
|
||||
})
|
||||
});
|
||||
}
|
||||
|
||||
#[test]
|
||||
|
|
@ -808,7 +808,7 @@ mod tests {
|
|||
let mut n_cases = 0;
|
||||
|
||||
timeago_table.entries.iter().for_each(|(lang, entries)| {
|
||||
entries.iter().for_each(|(t, entry)| {
|
||||
for (t, entry) in entries {
|
||||
entry.cases.iter().for_each(|(txt, n)| {
|
||||
let timeago = parse_timeago(*lang, txt);
|
||||
assert_eq!(
|
||||
|
|
@ -818,11 +818,11 @@ mod tests {
|
|||
);
|
||||
|
||||
n_cases += 1;
|
||||
})
|
||||
});
|
||||
});
|
||||
}
|
||||
});
|
||||
|
||||
assert_eq!(n_cases, 1065)
|
||||
assert_eq!(n_cases, 1065);
|
||||
}
|
||||
|
||||
#[rstest]
|
||||
|
|
@ -837,7 +837,7 @@ mod tests {
|
|||
)]
|
||||
#[case(
|
||||
Language::Bn,
|
||||
"যোগ দিয়েছেন 24 সেপ, 2013",
|
||||
"যোগ দিয়েছেন 24 সেপ, 2013",
|
||||
Some(ParsedDate::Absolute(date!(2013-9-24)))
|
||||
)]
|
||||
fn t_parse_date(
|
||||
|
|
@ -941,7 +941,7 @@ mod tests {
|
|||
Some(ParsedDate::Absolute(date!(2021 - 12 - 24))),
|
||||
"lang: {lang}"
|
||||
);
|
||||
})
|
||||
});
|
||||
}
|
||||
|
||||
#[test]
|
||||
|
|
@ -951,15 +951,15 @@ mod tests {
|
|||
let date_samples: BTreeMap<Language, BTreeMap<String, u32>> =
|
||||
serde_json::from_reader(BufReader::new(json_file)).unwrap();
|
||||
|
||||
date_samples.iter().for_each(|(lang, samples)| {
|
||||
for (lang, samples) in &date_samples {
|
||||
samples.iter().for_each(|(txt, duration)| {
|
||||
assert_eq!(
|
||||
parse_video_duration(*lang, txt),
|
||||
Some(*duration),
|
||||
"lang: {lang}; txt: `{txt}`"
|
||||
);
|
||||
})
|
||||
});
|
||||
});
|
||||
}
|
||||
}
|
||||
|
||||
#[rstest]
|
||||
|
|
@ -985,12 +985,4 @@ mod tests {
|
|||
let now = OffsetDateTime::now_utc();
|
||||
assert_eq!(date.year(), now.year() - 1);
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn tx() {
|
||||
let s = "Abcdef";
|
||||
let lc: (usize, char) = s.char_indices().last().unwrap();
|
||||
let t = &s[(lc.0 + lc.1.len_utf8())..];
|
||||
dbg!(&t);
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Reference in a new issue