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

439 / Документация по конструктору объекта и движений #461

Merged
merged 6 commits into from
Dec 10, 2024

Conversation

alkoleft
Copy link
Member

@alkoleft alkoleft commented Dec 10, 2024

Описание изменений


  • Добавлены тесты
  • Изменения отражены в документации (при необходимости)

Close #439

Summary by CodeRabbit

  • Новые функции

    • Обновлена документация по генерации тестовых данных, включая методы и примеры использования.
    • Добавлен новый инструмент "Конструктор Объектов" для гибкой генерации тестовых данных.
    • Введен "Конструктор Движений Документов" для упрощения создания движений документов.
    • Добавлена функция ДобавитьЗапись для записи новых объектов в процессе создания тестовых данных.
  • Исправления ошибок

    • Уточнены описания параметров в документации для повышения ясности.
  • Документация

    • Структурированы разделы и добавлены новые рекомендации по управлению тестовыми данными.
    • Обновлены ссылки на документацию для лучшей организации материалов.

Copy link

coderabbitai bot commented Dec 10, 2024

Caution

Review failed

The pull request is closed.

Walkthrough

Документация по генерации тестовых данных была значительно обновлена. Включены новые разделы, такие как "Конструктор объекта" и "Конструктор движений документов", с подробными описаниями методов и примерами использования. Также улучшена структура документации, добавлены ссылки и рекомендации по использованию генераторов случайных значений. Внесены изменения в существующие документы для повышения ясности и организации информации. Обновлены комментарии в коде для улучшения понимания параметров функций.

Changes

Файл Изменения
documentation/docs/features/test-data/data-generation/index.md Обновлено введение, добавлены категории инструментов, детализированы генераторы данных.
documentation/docs/features/test-data/data-generation/object-builder.md Новый файл с описанием конструктора объектов, его возможностей и примеров использования.
documentation/docs/features/test-data/data-generation/register-records-builder.md Новый файл с описанием конструктора движений документов и его функциональности.
documentation/docs/features/test-data/test-data.md Обновлена структура, добавлены новые разделы и ссылки на документацию по генерации данных.
exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/ObjectModule.bsl Обновлены комментарии к функциям для уточнения описания параметров.
exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/ObjectModule.bsl Добавлен новый метод ДобавитьЗапись, обновлены комментарии для уточнения функциональности.

Assessment against linked issues

