Испавление чтения настроек json на платформенное, если возможно.

[pumbaEO]

Привел файл настроек к стандартам json.

@silverbulleters/collaborator - просьба прокомментировать и проверить

[pumbaEO]

Пошла сборка https://travis-ci.org/silverbulleters/add/jobs/369024697

[artbear]

Тревис стал зеленым!
УРА!
Сейчас сделаю код-ревью

[artbear]

@pumbaEO Правильно я понимаю, что сейчас сломана совместимость со старыми корявыми VBParams.json от VB ?

[pumbaEO]

Да, как оно до этого работало нормально, даже не представляю. Любое форматирование могло убить парсинг в самый неожиданный момент.

[artbear]

@pumbaEO Хочу понять, что получат пользователи, переходящие на ADD.

А сейчас какое поведение в случае получения "старых" VBParams.json ?

БДД-раннер выдаст ошибку и завершится или все-таки будет то самое исключение с модальным окном?

[pumbaEO]

Исключение. Но лучше исключение, чем загрузка части настроек и последующее успешное выполнение пустого набора тестов.

[artbear]

Исключение. Но лучше исключение, чем загрузка части настроек и последующее успешное выполнение пустого набора тестов.

сейчас проблема в том, что зависнет выполнение фич на CI :(

Может быть, при запуске через ком.строку добавим

• перехват исключения
• вывод его во все логи, которые уже подключены - ЖР, 1С, лог-файл
• завершение работы 1С

?

тогда

• и сами не получим проблемы с зависанием
• и пользователи продукта не получат неожиданные проблемы при переходе на ADD
• и общее первичное впечатление о продукте не снизится :)

[artbear]

@pumbaEO Если нет возражений, я могу сделать вечером или в выходные или ты сам сделаешь?

[artbear]

у нас уже первые пользователи просят новый релиз :)
т.к. есть отсутствующие шаги из #112