LucaCappelletti94 commented on code in PR #2250:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/2250#discussion_r3207480651
##########
tests/sqlparser_postgres.rs:
##########
@@ -7932,6 +7932,73 @@ fn parse_alter_operator_class() {
.is_err());
}
+#[test]
+fn parse_create_and_alter_text_search_failure_cases() {
+ let sql_cases = [
+ "CREATE TEXT SEARCH DICTIONARY alt_ts_dict1 (template=simple)",
+ "CREATE TEXT SEARCH DICTIONARY alt_ts_dict2 (template=simple)",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict1 RENAME TO alt_ts_dict2",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict1 RENAME TO alt_ts_dict3",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict2 OWNER TO
regress_alter_generic_user2",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict2 OWNER TO
regress_alter_generic_user3",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict2 SET SCHEMA alt_nsp2",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict3 RENAME TO alt_ts_dict4",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict1 RENAME TO alt_ts_dict4",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict3 OWNER TO
regress_alter_generic_user2",
+ "ALTER TEXT SEARCH DICTIONARY alt_ts_dict3 SET SCHEMA alt_nsp2",
+ "CREATE TEXT SEARCH CONFIGURATION alt_ts_conf1 (copy=english)",
+ "CREATE TEXT SEARCH CONFIGURATION alt_ts_conf2 (copy=english)",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf1 RENAME TO alt_ts_conf2",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf1 RENAME TO alt_ts_conf3",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf2 OWNER TO
regress_alter_generic_user2",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf2 OWNER TO
regress_alter_generic_user3",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf2 SET SCHEMA alt_nsp2",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf3 RENAME TO alt_ts_conf4",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf1 RENAME TO alt_ts_conf4",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf3 OWNER TO
regress_alter_generic_user2",
+ "ALTER TEXT SEARCH CONFIGURATION alt_ts_conf3 SET SCHEMA alt_nsp2",
+ "CREATE TEXT SEARCH TEMPLATE alt_ts_temp1 (lexize=dsimple_lexize)",
+ "CREATE TEXT SEARCH TEMPLATE alt_ts_temp2 (lexize=dsimple_lexize)",
+ "ALTER TEXT SEARCH TEMPLATE alt_ts_temp1 RENAME TO alt_ts_temp2",
+ "ALTER TEXT SEARCH TEMPLATE alt_ts_temp1 RENAME TO alt_ts_temp3",
+ "ALTER TEXT SEARCH TEMPLATE alt_ts_temp2 SET SCHEMA alt_nsp2",
+ "CREATE TEXT SEARCH TEMPLATE tstemp_case (\"Init\" = init_function)",
+ "CREATE TEXT SEARCH PARSER alt_ts_prs1 (start = prsd_start, gettoken =
prsd_nexttoken, end = prsd_end, lextypes = prsd_lextype)",
+ "CREATE TEXT SEARCH PARSER alt_ts_prs2 (start = prsd_start, gettoken =
prsd_nexttoken, end = prsd_end, lextypes = prsd_lextype)",
+ "ALTER TEXT SEARCH PARSER alt_ts_prs1 RENAME TO alt_ts_prs2",
+ "ALTER TEXT SEARCH PARSER alt_ts_prs1 RENAME TO alt_ts_prs3",
+ "ALTER TEXT SEARCH PARSER alt_ts_prs2 SET SCHEMA alt_nsp2",
+ "CREATE TEXT SEARCH PARSER tspars_case (\"Start\" = start_function)",
+ ];
+
+ for sql in sql_cases {
+ if let Err(err) = pg().parse_sql_statements(sql) {
Review Comment:
Done. The positive text-search cases now use
`pg_and_generic().verified_stmt(...)`. The remaining `pg()` calls are negative
parser checks where we assert an error.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
