xUnitFor1C icon indicating copy to clipboard operation
xUnitFor1C copied to clipboard

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

Open git22ura opened this issue 8 years ago • 12 comments

Версия 4.1.0.1

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

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

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

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

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

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

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

git22ura avatar Jan 25 '17 08:01 git22ura

Похоже на баг

nixel2007 avatar Jan 25 '17 08:01 nixel2007

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

artbear avatar Jan 25 '17 09:01 artbear

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

git22ura avatar Jan 25 '17 10:01 git22ura

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

git22ura avatar Jan 25 '17 10:01 git22ura

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

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

git22ura avatar Jan 25 '17 11:01 git22ura

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

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

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

git22ura avatar Jan 25 '17 11:01 git22ura

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

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

Ndochp avatar Jan 25 '17 11:01 Ndochp

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

artbear avatar Jan 25 '17 12:01 artbear

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

git22ura avatar Jan 26 '17 10:01 git22ura

Спасибо

git22ura avatar Jan 27 '17 09:01 git22ura

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

artbear avatar Mar 26 '17 18:03 artbear