Merge pull request #370 from bia-technologies/feature/fix-sq-issues

Исправление замечаний SQ
bia-technologies · Sep 19, 2024 · 755cbf8 · 755cbf8
2 parents a9f334e + eb1ea74
commit 755cbf8
Show file tree

Hide file tree

Showing 26 changed files with 218 additions and 125 deletions.
diff --git a/exts/yaxunit/.settings/ManagedEnvironments.prefs b/exts/yaxunit/.settings/ManagedEnvironments.prefs
@@ -1,4 +1,4 @@
-CLIENT=false
+CLIENT=true
 EXTERNAL_CONN=false
 MNG_CLIENT=true
 MOBILE_AUTONOMOUS_SERVER=false

diff --git a/exts/yaxunit/src/CommonModules/МокитоОбучение/Module.bsl b/exts/yaxunit/src/CommonModules/МокитоОбучение/Module.bsl
@@ -1,6 +1,6 @@
 //©///////////////////////////////////////////////////////////////////////////©//
 //
-// Copyright 2021-2023 BIA-Technologies Limited Liability Company
+// Copyright 2021-2024 BIA-Technologies Limited Liability Company
 //
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.
@@ -137,7 +137,12 @@
 // Указывает, что при соблюдении условий (см. Когда) метод должен выполняться.
 // Используется для случаев, когда необходимо задать исключения для другого правила на этом методе.
 // 
-// TODO: Нужны примеры
+// Пример:
+// Мокито.Обучение(Документы.ПКО)
+// // По умолчанию метод выбрасывает исключение
+// .Когда("СформироватьПроводки").ВыброситьИсключение("Упал")
+// // При вызове для конкретного документа исключение не будет выброшено и выполняется метод конфигурации
+// .Когда("СформироватьПроводки", Мокито.МассивПараметров(Ссылка)).ВыполнитьМетод()
 // 
 // Возвращаемое значение:
 // ОбщийМодуль - Этот же модуль, для замыкания

diff --git a/exts/yaxunit/src/CommonModules/МокитоСлужебный/Module.bsl b/exts/yaxunit/src/CommonModules/МокитоСлужебный/Module.bsl
@@ -385,7 +385,7 @@
 // Возвращаемое значение:
 // Структура - Описание параметров перехвата объекта:
 // * Объект - Произвольный
-// * Методы - Структура 
+// * Методы - Структура
 Функция ОписаниеПараметровПерехватаОбъекта(Объект) Экспорт
 
  Возврат Новый Структура("Объект, Методы", Объект, Новый Структура);

diff --git a/exts/yaxunit/src/CommonModules/ЮТАсинхроннаяОбработкаСлужебныйКлиент/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТАсинхроннаяОбработкаСлужебныйКлиент/Module.bsl
@@ -78,6 +78,8 @@
 
 КонецФункции
 
+// BSLLS:MissingParameterDescription-off
+
 //@skip-check module-empty-method
 //@skip-check doc-comment-parameter-section
 Процедура ПустойОбработчик1(Параметр1) Экспорт
@@ -96,4 +98,6 @@
 
 КонецПроцедуры
 
+// BSLLS:MissingParameterDescription-on
+
 #КонецОбласти
diff --git a/exts/yaxunit/src/CommonModules/ЮТЗависимостиСлужебный/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТЗависимостиСлужебный/Module.bsl
@@ -50,13 +50,6 @@
 
 КонецФункции
 
-// Служебный метод, используется только в модуле см. ЮТЗависимости.
-// 
-// Параметры:
-// ОписаниеЗависимости - см. ЮТФабрика.НовоеОписаниеЗависимости 
-// 
-// Возвращаемое значение:
-// CommonModule.ЮТТесты
 Функция ДобавитьЗависимостьПриРегистрации(ОписаниеЗависимости) Экспорт
 
  Если ЮТКонтекстСлужебный.ТекущийЭтапПрогона() <> ЮТФабрика.ЭтапыПрогона().ЗагрузкаТестов Тогда

diff --git a/exts/yaxunit/src/CommonModules/ЮТЗапросы/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТЗапросы/Module.bsl
@@ -200,12 +200,6 @@
 
 КонецФункции
 
-Функция НовыйОписаниеЗапроса(ИмяТаблицы, ПредикатыУсловия, ВыбираемыеПоля) Экспорт
-
- Возврат ЮТЗапросыСлужебныйКлиентСервер.ОписаниеЗапроса(ИмяТаблицы, ПредикатыУсловия, ВыбираемыеПоля);
-
-КонецФункции
-
 // Функция получает список движений по указанному документу и регистру. 
 // Если документ не делает движений по указанному регистру, функция вызывает исключение. 
 // Результатом работы функции является список записей регистра, отсортированных по номеру строки.

