From 180a2bf94ce6a93251f712db20dbdbf49dad1361 Mon Sep 17 00:00:00 2001 From: Jeff Baskin Date: Sun, 3 May 2026 10:39:14 -0400 Subject: [PATCH] Removed warnings from add tests. --- tests/add_test.rs | 66 +++++++++++++++++++++++++++-------------------- 1 file changed, 38 insertions(+), 28 deletions(-) diff --git a/tests/add_test.rs b/tests/add_test.rs index c5b9d5f..a81d8ef 100644 --- a/tests/add_test.rs +++ b/tests/add_test.rs @@ -3,8 +3,8 @@ mod support; use chrono::Utc; use morethantext::{ action::{Addition, DocDef, Field, FieldType, Query}, - Action, CalcValue, Calculation, ErrorID, Include, IndexType, MTTError, MoreThanText, Name, - Operand, Path, TestMoreThanText, + Action, Calculation, ErrorID, Include, IndexType, MTTError, MoreThanText, Name, Operand, Path, + TestMoreThanText, }; use std::{collections::HashSet, time::Duration}; use support::{random_name, TestDocument}; @@ -13,7 +13,7 @@ use uuid::Uuid; #[test] fn can_new_documents_be_added() { let count = 5; - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let doc_name = random_name(); let field_name = random_name(); @@ -23,7 +23,7 @@ fn can_new_documents_be_added() { } let mut docdef = DocDef::new(doc_name.clone()); docdef.add_field(vec![field_name.clone()], FieldType::Integer); - client.create_document(docdef); + client.create_document(docdef).unwrap(); for item in data.iter() { let mut add = Addition::new(doc_name.clone()); add.add_field(field_name.clone(), item.clone()); @@ -47,7 +47,7 @@ fn can_new_documents_be_added() { #[test] fn does_it_error_on_a_bad_document_name() { - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let doc_name = Name::english("empty"); let mut expected = MTTError::new(ErrorID::NameNotFound(doc_name.clone().into())); @@ -59,12 +59,12 @@ fn does_it_error_on_a_bad_document_name() { #[test] fn does_it_error_on_bad_field_name() { - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let doc_name = Name::english("holder"); let field_name = Name::english("missing"); let docdef = DocDef::new(doc_name.clone()); - client.create_document(docdef); + client.create_document(docdef).unwrap(); let mut add = Addition::new(doc_name.clone()); add.add_field(field_name.clone(), "something"); let mut expected = MTTError::new(ErrorID::NameNotFound(field_name.clone().into())); @@ -76,10 +76,10 @@ fn does_it_error_on_bad_field_name() { #[test] fn does_it_error_on_bad_field_type() { - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let test_doc = TestDocument::new(vec![FieldType::Uuid]); - client.create_document(test_doc.get_docdef()); + client.create_document(test_doc.get_docdef()).unwrap(); let mut add = Addition::new(test_doc.get_doc_name().clone()); add.add_field(test_doc.get_field_name(0), "something"); let mut expected = MTTError::new(ErrorID::FieldTypeExpected(FieldType::Uuid)); @@ -92,10 +92,10 @@ fn does_it_error_on_bad_field_type() { #[test] #[ignore = "requires session to store language preference"] fn does_it_error_on_missing_fields() { - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let test_doc = TestDocument::new(vec![FieldType::Integer, FieldType::Integer]); - client.create_document(test_doc.get_docdef()); + client.create_document(test_doc.get_docdef()).unwrap(); let mut add = Addition::new(test_doc.get_doc_name().clone()); add.add_field(test_doc.get_field_name(0), 1); let mut expected = MTTError::new(ErrorID::FieldInvalidNone); @@ -107,13 +107,15 @@ fn does_it_error_on_missing_fields() { #[test] fn can_default_values_be_used() { - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let ftype = FieldType::StaticString; let test_doc = TestDocument::new(vec![ftype.clone()]); let mut docdef = test_doc.get_docdef(); - docdef.set_default(&test_doc.get_field_name(0), ftype.clone()); - client.create_document(docdef); + docdef + .set_default(&test_doc.get_field_name(0), ftype.clone()) + .unwrap(); + client.create_document(docdef).unwrap(); let add = Addition::new(test_doc.get_doc_name().clone()); let results = client.records(add).unwrap(); let rec = results.iter().last().unwrap(); @@ -122,14 +124,16 @@ fn can_default_values_be_used() { #[test] fn can_default_values_be_set() { - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let ftype = FieldType::StaticString; let fdefault = Uuid::new_v4().to_string(); let test_doc = TestDocument::new(vec![ftype.clone()]); let mut docdef = test_doc.get_docdef(); - docdef.set_default(&test_doc.get_field_name(0), fdefault.clone()); - client.create_document(docdef); + docdef + .set_default(&test_doc.get_field_name(0), fdefault.clone()) + .unwrap(); + client.create_document(docdef).unwrap(); let add = Addition::new(test_doc.get_doc_name().clone()); let results = client.records(add).unwrap(); let rec = results.iter().last().unwrap(); @@ -141,15 +145,17 @@ fn can_default_values_be_set() { #[test] fn can_default_values_be_overwritten() { - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let ftype = FieldType::StaticString; let fdefault = Uuid::new_v4().to_string(); let used = "something"; let test_doc = TestDocument::new(vec![ftype.clone()]); let mut docdef = test_doc.get_docdef(); - docdef.set_default(&test_doc.get_field_name(0), fdefault.clone()); - client.create_document(docdef); + docdef + .set_default(&test_doc.get_field_name(0), fdefault.clone()) + .unwrap(); + client.create_document(docdef).unwrap(); let mut add = Addition::new(test_doc.get_doc_name().clone()); add.add_field(test_doc.get_field_name(0), used); let results = client.records(add).unwrap(); @@ -160,14 +166,16 @@ fn can_default_values_be_overwritten() { #[test] fn can_default_values_be_calculated() { let duration = Duration::from_secs(300); - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let test_doc = TestDocument::new(vec![FieldType::DateTime]); let mut docdef = test_doc.get_docdef(); let mut calc = Calculation::new(Operand::Add); - calc.add_value(FieldType::DateTime); - calc.add_value(duration.clone()); - docdef.set_default(&test_doc.get_field_name(0), calc); + calc.add_value(FieldType::DateTime).unwrap(); + calc.add_value(duration.clone()).unwrap(); + docdef + .set_default(&test_doc.get_field_name(0), calc) + .unwrap(); client.create_document(docdef).unwrap(); let add = Addition::new(test_doc.get_doc_name()); let start = Utc::now() + duration; @@ -182,12 +190,14 @@ fn can_default_values_be_calculated() { #[test] fn are_unique_indexes_maintained_with_additions() { let data = 1; - let mut mtt = MoreThanText::new(); + let mtt = MoreThanText::new(); let client = mtt.client(); let test_doc = TestDocument::new(vec![FieldType::Integer]); let mut docdef = test_doc.get_docdef(); - docdef.add_index(&test_doc.get_field_name(0), IndexType::Unique); - client.create_document(docdef); + docdef + .add_index(&test_doc.get_field_name(0), IndexType::Unique) + .unwrap(); + client.create_document(docdef).unwrap(); let mut add = Addition::new(test_doc.get_doc_name()); add.add_field(test_doc.get_field_name(0), data.clone()); client.records(add.clone()).unwrap(); @@ -201,7 +211,7 @@ fn are_unique_indexes_maintained_with_additions() { #[test] fn does_addition_send_on_query_message() { let mut test_env = TestMoreThanText::new(); - let mut mtt = test_env.get_morethantext(); + let mtt = test_env.get_morethantext(); let client = mtt.client(); let test_doc = TestDocument::new(vec![FieldType::Integer]); client.create_document(test_doc.get_docdef()).unwrap();