#1224 Проверяет отсутствие кода после асинхронного вызова

[iArtemv]

Что сделано

• Добавлена проверка отсуствия кода после следующих асинхронного вызова:

Методы глобального контекста

1. ВвестиДатуАсинх,
2. ВвестиЗначениеАсинх,
3. ВвестиСтрокуАсинх,
4. ВвестиЧислоАсинх,
5. ВопросАсинх,
6. ВыполнитьОбработкуОповещения,
7. ДокументЗащищенПаролемАсинх,
8. ЗапроситьРазрешениеПользователяАсинх,
9. ЗапуститьПриложениеАсинх,
10. КаталогБиблиотекиМобильногоУстройстваАсинх,
11. КаталогВременныхФайловАсинх,
12. КаталогДокументовАсинх,
13. КопироватьФайлАсинх,
14. НайтиФайлыАсинх,
15. НачатьЗапросРазрешенияПользователя,
16. НачатьЗапускПриложения,
17. НачатьКопированиеФайла,
18. НачатьПеремещениеФайла,
19. НачатьПодключениеВнешнейКомпоненты,
20. НачатьПодключениеРасширенияПолученияИнформацииОКомпьютере,
21. НачатьПодключениеРасширенияРаботыСКриптографией,
22. НачатьПодключениеРасширенияРаботыСФайлами,
23. НачатьПоискФайлов,
24. НачатьПолучениеИнформацииОСетевыхАдаптерах,
25. НачатьПолучениеКаталогаБиблиотекиМобильногоУстройства,
26. НачатьПолучениеКаталогаВременныхФайлов,
27. НачатьПолучениеКаталогаДокументов,
28. НачатьПолучениеРабочегоКаталогаДанныхПользователя,
29. НачатьПолучениеФайлаССервера,
30. НачатьПолучениеФайлов,
31. НачатьПолучениеФайловССервера,
32. НачатьПомещениеФайла,
33. НачатьПомещениеФайлаНаСервер,
34. НачатьПомещениеФайлов,
35. НачатьПомещениеФайловНаСервер,
36. НачатьСозданиеДвоичныхДанныхИзФайла,
37. НачатьСозданиеКаталога,
38. НачатьУдалениеФайлов,
39. НачатьУстановкуВнешнейКомпоненты,
40. НачатьУстановкуРасширенияПолученияИнформацииОКомпьютере,
41. НачатьУстановкуРасширенияРаботыСКриптографией,
42. НачатьУстановкуРасширенияРаботыСФайлами,
43. ОткрытьЗначениеАсинх,
44. ОткрытьФорму,
45. ПереместитьФайлАсинх,
46. ПодключитьВнешнююКомпонентуАсинх,
47. ПодключитьРасширениеПолученияИнформацииОКомпьютереАсинх,
48. ПодключитьРасширениеРаботыСКриптографиейАсинх,
49. ПодключитьРасширениеРаботыСФайламиАсинх,
50. ПоказатьВводДаты,
51. ПоказатьВводЗначения,
52. ПоказатьВводСтроки,
53. ПоказатьВводЧисла,
54. ПоказатьВопрос,
55. ПоказатьЗначение,
56. ПоказатьОповещениеПользователя,
57. ПоказатьПредупреждение,
58. ПолучитьИнформациюОСетевыхАдаптерахАсинх,
59. ПолучитьФайлССервераАсинх,
60. ПолучитьФайлыССервераАсинх,
61. ПоместитьФайлНаСерверАсинх,
62. ПоместитьФайлыНаСерверАсинх,
63. ПредупреждениеАсинх,
64. РабочийКаталогДанныхПользователяАсинх,
65. СоздатьДвоичныеДанныеИзФайлаАсинх,
66. СоздатьКаталогАсинх,
67. УдалитьФайлыАсинх,
68. УстановитьВнешнююКомпонентуАсинх,
69. УстановитьРасширениеПолученияИнформацииОКомпьютереАсинх,
70. УстановитьРасширениеРаботыСКриптографиейАсинх,
71. УстановитьРасширениеРаботыСФайламиАсинх

