fix vistior tests and address comment

Author: ayman-sigma

src/ast/mod.rs

@@ -166,10 +166,6 @@ impl Ident {
             quote_style: Some(quote),
         }
     }
-
-    pub fn to_object_name_part(&self) -> ObjectNamePart {
-        ObjectNamePart::Identifier(self.clone())
-    }
 }
 
 impl From<&str> for Ident {

src/ast/visitor.rs

@@ -403,15 +403,15 @@ where
 /// ```
 /// # use sqlparser::parser::Parser;
 /// # use sqlparser::dialect::GenericDialect;
-/// # use sqlparser::ast::{ObjectName, visit_relations_mut};
+/// # use sqlparser::ast::{ObjectName, ObjectNamePart, Ident, visit_relations_mut};
 /// # use core::ops::ControlFlow;
 /// let sql = "SELECT a FROM foo";
 /// let mut statements = Parser::parse_sql(&GenericDialect{}, sql)
 ///    .unwrap();
 ///
 /// // visit statements, renaming table foo to bar
 /// visit_relations_mut(&mut statements, |table| {
-///   table.0[0].value = table.0[0].value.replace("foo", "bar");
+///   table.0[0] = ObjectNamePart::Identifier(Ident::new("bar"));
 ///   ControlFlow::<()>::Continue(())
 /// });
 ///
@@ -529,7 +529,7 @@ where
 ///   if matches!(expr, Expr::Identifier(col_name) if col_name.value == "x") {
 ///     let old_expr = std::mem::replace(expr, Expr::Value(Value::Null));
 ///     *expr = Expr::Function(Function {
-///           name: ObjectName(vec![Ident::new("f")]),
+///           name: ObjectName::from(vec![Ident::new("f")]),
 ///           args: FunctionArguments::List(FunctionArgumentList {
 ///               duplicate_treatment: None,
 ///               args: vec![FunctionArg::Unnamed(FunctionArgExpr::Expr(old_expr))],

src/parser/mod.rs

@@ -12716,7 +12716,7 @@ impl<'a> Parser<'a> {
                 {
                     parent_name
                         .0
-                        .insert(0, self.parse_identifier(false)?.to_object_name_part());
+                        .insert(0, ObjectNamePart::Identifier(self.parse_identifier(false)?));
                 }
                 (None, Some(parent_name))
             }