Commit 4a40ac0
committed
fix(tests): Use record.message instead of record.msg in log assertions
Root cause: When we converted f-strings to %s format strings in logger.warning(),
the log record behavior changed:
- f-strings: record.msg contains the fully formatted string
- %s format: record.msg contains the template with %s placeholders
record.message contains the formatted result
The test was checking record.msg which now contains the unformatted template
like 'Cannot set environment for new %s.' instead of the formatted message
like 'Cannot set environment for new windows.'
Fix: Changed assertions to use record.message (the proper way to access
formatted log messages) instead of record.msg.1 parent fea0198 commit 4a40ac0
1 file changed
+3
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
457 | 457 | | |
458 | 458 | | |
459 | 459 | | |
460 | | - | |
| 460 | + | |
461 | 461 | | |
462 | 462 | | |
463 | 463 | | |
| |||
467 | 467 | | |
468 | 468 | | |
469 | 469 | | |
470 | | - | |
| 470 | + | |
471 | 471 | | |
472 | 472 | | |
473 | 473 | | |
| |||
476 | 476 | | |
477 | 477 | | |
478 | 478 | | |
479 | | - | |
| 479 | + | |
480 | 480 | | |
481 | 481 | | |
482 | 482 | | |
| |||
0 commit comments