From df9aaccb46a2a3c069550df27da8824b731813a4 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Joakim=20Frosteg=C3=A5rd?= Date: Sat, 30 Oct 2021 17:06:28 +0200 Subject: [PATCH] aquatic_http_protocol: fix serde_quickcheck_identity_test Since empty info hash Vec is no longer allowed, don't allow it in test data and expect parsing to succeed. --- aquatic_http_protocol/src/request.rs | 21 +++++++++++++++------ 1 file changed, 15 insertions(+), 6 deletions(-) diff --git a/aquatic_http_protocol/src/request.rs b/aquatic_http_protocol/src/request.rs index 9b8e2ee..eeefa12 100644 --- a/aquatic_http_protocol/src/request.rs +++ b/aquatic_http_protocol/src/request.rs @@ -355,13 +355,22 @@ mod tests { #[test] fn quickcheck_serde_identity_request() { fn prop(request: Request) -> TestResult { - if let Request::Announce(AnnounceRequest { - key: Some(ref key), .. - }) = request - { - if key.len() > 30 { - return TestResult::discard(); + match request { + Request::Announce(AnnounceRequest { + key: Some(ref key), .. + }) => { + if key.len() > 30 { + return TestResult::discard(); + } } + Request::Scrape(ScrapeRequest { + ref info_hashes, + }) => { + if info_hashes.is_empty() { + return TestResult::discard(); + } + } + _ => {} } let mut bytes = Vec::new();