fix: some clippy warnings and tests

versatiles-org · Jan 23, 2025 · d988107 · d988107
1 parent 91a9f81
commit d988107
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 8 deletions.
diff --git a/Cargo.toml b/Cargo.toml
@@ -86,6 +86,7 @@ lazy_static = { version = "1.5.0", default-features = false }
 log = { version = "0.4.25", default-features = false }
 num_cpus = { version = "1.16.0", default-features = false }
 regex = { version = "1.11.1", default-features = false, features = [
+	"std",
 	"unicode-case",
 	"unicode-perl",
 ] }

diff --git a/versatiles_container/Cargo.toml b/versatiles_container/Cargo.toml
@@ -33,6 +33,7 @@ assert_fs.workspace = true
 wildmatch.workspace = true
 
 versatiles.workspace = true
+versatiles_core = { workspace = true, features = ["test"] }
 
 [features]
 default = []

diff --git a/versatiles_container/src/container/versatiles/reader.rs b/versatiles_container/src/container/versatiles/reader.rs
@@ -1,3 +1,5 @@
+#![allow(dead_code)]
+
 //! Module for reading `versatiles` tile containers.
 //!
 //! This module provides the `VersaTilesReader` struct, which implements the `TilesReader` trait for reading tile data from a `versatiles` container. It supports reading metadata, tile data, and probing the container for debugging purposes.
@@ -63,7 +65,6 @@ pub struct VersaTilesReader {
 	tilejson: TileJSON,
 }
 
-#[allow(dead_code)]
 impl VersaTilesReader {
 	/// Opens a `versatiles` container from a file path.
 	///

diff --git a/versatiles_core/src/utils/pretty_print.rs b/versatiles_core/src/utils/pretty_print.rs
@@ -102,11 +102,6 @@ impl PrettyPrint {
 	pub async fn as_string(&self) -> String {
 		self.printer.as_string().await
 	}
-
-	#[allow(dead_code)]
-	pub async fn add_str(&self, text: String) {
-		self.printer.write(text).await;
-	}
 }
 
 impl Default for PrettyPrint {
@@ -160,14 +155,13 @@ mod tests {
 		printer.add_warning("test_warning_1").await;
 		let mut cat = printer.get_category("test_category_1").await;
 		cat.get_list("test_list_1").await.add_key_value("string_1", &4).await;
-		cat.add_str(String::from("string_2")).await;
 		cat.add_warning("test_warning_2").await;
 		printer.add_warning("test_warning_3").await;
 
 		let result = printer.as_string().await;
 		assert_eq!(
 			&result,
-			"test_warning_1\ntest_category_1:\n   test_list_1:\n      string_1: 4\nstring_2   test_warning_2\ntest_warning_3\n"
+			"test_warning_1\ntest_category_1:\n   test_list_1:\n      string_1: 4\n   test_warning_2\ntest_warning_3\n"
 		);
 	}