Skip to content

Commit

Permalink
fix(sonar): исправление замечаний
Browse files Browse the repository at this point in the history
  • Loading branch information
alkoleft committed Sep 19, 2024
1 parent 0dbd7a5 commit e634f77
Show file tree
Hide file tree
Showing 14 changed files with 138 additions and 56 deletions.
2 changes: 1 addition & 1 deletion exts/yaxunit/.settings/ManagedEnvironments.prefs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
CLIENT=false
CLIENT=true
EXTERNAL_CONN=false
MNG_CLIENT=true
MOBILE_AUTONOMOUS_SERVER=false
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -150,7 +150,11 @@
#Если Сервер ИЛИ ТолстыйКлиентОбычноеПриложение ИЛИ ТолстыйКлиентУправляемоеПриложение Тогда
ВТранзакции = ЮТНастройкиВыполнения.ВТранзакции();
Если ВТранзакции Тогда
// BSLLS:PairingBrokenTransaction-off
// BSLLS:BeginTransactionBeforeTryCatch-off
НачатьТранзакцию(); // Отмена транзакции в см. ПослеКаждогоТеста
// BSLLS:PairingBrokenTransaction-on
// BSLLS:BeginTransactionBeforeTryCatch-on
КонецЕсли;
#КонецЕсли

Expand Down Expand Up @@ -188,6 +192,8 @@
Процедура ОтменитьТранзакциюТеста(Тест, ВТранзакции)

#Если Сервер ИЛИ ТолстыйКлиентОбычноеПриложение ИЛИ ТолстыйКлиентУправляемоеПриложение Тогда
// BSLLS:PairingBrokenTransaction-off
// BSLLS:WrongUseOfRollbackTransactionMethod-off
Если ВТранзакции Тогда
Если ТранзакцияАктивна() Тогда
ОтменитьТранзакцию();
Expand All @@ -200,6 +206,8 @@
ОтменитьТранзакцию();
ЮТРегистрацияОшибокСлужебный.ЗарегистрироватьПростуюОшибкуВыполнения(Тест, "Обнаружена незакрытая транзакция");
КонецЦикла;
// BSLLS:PairingBrokenTransaction-on
// BSLLS:WrongUseOfRollbackTransactionMethod-on
#КонецЕсли

КонецПроцедуры
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -148,7 +148,7 @@

// Обработка события "ПослеЧтенияСценариев"
// Параметры:
// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей, которые содержат информацию о запускаемых тестах
// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей.
Процедура ПослеЧтенияСценариев(Сценарии) Экспорт

ЮТЛогирование.Информация(СтрШаблон("Загрузка сценариев завершена. %1 сценариев.", Сценарии.Количество()));
Expand All @@ -174,7 +174,7 @@

КонецЦикла;

ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(ИмяКонтекстаЛогирования() + ".ОбщееКоличествоТестов", Количество, Истина);
ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(КлючКонтекстаОбщееКоличествоТестов(), Количество, Истина);

КонецПроцедуры

Expand All @@ -187,7 +187,7 @@
#Если Клиент Тогда
Контекст = Контекст();
ПрогрессКлиент = Контекст.КоличествоВыполненныхТестов;
ПрогрессСервер = ЮТКонтекстСлужебный.ЗначениеКонтекста(ИмяКонтекстаЛогирования() + ".КоличествоВыполненныхТестов", Истина);
ПрогрессСервер = ЮТКонтекстСлужебный.ЗначениеКонтекста(КлючКонтекстаКоличествоВыполненныхТестов(), Истина);

Если ПрогрессКлиент < ПрогрессСервер Тогда
Контекст.КоличествоВыполненныхТестов = ПрогрессСервер;
Expand Down Expand Up @@ -255,7 +255,7 @@
Возврат;
КонецЕсли;
#Если Клиент Тогда
ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(ИмяКонтекстаЛогирования() + ".КоличествоВыполненныхТестов", Контекст.КоличествоВыполненныхТестов, Истина);
ЮТКонтекстСлужебный.УстановитьЗначениеКонтекста(КлючКонтекстаКоличествоВыполненныхТестов(), Контекст.КоличествоВыполненныхТестов, Истина);
#КонецЕсли

МетаданныеМодуля = ОписаниеСобытия.Модуль.Метаданные;
Expand Down Expand Up @@ -351,6 +351,7 @@
// см. НовыйДанныеКонтекста
Функция Контекст()

//@skip-check constructor-function-return-section
Возврат ЮТКонтекстСлужебный.ЗначениеКонтекста(ИмяКонтекстаЛогирования());

КонецФункции
Expand All @@ -361,6 +362,18 @@

КонецФункции

Функция КлючКонтекстаОбщееКоличествоТестов()