Objective Addressed Explanation
Перечень всех методов конструктора (#439)
Примеры использования всех методов (#439)
Возможности для разных видов метаданных (#439)
Примеры разных кейсов с комментариями (#439) Примеры кейсов не были добавлены в документацию.

Possibly related PRs

🐇 В мире данных, где тесты живут,
Конструктор объектов теперь нас зовет.
С примерами ясными, как день,
Тесты готовим, как в сказке, без тени.
Случайные значения — наш верный друг,
В документации порядок, как в сказочном кругу! 🌟


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between a79253d and 5036595.

📒 Files selected for processing (4)
  • documentation/docs/features/test-data/data-generation/index.md (3 hunks)
  • documentation/docs/features/test-data/data-generation/register-records-builder.md (1 hunks)
  • exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/Forms/КлиентскийКонструктор/Module.bsl (3 hunks)
  • exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/ObjectModule.bsl (1 hunks)

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?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

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)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR. (Experiment)
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SonarQube Quality Gate failed! failed

bug 0 Bugs
vulnerability 0 Vulnerabilities
code_smell 1 Code Smells

No data Coverage
duplication_lt_20 50.00% Duplication

@@ -217,7 +217,7 @@

КонецФункции

// Создает новый объект.
// Записывает новый объект, но в отличии от методов см. Записать или см. Провести возвращает конструктор, что позволяет продолжить работу по созданию тестовых данных.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔗Длина строки 166 превышает максимально допустимую 150

CODE_SMELL Codesmell  MINOR Minor

standard badpractice parameters  Why is this an issue?

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Outside diff range and nitpick comments (5)
documentation/docs/features/test-data/data-generation/register-records-builder.md (1)

168-168: Исправьте грамматическую ошибку в описании параметра

В описании параметра ИменаРеквизитов некорректно использовано сочетание "Массив из Строка".

Предлагаемое исправление:

-`ИменаРеквизитов` (Строка или Массив из Строка): Имена реквизитов записи
+`ИменаРеквизитов` (Строка или Массив из Строки): Имена реквизитов записи
🧰 Tools
🪛 LanguageTool

[uncategorized] ~168-~168: Предлог «из» предполагает употребление существительного в родительном падеже: «из строки» или «Строка»
Context: ... - ИменаРеквизитов (Строка или Массив из Строка): Имена реквизитов записи, разделенные ...

(PREP_U_and_Noun)

documentation/docs/features/test-data/data-generation/object-builder.md (1)

28-28: Необходимы небольшие правки форматирования

  1. Уровень заголовка "Создание нового элемента справочника 'Контрагенты'" должен быть h3 вместо h4
  2. В описании параметра ИменаРеквизитов требуется корректировка падежа: "Массив из Строк"

Also applies to: 211-211

🧰 Tools
🪛 Markdownlint (0.35.0)

28-28: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time

(MD001, heading-increment)

documentation/docs/features/test-data/data-generation/index.md (3)

144-146: Исправьте дублирование слова "описание"

Текущий текст содержит повторение слова "описание". Предлагаю упростить формулировку.

-:::info Полное описание
-Описание всех методов и примеры можно посмотреть в [описании конструктора объекта](./object-builder.md)
+:::info Подробнее
+Все методы и примеры можно посмотреть в [описании конструктора объекта](./object-builder.md)
🧰 Tools
🪛 LanguageTool

[duplication] ~144-~144: Возможная опечатка: повтор слова
Context: ...(test-data-deletion.md) :::info Полное описание Описание всех методов и примеры можно посмотреть...

(WORD_REPEAT_RULE)


181-183: Исправьте дублирование слова "описание"

Аналогичное повторение слова "описание" в блоке info.

-:::info Полное описание
-Описание всех методов и примеры можно посмотреть в [описании конструктора движений](./register-records-builder.md)
+:::info Подробнее
+Все методы и примеры можно посмотреть в [описании конструктора движений](./register-records-builder.md)
🧰 Tools
🪛 LanguageTool

[duplication] ~181-~181: Возможная опечатка: повтор слова
Context: ...test-data-deletion.md). :::info Полное описание Описание всех методов и примеры можно посмотреть...

(WORD_REPEAT_RULE)


217-217: Добавьте краткое описание возможностей работы с макетами

Рекомендуется добавить краткое описание возможностей работы с макетами перед ссылкой на отдельную статью, чтобы читатель понимал, стоит ли переходить по ссылке.

+Макеты позволяют подготовить эталонные наборы данных для тестирования. Они особенно полезны, когда требуется воспроизвести определенные бизнес-сценарии или протестировать граничные случаи.
+
 Работа с макетами описана в отдельной [статье](../load-from-templates.md)
📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL

📥 Commits

Reviewing files that changed from the base of the PR and between 73b3bfa and a79253d.

📒 Files selected for processing (6)
  • documentation/docs/features/test-data/data-generation/index.md (3 hunks)
  • documentation/docs/features/test-data/data-generation/object-builder.md (1 hunks)
  • documentation/docs/features/test-data/data-generation/register-records-builder.md (1 hunks)
  • documentation/docs/features/test-data/test-data.md (1 hunks)
  • exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/ObjectModule.bsl (3 hunks)
  • exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/ObjectModule.bsl (1 hunks)
🧰 Additional context used
🪛 LanguageTool
documentation/docs/features/test-data/data-generation/index.md

[duplication] ~144-~144: Возможная опечатка: повтор слова
Context: ...(test-data-deletion.md) :::info Полное описание Описание всех методов и примеры можно посмотреть...

(WORD_REPEAT_RULE)


[duplication] ~181-~181: Возможная опечатка: повтор слова
Context: ...test-data-deletion.md). :::info Полное описание Описание всех методов и примеры можно посмотреть...

(WORD_REPEAT_RULE)

documentation/docs/features/test-data/data-generation/object-builder.md

[uncategorized] ~211-~211: Предлог «из» предполагает употребление существительного в родительном падеже: «из строки» или «Строка»
Context: ...* - ИменаРеквизитов (Строка, Массив из Строка): Имена реквизитов объекта или таблично...

(PREP_U_and_Noun)

documentation/docs/features/test-data/data-generation/register-records-builder.md

[uncategorized] ~168-~168: Предлог «из» предполагает употребление существительного в родительном падеже: «из строки» или «Строка»
Context: ... - ИменаРеквизитов (Строка или Массив из Строка): Имена реквизитов записи, разделенные ...

(PREP_U_and_Noun)

🪛 Markdownlint (0.35.0)
documentation/docs/features/test-data/data-generation/object-builder.md

28-28: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time

(MD001, heading-increment)

🔇 Additional comments (6)
documentation/docs/features/test-data/test-data.md (1)

51-51: Корректное обновление ссылки

Обновление ссылки на документацию по генерации данных улучшает навигацию между разделами документации.

documentation/docs/features/test-data/data-generation/register-records-builder.md (1)

1-271: Отличная документация по конструктору движений

Документация хорошо структурирована, содержит понятные примеры использования и подробное описание всех методов конструктора.

🧰 Tools
🪛 LanguageTool

[uncategorized] ~168-~168: Предлог «из» предполагает употребление существительного в родительном падеже: «из строки» или «Строка»
Context: ... - ИменаРеквизитов (Строка или Массив из Строка): Имена реквизитов записи, разделенные ...

(PREP_U_and_Noun)

exts/yaxunit/src/DataProcessors/ЮТКонструкторДвижений/ObjectModule.bsl (1)

50-51: Улучшена документация параметров функций!

Уточнения в описании параметров делают документацию более точной и понятной, явно указывая на работу с реквизитами записи.

Also applies to: 85-85, 110-111

exts/yaxunit/src/DataProcessors/ЮТКонструкторТестовыхДанных/ObjectModule.bsl (1)

Line range hint 220-242: Отличное дополнение к API конструктора!

Метод ДобавитьЗапись удачно расширяет функциональность конструктора:

  • Позволяет продолжить работу после записи объекта
  • Поддерживает флаг загрузки данных обмена
  • Сохраняет консистентность с другими методами класса
documentation/docs/features/test-data/data-generation/object-builder.md (1)

1-493: Отличная документация с подробным описанием API!

Документация хорошо структурирована и содержит:

  • Подробное описание всех методов
  • Практические примеры использования
  • Четкое описание параметров и возвращаемых значений
🧰 Tools
🪛 LanguageTool

[uncategorized] ~211-~211: Предлог «из» предполагает употребление существительного в родительном падеже: «из строки» или «Строка»
Context: ...* - ИменаРеквизитов (Строка, Массив из Строка): Имена реквизитов объекта или таблично...

(PREP_U_and_Noun)

🪛 Markdownlint (0.35.0)

28-28: Expected: h3; Actual: h4
Heading levels should only increment by one level at a time

(MD001, heading-increment)

documentation/docs/features/test-data/data-generation/index.md (1)

Line range hint 1-400: Документация хорошо структурирована и содержательна

Документация предоставляет исчерпывающее описание возможностей генерации тестовых данных с хорошими примерами кода. Структура логична и последовательна.

🧰 Tools
🪛 LanguageTool

[duplication] ~144-~144: Возможная опечатка: повтор слова
Context: ...(test-data-deletion.md) :::info Полное описание Описание всех методов и примеры можно посмотреть...

(WORD_REPEAT_RULE)

@alkoleft alkoleft merged commit 4a0a58a into develop Dec 10, 2024
1 check passed
@alkoleft alkoleft deleted the feature/439 branch December 10, 2024 22:08
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

Successfully merging this pull request may close these issues.

[Doc]: Развить документацию по работе с конструктором объекта
1 participant