Refactored requests from PR

Author: LucaCappelletti94

src/ast/mod.rs

@@ -3967,18 +3967,7 @@ pub enum Statement {
         sequence_options: Vec<SequenceOptions>,
         owned_by: Option<ObjectName>,
     },
-    /// ```sql
-    /// CREATE DOMAIN name [ AS ] data_type
-    ///         [ COLLATE collation ]
-    ///         [ DEFAULT expression ]
-    ///         [ domain_constraint [ ... ] ]
-    ///
-    ///     where domain_constraint is:
-    ///
-    ///     [ CONSTRAINT constraint_name ]
-    ///     { NOT NULL | NULL | CHECK (expression) }
-    /// ```
-    /// See [PostgreSQL](https://www.postgresql.org/docs/current/sql-createdomain.html)
+    /// A `CREATE DOMAIN` statement.
     CreateDomain(CreateDomain),
     /// ```sql
     /// CREATE TYPE <name>

src/parser/mod.rs

@@ -5896,20 +5896,8 @@ impl<'a> Parser<'a> {
         Ok(owner)
     }
 
-    /// ```sql
-    ///     CREATE DOMAIN name [ AS ] data_type
-    ///             [ COLLATE collation ]
-    ///             [ DEFAULT expression ]
-    ///             [ domain_constraint [ ... ] ]
-    ///     
-    ///         where domain_constraint is:
-    ///     
-    ///         [ CONSTRAINT constraint_name ]
-    ///         { NOT NULL | NULL | CHECK (expression) }
-    /// ```
-    ///
-    /// [PostgreSQL Documentation](https://www.postgresql.org/docs/current/sql-createdomain.html)
-    pub fn parse_create_domain(&mut self) -> Result<Statement, ParserError> {
+    /// Parses a [Statement::CreateDomain] statement.
+    fn parse_create_domain(&mut self) -> Result<Statement, ParserError> {
         let name = self.parse_object_name(false)?;
         self.expect_keyword_is(Keyword::AS)?;
         let data_type = self.parse_data_type()?;

tests/sqlparser_postgres.rs

@@ -5099,10 +5099,7 @@ fn parse_create_domain() {
     });
 
     assert_eq!(pg().verified_stmt(sql1), expected);
-}
 
-#[test]
-fn parse_create_domain_with_collation() {
     let sql2 = "CREATE DOMAIN my_domain AS INTEGER COLLATE \"en_US\" CHECK (VALUE > 0)";
     let expected = Statement::CreateDomain(CreateDomain {
         name: ObjectName::from(vec![Ident::new("my_domain")]),
@@ -5120,10 +5117,7 @@ fn parse_create_domain_with_collation() {
     });
 
     assert_eq!(pg().verified_stmt(sql2), expected);
-}
 
-#[test]
-fn parse_create_domain_with_default() {
     let sql3 = "CREATE DOMAIN my_domain AS INTEGER DEFAULT 1 CHECK (VALUE > 0)";
     let expected = Statement::CreateDomain(CreateDomain {
         name: ObjectName::from(vec![Ident::new("my_domain")]),
@@ -5141,10 +5135,7 @@ fn parse_create_domain_with_default() {
     });
 
     assert_eq!(pg().verified_stmt(sql3), expected);
-}
 
-#[test]
-fn parse_create_domain_with_default_and_collation() {
     let sql4 = "CREATE DOMAIN my_domain AS INTEGER COLLATE \"en_US\" DEFAULT 1 CHECK (VALUE > 0)";
     let expected = Statement::CreateDomain(CreateDomain {
         name: ObjectName::from(vec![Ident::new("my_domain")]),
@@ -5162,10 +5153,7 @@ fn parse_create_domain_with_default_and_collation() {
     });
 
     assert_eq!(pg().verified_stmt(sql4), expected);
-}
 
-#[test]
-fn parse_create_domain_with_named_constraint() {
     let sql5 = "CREATE DOMAIN my_domain AS INTEGER CONSTRAINT my_constraint CHECK (VALUE > 0)";
     let expected = Statement::CreateDomain(CreateDomain {
         name: ObjectName::from(vec![Ident::new("my_domain")]),