Md reference intergrity quick fix #36
Conversation
Добавлена проверка на ссылочную целостность MD-объектов в конфигурации.
md-reference-intergrity.
md-reference-intergrity.
git@github.com:1C-Company/dt-project-checks.git into md-reference-intergrity-quick-fix
CHANGELOG.md
Outdated
| ## 0.4.0 | ||
|
|
||
| - Добавлено быстрое исправление для проверки целостности ссылок объектов метаданных путем удаления неправильной ссылки. | ||
|
|
There was a problem hiding this comment.
текущая версия в мастере - 0.3
...heck.md.itests/src/com/e1c/dt/check/md/fix/itests/MdReferenceIntegrityCheckQuickFixTest.java
Outdated
Show resolved
Hide resolved
tests/com.e1c.dt.check.md.itests/src/com/e1c/dt/check/md/fix/itests/AbstractFixTest.java
Show resolved
Hide resolved
| MdReferenceIntegrity_FixTitle=\u0423\u0434\u0430\u043B\u0435\u043D\u0438\u0435 \u043F\u0440\u043E\u043A\u0441\u0438-\u0441\u0441\u044B\u043B\u043A\u0438 | ||
| MdReferenceIntegrity_FixDescription=\u0411\u0443\u0434\u0435\u0442 \u0443\u0434\u0430\u043B\u0435\u043D\u0430 \u0441\u0441\u044B\u043B\u043A\u0430 \u043D\u0430 \u043F\u0440\u043E\u043A\u0441\u0438 |
There was a problem hiding this comment.
Неправильный файл - неправильная кодировка -> UTF-8 надо
| * @author Alexandr Sanarov | ||
| * | ||
| */ | ||
| final class Messages |
There was a problem hiding this comment.
В тестах локализация вообще не нужна - достаточно использовать английский язык
| MdReferenceIntegrity_FixTitle=Proxy reference deletion | ||
| MdReferenceIntegrity_FixDescription=Proxy reference will be removed from contained object |
There was a problem hiding this comment.
ненужная локализация в тестах
| { | ||
|
|
||
| @ClassRule | ||
| public static TestingPlatformSupport testingPlatformSupport = new TestingPlatformSupport(Version.V8_3_20); |
There was a problem hiding this comment.
а зачем это? если нигде не используется
| * | ||
| * @param marker {@link Marker} marker with error. May not be {@code null}. | ||
| * @param dtProject {@link IDtProject} EDT project. May not be {@code null}. Should match the sub-folder name of the 'workspaces' folder | ||
| * @param targetFixDescription {@link String} Fix title. May not be {@code null}. |
There was a problem hiding this comment.
А можно ли как-то указать фикс не через описание, а как-то по-другому. Описание сидит в .properties файле и его могут поменять. К логике это никакого отношения иметь не будет, а тест сломается.
Например, причесали запятые или привели формулировки к общему виду. Тест по идее сломаться не должен, а скорее всего сломается.
tests/com.e1c.dt.check.md.itests/src/com/e1c/dt/check/md/fix/itests/AbstractFixTest.java
Outdated
Show resolved
Hide resolved
tests/com.e1c.dt.check.md.itests/src/com/e1c/dt/check/md/fix/itests/AbstractFixTest.java
Outdated
Show resolved
Hide resolved
|
Kudos, SonarCloud Quality Gate passed! |
Что сделано
Добавлено быстрое исправление для проверки целостности ссылок путем удаления proxy-ссылки
Чек-лист
Общее:
masterи нет конфликтовЕсли применимо:
Закрываемые задачи
Closes #??
@tretyakevich прошу сделать аудит