From 6be301ebd8e42aa78f99619f80675e131ee3707f Mon Sep 17 00:00:00 2001
From: Vlasislav Kashin <99754299+vloldik@users.noreply.github.com>
Date: Sun, 13 Jul 2025 21:29:41 +0300
Subject: [PATCH] Refactor names and imports

---
 src/lib.rs                             |  8 +++++++-
 src/phonenumberutil/errors.rs          |  6 +++---
 src/phonenumberutil/phonenumberutil.rs |  4 ++--
 src/regex_based_matcher.rs             |  4 ++--
 src/regexp_cache.rs                    |  4 ++--
 src/tests/phonenumberutil_tests.rs     | 21 ++++++++++++++++-----
 6 files changed, 32 insertions(+), 15 deletions(-)

diff --git a/src/lib.rs b/src/lib.rs
index cedf7e4..34304dc 100644
--- a/src/lib.rs
+++ b/src/lib.rs
@@ -44,7 +44,13 @@ pub use phonenumberutil::{
     errors::{*},
     enums::{*},
 };
-pub use generated::proto::phonemetadata::{*};
+pub use generated::proto::phonemetadata;
 pub use generated::proto::phonenumber::PhoneNumber;
 pub use generated::proto::phonenumber::phone_number::CountryCodeSource;
+pub use regexp_cache::InvalidRegexError;
 mod tests;
+
+#[cfg(test)]
+mod test {
+
+}
\ No newline at end of file
diff --git a/src/phonenumberutil/errors.rs b/src/phonenumberutil/errors.rs
index c823eb3..4a6aae8 100644
--- a/src/phonenumberutil/errors.rs
+++ b/src/phonenumberutil/errors.rs
@@ -17,12 +17,12 @@ use std::num::ParseIntError;
 
 use thiserror::Error;
 
-use crate::regexp_cache::ErrorInvalidRegex;
+use crate::regexp_cache::InvalidRegexError;
 
 #[derive(Debug, PartialEq, Error)]
 pub enum InternalLogicError {
     #[error("{0}")]
-    InvalidRegex(#[from] ErrorInvalidRegex),
+    InvalidRegex(#[from] InvalidRegexError),
     
     #[error("{0}")]
     InvalidMetadataForValidRegion(#[from] InvalidMetadataForValidRegionError)
@@ -43,7 +43,7 @@ pub enum ParseError {
     #[error("Too long nsn")]
     TooLongNsn, // TOO_LONG in the java version.
     #[error("{0}")]
-    InvalidRegex(#[from] ErrorInvalidRegex),
+    InvalidRegex(#[from] InvalidRegexError),
 }
 
 #[derive(Debug, PartialEq, Error)]
diff --git a/src/phonenumberutil/phonenumberutil.rs b/src/phonenumberutil/phonenumberutil.rs
index 4e39abe..e9684eb 100644
--- a/src/phonenumberutil/phonenumberutil.rs
+++ b/src/phonenumberutil/phonenumberutil.rs
@@ -45,7 +45,7 @@ use crate::{
         phonenumber::{phone_number::CountryCodeSource, PhoneNumber}
     },
     region_code::RegionCode, interfaces::MatcherApi, macros::owned_from_cow_or,
-    regex_based_matcher::RegexBasedMatcher, regex_util::{RegexConsume, RegexFullMatch}, regexp_cache::ErrorInvalidRegex, string_util::strip_cow_prefix
+    regex_based_matcher::RegexBasedMatcher, regex_util::{RegexConsume, RegexFullMatch}, regexp_cache::InvalidRegexError, string_util::strip_cow_prefix
 };
 
 use dec_from_char::DecimalExtended;
@@ -54,7 +54,7 @@ use regex::Regex;
 
 // Helper type for Result
 
-pub type RegexResult<T> = std::result::Result<T, ErrorInvalidRegex>;
+pub type RegexResult<T> = std::result::Result<T, InvalidRegexError>;
 
 pub type ParseResult<T> = std::result::Result<T, ParseError>;
 
diff --git a/src/regex_based_matcher.rs b/src/regex_based_matcher.rs
index a4a2d12..9e4880f 100644
--- a/src/regex_based_matcher.rs
+++ b/src/regex_based_matcher.rs
@@ -17,7 +17,7 @@
 use log::{error};
 use super::regex_util::{RegexFullMatch, RegexConsume};
 
-use crate::{interfaces, generated::proto::phonemetadata::PhoneNumberDesc, regexp_cache::{ErrorInvalidRegex, RegexCache}};
+use crate::{interfaces, generated::proto::phonemetadata::PhoneNumberDesc, regexp_cache::{InvalidRegexError, RegexCache}};
 
 pub struct RegexBasedMatcher {
     cache: RegexCache,   
@@ -32,7 +32,7 @@ impl RegexBasedMatcher {
         &self, phone_number: &str, 
         number_pattern: &str,
         allow_prefix_match: bool
-    ) -> Result<bool, ErrorInvalidRegex> {
+    ) -> Result<bool, InvalidRegexError> {
         let regexp = self.cache.get_regex(number_pattern)?;
 
         // find first occurrence
diff --git a/src/regexp_cache.rs b/src/regexp_cache.rs
index a04d6cb..2a6938c 100644
--- a/src/regexp_cache.rs
+++ b/src/regexp_cache.rs
@@ -20,7 +20,7 @@ use thiserror::Error;
 
 #[derive(Debug, PartialEq, Error)]
 #[error("An error occurred while trying to create regex: {0}")]
-pub struct ErrorInvalidRegex(#[from] regex::Error);
+pub struct InvalidRegexError(#[from] regex::Error);
 
 pub struct RegexCache {
     cache: DashMap<String, Arc<regex::Regex>>
@@ -34,7 +34,7 @@ impl RegexCache {
         }
     }
 
-    pub fn get_regex(&self, pattern: &str) -> Result<Arc<regex::Regex>, ErrorInvalidRegex> {
+    pub fn get_regex(&self, pattern: &str) -> Result<Arc<regex::Regex>, InvalidRegexError> {
         if let Some(regex) = self.cache.get(pattern) {
             Ok(regex.value().clone())
         } else {
diff --git a/src/tests/phonenumberutil_tests.rs b/src/tests/phonenumberutil_tests.rs
index dbe2c6a..6473785 100644
--- a/src/tests/phonenumberutil_tests.rs
+++ b/src/tests/phonenumberutil_tests.rs
@@ -1,11 +1,22 @@
 use protobuf::{Message, MessageField};
 
 use crate::{
-    PhoneNumberFormat, PhoneNumberType, NumberLengthType,
-    ParseError, ValidationError,
-    NumberFormat, PhoneMetadata, PhoneMetadataCollection, PhoneNumberDesc,
-    CountryCodeSource, PhoneNumber,
-    PhoneNumberUtil,
+    phonenumberutil::{
+        enums::{
+            PhoneNumberFormat, PhoneNumberType, NumberLengthType,
+        },
+        errors::{
+            ParseError, ValidationError
+        }
+    },
+    generated::proto::{
+        phonemetadata::{
+            NumberFormat, PhoneMetadata, PhoneMetadataCollection, PhoneNumberDesc
+        },
+        phonenumber::phone_number::CountryCodeSource, 
+        phonenumber::PhoneNumber,
+    },
+    phonenumberutil::phonenumberutil::PhoneNumberUtil,
 };
 
 use super::region_code::RegionCode;