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

Добавить возможность исключения планов обмена из теста командного интерфейса #746

Closed
GladilovDmitry opened this issue May 19, 2020 · 7 comments · Fixed by #791
Labels
Milestone

Comments

@GladilovDmitry
Copy link
Contributor

Существует проблема
В ЕРПУХ при тестировании командного интерфейса получаем ошибки в большом числе тестов из-за модального окна получаемого при закрытии формы настройки обмена из плана обмена НалоговыйМониторинг: Налоговый мониторинг->Публикация данных налогового мониторинга: проверка элемента командного интерфейса.

Хотелось бы иметь следующую функциональность
поддерживаемую секцию "Планы обмена" в vanessaTddConf.json

@artbear
Copy link
Collaborator

artbear commented May 19, 2020

@GladilovDmitry @VladFrost а может быть, научиться такое модальное окно научиться закрывать?
исключения все-таки большое зло.

@GladilovDmitry можешь привести данные по этому модальному окну?
из бдд-режима и Исследователя форм, например?
текст заголовка, имя формы, текст предупреждения, наличие кнопок и т.п.

@artbear artbear added the smoke label May 19, 2020
@VladFrost
Copy link
Contributor

Исключения – это зло, но это неизбежное зло. Лучше всего, конечно, баг вендору зарегить, и сослаться на стандарт, который нарушен в модальном диалоге. Стандарты опубликованы https://its.1c.ru/db/v8std

  1. Типовые исключения можно прям в коде зашивать, как здесь https://github.com/vanessa-opensource/add/blob/master/tests/smoke/тесты_КомандныйИнтерфейс/тесты_КомандныйИнтерфейс/Forms/Форма/Ext/Form/Module.bsl#L271

  2. В плагине "ТестКлиенты" есть алгоритм, который нажимает подходящую кнопку, можно его попробовать допилить, чтобы и в случае с налоговым мониторингом срабатывал https://github.com/vanessa-opensource/add/blob/master/plugins/ТестКлиенты/ТестКлиенты/Forms/Форма/Ext/Form/Module.bsl#L317

@artbear artbear added this to the 6.7.0 milestone May 19, 2020
@GladilovDmitry
Copy link
Contributor Author

GladilovDmitry commented May 19, 2020

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

Так стандарт-то вроде не нарушен. Модальный диалог выводится при попытке закрытия формы, т.к. (в вольном изложении) "настройка обмена не завершена. Вы уверены что хотите закрыть?". Или я не нашел нарушенный стандарт?

@GladilovDmitry
Copy link
Contributor Author

@GladilovDmitry можешь привести данные по этому модальному окну?
из бдд-режима и Исследователя форм, например?
текст заголовка, имя формы, текст предупреждения, наличие кнопок и т.п.

Безымянный
Безымянный2
Screenshot_1

@GladilovDmitry
Copy link
Contributor Author

Типовые исключения можно прям в коде зашивать, как здесь https://github.com/vanessa-opensource/add/blob/master/tests/smoke/тесты_КомандныйИнтерфейс/тесты_КомандныйИнтерфейс/Forms/Форма/Ext/Form/Module.bsl#L271

Можно-то можно, да только для каждой конфы они разные. Писать вообще все возможные?
Вообще мне не нравится зашивать что-то жестко в код, т.к. тогда нельзя просто "воткнуть"новую версию vanessa, надо где-то хранить свои версии обработок, мержить их. Брр.
Лучше хранить их в том же vanessaTddConf.json. Про что по-сути и идет речь в данном issue. Кстати я бы и уже зашитые исключения вынес в поставляемый vanessaTddConf.json и удалил из кода обработки.

@artbear
Copy link
Collaborator

artbear commented May 19, 2020

вроде бы в 1С-конфах есть типовой вопрос "Завершить работу с помощником?"
в демке БСП часто вижу его.

может быть, нам добавить обработку этого модального окна?

@VladFrost
Copy link
Contributor

может быть, нам добавить обработку этого модального окна?

Да, хороший вариант.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants