Добавлен метод ФикцияНезаполненных #479
Conversation
WalkthroughИзменения направлены на добавление нового метода Changes
Assessment against linked issues
Possibly related PRs
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/Forms/КлиентскийКонструктор/Module.bsl (1)
144-145: Вынос кода инициализации
ВозвратЭтотОбъектобеспечивает цепочечные вызовы, сохраняя общий стиль. Убедитесь, что метод корректно документирован для других разработчиков.exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/Forms/КлиентскийКонструктор/Module.bsl (2)
113-115: Консистентность параметров
ПередачаЛожьвФикцияРеквизитовсогласована с аналогичными методами в других модулях. Рекомендуется единообразное описание в комментариях для простоты сопровождения.
135-137: Уточнение документации
Функция возвращает объект для продолжения построения. Это верно в обоих сценариях (заполненные и незаполненные реквизиты). Убедитесь, что данный аспект описан в Wiki проекта, чтобы другие разработчики понимали цепочку вызовов.tests/src/CommonModules/Обр_ЮТКонструкторДвижений_МО/Module.bsl (1)
219-225: Уточнить формулировку сообщений в проверках
Сообщения вида «Товар должен быть заполнен» и «Количество должно быть заполнено» выглядят корректно. При необходимости можно добавить более детальные разъяснения (например, «Товар должен быть заполнен справочной ссылкой»), если это часто вызывает путаницу у разработчиков или тестировщиков.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (7)
exts/yaxunit/src/CommonModules/ЮТКонструкторТестовыхДанныхСлужебный/Module.bsl(2 hunks)exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/Forms/КлиентскийКонструктор/Module.bsl(1 hunks)exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/ObjectModule.bsl(1 hunks)exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/Forms/КлиентскийКонструктор/Module.bsl(1 hunks)exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/ObjectModule.bsl(1 hunks)tests/src/CommonModules/ОМ_ЮТКонструкторТестовыхДанных/Module.bsl(2 hunks)tests/src/CommonModules/Обр_ЮТКонструкторДвижений_МО/Module.bsl(3 hunks)
🔇 Additional comments (15)
exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/ObjectModule.bsl (3)
117-119: Уточните назначение логического параметра
Строка 117 добавляет вызовФикцияРеквизитовс параметромЛожь. Убедитесь, что это поведение согласовано с другими вызовами, а также что документация описывает, как именно использованиеЛожьвлияет на логику заполнения.
122-136: Метод добавляет полезную функциональность
Добавленный метод «ФикцияНезаполненных» четко разделяет логику заполнения реквизитов: заполняет только ещё не заполненные. Это повышает гибкость конструкторов.
137-138: Проверка соответствия требованиям
ВызовФикцияРеквизитовсИстинакорректно отражает заданную функциональность, но стоит убедиться, что вызываемый метод должным образом игнорирует уже заполненные поля. Тесты на это поведение крайне рекомендуются.exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/Forms/КлиентскийКонструктор/Module.bsl (2)
123-125: Изменение параметра для «ФикцияРеквизитов»
Обращение с явной передачейЛожьуточняет, что все поля должны быть заполнены, а не только пустые. Это помогает избежать скрытых ошибок при массовом заполнении.
128-143: Новый метод «ФикцияНезаполненных»
Метод логично дополняет существующий функционал, разделяя сценарии заполнения. Рекомендуется добавить в тест-кейсы проверку, чтобы гарантировать, что изначально уже установленные реквизиты не перезаписываются.exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/Forms/КлиентскийКонструктор/Module.bsl (1)
118-134: Новое поведение для незаполненных реквизитов
«ФикцияНезаполненных» обеспечивает точечную генерацию данных там, где реквизиты пусты. Советуем проверить, как правило, обрабатываются составные типы, если поля уже частично заполнены.exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/ObjectModule.bsl (3)
108-110: Передача параметра «Ложь»
ВызовФикцияРеквизитов(..., Ложь)указывает на заполнение всех указанных реквизитов. Убедитесь, что в случаях, когда требуется дифференцировать только незаполненные поля, используется корректный метод.
113-129: Пояснение логики «ФикцияНезаполненных»
Вся логика (строки 113–129) грамотно описывает назначение и поведение нового метода, а добавленный комментарий увеличивает понятность для будущих читателей.
130-131: Инициализация реквизитов
Убедитесь, что при вызовеФикцияРеквизитов(..., Истина)действительно игнорируются поля, уже заданные явно. Рекомендуется проверить множество тестовых сценариев.tests/src/CommonModules/Обр_ЮТКонструкторДвижений_МО/Module.bsl (2)
31-31: Добавлен новый тест для проверки незаполненных реквизитов
Отлично, что добавили тест «ФикцияНезаполненных» в список исполняемых сценариев. Это повышает покрытие тестами и помогает контролировать корректное заполнение полей.
236-275: Новая процедура «ФикцияНезаполненных»
Код процедуры логичен: при передаче поля «Склад» со значением Неопределено метод «ФикцияНезаполненных» корректно заполняет его фиктивным значением. Итоговые проверки (Товар, Склад, Количество) реализованы последовательно и помогают убедиться, что незаполненные поля действительно были заполнены автоматикой.tests/src/CommonModules/ОМ_ЮТКонструкторТестовыхДанных/Module.bsl (2)
35-35: Добавлен тест «ФикцияНезаполненных» в контексте конструктора тестовых данных
Регистрировать тест в процедуре «ИсполняемыеСценарии» — правильный шаг, чтобы удобно группировать проверку логики заполнения реквизитов и поддерживать однородную архитектуру тестов.
308-327: Реализация процедуры «ФикцияНезаполненных» для курсов валют
- При установке «Курс=Неопределено» и вызове «ФикцияНезаполненных("Валюта, Курс")» подтверждается, что заполнение реквизитов происходит только для незаполненных значений.
- Проверка свойства «Период» и соответствие «Валюта» исходным данным обеспечивают полноту теста.
Все выглядит целостно и соответствует назначению процедуры.exts/yaxunit/src/CommonModules/ЮТКонструкторТестовыхДанныхСлужебный/Module.bsl (2)
86-86: Добавлен параметр «ТолькоНезаполненные»
Расширение сигнатуры процедуры «ФикцияРеквизитов» новым параметром повышает гибкость при заполнении полей. Хорошо, что есть возможность пропускать те реквизиты, которые уже имеют значение.
100-106: Проверка «ТолькоНезаполненные»
ЛогикаЕсли ЗначениеЗаполнено(...) Тогда Продолжить;корректна и гарантирует, что уже установленное поле не будет перезаписано фиктивным значением. Это упрощает тестовые сценарии и уменьшает риск перезатирания значений.
Описание изменений
Добавлен метод
ФикцияНезаполненныхдля конструктора движений и конструктора тестовых данныхClose #462
Summary by CodeRabbit
Новые возможности
ФикцияНезаполненныхдля селективного заполнения атрибутовУлучшения тестирования