Commit 3ad6c16
authored
to 2.2: fix failed BVT cases. (#22095)
### **User description**
## What type of PR is this?
- [ ] API-change
- [x] BUG
- [ ] Improvement
- [ ] Documentation
- [ ] Feature
- [ ] Test and CI
- [ ] Code Refactoring
## Which issue(s) this PR fixes:
issue ##21092
## What this PR does / why we need it:
fix failed BVT cases.
___
### **PR Type**
Bug fix, Tests
___
### **Description**
- Fix BVT test failures by updating expected results
- Remove placeholder error messages from test outputs
- Correct session IDs and snapshot references
- Update role privilege test expectations
___
### **Changes diagram**
```mermaid
flowchart LR
A["Test Files"] --> B["Remove Error Placeholders"]
B --> C["Update Expected Results"]
C --> D["Fix Session IDs"]
D --> E["Correct Snapshot References"]
E --> F["Update Role Privileges"]
```
___
### **Changes walkthrough** 📝
<table><thead><tr><th></th><th align="left">Relevant
files</th></tr></thead><tbody><tr><td><strong>Tests</strong></td><td><details><summary>7
files</summary><table>
<tr>
<td><strong>pub_sub_improvement.result</strong><dd><code>Remove
duplicate database drop error messages</code>
</dd></td>
<td><a
href="https://github.com/matrixorigin/matrixone/pull/22095/files#diff-6715b97754ed50c80550019651625cd1116c4d0e70721d73af17ec7ade05a60f">+0/-2</a>
</td>
</tr>
<tr>
<td><strong>restore_dropped_account.result</strong><dd><code>Replace
error placeholders with actual query results</code>
</dd></td>
<td><a
href="https://github.com/matrixorigin/matrixone/pull/22095/files#diff-8418a455da56c4a8918f4163f549c5f83fab4a2b505bd3a26a18c4dba34f752c">+101/-121</a></td>
</tr>
<tr>
<td><strong>restore_dropped_account.sql</strong><dd><code>Fix session
IDs and remove issue markers</code>
</dd></td>
<td><a
href="https://github.com/matrixorigin/matrixone/pull/22095/files#diff-416c07871963f456057dc1ed8c9f9651ee2de6e617490646ecef78bf4ca08bab">+15/-19</a>
</td>
</tr>
<tr>
<td><strong>restore_pub_to_new.result</strong><dd><code>Update database
listing expectations after restore</code>
</dd></td>
<td><a
href="https://github.com/matrixorigin/matrixone/pull/22095/files#diff-2374d3095ff5490f152abd992a4996899c85c9e12e341fad2e960a08e0c712cb">+0/-2</a>
</td>
</tr>
<tr>
<td><strong>restore_pub_to_new.sql</strong><dd><code>Correct session ID
numbering sequence</code>
</dd></td>
<td><a
href="https://github.com/matrixorigin/matrixone/pull/22095/files#diff-e8e32a24f43273269abed002f2e8ea1a6c60df0f2980c043bd626c8733febe05">+4/-4</a>
</td>
</tr>
<tr>
<td><strong>grant_privs_role.result</strong><dd><code>Update role
privilege test expected outputs</code>
</dd></td>
<td><a
href="https://github.com/matrixorigin/matrixone/pull/22095/files#diff-3f3a3f3711bbf338daab744a0850691be546290786b88ceb24f8c8cc3b18c51b">+40/-31</a>
</td>
</tr>
<tr>
<td><strong>grant_privs_role.sql</strong><dd><code>Fix table creation
and role naming</code>
</dd></td>
<td><a
href="https://github.com/matrixorigin/matrixone/pull/22095/files#diff-070e5a012987611b57a8d475c74e90b11ee1ac2bea10e499ad592afdf1b77921">+4/-4</a>
</td>
</tr>
</table></details></td></tr></tr></tbody></table>
___
> <details> <summary> Need help?</summary><li>Type <code>/help how to
...</code> in the comments thread for any questions about Qodo Merge
usage.</li><li>Check out the <a
href="https://qodo-merge-docs.qodo.ai/usage-guide/">documentation</a>
for more information.</li></details>1 parent 67c7c0f commit 3ad6c16
File tree
7 files changed
+164
-183
lines changed- test/distributed/cases
- publication_subscription
- snapshot
- zz_accesscontrol
7 files changed
+164
-183
lines changedLines changed: 0 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
233 | 233 | | |
234 | 234 | | |
235 | 235 | | |
236 | | - | |
237 | 236 | | |
238 | | - | |
239 | 237 | | |
240 | 238 | | |
241 | 239 | | |
| |||
Lines changed: 101 additions & 121 deletions
Large diffs are not rendered by default.
Lines changed: 15 additions & 19 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
19 | | - | |
20 | | - | |
21 | | - | |
22 | 19 | | |
23 | 20 | | |
24 | 21 | | |
25 | | - | |
| 22 | + | |
26 | 23 | | |
27 | 24 | | |
28 | 25 | | |
| |||
222 | 219 | | |
223 | 220 | | |
224 | 221 | | |
225 | | - | |
226 | | - | |
227 | | - | |
228 | | - | |
229 | 222 | | |
230 | 223 | | |
231 | 224 | | |
| |||
363 | 356 | | |
364 | 357 | | |
365 | 358 | | |
366 | | - | |
| 359 | + | |
367 | 360 | | |
368 | 361 | | |
369 | 362 | | |
| |||
372 | 365 | | |
373 | 366 | | |
374 | 367 | | |
375 | | - | |
| 368 | + | |
376 | 369 | | |
377 | 370 | | |
378 | 371 | | |
379 | 372 | | |
380 | 373 | | |
381 | 374 | | |
382 | 375 | | |
383 | | - | |
| 376 | + | |
384 | 377 | | |
385 | 378 | | |
386 | 379 | | |
| |||
393 | 386 | | |
394 | 387 | | |
395 | 388 | | |
396 | | - | |
397 | | - | |
398 | 389 | | |
399 | 390 | | |
400 | 391 | | |
401 | | - | |
| 392 | + | |
402 | 393 | | |
403 | | - | |
404 | | - | |
| 394 | + | |
| 395 | + | |
405 | 396 | | |
406 | | - | |
| 397 | + | |
407 | 398 | | |
| 399 | + | |
| 400 | + | |
| 401 | + | |
408 | 402 | | |
409 | | - | |
410 | | - | |
| 403 | + | |
| 404 | + | |
| 405 | + | |
| 406 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
91 | | - | |
92 | | - | |
93 | 91 | | |
94 | 92 | | |
95 | 93 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
55 | 55 | | |
56 | 56 | | |
57 | 57 | | |
58 | | - | |
| 58 | + | |
59 | 59 | | |
60 | 60 | | |
61 | 61 | | |
| |||
64 | 64 | | |
65 | 65 | | |
66 | 66 | | |
67 | | - | |
| 67 | + | |
68 | 68 | | |
69 | 69 | | |
70 | 70 | | |
| |||
83 | 83 | | |
84 | 84 | | |
85 | 85 | | |
86 | | - | |
| 86 | + | |
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
91 | 91 | | |
92 | 92 | | |
93 | 93 | | |
94 | | - | |
| 94 | + | |
95 | 95 | | |
96 | 96 | | |
97 | 97 | | |
| |||
Lines changed: 40 additions & 31 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
264 | 264 | | |
265 | 265 | | |
266 | 266 | | |
267 | | - | |
268 | 267 | | |
| 268 | + | |
269 | 269 | | |
270 | | - | |
271 | 270 | | |
272 | | - | |
| 271 | + | |
| 272 | + | |
| 273 | + | |
| 274 | + | |
| 275 | + | |
273 | 276 | | |
274 | 277 | | |
275 | 278 | | |
| |||
279 | 282 | | |
280 | 283 | | |
281 | 284 | | |
282 | | - | |
283 | 285 | | |
284 | | - | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
285 | 289 | | |
286 | | - | |
287 | 290 | | |
288 | 291 | | |
289 | 292 | | |
| |||
334 | 337 | | |
335 | 338 | | |
336 | 339 | | |
337 | | - | |
338 | | - | |
| 340 | + | |
339 | 341 | | |
340 | 342 | | |
341 | 343 | | |
| |||
365 | 367 | | |
366 | 368 | | |
367 | 369 | | |
368 | | - | |
369 | | - | |
370 | | - | |
371 | | - | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
372 | 374 | | |
373 | 375 | | |
374 | 376 | | |
375 | 377 | | |
376 | 378 | | |
377 | 379 | | |
378 | | - | |
379 | | - | |
380 | | - | |
| 380 | + | |
381 | 381 | | |
382 | | - | |
383 | | - | |
384 | | - | |
385 | 382 | | |
386 | | - | |
387 | | - | |
388 | | - | |
389 | 383 | | |
| 384 | + | |
| 385 | + | |
| 386 | + | |
| 387 | + | |
390 | 388 | | |
| 389 | + | |
| 390 | + | |
| 391 | + | |
391 | 392 | | |
| 393 | + | |
| 394 | + | |
| 395 | + | |
392 | 396 | | |
393 | 397 | | |
394 | 398 | | |
| |||
400 | 404 | | |
401 | 405 | | |
402 | 406 | | |
| 407 | + | |
403 | 408 | | |
404 | 409 | | |
405 | 410 | | |
| |||
411 | 416 | | |
412 | 417 | | |
413 | 418 | | |
| 419 | + | |
414 | 420 | | |
415 | 421 | | |
416 | 422 | | |
417 | 423 | | |
| 424 | + | |
418 | 425 | | |
419 | 426 | | |
420 | 427 | | |
| |||
431 | 438 | | |
432 | 439 | | |
433 | 440 | | |
434 | | - | |
435 | | - | |
436 | | - | |
| 441 | + | |
437 | 442 | | |
438 | | - | |
439 | | - | |
440 | | - | |
441 | 443 | | |
442 | | - | |
443 | | - | |
444 | | - | |
445 | 444 | | |
| 445 | + | |
| 446 | + | |
| 447 | + | |
| 448 | + | |
446 | 449 | | |
| 450 | + | |
| 451 | + | |
| 452 | + | |
447 | 453 | | |
| 454 | + | |
| 455 | + | |
| 456 | + | |
448 | 457 | | |
449 | 458 | | |
450 | | - | |
| 459 | + | |
451 | 460 | | |
452 | 461 | | |
453 | 462 | | |
| |||
Lines changed: 4 additions & 4 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
253 | 253 | | |
254 | 254 | | |
255 | 255 | | |
256 | | - | |
| 256 | + | |
257 | 257 | | |
258 | 258 | | |
259 | 259 | | |
260 | 260 | | |
261 | 261 | | |
262 | 262 | | |
263 | 263 | | |
264 | | - | |
265 | | - | |
| 264 | + | |
| 265 | + | |
266 | 266 | | |
267 | 267 | | |
268 | 268 | | |
| |||
288 | 288 | | |
289 | 289 | | |
290 | 290 | | |
291 | | - | |
| 291 | + | |
292 | 292 | | |
293 | 293 | | |
294 | 294 | | |
| |||
0 commit comments