Методы различных объектов

1. ВидеоконференцияАсинх,
2. ВыбратьАсинх,
3. ВыбратьДействиеАсинх,
4. ВыбратьИзМенюАсинх,
5. ВыбратьИзСпискаАсинх,
6. ВыбратьЭлементАсинх,
7. ВыгрузитьАсинх,
8. ВызватьHTTPМетодАсинх,
9. ВыполнитьРегистрациюИнформационнойБазыАсинх,
10. ДобавитьАрхивнуюМеткуВремениАсинх,
11. ДобавитьАсинх,
12. ДобавитьОтображаемыйОбъектАсинх,
13. ДобавитьПодписьАсинх,
14. ЗагрузитьВидеообъявлениеСВознаграждениемАсинх,
15. ЗагрузитьПолноэкраннуюРекламуАсинх,
16. ЗагрузитьРекламныйБаннерАсинх,
17. ЗакрытьАсинх,
18. ЗаписатьАсинх,
19. ЗаписатьБайтАсинх,
20. ЗаписатьБуферДвоичныхДанныхАсинх,
21. ЗаписатьВложенияАсинх,
22. ЗаписатьОтображаемыйОбъектАсинх,
23. ЗаписатьПодписьАсинх,
24. ЗаписатьСимволыАсинх,
25. ЗаписатьСтрокуАсинх,
26. ЗаписатьУдалениеВложенийАсинх,
27. ЗаписатьФайлДляПечатиАсинх,
28. ЗаписатьЦелое16Асинх,
29. ЗаписатьЦелое32Асинх,
30. ЗаписатьЦелое64Асинх,
31. ЗашифроватьАсинх,
32. ИзменитьАсинх,
33. ИмяМетодаАсинх,
34. ИнициализироватьАсинх,
35. КопироватьВАсинх,
36. НайтиПоОтпечаткуАсинх,
37. НайтиПоСерийномуНомеруАсинх,
38. НайтиПоСубъектуАсинх,
39. ОбновитьСпискиОтзываСертификатовАсинх,
40. ОтключитьОбработчикНовыхСообщенийАсинх,
41. ОткрытьАсинх,
42. ОткрытьДляДописыванияАсинх,
43. ОткрытьДляЗаписиАсинх,
44. ОткрытьДляЧтенияАсинх,
45. ОткрытьПотокДляЧтенияАсинх,
46. ОтменитьРегистрациюИнформационнойБазыАсинх,
47. ОтметитьЭлементыАсинх,
48. ОтправитьДляОбработкиАсинх,
49. ПерейтиАсинх,
50. ПодключитьОбработчикНовыхСообщенийАсинх,
51. ПодписатьАсинх,
52. ПолучитьАсинх,
53. ПолучитьБуферДвоичныхДанныхАсинх,
54. ПолучитьВложенияАсинх,
55. ПолучитьВремяИзмененияАсинх,
56. ПолучитьВсеАсинх,
57. ПолучитьДанныеАсинх,
58. ПолучитьДвоичныеДанныеАсинх,
59. ПолучитьЗаголовкиАсинх,
60. ПолучитьИмяСвойстваАсинх,
61. ПолучитьИнформациюМодуляКриптографииАсинх,
62. ПолучитьКартинкуПредставленияФайлаБиблиотекиМобильногоУстройстваАсинх,
63. ПолучитьКонтейнерПодписейКриптографииАсинх,
64. ПолучитьНевидимостьАсинх,
65. ПолучитьОбсуждениеАсинх,
66. ПолучитьОписанияПодписейАсинх,
67. ПолучитьСертификатыИзПодписиАсинх,
68. ПолучитьТолькоЧтениеАсинх,
69. ПолучитьУниверсальноеВремяИзмененияАсинх,
70. ПолучитьХранилищеСертификатовАсинх,
71. ПоместитьДанныеАсинх,
72. ПреобразоватьВОттенкиСерогоАсинх,
73. ПриобрестиАсинх,
74. ПроверитьАсинх,
75. ПроверитьМеткуВремениАсинх,
76. ПроверитьПодписиАсинх,
77. ПроверитьПодписьАсинх,
78. ПроверитьСертификатАсинх,
79. ПропуститьАсинх,
80. ПропуститьДоАсинх,
81. ПрочитатьАсинх,
82. ПрочитатьБайтАсинх,
83. ПрочитатьВБуферДвоичныхДанныхАсинх,
84. ПрочитатьДоАсинх,
85. ПрочитатьСимволыАсинх,
86. ПрочитатьСтрокуАсинх,
87. ПрочитатьЦелое16Асинх,
88. ПрочитатьЦелое32Асинх,
89. ПрочитатьЦелое64Асинх,
90. РазделитьАсинх,
91. РазделитьНаЧастиПоАсинх,
92. РазмерАсинх,
93. РасшифроватьАсинх,
94. РедактироватьАсинх,
95. СброситьБуферыАсинх,
96. СканироватьДокументыАсинх,
97. СоздатьАсинх,
98. СоздатьВременныйФайлАсинх,
99. СуществуетАсинх,
100. УдалитьАсинх,
101. УдалитьДанныеАсинх,
102. УсовершенствоватьПодписьАсинх,
103. УстановитьВремяИзмененияАсинх,
104. УстановитьИмяСвойстваАсинх,
105. УстановитьНевидимостьАсинх,
106. УстановитьРазмерАсинх,
107. УстановитьТолькоЧтениеАсинх,
108. УстановитьУниверсальноеВремяИзмененияАсинх,
109. ЭтоКаталогАсинх,
110. ЭтоФайлАсинх