diff --git a/exts/yaxunit/src/CommonModules/ЮТЗапросыСлужебныйВызовСервера/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТЗапросыСлужебныйВызовСервера/Module.bsl
@@ -53,11 +53,12 @@
 // Возвращаемое значение:
 // - Структура Из Произвольный - Значения реквизитов ссылки при получении значений множества реквизитов
 // - Произвольный - Значение реквизита ссылки при получении значения одного реквизита
+// - Неопределено - Запрос вернул пустой результат
 Функция ЗначенияРеквизитов(Знач Ссылка, Знач ИменаРеквизитов, Знач ОдинРеквизит) Экспорт
 
  ИмяТаблицы = Ссылка.Метаданные().ПолноеИмя();
 
- ТекстЗапроса = СтрШаблон("ВЫБРАТЬ ПЕРВЫЕ 1 %1 ИЗ %2 ГДЕ Ссылка = &Ссылка", ИменаРеквизитов, ИмяТаблицы);
+ ТекстЗапроса = СтрШаблон("ВЫБРАТЬ ПЕРВЫЕ 1 %1 ИЗ %2 ГДЕ Ссылка = &Ссылка", ИменаРеквизитов, ИмяТаблицы); // BSLLS:QueryParseError-off
  Запрос = Новый Запрос(ТекстЗапроса);
  Запрос.УстановитьПараметр("Ссылка", Ссылка);
 

diff --git a/exts/yaxunit/src/CommonModules/ЮТИсключения/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТИсключения/Module.bsl
@@ -28,7 +28,7 @@
 // Строка - Текст исключения
 Функция НекорректныеПараметрыМетода(ИмяМетода, Пояснение = Неопределено) Экспорт
 
- Сообщение = СтрШаблон("Некорректные параметры метода `%1`");
+ Сообщение = СтрШаблон("Некорректные параметры метода `%1`", ИмяМетода);
  Возврат ЮТСтроки.ДобавитьСтроку(Сообщение, Пояснение, ", ");
 
 КонецФункции

diff --git a/exts/yaxunit/src/CommonModules/ЮТИсполнительСлужебныйКлиентСервер/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТИсполнительСлужебныйКлиентСервер/Module.bsl
@@ -150,7 +150,11 @@
 #Если Сервер ИЛИ ТолстыйКлиентОбычноеПриложение ИЛИ ТолстыйКлиентУправляемоеПриложение Тогда
  ВТранзакции = ЮТНастройкиВыполнения.ВТранзакции();
  Если ВТранзакции Тогда
- НачатьТранзакцию();
+ // BSLLS:PairingBrokenTransaction-off
+ // BSLLS:BeginTransactionBeforeTryCatch-off
+ НачатьТранзакцию(); // Отмена транзакции в см. ПослеКаждогоТеста
+ // BSLLS:PairingBrokenTransaction-on
+ // BSLLS:BeginTransactionBeforeTryCatch-on
  КонецЕсли;
 #КонецЕсли
 
@@ -188,6 +192,8 @@
 Процедура ОтменитьТранзакциюТеста(Тест, ВТранзакции)
 
 #Если Сервер ИЛИ ТолстыйКлиентОбычноеПриложение ИЛИ ТолстыйКлиентУправляемоеПриложение Тогда
+ // BSLLS:PairingBrokenTransaction-off
+ // BSLLS:WrongUseOfRollbackTransactionMethod-off
  Если ВТранзакции Тогда
  Если ТранзакцияАктивна() Тогда
  ОтменитьТранзакцию();
@@ -200,6 +206,8 @@
  ОтменитьТранзакцию();
  ЮТРегистрацияОшибокСлужебный.ЗарегистрироватьПростуюОшибкуВыполнения(Тест, "Обнаружена незакрытая транзакция");
  КонецЦикла;
+ // BSLLS:PairingBrokenTransaction-on
+ // BSLLS:WrongUseOfRollbackTransactionMethod-on
 #КонецЕсли
 
 КонецПроцедуры

diff --git a/exts/yaxunit/src/CommonModules/ЮТКоллекции/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТКоллекции/Module.bsl
@@ -320,13 +320,13 @@
 // 
 // Параметры:
 // Значение1 - Произвольный
