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.

Sign up for GitHub

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

Jump to bottom

33 Валидация модели состава автономной конфигурации и квик-фиксы #34

Closed
wants to merge 1 commit into from
Closed

33 Валидация модели состава автономной конфигурации и квик-фиксы #34

wants to merge 1 commit into from

Conversation

wolfandw
Copy link
Collaborator

@wolfandw wolfandw commented Jul 12, 2022 •
edited

Что сделано

  • Проверка модели состава автономной конфигурации
  • Квик-фикс для исправления некорректных элементов состава автономной конфигурации

Чек-лист

Общее:

  • ветка PR обновлена из master и нет конфликтов
  • Тесты-кейсы, JUnit тесты правильного и неправильного состояния
  • Измененные Вами исходники отформатированы в соответствии с конвенцией
  • Авто-аудит (SonarQube и CheckStyle) пройден, покрытие кода хорошее, ошибок нет, плохой код устранен
  • Добавлена запись в ИСТОРИЮ ИЗМЕНЕНИЯ, включаемая в пользовательскую документацию плагина

Если применимо:

  • Пользовательская документация на доп.инструменты написана (на русском)
  • Описание проверок - на двух языках

Закрываемые задачи

Closes #??

@wolfandw wolfandw force-pushed the 33-standalone-content-validation-quick-fix branch 3 times, most recently from 6606566 to cf5de86 Compare July 12, 2022 10:24
@marmyshev marmyshev marked this pull request as draft July 14, 2022 14:27
@wolfandw wolfandw force-pushed the 33-standalone-content-validation-quick-fix branch from cf5de86 to c52c2db Compare July 20, 2022 16:10
@wolfandw wolfandw force-pushed the 33-standalone-content-validation-quick-fix branch from c52c2db to 72c7549 Compare August 1, 2022 12:29
@wolfandw wolfandw marked this pull request as ready for review August 2, 2022 09:32
@wolfandw wolfandw force-pushed the 33-standalone-content-validation-quick-fix branch from 72c7549 to 26af7a6 Compare August 3, 2022 13:23
marmyshev
marmyshev requested changes Aug 3, 2022
View reviewed changes
CHANGELOG.md Outdated
Comment on lines 5 to 9
## 0.4.0

- Проверка модели состава автономной конфигурации
- Квик-фикс для исправления некорректных элементов состава автономной конфигурации

Copy link
Collaborator

Choose a reason for hiding this comment

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

нужно перенести в секцию ниже - текущая версия 0.3 в мастере

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

перенес

....dt.check.md/src/com/e1c/dt/check/internal/md/check/ConfigurationStandaloneContentCheck.java
Comment on lines +70 to +74
ConfigurationStandaloneContentValidator.validate(standaloneContent, version)
.entrySet()
.forEach(
error -> error.getValue()
.forEach(errorMessage -> resultAceptor.addIssue(errorMessage, error.getKey())));
Copy link
Collaborator

Choose a reason for hiding this comment

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

выглядит код странно - а где логика проверки?

кажется ConfigurationStandaloneContentValidator - запрещено теперь создавать такие классы в ЕДТ.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

логика в валидаторе. используется в разных местах. это не модельный валидатор - просто название такое.

...check.md/src/com/e1c/dt/check/internal/md/check/ConfigurationMobileApplicationUrlsCheck.java
@@ -10,7 +10,7 @@
* Contributors:
* 1C-Soft LLC - initial API and implementation
*******************************************************************************/
package com.e1c.dt.check.md;
package com.e1c.dt.check.internal.md.check;
Copy link
Collaborator

Choose a reason for hiding this comment

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

как-то не правильно выглядит пакет. Если публичный делать - тогда будет com.e1c.dt.check.md.check;? Выглядит этот check.check не очень.

предлагаю com.e1c.dt.check.internal.md

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

это пакет для чеков. а есть еще пакет для квикфиксов com.e1c.dt.check.internal.md.fix

Copy link
Collaborator

Choose a reason for hiding this comment

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

А чем тебе не понравились чеки в корне бандла для чеков? это вообще весь проект для чеков и фиксов. как-бы ничего другого тут не предполагается.

@wolfandw wolfandw force-pushed the 33-standalone-content-validation-quick-fix branch from 26af7a6 to 9d26a05 Compare August 3, 2022 14:42
@sonarcloud
Copy link

sonarcloud bot commented Aug 3, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 4 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 5 Code Smells

93.9% 93.9% Coverage
0.0% 0.0% Duplication

@wolfandw wolfandw closed this Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marmyshev marmyshev marmyshev requested changes

@MaksimDzyuba MaksimDzyuba Awaiting requested review from MaksimDzyuba

@tretyakevich tretyakevich Awaiting requested review from tretyakevich

Assignees

@wolfandw wolfandw

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@wolfandw @marmyshev