Правило "Обработчик регламентного задания" - ГОТОВО

[artbear]

Описание

• реализовал правило
• исправил ошибку в AbstractMetadataDiagnostic - в метод checkMetadata в некоторых случаях попадали метаданные, не заданные фильтром
  • обнаружил при запуске анализа правила на БСП 3.1 через ком.строку
• подключил mdclasses 0.10.2

Связанные задачи

Closes #2861

Чеклист

Общие

• [x] Ветка PR обновлена из develop
• [x] Отладочные, закомментированные и прочие, не имеющие смысла участки кода удалены
• [x] Изменения покрыты тестами
• [x] Обязательные действия перед коммитом выполнены (запускал команду gradlew precommit)

Для диагностик

• [x] Описание диагностики заполнено для обоих языков (присутствуют файлы для обоих языков, для русского заполнено все подробно, перевод на английский можно опустить)

Дополнительно

[artbear]

@nixel2007 все тесты прошли, только шаг Sentry упал. ИМХО не по моей вине )

Error: Environment variable SENTRY_ORG is missing an organization slug

https://github.com/artbear/bsl-language-server/runs/7487025826?check_suite_focus=true

[nixel2007]

Да, там надо gatekeeper подключить

[artbear]

• исправил ошибку в AbstractMetadataDiagnostic - в метод checkMetadata в некоторых случаях попадали метаданные, не заданные фильтром
  • обнаружил при запуске анализа правила на БСП 3.1 через ком.строку

[sonarqubecloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

100.0% Coverage
0.0% Duplication

[artbear]

Правило готово.

Проверено на БСП 3.1 и КА 2.4

В КА найдены правильные срабатывания на пустые методы регл.заданий и даже на дубль обработчика в двух регл.заданиях

[artbear]

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

[nixel2007]

Я бы подождал/сделал доработки в mdclasses. Фпшить на ровном месте не хочется.

[artbear]

Я бы подождал/сделал доработки в mdclasses. Фпшить на ровном месте не хочется.

зачем затягивать с полезным правилом?

давай его смержим, а уже дальше доработаем.

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

[nixel2007]

Добавить одно поле в mdclasses - это не затягивать.

[artbear]

Добавить одно поле в mdclasses - это не затягивать.

кто и когда его добавит? )

[artbear]

Добавить одно поле в mdclasses - это не затягивать.

кто и когда его добавит? )

ну и самое главное - когда выйдет релиз с поддержкой этой фичи уже после вливания МР с доработкой?

@theshadowco сможешь оперативно выпустить релиз , если я сделаю доработку?

[artbear]

@nixel2007 @theshadowco я реализовал поддержку необходимых атрибутов регл.заданий

смотрите ПР https://github.com/1c-syntax/mdclasses/pull/364

[theshadowco]

@artbear https://github.com/1c-syntax/mdclasses/releases/tag/v0.10.1

[theshadowco]

@artbear Точнее https://github.com/1c-syntax/mdclasses/releases/tag/v0.10.2 ибо прошлая реализация была с ошибкой и я ХЗ как тебе это удалось

[sonarqubecloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

98.9% Coverage
0.0% Duplication

[artbear]

@theshadowco @nixel2007 правило готово. примите его, если нет возражений.

подключил релиз mdclasses 0.10.2 тесты прошли

[sonarqubecloud[bot]]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells

98.0% Coverage
0.0% Duplication

[artbear]

Наконец-то мерж! Большое спасибо @otymko !