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

fix smoketest Тесты_ПроверкаРеквизитовДокументовКомментарийОтветственный #816

Conversation

SidnevAV
Copy link
Contributor

@SidnevAV SidnevAV commented Nov 15, 2020

Исправления по тесту Тесты_ПроверкаРеквизитовДокументовКомментарийОтветственный

сделанные изменения:

  1. Оптимизация работы теста
  2. Добавлены новые настройки:
    • "Используется",
    • "ВыводитьИсключения",
    • "ПропускатьОбъектыСПрефиксомУдалить",
    • "ОпределяемыйТипКомментарий": Имя определяемого типа "Комментарий" для проверки типа реквизита "Комментарий" документов,
    • "ОпределяемыйТипОтветственный": Имя определяемого типа "Ответственный" для проверки типа реквизита "Ответственный" документов,

@vanessa-opensource/Collaborators - просьба прокомментировать и проверить

@artbear artbear added the smoke label Nov 16, 2020
@artbear artbear added this to the 6.7.0 milestone Nov 16, 2020
@artbear
Copy link
Collaborator

artbear commented Nov 16, 2020

@SidnevAV я вмержил твои предыдущие ПР, и теперь конфликты в файле tools/JSON/xUnitParams.json

поправь их.

Copy link
Collaborator

@artbear artbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

есть предложения

Copy link
Collaborator

@artbear artbear left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Предлагаю немного уточнить поведения для избежания ложных срабатываний.

Если РеквизитКомментарий = Неопределено Тогда
ШаблонСообщения = НСтр("ru = '%1%2 не указан реквизит ""Комментарий""'");
ДобавитьСообщениеВРезультат(СтроковыеУтилиты, Результат, ШаблонСообщения)
КонецЕсли;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Предлагаю сразу пропускать тест, если у документа нет реквизита Комментарий

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

не во всех документах есть этот реквизит.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

готово

Если РеквизитОтветственный = Неопределено Тогда
ШаблонСообщения = НСтр("ru = '%1%2 не указан реквизит ""Ответственный""'");
ДобавитьСообщениеВРезультат(СтроковыеУтилиты, Результат, ШаблонСообщения)
КонецЕсли;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

аналогично пропускать по реквизиту Ответственный.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

готово

@artbear
Copy link
Collaborator

artbear commented Nov 30, 2020

смержил твой же соседний ПР #814

и опять конфликты все в том же tools/JSON/xUnitParams.json

поправишь?

@artbear artbear merged commit 589773c into vanessa-opensource:develop Dec 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants