Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Не работает xunit из состава ADD в обычном приложении на УПП 1.3.104 и старше #405

Closed
Begemoth2 opened this issue Feb 6, 2019 · 8 comments
Milestone

Comments

@Begemoth2
Copy link

При запуске обработки xddTestRunner в обычном приложении в УПП 1.3.104.2 ошибка
{ВнешняяОбработка.СтроковыеУтилиты.МодульОбъекта(952,9)}: Процедура или функция с указанным именем уже определена (ФорматированнаяСтрока)
Функция <<?>>ФорматированнаяСтрока(Знач Строка) Экспорт (Проверка: Толстый клиент (обычное приложение))

Причина в том, что в УПП начиная с определенной версии добавили Глобальный общий модуль ИнтеграцияЕГАИСУТКлиентСерверГлобальный где есть функция ФорматированнаяСтрока.

Как следствие не можем использовать тесты на более-менее свежих версиях УПП.

В качестве способа обхода в нашем форке ADD в плагине СтроковыеУтилиты обрамили функцию ФорматированнаяСтрока в секцию
#Если ТолстыйКлиентОбычноеПриложение Тогда
#Иначе
чтобы она не компилировалась в обычном приложении

@Begemoth2
Copy link
Author

Примите ли наш вариант исправления как пулреквест?

@artbear
Copy link
Collaborator

artbear commented Feb 6, 2019

Да, подобное исправление примем.

@artbear artbear added the tdd label Feb 6, 2019
@sfaqer
Copy link
Contributor

sfaqer commented Feb 7, 2019

@artbear а чем этот ишуз принципиально отличается от #322 ? Ну т.е. там отклонено с обоснованием "нельзя подгонять фреймворк под конкретную конфу", а тут уже можно?)

@artbear
Copy link
Collaborator

artbear commented Feb 8, 2019

@sfaqer Хороший вопрос, спасибо.
в обоих случаях мы "допилили фреймворк" под конкретные случаи и это не потребовало сильных изменений в проекте.

@sfaqer
Copy link
Contributor

sfaqer commented Feb 11, 2019

@artbear, беглый поиск по проекту показывает что данный функционал нигде не используется (кроме тестов которые проверяют работу этого функционала), так что изменение предложенное выше не повлияет на работоспособность проекта, однако пользовательские тест, которые используют данный функционал, полетят, в случае если тестируемая конфигурация на обычных формах, и это не актуальная УПП. Всё таки УПП не единственная конфигурация на ОФ, да и далеко не у всех обновляется.

@sfaqer
Copy link
Contributor

sfaqer commented Feb 12, 2019

@artbear, блин, только сейчас обратил внимание что в моём позапрошлом комменте гитхаб вставил ссылку на ПР а не на ишуз, под "322" понимался вот этот ишуз: #332

@artbear
Copy link
Collaborator

artbear commented Feb 12, 2019

@sfaqer @yadimetrio Да, #322 и текущий ишуз имеют много общего.

В сентябре у меня было другое мнение, сейчас оно изменилось.

Мотивация такая - на обычных формах юзаются фактически только "старые и заслуженные монстры" (УПП, КА и какие-то самописки), все остальные давно на упр.формах и на новых типовых.

В новых конфигурациях нет проблем с ФорматированнаяСтрока.
Поэтому для пользователей в упр.режиме ничего не поменяется,
а для пользователей обычного режима можно пойти навстречу и дать возможность юзать и свою конфигурацию, и наш фреймворк.

@artbear
Copy link
Collaborator

artbear commented Apr 23, 2019

Выполнено в #461

@artbear artbear closed this as completed Apr 23, 2019
@artbear artbear added this to the 5.9.0 milestone Apr 23, 2019
@artbear artbear added the smoke label Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants