Падают тесты из ветки develop из-за проблем с доступом к лог-файлам

[artbear]

Упали тесты. Количество 4 шт :
    тест №1 : Тест_ДолженРазложитьКаталогСВложеннымиКаталогами : путь файла <C:\projects\precommit1c\tests\v8files-extractor-test.os>
    тест №2 : Тест_ДолженРазобратьФайлыПоЖурналуИзмененийГит : путь файла <C:\projects\precommit1c\tests\v8files-extractor-test.os>
    тест №3 : Тест_ДолженПроверитьНастройкиРепозитарияГит : путь файла <C:\projects\precommit1c\tests\v8files-extractor-test.os>
    тест №4 : Тест_ДолженСобратьФайлОбработкиИзЗаданнойПапки : путь файла <C:\projects\precommit1c\tests\v8files-extractor-test.os>

ОШИБКА: Упал тест <Тест_ДолженСобратьФайлОбработкиИзЗаданнойПапки>, файл <C:\projects\precommit1c\tests\v8files-extractor-test.os>:
    {Модуль C:\Program Files (x86)\OneScript\lib\logos\src\file-appender.os / Ошибка в строке: 10 / Внешнее исключение: Процесс не может получить доступ к файлу "C:\projects\precommit1c\v8files-extractor.log", так как этот файл используется
 другим процессом.}
        мФайлЛога = Новый ЗаписьТекста(Путь, Кодировка,,Добавлять);

Похоже, что проблема при одновременной записи в файл порождается в файле oscript-library\src\logos\src\file-appender.os, т.к. разные экземпляры класса ВыводЛогаВФайл пишут в один тот же файл, что является ошибкой.

[artbear]

Баг обнаружен в #84

[artbear]

@EvilBeaver провел расследование.
Похоже, что баг из-за отсутствия вызова виртуального деструктора при удалении способов вывода, из-за чего файл остается залоченным.

Есть код добавления новых способов вывода

Аппендер = Новый ВыводЛогаВФайл();
Аппендер.ОткрытьФайл(ОбъединитьПути(КаталогПроекта(), ИмяСкрипта()+".cmdline.log"));
Лог_cmdline.ДобавитьСпособВывода(Аппендер);

смотрю метод Лог_cmdline.ДобавитьСпособВывода(Аппендер);

Процедура ДобавитьСпособВывода(Знач СпособВывода) Экспорт

    Если Не мСпособВыводаЗаданВручную Тогда
        мСпособыВывода.Очистить();// !!!
        мСпособВыводаЗаданВручную = Истина;
    КонецЕсли;

    мСпособыВывода.Добавить(СпособВывода);

КонецПроцедуры

Т.е. старые способы вывода мы просто очищаем.
Но у наших способов вывода есть виртуальный деструктор ( метод Закрыть )
А он не вызывается, вот и получаем баг с залоченным файлом :(

Исправление:

• вместо мСпособыВывода.Очистить(); нужно вызывать метод Закрыть(), который и вызывает закрытие.
• в метод УдалитьСпособВывода также добавить вызов закрытия способа вывода
• нужно документировать интерфейс способов вывода, сейчас этого нет.

[artbear]

В oscript-library также пофиксил
EvilBeaver/oscript-library@2a5ca6e