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

Ошибка при открытии формы групп справочников, нужно при открытии формы группы передавать параметр ЭтоГруппа = Истина #675

Open
git22ura opened this issue Jan 25, 2017 · 12 comments

Comments

@git22ura
Copy link

Версия 4.1.0.1

Как интерпретировать результаты дымового теста. На примере конфы Управление холдингом (хотя не суть важно)
Допустим, у меня 2 из 20 тестов упали.
Начинаю разбор.

Новые : Справочник.Банки.Форма.ФормаГруппы, путь УпрФорма ....тесты_ОткрытиеФормКонфигурации.epf

"{ОбщийМодуль.РаботаСБанкамиБП.Модуль(1238)}: Поле объекта не обнаружено (КоррСчет)
Элементы.КоррСчет.Доступность = МожноРедактировать;"
Кусок кода:

Если НЕ Форма.Объект.ЭтоГруппа Тогда
Элементы.КоррСчет.Доступность = МожноРедактировать;
КонецЕсли;

Начинаю руками открывать форму группы, вижу что все открывается корректно. Дебажу. Вижу, что дымовой тест открывает форму с Параметры.ЭтоГруппа = Ложь, а надо с Истина.

Вручную прогнать дымовой тест и убедиться дополнительно, что такие формы совсем не ошибка - не проблема.
Но поскольку тесты будут прогоняться на дженкинсе, стоит задача чтобы тесты был зелеными полностью.

Как поправить ситуацию? Какой правильный путь? Пытаться вставить параметры в этот тест или добавлять как-то тест в исключение, писать свой?

@nixel2007
Copy link
Member

Похоже на баг

@artbear
Copy link
Member

artbear commented Jan 25, 2017

@git22ura Т.е у тебя падает тест открытия формы для новой группы, верно?
типовая конфа? если да, укажи ее название.

@git22ura
Copy link
Author

@artbear, верно.
Платформа: 1С:Предприятие 8.3 (8.3.9.1850)
Конфигурация: Управление холдингом, редакция 1.3 (1.3.4.25)
Режим: Серверный (сжатие: усиленное)
Приложение: Тонкий клиент
Вариант интерфейса: Такси

@git22ura
Copy link
Author

Попробовал на демо бухгалтерии - тоже самое. Бухгалтерия предприятия (демо) 3.0.42.85

@git22ura
Copy link
Author

Но меня больше волнует вытекающий вопрос:

Как поправить ситуацию? Какой правильный путь? Пытаться вставить параметры в этот тест или добавлять как-то тест в исключение, писать свой?

@git22ura
Copy link
Author

Для примера есть правильное падение теста открытия формы:

{ВнешняяОбработка.БазовыеУтверждения.Форма.Форма.Форма(327)}: [Failed] Переданный параметр (ложь) не является Истиной, а хотели, чтобы являлся.
ТестируемаяФорма "Справочник.ВариантыОтчетов.Форма.ФормаЭлемента" не открылась, а должна была открыться
ВызватьИсключение Префикс + " " + СообщениеОшибки;

Но я не хочу обращать внимание на это в дальнейших автопроверках.

@Ndochp
Copy link

Ndochp commented Jan 25, 2017

Тут ИМХО два разных момента:

  1. При открытии формы группы должен стоять параметр ЭтоГруппа = Истина
  2. Дымовой тест на произвольной конфигурации будет падать на части форм и это не будет ошибкой. Соответственно для конфигурации нужно вести список исключений. Логично их втыкать в генерацию шагов, то есть дымовой дымовым, но некоторые формы не проверяем. Кажется это было предусмотрено где-то внутри теста в третьей версии, четвертую не смотрел.

@artbear
Copy link
Member

artbear commented Jan 25, 2017

В коде теста есть спец.функция "Исключения_ХХХ", в которых можно прописать исключения. Просто добавить имена метаданных (например, "ВариантыОтчетов")

@artbear artbear added this to the 4.1.0.0 milestone Jan 25, 2017
@git22ura
Copy link
Author

@artbear
Не нашел указанной функции (и даже по слову "Исключения") в тесте.

@artbear
Copy link
Member

artbear commented Jan 26, 2017

Посмотри Функция ПолучитьСписокИсключений_ХХХ

@git22ura
Copy link
Author

Спасибо

@artbear
Copy link
Member

artbear commented Mar 26, 2017

Появилась возможность внешней настройки через json-файл, а не через доработку модуля объекта теста
Смотреть #686
Уже в ветке develop

@artbear artbear changed the title Интерпретация дымового теста или как заставить тест позеленеть Ошибка при открытии формы групп справочников, нужно при открытии формы группы передавать параметр ЭтоГруппа = Истина Mar 26, 2017
@artbear artbear added bug and removed question labels Mar 26, 2017
@artbear artbear modified the milestones: 4.2.0.0, 4.3.0.0 Jul 5, 2017
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

4 participants