Skip to content

Commit 21d7e67

Browse files
Merge pull request #689 from Mark-Simulacrum/fix-686
Avoid panicking for empty result set queries
2 parents 31af4d0 + 1e4e12c commit 21d7e67

File tree

1 file changed

+14
-1
lines changed

1 file changed

+14
-1
lines changed

site/src/server.rs

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -712,7 +712,20 @@ pub async fn handle_self_profile(
712712
let mut sp_responses = data
713713
.query::<Option<selector::SelfProfileData>>(query.clone(), commits.clone())
714714
.await?;
715-
assert_eq!(sp_responses.len(), 1, "all selectors are exact");
715+
716+
if sp_responses.is_empty() {
717+
return Err(format!("no results found for {:?} in {:?}", query, commits));
718+
}
719+
720+
assert_eq!(
721+
sp_responses.len(),
722+
1,
723+
"all selectors are exact, paths: {:?}",
724+
sp_responses
725+
.iter()
726+
.map(|v| format!("{:?}", v.path))
727+
.collect::<Vec<_>>()
728+
);
716729
let mut sp_response = sp_responses.remove(0).series;
717730

718731
let mut cpu_responses = data

0 commit comments

Comments
 (0)