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

TDD Плагин Настройки Метод ЕстьНастройка работает не в соответствии со своим описанием #793

Closed
petypen opened this issue Oct 21, 2020 · 4 comments

Comments

@petypen
Copy link
Contributor

petypen commented Oct 21, 2020

Описание ошибки
Для экспортного метода ЕстьНастройка плагина Настройки в описании заявлено:

// Позволяет одним вызовом получить значение из вложенных друг в друга структур

// Пример:
// Структура = Новый Структура("Ключ1", Новый Структура("Ключ2", Новый Структура("Ключ3", 42)));
// РезультатПроверки = ЕстьНастройка("Ключ1.Ключ2.Ключ3", ВремЗнач);
// В результате получим ВремЗнач == 42

однако, метод всегда возвращает структуру, даже тогда, когда можно вернуть значение элемента структуры.

Сценарий воспроизведения
Шаги по воспроизведению:

В обработку -тест добавить плагин Настройки.
Попробовать вызвать метод ЕстьНастройки по коду из примера выше
Ожидаем получить число 42 а получаем Структуру

Ожидаемое поведение, если бы ошибки не было

Скриншоты
если применимо, приложите скриншоты

Окружение:

Дополнительный контекст

  • Типовая БУХ 30
  • Режим защиты от опасных действий выключен
@petypen
Copy link
Contributor Author

petypen commented Oct 21, 2020

Техническая информация:
- Версия Vanessa-ADD: 6.7.0
- Имя конфигурации: БухгалтерияПредприятия
- Синоним конфигурации: Бухгалтерия предприятия, редакция 3.0
- Версия конфигурации: 3.0.82.34
- Платформа 1С: 8.3.17.1549
- Тип базы: Файловая ИБ
- Режим запуска: Управляемое приложение (Тонкий клиент)
- Режим совместимости (для TestManager): Версия8_3_14
- Режим совместимости интерфейса (для TestManager): ТаксиРазрешитьВерсия8_2
- Режим синхронности: выключен
- Защита от опасных действий: выключен
- Язык (для TestManager): Русский
- Локализация (для TestManager): ru
- Операционная система: Windows x86-64 Microsoft Windows 10 version 10.0 (Build 19041)

@petypen
Copy link
Contributor Author

petypen commented Oct 21, 2020

@artbear Артур, я исправил это поведение в плагине Настройка и хотел бы сделать PR
Нужен твой совет.
Сейчас в плагине есть метод ЕстьНастройка, с описанным неправильным поведением.
Можно изменить его поведение (сигнатура не изменится), но я боюсь, что кто то уже использует этот метод и адаптировал свои тесты к текущему поведению. В результате у человека начнут валиться тесты.
Можно добавить еще один метод, например ЕстьНастройка2 не трогая старый метод.

@artbear
Copy link
Collaborator

artbear commented Oct 22, 2020

Да. скорее всего, текущее поведение этого метода уже юзается (
предлагаю исправить описание этого метода

@artbear
Copy link
Collaborator

artbear commented Nov 8, 2020

выполнено

@artbear artbear closed this as completed Nov 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants