Skip to content

Commit d20c5d6

Browse files
authored
fix : snapshot to the modern multiline format (#19517)
## Which issue does this PR close? N/A ## Rationale for this change A legacy inline insta snapshot was using an outdated format that does not start and end with a newline. Newer versions of insta warn about this format and will fail to match such snapshots in the future. Updating the snapshot to the modern multiline format ensures forward compatibility and avoids future snapshot failures. ## What changes are included in this PR? - Converted a legacy inline insta snapshot to the modern multiline raw snapshot format. - Added leading and trailing newlines as required by current insta snapshot conventions. - No changes were made to the underlying logic or behavior. ## Are these changes tested? Yes ## Are there any user-facing changes? No
1 parent e5ca510 commit d20c5d6

File tree

1 file changed

+4
-1
lines changed

1 file changed

+4
-1
lines changed

datafusion/sql/tests/sql_integration.rs

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4508,7 +4508,10 @@ fn test_parse_escaped_string_literal_value() {
45084508
let plan = logical_plan(sql).unwrap();
45094509
assert_snapshot!(
45104510
plan,
4511-
@"Projection: character_length(Utf8(\"%\")) AS len, Utf8(\"K\") AS hex, Utf8(\"\u{1}\") AS unicode\n EmptyRelation: rows=1"
4511+
@r#"
4512+
Projection: character_length(Utf8("%")) AS len, Utf8("K") AS hex, Utf8("") AS unicode
4513+
EmptyRelation: rows=1
4514+
"#
45124515
);
45134516

45144517
let sql = r"SELECT character_length(E'\000') AS len";

0 commit comments

Comments
 (0)