Skip to content

Commit 31b02be

Browse files
jyn514Joshua Nelson
authored and
Joshua Nelson
committed
Don't crash on empty search queries
1 parent bf33ec6 commit 31b02be

File tree

1 file changed

+14
-0
lines changed

1 file changed

+14
-0
lines changed

src/web/releases.rs

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,9 @@ fn get_search_results(
229229
limit: i64,
230230
) -> Result<(i64, Vec<Release>), failure::Error> {
231231
query = query.trim();
232+
if query.is_empty() {
233+
return Ok((0, Vec::new()));
234+
}
232235
let offset = (page - 1) * limit;
233236

234237
let statement = "
@@ -1266,4 +1269,15 @@ mod tests {
12661269
Ok(())
12671270
});
12681271
}
1272+
1273+
#[test]
1274+
fn test_empty_query() {
1275+
wrapper(|env| {
1276+
let mut conn = env.db().conn();
1277+
let (num_results, results) = get_search_results(&mut conn, "", 0, 0).unwrap();
1278+
assert_eq!(num_results, 0);
1279+
assert!(results.is_empty());
1280+
Ok(())
1281+
})
1282+
}
12691283
}

0 commit comments

Comments
 (0)