Возврат ИмяКонтекстаЛогирования() + ".ОбщееКоличествоТестов";

КонецФункции

Функция КлючКонтекстаКоличествоВыполненныхТестов()

Возврат ИмяКонтекстаЛогирования() + ".КоличествоВыполненныхТестов";

КонецФункции

// Новый данные контекста.
//
// Возвращаемое значение:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@
// ИмяНастройки - Строка - Имя настройки, см. ЮТФабрика.ПараметрыИсполненияТеста
// ЗначениеПоУмолчанию - Произвольный - Значение по умолчанию
// СтрогийУровеньИсполнения - Булево - Признак, стоит ли проверять наличие настройки у родительских элементов.
// Ложь - По умолчанию, будет выполнен поиск и получение значения для родителей (набор, модуль),
// Ложь - По умолчанию, будет выполнен поиск и получение значения для родителей (набор, модуль),
// если значения для текущего элемента не установлено.
// Истина - Получение настройки только для текущего элемента.
//
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -116,9 +116,13 @@
Результат.Успешно = РезультатПередачиФайлов <> Неопределено;

Если Результат.Успешно Тогда
Результат.СохраняемыйКонтекстКлиент = Новый Структура("РабочийКаталог, ПолноеИмя", ЮТНастройкиВыполнения.КаталогПроекта(), Параметры.ПолныйПуть);
Результат.СохраняемыйКонтекстКлиент = Новый Структура("РабочийКаталог, ПолноеИмя",
ЮТНастройкиВыполнения.КаталогПроекта(),
Параметры.ПолныйПуть);

ПолноеИмяСервер = ЮТФайлы.ОбъединитьПути(Результат.СохраняемыйКонтекстСервер.РабочийКаталог, Параметры.ПутьКФайлуПроекта);
Результат.СохраняемыйКонтекстСервер = РезультатПередачиФайлов;
Результат.СохраняемыйКонтекстСервер.Вставить("ПолноеИмя", ЮТФайлы.ОбъединитьПути(Результат.СохраняемыйКонтекстСервер.РабочийКаталог, Параметры.ПутьКФайлуПроекта));
Результат.СохраняемыйКонтекстСервер.Вставить("ПолноеИмя", ПолноеИмяСервер);
КонецЕсли;

ЮТАсинхроннаяОбработкаСлужебныйКлиент.ВызватьСледующийОбработчик(Параметры, Результат);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@
// Регистрирует ошибку загрузки тестов
//
// Параметры:
// Объект - Структура - см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля или см. ЮТФабрикаСлужебный.ОписаниеТестовогоНабора или см. ЮТФабрикаСлужебный.ОписаниеТеста
// Объект - Структура - Описание регистрируемого модуля, набора или теста
// Описание - Строка - Описания ошибки, места возникновения
// Ошибка - ИнформацияОбОшибке
Процедура ЗарегистрироватьОшибкуЧтенияТестов(Объект, Описание, Ошибка) Экспорт
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -226,7 +226,7 @@

// Вызов события "ПослеЧтенияСценариев"
// Параметры:
// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей, которые содержат информацию о запускаемых тестах
// Сценарии - Массив из см. ЮТФабрикаСлужебный.ОписаниеТестовогоМодуля - Набор описаний тестовых модулей
Процедура ПослеЧтенияСценариев(Сценарии) Экспорт