-// Значение2 - Произвольный 
-// Значение3 - Произвольный 
-// Значение4 - Произвольный 
-// Значение5 - Произвольный 
-// Значение6 - Произвольный 
-// Значение7 - Произвольный 
-// Значение8 - Произвольный 
+// Значение2 - Произвольный
+// Значение3 - Произвольный
+// Значение4 - Произвольный
+// Значение5 - Произвольный
+// Значение6 - Произвольный
+// Значение7 - Произвольный
+// Значение8 - Произвольный
 // Значение9 - Произвольный 
 // Значение10 - Произвольный
 // 

diff --git a/exts/yaxunit/src/CommonModules/ЮТЛогированиеСлужебный/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТЛогированиеСлужебный/Module.bsl
@@ -148,7 +148,7 @@
 
 // Обработка события "ПослеЧтенияСценариев"
 // Параметры:
-// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей, которые содержат информацию о запускаемых тестах
+// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей.
 Процедура ПослеЧтенияСценариев(Сценарии) Экспорт
 
  ЮТЛогирование.Информация(СтрШаблон("Загрузка сценариев завершена. %1 сценариев.", Сценарии.Количество()));
@@ -174,7 +174,7 @@
 
  КонецЦикла;
 
- ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(ИмяКонтекстаЛогирования() + ".ОбщееКоличествоТестов", Количество, Истина);
+ ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(КлючКонтекстаОбщееКоличествоТестов(), Количество, Истина);
 
 КонецПроцедуры
 
@@ -187,7 +187,7 @@
 #Если Клиент Тогда
  Контекст = Контекст();
  ПрогрессКлиент = Контекст.КоличествоВыполненныхТестов;
- ПрогрессСервер = ЮТКонтекстСлужебный.ЗначениеКонтекста(ИмяКонтекстаЛогирования() + ".КоличествоВыполненныхТестов", Истина);
+ ПрогрессСервер = ЮТКонтекстСлужебный.ЗначениеКонтекста(КлючКонтекстаКоличествоВыполненныхТестов(), Истина);
 
  Если ПрогрессКлиент < ПрогрессСервер Тогда
  Контекст.КоличествоВыполненныхТестов = ПрогрессСервер;
@@ -255,7 +255,7 @@
  Возврат;
  КонецЕсли;
 #Если Клиент Тогда
- ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(ИмяКонтекстаЛогирования() + ".КоличествоВыполненныхТестов", Контекст.КоличествоВыполненныхТестов, Истина);
+ ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(КлючКонтекстаКоличествоВыполненныхТестов(), Контекст.КоличествоВыполненныхТестов, Истина);
 #КонецЕсли
 
  МетаданныеМодуля = ОписаниеСобытия.Модуль.Метаданные;
@@ -351,6 +351,7 @@
 // см. НовыйДанныеКонтекста
 Функция Контекст()
 
+ //@skip-check constructor-function-return-section
  Возврат ЮТКонтекстСлужебный.ЗначениеКонтекста(ИмяКонтекстаЛогирования());
 
 КонецФункции
@@ -361,6 +362,18 @@
 
 КонецФункции
 
+Функция КлючКонтекстаОбщееКоличествоТестов()
+
+ Возврат ИмяКонтекстаЛогирования() + ".ОбщееКоличествоТестов";
+
+КонецФункции
+
+Функция КлючКонтекстаКоличествоВыполненныхТестов()
+
+ Возврат ИмяКонтекстаЛогирования() + ".КоличествоВыполненныхТестов";
+
+КонецФункции
+
 // Новый данные контекста.
 //
 // Возвращаемое значение:

diff --git a/exts/yaxunit/src/CommonModules/ЮТНастройкиВыполнения/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТНастройкиВыполнения/Module.bsl
@@ -76,7 +76,8 @@
 // ИмяНастройки - Строка - Имя настройки, см. ЮТФабрика.ПараметрыИсполненияТеста
 // ЗначениеПоУмолчанию - Произвольный - Значение по умолчанию
 // СтрогийУровеньИсполнения - Булево - Признак, стоит ли проверять наличие настройки у родительских элементов.
-// Ложь - По умолчанию, будет выполнен поиск и получение значения для родителей (набор, модуль), если значения для текущего элемента не установлено.
+// Ложь - По умолчанию, будет выполнен поиск и получение значения для родителей (набор, модуль),
+// если значения для текущего элемента не установлено.
 // Истина - Получение настройки только для текущего элемента.
 // 
 // Возвращаемое значение:

diff --git a/exts/yaxunit/src/CommonModules/ЮТОбщий/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТОбщий/Module.bsl
@@ -801,7 +801,7 @@
  Иначе
 
  Представление = Строка(Объект);