Чек-лист

Общее:

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

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

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

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

Closes #1224

[AlmazNasibullin]

Я тут совсем не уверен, но как-то из наших платформенных артефактов этот список программно нельзя получать? Появятся новые асинхронные метода в платформе, мы их забудем сюда добавить.

[iArtemv]

Вроде нет

[marmyshev]

Можно по контексту платформы собрать все методы у которых есть ОписаниеОповещения, или которые заканчиваются на Async - вынести это в отдельный Кэш-провайдер который по-требованию будут собирать из контекста по версии проверка и кэшировать на время вызова.

Не следует так же забывать про существование асинхронных методов у объектов платформы.

interface AsyncInvocationProvider
{

   // Методы глобального контекста
   Collection<String> getAsyncInvocationNames(Version version);

   // Методы со списком типов в которых они используются
   Map<String, Collection<String>> getAsyncTypeMethodNames(Version version);

}

Согласен с @AlmazNasibullin что асинхронные методы появляются в каждой новой версии платформы.

[AlmazNasibullin]

Можно вместо этого метода использовать метод EcoreUtil2.getContainerOfType(EObject, Class<T>).

[iArtemv]

да, только в данном случае получается что неизвестно какой там тип будет

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

94.1% Coverage
0.0% Duplication

[marmyshev]

Можно по контексту платформы собрать все методы у которых есть ОписаниеОповещения, или которые заканчиваются на Async - вынести это в отдельный Кэш-провайдер который по-требованию будут собирать из контекста по версии проверка и кэшировать на время вызова.

Не следует так же забывать про существование асинхронных методов у объектов платформы.

interface AsyncInvocationProvider
{

   // Методы глобального контекста
   Collection<String> getAsyncInvocationNames(Version version);

   // Методы со списком типов в которых они используются
   Map<String, Collection<String>> getAsyncTypeMethodNames(Version version);

}

Согласен с @AlmazNasibullin что асинхронные методы появляются в каждой новой версии платформы.

[marmyshev]

Подозрительный код.

Зачем на вычислять типы значений передаваемых в параметр, если можно взять сразу тип параметра из контекста платформы? Если код нетипизированный в части передаваемых параметров - то тут не сработает, но как бы ожидалось что сработает...

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

[iArtemv]

Здесь проверяется условие соответствия опции, что в параметр фактически передан ОписаниеОповвещения, если тип отличный, не задан парамер или задан Неопределено - то под условие опции не попадает.

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

86.8% Coverage
0.0% Duplication