Дымовые тесты, в чем необходимость пропуска для УФ #617

pumbaEO · 2016-04-20T09:58:33Z

В чем необходимость пропуска управляемых форм?
https://github.com/xDrivenDevelopment/xUnitFor1C/blob/develop/src/Tests/Smoke/%D1%82%D0%B5%D1%81%D1%82%D1%8B_%D0%9E%D1%82%D0%BA%D1%80%D1%8B%D1%82%D0%B8%D0%B5%D0%A4%D0%BE%D1%80%D0%BC%D0%9A%D0%BE%D0%BD%D1%84%D0%B8%D0%B3%D1%83%D1%80%D0%B0%D1%86%D0%B8%D0%B8/ObjectModule.bsl#L296

artbear · 2016-04-20T10:13:15Z

@pumbaEO
Там сверху написано
ИспользоватьУФ = Метаданные.ИспользоватьУправляемыеФормыВОбычномПриложении;
Соответственно, если этот флаг в конфигурации выключен, УФ нет смысла открывать, т.к. они все равно не откроются или откроются пустыми (без выполнения обработчиков)

ИспользоватьУФ = Метаданные.ИспользоватьУправляемыеФормыВОбычномПриложении; // !!
    МетаФормы = Новый Массив();
    Для Каждого ИмяОсновнойФормы Из ИменаОсновныхФорм Цикл
        Попытка
            МетаФорма = МетаОбъект[ИмяОсновнойФормы];
        Исключение
            Продолжить;
        КонецПопытки;
        Если МетаФорма = Неопределено Тогда
            Продолжить;
        КонецЕсли; 
        Если МетаФормы.Найти(МетаФорма) <> Неопределено Тогда
            Продолжить;
        КонецЕсли; 
        Если НЕ ИспользоватьУФ И МетаФорма.ТипФормы = Метаданные.СвойстваОбъектов.ТипФормы.Управляемая Тогда
            Продолжить;
        КонецЕсли;
        МетаФормы.Добавить(МетаФорма);
    КонецЦикла;
    Возврат МетаФормы;```

pumbaEO · 2016-04-20T11:14:53Z

В зависимости от режима совместимости этой настройки в новых версиях нет http://prntscr.com/aujqnl

Irekovich · 2016-04-20T11:20:46Z

А так?
http://uploads.ru/e75ZV.png

pumbaEO · 2016-04-20T12:01:04Z

Да, так показывает. Но это не объясняет почему он откроются пустыми. Т.к. для той же ERP эти флаги стоят и в управляемом режиме все открывается и запускается.
http://prntscr.com/aukc5i

artbear · 2016-04-20T13:49:48Z

@pumbaEO

Но это не объясняет почему он откроются пустыми. Т.к. для той же ERP эти флаги стоят и в управляемом режиме все открывается и запускается.

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

Или я что-то не знаю?

artbear · 2016-04-20T18:57:34Z

@pumbaEO Вопрос решен, закрываем?

naa4sb · 2016-06-06T13:57:13Z

ИспользоватьУФ = Ложь
ТекущийРежимЗапуска() = Управляемое приложение
настройки конфигурации такие
http://take.ms/ZFMQa

Из-за этого все управляемые формы пропускаются:

        Если НЕ ИспользоватьУФ И МетаФорма.ТипФормы = Метаданные.СвойстваОбъектов.ТипФормы.Управляемая Тогда
            Продолжить;
        КонецЕсли;

Или я что-то не так запускаю?

artbear · 2016-06-06T17:12:44Z

@naa4sb Ага, теперь понял проблему. Подумаю, как поправить

naa4sb · 2016-06-06T17:59:03Z

@artbear пока вот так работает,
ИспользоватьУФ = Метаданные.ИспользоватьУправляемыеФормыВОбычномПриложении ИЛИ (ТекущийРежимЗапуска() <> РежимЗапускаКлиентскогоПриложения.ОбычноеПриложение);

artbear · 2016-06-06T18:15:31Z

@naa4sb Сможешь пулл-реквест сделать?

das999 · 2016-06-06T18:47:12Z

я - нуб, но попробую) тестов вот только не обещаю, хотя теоретически в
новых версиях платформы поведение метода ТекущийРежимЗапуска() может
измениться, и это надо тестить.

2016-06-06 21:15 GMT+03:00 Artur Ayukhanov [email protected]:

@naa4sb https://github.com/naa4sb Сможешь пулл-реквест сделать?

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#617 (comment),
or mute the thread
https://github.com/notifications/unsubscribe/AQqbQ_BbtJ3AQaGB5ZUD1fO2AkMlvxwFks5qJGPGgaJpZM4ILiDc
.

naa4sb · 2016-06-08T09:07:21Z

@artbear Немножко вопросов)

Прогнал тесты на cf до внесения своих изменений
Запущено тестов - 288, ошибочных тестов - 0, не прошло проверку - 0, пропущено - 1
ТестДолжен_ПроверитьРавенствоФайловСУчетомПереданныхПараметровСравнения, путь C:\GIT\naa4sb\xUnitFor1C\Tests\Plugins\ТестыПроверитьРавенствоФайлов.epf
Отсутствует реализация тестового метода (ТестДолжен_ПроверитьРавенствоФайловСУчетомПереданныхПараметровСравнения).
Не обращать внимания?
Надо ли менять минорную версию xddTestRunner с учетом того, что изменяется совсем другая epf? Или версия xddTestRunner - это версия всего продукта в целом, а не конкретно обработки xddTestRunner, и ее надо менять, если меняется какая-то часть продукта?

artbear · 2016-06-08T10:56:19Z

@naa4sb

Да, ТестДолженПроверитьРавенствоФайловСУчетомПереданныхПараметровСравнения) пропускаем.
версия браузере - это как версия всего продукта. менять не нужно.
ты сохранял файлы только в 8.3 или в 8.2 ?
Тесты прогонял на УФ (тонкий и толстый) и ОФ ? :)

artbear · 2016-06-08T10:59:57Z

Командой git diff develop YPermitin/develop-ypermitin --name-only *.bsl
смотрим все изменения модулей между твоей и общей веткой
все остальные файлы (внешние обработки и исходники) нужно откатить

src/Plugins/ГенераторОтчетаMXL/ObjectModule.bsl
src/Plugins/ЗагрузчикИзПодсистемКонфигурации/Form/УправляемаяФорма/Форма.bsl
src/Plugins/ЗагрузчикКаталога/ObjectModule.bsl
src/Plugins/ЗагрузчикФайла/Form/Форма/Форма.bsl
src/Plugins/ЗагрузчикФайла/ObjectModule.bsl
src/Plugins/ПарсерКоманднойСтроки/ObjectModule.bsl
src/Plugins/СтроковыеУтилиты/ObjectModule.bsl
src/PluginsTemplates/ШаблонГенератораОтчета/Form/Форма/Форма.bsl
src/PluginsTemplates/ШаблонГенератораОтчета/ObjectModule.bsl
src/PluginsTemplates/ШаблонЗагрузчика/Form/Форма/Форма.bsl
src/PluginsTemplates/ШаблонЗагрузчика/ObjectModule.bsl
src/PluginsTemplates/ШаблонУтилиты/Form/Форма/Форма.bsl
src/PluginsTemplates/ШаблонУтилиты/ObjectModule.bsl
src/Tests/Core/Тесты_ЗапускВПакетномРежиме/ObjectModule.bsl
src/Tests/Core/Тесты_СистемаПлагинов/Form/Форма/Форма.bsl
src/Tests/Core/Тесты_СистемаПлагинов/ObjectModule.bsl
src/Tests/Gui83/юнитТест_UI_83/Form/Форма/Форма.bsl
src/Tests/Plugins/Тесты_ГенераторОтчетаJUnitXML/ObjectModule.bsl
src/Tests/Plugins/Тесты_ГенераторОтчетаMXL/ObjectModule.bsl
src/Tests/Plugins/Тесты_ЗагрузчикФайла/Form/Форма/Форма.bsl
src/Tests/Plugins/Тесты_ЗагрузчикФайла/ObjectModule.bsl
src/Tests/Utils/Тест_КонвертацияТестов_в_версию4/ObjectModule.bsl
src/Utils/xddTestsConvertIntoRebornFormat/ObjectModule.bsl
src/xddTestRunner/Form/УправляемаяФорма/Форма.bsl
src/xddTestRunner/Form/Форма/module.bsl
src/xddTestRunner/ObjectModule.bsl

artbear · 2016-06-08T11:00:38Z

А с этими файлами нужно разобраться и устранить ненужные изменения.

naa4sb · 2016-06-08T13:34:15Z

@artbear Это не мои изменения, а Юры Пермитина)
По моим получается строчка кода изменений в дымовом, если версию в браузере тестов не надо менять.
С тестами каюсь, схалтурил, ибо руками пока) Только 8.3.8, тонкий и УФ. Прогоню по остальным)
Вопрос: имеет значение в какой версии платформы сохраняется измененный бинарник (epf)? Если имеет, то в какой надо сохранять? Сейчас была 8.3.8.

nixel2007 · 2016-06-08T13:58:57Z

@naa4sb

Вопрос: имеет значение в какой версии платформы сохраняется измененный бинарник (epf)? Если имеет, то в какой надо сохранять? Сейчас была 8.3.8.

Да, имеет. Надо сохранять на 8.2, так как 8.3 сохраняет картинки в обычных формах не так, как было до 8.2.19. Заявленная же поддержка начинается от 8.2.14

rsyuzyov · 2016-06-08T14:58:40Z

@nixel2007 Извините, дяденьки, что встреваю, но

И имхо пора поднять хотя бы до 8.2.19...

nixel2007 · 2016-06-08T15:09:02Z

@rsyuzyov да ладно уж, дяденька...)

Это вопрос к @artbear

На мой взгляд, если люди, сидящие на 8.2 (sic!) до сих пор не перешли на 8.2.19 (sic!), то им можно вообще ничего не тестировать, и так все плохо.

rsyuzyov · 2016-06-08T15:13:04Z

Ну вот и я о чем :)

rsyuzyov · 2016-06-08T15:13:50Z

Призываю @artbear - допустимо ли?

ghost · 2016-06-08T23:03:17Z

@nixel2007 я просто также оставлю это здесь

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

YPermitin · 2016-06-09T03:48:45Z

@nixel2007 все же с 8.2.14?

Вчера проверил, с этой версии точно не работает. В PR #622 в конце отписался по ошибкам, которые нашел на первых порах.

artbear · 2016-06-10T10:54:21Z

ИМХО Совместимость с 8.2.17 пусть пока остается.
Усилия по ее поддержке минимальны, только сохранять файлы в 8.2.19, а не в 8.3

artbear added question 1С:Предприятие Дымовые тесты labels Apr 20, 2016

artbear self-assigned this Apr 20, 2016

artbear added this to the 4.0.1.0 milestone Jun 6, 2016

artbear removed this from the 4.0.1.0 milestone Sep 9, 2016

artbear modified the milestones: 4.1.0.0, 4.0.1.0 Sep 9, 2016

naa4sb mentioned this issue Sep 9, 2016

Задача #617. Исправлено заполнение тестов для управляемых форм. #624

Closed

artbear closed this as completed in 2424b60 Sep 9, 2016

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Дымовые тесты, в чем необходимость пропуска для УФ #617

Дымовые тесты, в чем необходимость пропуска для УФ #617

pumbaEO commented Apr 20, 2016

artbear commented Apr 20, 2016

pumbaEO commented Apr 20, 2016

Irekovich commented Apr 20, 2016 •

edited

Loading

pumbaEO commented Apr 20, 2016

artbear commented Apr 20, 2016

artbear commented Apr 20, 2016

naa4sb commented Jun 6, 2016

artbear commented Jun 6, 2016

naa4sb commented Jun 6, 2016

artbear commented Jun 6, 2016

das999 commented Jun 6, 2016

naa4sb commented Jun 8, 2016

artbear commented Jun 8, 2016

artbear commented Jun 8, 2016

artbear commented Jun 8, 2016

naa4sb commented Jun 8, 2016

nixel2007 commented Jun 8, 2016

rsyuzyov commented Jun 8, 2016

nixel2007 commented Jun 8, 2016

rsyuzyov commented Jun 8, 2016

rsyuzyov commented Jun 8, 2016

ghost commented Jun 8, 2016

YPermitin commented Jun 9, 2016

artbear commented Jun 10, 2016

Дымовые тесты, в чем необходимость пропуска для УФ #617

Дымовые тесты, в чем необходимость пропуска для УФ #617

Comments

pumbaEO commented Apr 20, 2016

artbear commented Apr 20, 2016

pumbaEO commented Apr 20, 2016

Irekovich commented Apr 20, 2016 • edited Loading

pumbaEO commented Apr 20, 2016

artbear commented Apr 20, 2016

artbear commented Apr 20, 2016

naa4sb commented Jun 6, 2016

artbear commented Jun 6, 2016

naa4sb commented Jun 6, 2016

artbear commented Jun 6, 2016

das999 commented Jun 6, 2016

naa4sb commented Jun 8, 2016

artbear commented Jun 8, 2016

artbear commented Jun 8, 2016

artbear commented Jun 8, 2016

naa4sb commented Jun 8, 2016

nixel2007 commented Jun 8, 2016

rsyuzyov commented Jun 8, 2016

nixel2007 commented Jun 8, 2016

rsyuzyov commented Jun 8, 2016

rsyuzyov commented Jun 8, 2016

ghost commented Jun 8, 2016

YPermitin commented Jun 9, 2016

artbear commented Jun 10, 2016

Irekovich commented Apr 20, 2016 •

edited

Loading