- ПредставлениеТипа = ПредставлениеТипа(ТипОбъекта); // Для ссылочных
+ ПредставлениеТипа = ЮТТипыДанныхСлужебный.ПредставлениеТипа(ТипОбъекта); // Для ссылочных
 
  КонецЕсли;
 

diff --git a/exts/yaxunit/src/CommonModules/ЮТОтчетAllureСлужебный/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТОтчетAllureСлужебный/Module.bsl
@@ -144,7 +144,7 @@
  Если РезультатТеста.Статус = Статусы.Ожидание Тогда
  Описание.statusDetails.message = "Тест не был вызван";
  Иначе
- Описание.statusDetails.message = "Тест не успешен, но нет сообщений об ошибках"
+ Описание.statusDetails.message = "Тест не успешен, но нет сообщений об ошибках";
  КонецЕсли;
 
  КонецЕсли;

diff --git a/exts/yaxunit/src/CommonModules/ЮТОтчетСлужебный/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТОтчетСлужебный/Module.bsl
@@ -129,7 +129,7 @@
  Если Существует Тогда
  ОбработчикПроверки = Новый ОписаниеОповещения("ПослеПроверкиКаталога", ЭтотОбъект, Параметры);
  ЮТФайлы.ЭтоКаталог(ПутьКОтчету, ОбработчикПроверки);
- Возврат
+ Возврат;
  КонецЕсли;
 
  Формат = Параметры.ПараметрыФормирования.Формат;

diff --git a/exts/yaxunit/src/CommonModules/ЮТРазрешениеЗависимостейСлужебныйКлиент/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТРазрешениеЗависимостейСлужебныйКлиент/Module.bsl
@@ -116,9 +116,13 @@
  Результат.Успешно = РезультатПередачиФайлов <> Неопределено;
 
  Если Результат.Успешно Тогда
- Результат.СохраняемыйКонтекстКлиент = Новый Структура("РабочийКаталог, ПолноеИмя", ЮТНастройкиВыполнения.КаталогПроекта(), Параметры.ПолныйПуть);
+ Результат.СохраняемыйКонтекстКлиент = Новый Структура("РабочийКаталог, ПолноеИмя",
+ ЮТНастройкиВыполнения.КаталогПроекта(),
+ Параметры.ПолныйПуть);
+
+ ПолноеИмяСервер = ЮТФайлы.ОбъединитьПути(Результат.СохраняемыйКонтекстСервер.РабочийКаталог, Параметры.ПутьКФайлуПроекта);
  Результат.СохраняемыйКонтекстСервер = РезультатПередачиФайлов;
- Результат.СохраняемыйКонтекстСервер.Вставить("ПолноеИмя", ЮТФайлы.ОбъединитьПути(Результат.СохраняемыйКонтекстСервер.РабочийКаталог, Параметры.ПутьКФайлуПроекта));
+ Результат.СохраняемыйКонтекстСервер.Вставить("ПолноеИмя", ПолноеИмяСервер);
  КонецЕсли;
 
  ЮТАсинхроннаяОбработкаСлужебныйКлиент.ВызватьСледующийОбработчик(Параметры, Результат);

diff --git a/exts/yaxunit/src/CommonModules/ЮТРегистрацияОшибокСлужебный/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТРегистрацияОшибокСлужебный/Module.bsl
@@ -48,7 +48,7 @@
 // Регистрирует ошибку загрузки тестов
 //
 // Параметры:
-// Объект - Структура - см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля или см. ЮТФабрикаСлужебный.ОписаниеТестовогоНабора или см. ЮТФабрикаСлужебный.ОписаниеТеста
+// Объект - Структура - Описание регистрируемого модуля, набора или теста
 // Описание - Строка - Описания ошибки, места возникновения
 // Ошибка - ИнформацияОбОшибке
 Процедура ЗарегистрироватьОшибкуЧтенияТестов(Объект, Описание, Ошибка) Экспорт

diff --git a/exts/yaxunit/src/CommonModules/ЮТСобытияСлужебный/Module.bsl b/exts/yaxunit/src/CommonModules/ЮТСобытияСлужебный/Module.bsl
@@ -226,7 +226,7 @@
 
 // Вызов события "ПослеЧтенияСценариев"
 // Параметры:
-// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей, которые содержат информацию о запускаемых тестах
+// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей
 Процедура ПослеЧтенияСценариев(Сценарии) Экспорт
 
  Параметры = ЮТКоллекции.ЗначениеВМассиве(Сценарии);