Skip to content

Commit ead8209

Browse files
kosiewkosiew
authored
update insta snapshots (#19381)
## Which issue does this PR close? <!-- We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax. For example `Closes #123` indicates that this PR will close issue #123. --> - Closes #19380. ## Rationale for this change <!-- Why are you proposing this change? If this is already explained clearly in the issue then this section is not needed. Explaining clearly why changes are proposed helps reviewers understand your changes and offer better suggestions for fixes. --> Snapshot test passes but the existing value is in a legacy format. Updated insta snapshots to new format by running `cargo insta test --force-update-snapshots` ## What changes are included in this PR? Snapshots in various directories. <!-- There is no need to duplicate the description in the issue here but it is sometimes worth providing a summary of the individual changes in this PR. --> ## Are these changes tested? <!-- We typically require tests for all PRs in order to: 1. Prevent the code from being accidentally broken by subsequent changes 2. Serve as another way to document the expected behavior of the code If tests are not included in your PR, please explain why (for example, are they covered by existing tests)? --> Yes ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> No <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> No
1 parent 8120da4 commit ead8209

File tree

70 files changed

+4151
-4263
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

70 files changed

+4151
-4263
lines changed

‎datafusion-cli/src/print_format.rs‎

Lines changed: 32 additions & 34 deletions
Original file line numberDiff line numberDiff line change
@@ -247,12 +247,12 @@ mod tests {
247247
.with_schema(three_column_schema())
248248
.with_batches(vec![])
249249
.run();
250-
assert_snapshot!(output, @r#"
250+
assert_snapshot!(output, @r"
251251
+---+---+---+
252252
| a | b | c |
253253
+---+---+---+
254254
+---+---+---+
255-
"#);
255+
");
256256
}
257257

258258
#[test]
@@ -262,11 +262,11 @@ mod tests {
262262
.with_batches(split_batch(three_column_batch()))
263263
.with_header(WithHeader::No)
264264
.run();
265-
assert_snapshot!(output, @r#"
265+
assert_snapshot!(output, @r"
266266
1,4,7
267267
2,5,8
268268
3,6,9
269-
"#);
269+
");
270270
}
271271

272272
#[test]
@@ -276,12 +276,12 @@ mod tests {
276276
.with_batches(split_batch(three_column_batch()))
277277
.with_header(WithHeader::Yes)
278278
.run();
279-
assert_snapshot!(output, @r#"
279+
assert_snapshot!(output, @r"
280280
a,b,c
281281
1,4,7
282282
2,5,8
283283
3,6,9
284-
"#);
284+
");
285285
}
286286

287287
#[test]
@@ -291,10 +291,10 @@ mod tests {
291291
.with_batches(split_batch(three_column_batch()))
292292
.with_header(WithHeader::No)
293293
.run();
294-
assert_snapshot!(output, @"
295-
1\t4\t7
296-
2\t5\t8
297-
3\t6\t9
294+
assert_snapshot!(output, @r"
295+
1 4 7
296+
2 5 8
297+
3 6 9
298298
")
299299
}
300300

@@ -305,11 +305,11 @@ mod tests {
305305
.with_batches(split_batch(three_column_batch()))
306306
.with_header(WithHeader::Yes)
307307
.run();
308-
assert_snapshot!(output, @"
309-
a\tb\tc
310-
1\t4\t7
311-
2\t5\t8
312-
3\t6\t9
308+
assert_snapshot!(output, @r"
309+
a b c
310+
1 4 7
311+
2 5 8
312+
3 6 9
313313
");
314314
}
315315

@@ -320,15 +320,15 @@ mod tests {
320320
.with_batches(split_batch(three_column_batch()))
321321
.with_header(WithHeader::Ignored)
322322
.run();
323-
assert_snapshot!(output, @r#"
323+
assert_snapshot!(output, @r"
324324
+---+---+---+
325325
| a | b | c |
326326
+---+---+---+
327327
| 1 | 4 | 7 |
328328
| 2 | 5 | 8 |
329329
| 3 | 6 | 9 |
330330
+---+---+---+
331-
"#);
331+
");
332332
}
333333
#[test]
334334
fn print_json() {
@@ -337,9 +337,7 @@ mod tests {
337337
.with_batches(split_batch(three_column_batch()))
338338
.with_header(WithHeader::Ignored)
339339
.run();
340-
assert_snapshot!(output, @r#"
341-
[{"a":1,"b":4,"c":7},{"a":2,"b":5,"c":8},{"a":3,"b":6,"c":9}]
342-
"#);
340+
assert_snapshot!(output, @r#"[{"a":1,"b":4,"c":7},{"a":2,"b":5,"c":8},{"a":3,"b":6,"c":9}]"#);
343341
}
344342

345343
#[test]
@@ -363,11 +361,11 @@ mod tests {
363361
.with_batches(split_batch(three_column_batch()))
364362
.with_header(WithHeader::No)
365363
.run();
366-
assert_snapshot!(output, @r#"
364+
assert_snapshot!(output, @r"
367365
1,4,7
368366
2,5,8
369367
3,6,9
370-
"#);
368+
");
371369
}
372370
#[test]
373371
fn print_automatic_with_header() {
@@ -376,12 +374,12 @@ mod tests {
376374
.with_batches(split_batch(three_column_batch()))
377375
.with_header(WithHeader::Yes)
378376
.run();
379-
assert_snapshot!(output, @r#"
377+
assert_snapshot!(output, @r"
380378
a,b,c
381379
1,4,7
382380
2,5,8
383381
3,6,9
384-
"#);
382+
");
385383
}
386384

387385
#[test]
@@ -396,15 +394,15 @@ mod tests {
396394
.with_maxrows(max_rows)
397395
.run();
398396
allow_duplicates! {
399-
assert_snapshot!(output, @r#"
397+
assert_snapshot!(output, @r"
400398
+---+
401399
| a |
402400
+---+
403401
| 1 |
404402
| 2 |
405403
| 3 |
406404
+---+
407-
"#);
405+
");
408406
}
409407
}
410408
}
@@ -416,7 +414,7 @@ mod tests {
416414
.with_batches(vec![one_column_batch()])
417415
.with_maxrows(MaxRows::Limited(1))
418416
.run();
419-
assert_snapshot!(output, @r#"
417+
assert_snapshot!(output, @r"
420418
+---+
421419
| a |
422420
+---+
@@ -425,7 +423,7 @@ mod tests {
425423
| . |
426424
| . |
427425
+---+
428-
"#);
426+
");
429427
}
430428

431429
#[test]
@@ -439,7 +437,7 @@ mod tests {
439437
])
440438
.with_maxrows(MaxRows::Limited(5))
441439
.run();
442-
assert_snapshot!(output, @r#"
440+
assert_snapshot!(output, @r"
443441
+---+
444442
| a |
445443
+---+
@@ -452,7 +450,7 @@ mod tests {
452450
| . |
453451
| . |
454452
+---+
455-
"#);
453+
");
456454
}
457455

458456
#[test]
@@ -464,15 +462,15 @@ mod tests {
464462
.with_format(PrintFormat::Table)
465463
.with_batches(vec![empty_batch.clone(), batch, empty_batch])
466464
.run();
467-
assert_snapshot!(output, @r#"
465+
assert_snapshot!(output, @r"
468466
+---+
469467
| a |
470468
+---+
471469
| 1 |
472470
| 2 |
473471
| 3 |
474472
+---+
475-
"#);
473+
");
476474
}
477475

478476
#[test]
@@ -486,12 +484,12 @@ mod tests {
486484
.with_batches(vec![empty_batch])
487485
.with_header(WithHeader::Yes)
488486
.run();
489-
assert_snapshot!(output, @r#"
487+
assert_snapshot!(output, @r"
490488
+---+
491489
| a |
492490
+---+
493491
+---+
494-
"#);
492+
");
495493

496494
// No output for empty batch when schema contains no columns
497495
let empty_batch = RecordBatch::new_empty(Arc::new(Schema::empty()));

‎datafusion-cli/tests/snapshots/cli_explain_environment_overrides@explain_plan_environment_overrides.snap‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ info:
77
- EXPLAIN SELECT 123
88
env:
99
DATAFUSION_EXPLAIN_FORMAT: pgjson
10-
snapshot_kind: text
1110
---
1211
success: true
1312
exit_code: 0

‎datafusion-cli/tests/snapshots/[email protected]‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
source: tests/cli_integration.rs
2+
source: datafusion-cli/tests/cli_integration.rs
33
info:
44
program: datafusion-cli
55
args:

‎datafusion-cli/tests/snapshots/[email protected]‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
source: tests/cli_integration.rs
2+
source: datafusion-cli/tests/cli_integration.rs
33
info:
44
program: datafusion-cli
55
args:

‎datafusion-cli/tests/snapshots/[email protected]‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
source: tests/cli_integration.rs
2+
source: datafusion-cli/tests/cli_integration.rs
33
info:
44
program: datafusion-cli
55
args:

‎datafusion-cli/tests/snapshots/[email protected]‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
source: tests/cli_integration.rs
2+
source: datafusion-cli/tests/cli_integration.rs
33
info:
44
program: datafusion-cli
55
args:

‎datafusion-cli/tests/snapshots/[email protected]‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
source: tests/cli_integration.rs
2+
source: datafusion-cli/tests/cli_integration.rs
33
info:
44
program: datafusion-cli
55
args:

‎datafusion-cli/tests/snapshots/[email protected]‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
source: tests/cli_integration.rs
2+
source: datafusion-cli/tests/cli_integration.rs
33
info:
44
program: datafusion-cli
55
args:

‎datafusion-cli/tests/snapshots/cli_quick_test@batch_size.snap‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
source: tests/cli_integration.rs
2+
source: datafusion-cli/tests/cli_integration.rs
33
info:
44
program: datafusion-cli
55
args:

‎datafusion-cli/tests/snapshots/cli_quick_test@default_explain_plan.snap‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ info:
55
args:
66
- "--command"
77
- EXPLAIN SELECT 123
8-
snapshot_kind: text
98
---
109
success: true
1110
exit_code: 0

0 commit comments

Comments
 (0)