Параметры = ЮТКоллекции.ЗначениеВМассиве(Сценарии);
Expand Down
32 changes: 22 additions & 10 deletions exts/yaxunit/src/CommonModules/ЮТСтроки/Module.bsl
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,13 @@
// Параметры:
// ИсходнаяСтрока - Строка - Строка, к которой будет добавлена дополнительная строка.
// ДополнительнаяСтрока - Строка - Строка, которую нужно добавить к исходной строке.
// Разделитель - Строка - Символ или набор символов, используемых для разделения исходной и дополнительной строк. По умолчанию используется точка с запятой (";").
// Разделитель - Строка - Символ или набор символов, используемых для разделения исходной и дополнительной строк.
// По умолчанию используется точка с запятой (";").
//
// Возвращаемое значение:
// Строка - Новая строка, образованная объединением исходной и дополнительной строк с помощью указанного разделителя. Если дополнительная строка пустая, то возвращается исходная строка. Если исходная строка пустая, а дополнительная нет, то возвращается дополнительная строка.
// Строка - Новая строка, образованная объединением исходной и дополнительной строк с помощью указанного разделителя.
// Если дополнительная строка пустая, то возвращается исходная строка.
// Если исходная строка пустая, а дополнительная нет, то возвращается дополнительная строка.
//
// Пример:
// ИсходнаяСтрока = "Привет";
Expand Down Expand Up @@ -57,14 +60,16 @@
// Ограничения:
// - Метод не изменяет исходную строку, а создает новый массив строк на ее основе.
// - В отличие от СтрРазделить, в качестве разделителя используется вся переданная строка, а не любой из символов, входящих в нее.
// - Отсекает незначащие символы, стоящие слева от первого значащего символа в строке, и пробелы, стоящие справа от последнего значащего символа в строке.
// - Отсекает незначащие символы, стоящие слева от первого значащего символа в строке,
// и пробелы, стоящие справа от последнего значащего символа в строке.
//
// Параметры:
// Значение - Строка - Строка, которую нужно разделить на подстроки.
// Разделитель - Строка - Символ или набор символов, используемых для разделения строки на подстроки. По умолчанию используется точка с запятой (";").
//
// Возвращаемое значение:
// Массив из Строка - Массив строк, содержащий подстроки, полученные в результате разделения исходной строки. Если исходная строка пустая, то возвращается пустой массив.
// Массив из Строка - Массив строк, содержащий подстроки, полученные в результате разделения исходной строки.
// Если исходная строка пустая, то возвращается пустой массив.
//
// Пример:
// Значение = "Привет; Мир; Как дела?";
Expand Down Expand Up @@ -108,11 +113,14 @@
// Возвращает строку, содержащую все русские буквы в указанном регистре.
//
// Параметры:
// НижнийРегистр - Булево - Если значение равно Истина, то в создаваемой строке будут использованы буквы в нижнем регистре. По умолчанию равно Истина.
// ВерхнийРегистр - Булево - Если значение равно Истина, то в создаваемой строке будут использованы буквы в верхнем регистре. По умолчанию равно Ложь.
// НижнийРегистр - Булево - Если значение равно `Истина`, то в создаваемой строке будут использованы буквы в нижнем регистре.
// По умолчанию равно `Истина`.
// ВерхнийРегистр - Булево - Если значение равно `Истина`, то в создаваемой строке будут использованы буквы в верхнем регистре.
// По умолчанию равно `Ложь`.
//
// Возвращаемое значение:
// Строка - Новая строка, содержащая все русские буквы в указанном регистре. Если оба параметра НижнийРегистр и ВерхнийРегистр равны Истина, то в создаваемой строке будут использованы буквы в обоих регистрах.
// Строка - Новая строка, содержащая все русские буквы в указанном регистре.
// Если оба параметра НижнийРегистр и ВерхнийРегистр равны ``Истина`, то в создаваемой строке будут использованы буквы в обоих регистрах.
//
// Пример:
// Результат = РусскиеБуквы(Истина, Ложь); // Результат будет равен "абвгдеёжзийклмнопрстуфхцчшщъыьэюя".
Expand All @@ -127,11 +135,14 @@
// Возвращает строку, содержащую все английские буквы в указанном регистре.
//
// Параметры:
// НижнийРегистр - Булево - Если значение равно Истина, то в создаваемой строке будут использованы буквы в нижнем регистре. По умолчанию равно Истина.
// ВерхнийРегистр - Булево - Если значение равно Истина, то в создаваемой строке будут использованы буквы в верхнем регистре. По умолчанию равно Ложь.
// НижнийРегистр - Булево - Если значение равно `Истина`, то в создаваемой строке будут использованы буквы в нижнем регистре.
// По умолчанию равно `Истина`.
// ВерхнийРегистр - Булево - Если значение равно `Истина`, то в создаваемой строке будут использованы буквы в верхнем регистре.
// По умолчанию равно `Ложь`.
//
// Возвращаемое значение:
// Строка - Новая строка, содержащая все английские буквы в указанном регистре. Если оба параметра НижнийРегистр и ВерхнийРегистр равны Истина, то в создаваемой строке будут использованы буквы в обоих регистрах.
// Строка - Новая строка, содержащая все английские буквы в указанном регистре.
// Если оба параметра НижнийРегистр и ВерхнийРегистр равны `Истина`, то в создаваемой строке будут использованы буквы в обоих регистрах.
//
// Пример:
// Результат = АнглийскиеБуквы(Истина, Ложь); // Результат будет равен "abcdefghijklmnopqrstuvwxyz".
Expand Down Expand Up @@ -320,6 +331,7 @@
// Строка - Локализованное сообщение платформы
Функция ЛокализованноеСообщениеПлатформы(СтрокаНаРазныхЯзыках) Экспорт

//@skip-check bsl-nstr-string-literal-format
Возврат НСтр(СтрокаНаРазныхЯзыках, ЮТОкружение.ЛокальПлатформы());

КонецФункции
Expand Down
Loading

0 comments on commit e634f77

Please sign in to comment.