move EXECUTE tests to sqlparser_common

addresses apache#1490

and apache#1490

Author: lovasoa

tests/sqlparser_common.rs

@@ -1395,6 +1395,10 @@ fn pg_and_generic() -> TestedDialects {
     ])
 }
 
+fn ms_and_generic() -> TestedDialects {
+    TestedDialects::new(vec![Box::new(MsSqlDialect {}), Box::new(GenericDialect {})])
+}
+
 #[test]
 fn parse_json_ops_without_colon() {
     use self::BinaryOperator::*;
@@ -9730,6 +9734,41 @@ fn parse_call() {
     );
 }
 
+#[test]
+fn parse_execute_stored_procedure() {
+    let expected = Statement::Execute {
+        name: ObjectName(vec![
+            Ident {
+                value: "my_schema".to_string(),
+                quote_style: None,
+            },
+            Ident {
+                value: "my_stored_procedure".to_string(),
+                quote_style: None,
+            },
+        ]),
+        parameters: vec![
+            Expr::Value(Value::NationalStringLiteral("param1".to_string())),
+            Expr::Value(Value::NationalStringLiteral("param2".to_string())),
+        ],
+        has_parentheses: false,
+        using: vec![],
+    };
+    assert_eq!(
+        // Microsoft SQL Server does not use parentheses around arguments for EXECUTE
+        ms_and_generic()
+            .verified_stmt("EXECUTE my_schema.my_stored_procedure N'param1', N'param2'"),
+        expected
+    );
+    assert_eq!(
+        ms_and_generic().one_statement_parses_to(
+            "EXEC my_schema.my_stored_procedure N'param1', N'param2';",
+            "EXECUTE my_schema.my_stored_procedure N'param1', N'param2'",
+        ),
+        expected
+    );
+}
+
 #[test]
 fn parse_create_table_collate() {
     pg_and_generic().verified_stmt("CREATE TABLE tbl (foo INT, bar TEXT COLLATE \"de_DE\")");

tests/sqlparser_mssql.rs

@@ -570,41 +570,6 @@ fn parse_substring_in_select() {
     }
 }
 
-#[test]
-fn parse_mssql_execute_stored_procedure() {
-    let expected = Statement::Execute {
-        name: ObjectName(vec![
-            Ident {
-                value: "my_schema".to_string(),
-                quote_style: None,
-            },
-            Ident {
-                value: "my_stored_procedure".to_string(),
-                quote_style: None,
-            },
-        ]),
-        parameters: vec![
-            Expr::Value(Value::NationalStringLiteral("param1".to_string())),
-            Expr::Value(Value::NationalStringLiteral("param2".to_string())),
-        ],
-        has_parentheses: false,
-        using: vec![],
-    };
-    assert_eq!(
-        ms().verified_stmt("EXECUTE my_schema.my_stored_procedure N'param1', N'param2'"),
-        expected
-    );
-    assert_eq!(
-        Parser::parse_sql(
-            &MsSqlDialect {},
-            "EXEC my_schema.my_stored_procedure N'param1', N'param2';"
-        )
-        .unwrap()[0],
-        expected,
-        "EXEC should be parsed the same as EXECUTE"
-    );
-}
-
 #[test]
 fn parse_mssql_declare() {
     let sql = "DECLARE @foo CURSOR, @bar INT, @baz AS TEXT = 'foobar';";