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

Open
Begemoth2 opened this Issue Feb 6, 2019 · 7 comments

Comments

Projects
None yet
3 participants
@Begemoth2
Copy link

Begemoth2 commented Feb 6, 2019

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

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

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

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

@Begemoth2

This comment has been minimized.

Copy link
Author

Begemoth2 commented Feb 6, 2019

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

@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Feb 6, 2019

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

@artbear artbear added the tdd label Feb 6, 2019

@sfaqer

This comment has been minimized.

Copy link
Contributor

sfaqer commented Feb 7, 2019

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

@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Feb 8, 2019

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

@sfaqer

This comment has been minimized.

Copy link
Contributor

sfaqer commented Feb 11, 2019

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

@sfaqer

This comment has been minimized.

Copy link
Contributor

sfaqer commented Feb 12, 2019

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

@artbear

This comment has been minimized.

Copy link
Collaborator

artbear commented Feb 12, 2019

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

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment