Merge branch 'master' into issue943

This commit is contained in:
gorogoroumaru 2020-09-02 13:09:37 +09:00 committed by GitHub
commit 0ca44b6a82
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 46 additions and 13 deletions

View file

@ -1466,7 +1466,6 @@ async fn test_facet_count() {
server.update_all_settings(body).await;
// same as before, but now facets are set:
test_post_get_search!(server, query, |response, _status_code|{
println!("{}", response);
assert!(response.get("exhaustiveFacetsCount").is_some());
assert_eq!(response.get("facetsDistribution").unwrap().as_object().unwrap().values().count(), 1);
// assert that case is preserved
@ -1696,3 +1695,37 @@ async fn update_documents_with_facet_distribution() {
let (response2, _) = server.search_post(search).await;
assert_json_eq!(expected_facet_distribution, response2["facetsDistribution"].clone());
}
#[actix_rt::test]
async fn test_facet_count_with_facet_filter() {
let mut server = common::Server::test_server().await;
let body = json!({
"attributesForFaceting": ["gender"]
});
server.update_all_settings(body).await;
let query = json!({
"q": "a",
"facetsDistribution": ["gender"],
"facetFilters": ["gender:male"],
});
test_post_get_search!(server, query, |response, _status_code|{
assert!(response.get("exhaustiveFacetsCount").is_some());
let facets_distribution = response.get("facetsDistribution").unwrap().as_object();
assert_eq!(facets_distribution.unwrap()["gender"]["male"], 37);
assert_eq!(facets_distribution.unwrap()["gender"]["female"], 39);
});
// facet distribution should remain the same when facet filter changes
let query = json!({
"q": "a",
"facetsDistribution": ["gender"],
"facetFilters": [["gender:male", "gender:female"]],
});
test_post_get_search!(server, query, |response, _status_code|{
assert!(response.get("exhaustiveFacetsCount").is_some());
let facets_distribution = response.get("facetsDistribution").unwrap().as_object();
assert_eq!(facets_distribution.unwrap()["gender"]["male"], 37);
assert_eq!(facets_distribution.unwrap()["gender"]["female"], 39);
});
}