Fix review comments

Author: git-hulk

src/ast/mod.rs

@@ -2164,7 +2164,7 @@ pub enum Statement {
         location: Option<HiveSetLocation>,
         /// ClickHouse dialect supports `ON CLUSTER` clause for ALTER TABLE
         /// For example: `ALTER TABLE table_name ON CLUSTER cluster_name ADD COLUMN c UInt32`
-        /// [ClickHouse](https://clickhouse.com/docs/en/sql-reference/statements/alter)
+        /// [ClickHouse](https://clickhouse.com/docs/en/sql-reference/statements/alter/update)
         on_cluster: Option<String>,
     },
     /// ```sql

tests/sqlparser_clickhouse.rs

@@ -25,7 +25,6 @@ use sqlparser::ast::Value::Number;
 use sqlparser::ast::*;
 use sqlparser::dialect::ClickHouseDialect;
 use sqlparser::dialect::GenericDialect;
-use sqlparser::parser::ParserError::ParserError;
 
 #[test]
 fn parse_map_access_expr() {
@@ -1092,27 +1091,6 @@ fn parse_create_table_on_commit_and_as_query() {
     }
 }
 
-#[test]
-fn test_alter_table_with_on_cluster() {
-    let sql = "ALTER TABLE t ON CLUSTER 'cluster' ADD CONSTRAINT bar PRIMARY KEY (baz)";
-    match clickhouse_and_generic().verified_stmt(sql) {
-        Statement::AlterTable {
-            name, on_cluster, ..
-        } => {
-            assert_eq!(name.to_string(), "t");
-            assert_eq!(on_cluster, Some("'cluster'".to_string()));
-        }
-        _ => unreachable!(),
-    }
-
-    let res = clickhouse_and_generic()
-        .parse_sql_statements("ALTER TABLE t ON CLUSTER 123 ADD CONSTRAINT bar PRIMARY KEY (baz)");
-    assert_eq!(
-        res.unwrap_err(),
-        ParserError("Expected: identifier or cluster literal, found: 123".to_string())
-    )
-}
-
 fn clickhouse() -> TestedDialects {
     TestedDialects {
         dialects: vec![Box::new(ClickHouseDialect {})],

tests/sqlparser_common.rs

@@ -3822,6 +3822,27 @@ fn parse_alter_table() {
     }
 }
 
+#[test]
+fn test_alter_table_with_on_cluster() {
+    let sql = "ALTER TABLE t ON CLUSTER 'cluster' ADD CONSTRAINT bar PRIMARY KEY (baz)";
+    match all_dialects().verified_stmt(sql) {
+        Statement::AlterTable {
+            name, on_cluster, ..
+        } => {
+            std::assert_eq!(name.to_string(), "t");
+            std::assert_eq!(on_cluster, Some("'cluster'".to_string()));
+        }
+        _ => unreachable!(),
+    }
+
+    let res = all_dialects()
+        .parse_sql_statements("ALTER TABLE t ON CLUSTER 123 ADD CONSTRAINT bar PRIMARY KEY (baz)");
+    std::assert_eq!(
+        res.unwrap_err(),
+        ParserError::ParserError("Expected: identifier or cluster literal, found: 123".to_string())
+    )
+}
+
 #[test]
 fn parse_alter_index() {
     let rename_index = "ALTER INDEX idx RENAME TO new_idx";