Skip to content

Тест: проверка наследования настроек аппендеров вложенными логгерами + исправление кодировки#47

Open
Copilot wants to merge 4 commits intomasterfrom
copilot/check-logger-settings
Open

Тест: проверка наследования настроек аппендеров вложенными логгерами + исправление кодировки#47
Copilot wants to merge 4 commits intomasterfrom
copilot/check-logger-settings

Conversation

Copy link

Copilot AI commented Mar 12, 2026

Existing hierarchy test (Тест_ДолженПроверитьПриемственностьИерархииЛогов) only validates that a single appender is inherited by dot-notation child loggers. It doesn't verify that per-appender level settings propagate correctly.

  • Added Тест_ДолженПроверитьПередачуНастроекАппендеровВИерархииЛогов — configures parentlog with two file appenders at different levels (DEBUG and ERROR), then creates parentlog.childlog and asserts:
    • DEBUG appender receives both messages (debug + error)
    • ERROR appender receives only the error message
logger.parentlog=DEBUG, fileDebug, fileError
appender.fileDebug.level=DEBUG
appender.fileError.level=ERROR

This confirms НайтиНастройкуВышеПоИерахии passes the full config (appender definitions + their individual levels) to child loggers, not just the log level.

  • Fixed encoding issue on Windows: file appender writes in UTF-8, but ЧтениеТекста without explicit encoding defaults to the system codepage (Windows-1251), causing Cyrillic text comparison failures. Added explicit "UTF-8" parameter to all Новый ЧтениеТекста(...) calls in file-reading tests (Тест_ДолженПроверитьВыводДвухЛоговВОДинФайл, Тест_ДолженПроверитьПриемственностьИерархииЛогов, and the new test).

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

…точку) логгерам

Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Copilot AI changed the title [WIP] Verify logger configurations for nested loggers Тест: проверка наследования настроек аппендеров вложенными логгерами Mar 12, 2026
Copilot AI requested a review from nixel2007 March 12, 2026 17:57
@nixel2007 nixel2007 marked this pull request as ready for review March 12, 2026 18:01
Copilot AI review requested due to automatic review settings March 12, 2026 18:01
@nixel2007
Copy link
Member

@copilot тест под виндой падает из-за кодировки.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

PR расширяет покрытие тестами иерархического наследования настроек логгеров: теперь проверяется, что вложенные (dot-notation) логгеры наследуют не только общий уровень, но и настройки отдельных аппендеров (включая их уровни).

Changes:

  • Добавлен тест Тест_ДолженПроверитьПередачуНастроекАппендеровВИерархииЛогов, который настраивает два file-аппендера на разных уровнях (DEBUG/ERROR) у родительского логгера и проверяет корректную фильтрацию сообщений у дочернего.
  • Тест добавлен в список выполняемых тестов.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Author

Copilot AI commented Mar 12, 2026

@copilot тест под виндой падает из-за кодировки.

Исправлено в 5d4dfe8. Файловый аппендер пишет в UTF-8, но ЧтениеТекста без указания кодировки на Windows использует системную кодировку (Windows-1251), из-за чего кириллица читалась как ОТЛАР"РљРђ вместо ОТЛАДКА. Добавил явное указание "UTF-8" во все 4 вызова Новый ЧтениеТекста(...) в трёх тестах (мой новый + два существующих, которые также падали).

Copilot AI changed the title Тест: проверка наследования настроек аппендеров вложенными логгерами Тест: проверка наследования настроек аппендеров вложенными логгерами + исправление кодировки Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants