From d2d6489b2f37599d6a94c734f12d9022472827a5 Mon Sep 17 00:00:00 2001 From: fys <40801205+fengys1996@users.noreply.github.com> Date: Tue, 12 Aug 2025 12:43:42 +0800 Subject: [PATCH] fix: unit test about trigger labels parse (#6716) --- src/sql/src/parsers/alter_parser/trigger.rs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/sql/src/parsers/alter_parser/trigger.rs b/src/sql/src/parsers/alter_parser/trigger.rs index 6f4fb01a9b..eaa22df859 100644 --- a/src/sql/src/parsers/alter_parser/trigger.rs +++ b/src/sql/src/parsers/alter_parser/trigger.rs @@ -607,7 +607,7 @@ mod tests { fn test_parse_alter_trigger_labels() { // Passed case: SET LABELS. // Note: "ALTER TRIGGER" is matched. - let sql = r#"test_trigger SET LABELS (Key1='value1', 'KEY2'='value2')"#; + let sql = r#"test_trigger SET LABELS (Key1='value1', 'KEY2'='VALUE2')"#; let mut ctx = ParserContext::new(&GreptimeDbDialect {}, sql).unwrap(); let stmt = ctx.parse_alter_trigger().unwrap(); let Statement::AlterTrigger(alter) = stmt else { @@ -616,8 +616,10 @@ mod tests { let Some(LabelOperations::ReplaceAll(labels)) = alter.operation.label_operations else { panic!("Expected ReplaceAll label operations"); }; + + assert_eq!(labels.len(), 2); assert_eq!(labels.get("key1"), Some(&"value1".to_string())); - assert_eq!(labels.get("KEY2"), Some(&"value2".to_string())); + assert_eq!(labels.get("key2"), Some(&"VALUE2".to_string())); // Passed case: multiple ADD/DROP/MODIFY LABELS. let sql = r#"test_trigger ADD LABELS (key1='value1') MODIFY LABELS (key2='value2') DROP LABELS ('key3')"#;