Что посмотреть на выходных? Обзор лучших докладов в свободном доступе. Часть вторая, JBreak 2017 / Comments / Habr

tagir_valeev Jan 15 2018 at 06:10

В старых инспекциях, после фикса они просто терялись, в новых — за этим стараются специально следить.

Сейчас мы, кстати, вышли на новый уровень фанатизма с комментариями. У нас гоняются рандомные проперти-тесты, где в различные джава-файлы вставляются комментарии в самые сумасшедшие места, потом применяются любые интеншны и квик-фиксы и проверяется, что все комментарии остались в коде. Многие старые инспекции уже починили на этот счёт. Сегодня мне свалилось, например, такое:

Source code:
if(fileType instanceof //simple end comment
  LanguageFileType) {
Warning: "Condition fileType instanceof LanguageFileType is always false"
Quick-fix: "Simplify fileType instanceof LanguageFileType to false"
Error: "Lost comment //simple end comment"

Может никому и не придёт в голову вставлять сюда комментарий, но мы это починим :-)

olegchir Jan 15 2018 at 14:08

Такой Chaos Monkey, но не для микросервисов? :)

А где лежит этот тест, чтобы найти в исходниках Community Edition и заценить?

tagir_valeev Jan 15 2018 at 14:14

А где лежит этот тест, чтобы найти в исходниках Community Edition и заценить?

JavaCodeInsightSanityTest#testPreserveComments и далее, там довольно развесистая система.

Такой Chaos Monkey, но не для микросервисов? :)

В некотором смысле да, даже названия классов намекают.