From c388dca5ec64ee49c78c93be6d3ff6a29ded501c Mon Sep 17 00:00:00 2001 From: Kerollmops Date: Thu, 30 Sep 2021 11:20:16 +0200 Subject: [PATCH] Check that invalid updates do not create an index with a primary key --- meilisearch-http/tests/index/create_index.rs | 18 +++++++++++++++++- 1 file changed, 17 insertions(+), 1 deletion(-) diff --git a/meilisearch-http/tests/index/create_index.rs b/meilisearch-http/tests/index/create_index.rs index 3c081f0ed..4813847ff 100644 --- a/meilisearch-http/tests/index/create_index.rs +++ b/meilisearch-http/tests/index/create_index.rs @@ -1,5 +1,5 @@ use crate::common::Server; -use serde_json::Value; +use serde_json::{json, Value}; #[actix_rt::test] async fn create_index_no_primary_key() { @@ -33,6 +33,22 @@ async fn create_index_with_primary_key() { assert_eq!(response.as_object().unwrap().len(), 5); } +#[actix_rt::test] +async fn create_index_with_invalid_primary_key() { + let document = json!([ { "id": 2, "title": "Pride and Prejudice" } ]); + + let server = Server::new().await; + let index = server.index("movies"); + let (_response, code) = index.add_documents(document, Some("title")).await; + assert_eq!(code, 202); + + index.wait_update_id(0).await; + + let (response, code) = index.get().await; + assert_eq!(code, 200); + assert_eq!(response["primaryKey"], Value::Null); +} + // TODO: partial test since we are testing error, amd error is not yet fully implemented in // transplant #[actix_rt::test]