Commit aee8f57
authored
fix: fix test JSON repr issues (#5719)
The `safeJsonStringify` we use in tests doesn't handle `Buffer`s
correctly. We normally would not convert them to JSON in production, but
the JSON representation is used for tests snapshots, so I replaced
`safeJsonStringify` with a custom one.
D1 doesn't actually return a Buffer so I just updated its snapshot.1 parent 9d980ce commit aee8f57
File tree
2 files changed
+38
-2
lines changed- libs/driver-adapters/executor/src/qc-test-worker
- query-engine/connector-test-kit-rs/query-engine-tests/tests/raw/sql
2 files changed
+38
-2
lines changedLines changed: 33 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
12 | 12 | | |
13 | 13 | | |
14 | 14 | | |
15 | | - | |
16 | 15 | | |
17 | 16 | | |
18 | 17 | | |
| |||
302 | 301 | | |
303 | 302 | | |
304 | 303 | | |
| 304 | + | |
| 305 | + | |
| 306 | + | |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
| 322 | + | |
| 323 | + | |
| 324 | + | |
| 325 | + | |
| 326 | + | |
| 327 | + | |
| 328 | + | |
| 329 | + | |
| 330 | + | |
| 331 | + | |
| 332 | + | |
| 333 | + | |
| 334 | + | |
| 335 | + | |
| 336 | + | |
Lines changed: 5 additions & 1 deletion
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
661 | 661 | | |
662 | 662 | | |
663 | 663 | | |
664 | | - | |
| 664 | + | |
| 665 | + | |
| 666 | + | |
| 667 | + | |
| 668 | + | |
665 | 669 | | |
666 | 670 | | |
667 | 671 | | |
| |||
0 commit comments