fix(sonar): исправлены замечания сонара

exts/yaxunit/src/CommonModules/ЮТЗапросыСлужебныйВызовСервера/Module.bsl

@@ -273,7 +273,7 @@
 
 КонецФункции
 
-Функция ШаблонУсловия(ВыражениеПредиката, ВыраженияПредикатов, ТипРеквизита)
+Функция ШаблонУсловия(ВыражениеПредиката, ВыраженияПредикатов, ТипРеквизита) // BSLLS:CognitiveComplexity-off
 	Выражение = ВыражениеПредиката.ВидСравнения;
 
 	Отрицание = ЮТПредикатыСлужебныйКлиентСервер.ЭтоВыраженияОтрицания(Выражение);

exts/yaxunit/src/CommonModules/ЮТПредикатыСлужебныйКлиентСервер/Module.bsl

@@ -132,7 +132,7 @@
 
 КонецФункции
 
-Функция ШаблонВыражения(Знач Выражение) Экспорт
+Функция ШаблонВыражения(Знач Выражение) Экспорт // BSLLS:CognitiveComplexity-off
 
 	Отрицание = СтрНачинаетсяС(Выражение, "Не");
 	Если Отрицание Тогда

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.
@@ -18,6 +18,15 @@
 
 #Область СлужебныйПрограммныйИнтерфейс
 
+// Проверить тип параметра.
+// 
+// Параметры:
+//  Значение - Произвольный
+//  ОжидаемыйТип - Тип, ОписаниеТипов - Ожидаемый тип или описание типа
+//               - Строка - Имена типов разделенные запятой
+//  ИмяМетода - Строка
+//  ИмяПараметра - Строка
+//  ЕслиУстановлен - Булево - Выполнять проверку только если значение не неопределено
 Процедура ПроверитьТипПараметра(Значение, ОжидаемыйТип, ИмяМетода, ИмяПараметра, ЕслиУстановлен = Ложь) Экспорт
 
 	Если ЕслиУстановлен И Значение = Неопределено Тогда

exts/yaxunit/src/CommonModules/ЮТСравнениеСлужебныйСервер/Module.bsl

@@ -63,57 +63,64 @@
 	// https://github.com/vanessa-opensource/add 
 	// перенесен из Functest
 
-	ТипТабличногоДокумента = ТипЗнч(ТабличныйДокумент);
-	Если ТипТабличногоДокумента <> Тип("ТабличныйДокумент") И ТипТабличногоДокумента <> Тип("ПолеТабличногоДокумента") Тогда
-		ВызватьИсключение "ПолучитьТаблицуЗначенийИзТабличногоДокумента: Требуется тип ТабличныйДокумент или ПолеТабличногоДокумента";
-	КонецЕсли;
+	ЮТПроверкиСлужебный.ПроверитьТипПараметра(ТабличныйДокумент,
+											  Тип("ТабличныйДокумент"),
+											  "ЮТСравнениеСлужебныйСервер.ПолучитьТаблицуЗначенийИзТабличногоДокумента",
+											  "ТабличныйДокумент");
 
 	Если УчитыватьТолькоВидимыеКолонки И УчитыватьТолькоВидимыеСтроки Тогда
-		ЮТТестовыеДанныеСлужебныйСервер.ТаблицаИзТабличногоДокумента(ТипТабличногоДокумента);
+		ЮТТестовыеДанныеСлужебныйСервер.ТаблицаИзТабличногоДокумента(ТабличныйДокумент);
 	КонецЕсли;
 
-	НомерПоследнейКолонки = ТабличныйДокумент.ШиринаТаблицы;
-	НомерПоследнейСтроки = ТабличныйДокумент.ВысотаТаблицы;
-
 	НоваяТаблицаЗначений = Новый ТаблицаЗначений;
-	Колонки = НоваяТаблицаЗначений.Колонки;
-	ТипСтрока = Новый ОписаниеТипов("Строка");
 
-	// TODO При определении видимости не учитывается наличие нескольких форматов строк, сейчас видимость колонки определяется по формату первой строки
-	УчитываемыеКолонки = Новый Массив;
-	Для НомерКолонки = 1 По НомерПоследнейКолонки Цикл
-		ОбластьКолонки = ТабличныйДокумент.Область(0, НомерКолонки, 1, НомерКолонки);
-
-		УчитыватьКолонку = Не УчитыватьТолькоВидимыеКолонки Или ОбластьКолонки.Видимость;
-		Если УчитыватьКолонку Тогда
-			УчитываемыеКолонки.Добавить(НомерКолонки);
-			ШиринаКолонки = ОбластьКолонки.ШиринаКолонки;
-			Если ШиринаКолонки <= 1 Тогда
-				ШиринаКолонки = 1;
-			КонецЕсли;
-			ИмяКолонки = "К" + Формат(Колонки.Количество() + 1, "ЧН=; ЧГ=0");
-			Колонки.Добавить(ИмяКолонки, ТипСтрока, ИмяКолонки, ШиринаКолонки);
-		КонецЕсли;
-	КонецЦикла;
+	УчитываемыеКолонки = УчитываемыеКолонки(ТабличныйДокумент, НоваяТаблицаЗначений, УчитыватьТолькоВидимыеКолонки);
 
 	ГраницаКолонок = УчитываемыеКолонки.ВГраница();
+	НомерПоследнейСтроки = ТабличныйДокумент.ВысотаТаблицы;
+
 	Для НомерСтроки = 1 По НомерПоследнейСтроки Цикл
 
-		Если УчитыватьТолькоВидимыеСтроки И Не ТабличныйДокумент.Область(НомерСтроки, , НомерСтроки).Видимость Тогда
+		УчитыватьСтроку = УчитыватьТолькоВидимыеСтроки И Не ТабличныйДокумент.Область(НомерСтроки, , НомерСтроки).Видимость;
+		Если УчитыватьСтроку Тогда
 			Продолжить;
 		КонецЕсли;
 
 		НоваяСтрока = НоваяТаблицаЗначений.Добавить();
 
 		Для Индекс = 0 По ГраницаКолонок Цикл
-			НомерКолонки = УчитываемыеКолонки[Индекс];
-			Область = ТабличныйДокумент.Область(НомерСтроки, НомерКолонки, НомерСтроки, НомерКолонки);
+			Область = ТабличныйДокумент.Область(НомерСтроки, УчитываемыеКолонки[Индекс]);
 			НоваяСтрока[Индекс] = Область.Текст;
 		КонецЦикла;
+
 	КонецЦикла;
 
 	Возврат НоваяТаблицаЗначений;
 
 КонецФункции
 
+Функция УчитываемыеКолонки(ТабличныйДокумент, ТаблицаЗначений, УчитыватьТолькоВидимыеКолонки)
+
+	ТипСтрока = Новый ОписаниеТипов("Строка");
+	НомерПоследнейКолонки = ТабличныйДокумент.ШиринаТаблицы;
+	Колонки = ТаблицаЗначений.Колонки;
+
+	// При определении видимости не учитывается наличие нескольких форматов строк, сейчас видимость колонки определяется по формату первой строки
+	УчитываемыеКолонки = Новый Массив;
+	Для НомерКолонки = 1 По НомерПоследнейКолонки Цикл
+
+		ОбластьКолонки = ТабличныйДокумент.Область(1, НомерКолонки);
+		УчитыватьКолонку = Не УчитыватьТолькоВидимыеКолонки Или ОбластьКолонки.Видимость;
+
+		Если УчитыватьКолонку Тогда
+			УчитываемыеКолонки.Добавить(НомерКолонки);
+			ИмяКолонки = "К" + Формат(Колонки.Количество() + 1, "ЧН=; ЧГ=0");
+			Колонки.Добавить(ИмяКолонки, ТипСтрока);
+		КонецЕсли;
+	КонецЦикла;
+
+	Возврат УчитываемыеКолонки;
+
+КонецФункции
+
 #КонецОбласти

exts/yaxunit/src/CommonModules/ЮТТестовыеДанные/Module.bsl

@@ -573,12 +573,15 @@
 // Результат = СлучайныйНомерТелефона(); // Результат будет содержать случайный номер телефона в формате "+7(XXX)XXX-XX-XX".
 // Результат = СлучайныйНомерТелефона("666"); // Результат будет содержать случайный номер телефона в формате "+666(XXX)XXX-XX-XX".
 Функция СлучайныйНомерТелефона(КодСтраны = "7") Экспорт
+	ТриЗнака = 999;
+	ДваЗнака = 99;
+
 	Результат = СтрШаблон("+%1(%2)%3-%4-%5",
 						  ?(ПустаяСтрока(КодСтраны), "7", КодСтраны),
-						  Формат(СлучайноеЧисло(0, 999), "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;"),
-						  Формат(СлучайноеЧисло(0, 999), "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;"),
-						  Формат(СлучайноеЧисло(0, 99), "ЧЦ=2; ЧН=00; ЧВН=; ЧГ=0;"),
-						  Формат(СлучайноеЧисло(0, 99), "ЧЦ=2; ЧН=00; ЧВН=; ЧГ=0;"));
+						  Формат(СлучайноеЧисло(0, ТриЗнака), "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;"),
+						  Формат(СлучайноеЧисло(0, ТриЗнака), "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;"),
+						  Формат(СлучайноеЧисло(0, ДваЗнака), "ЧЦ=2; ЧН=00; ЧВН=; ЧГ=0;"),
+						  Формат(СлучайноеЧисло(0, ДваЗнака), "ЧЦ=2; ЧН=00; ЧВН=; ЧГ=0;"));
 
 	Возврат Результат;
 

exts/yaxunit/src/CommonModules/ЮТТестовыеДанныеСлужебныйСервер/Module.bsl

@@ -828,15 +828,6 @@
 
 КонецФункции
 
-// Создать объект.
-//
-// Параметры: ОписаниеМенеджера -
-// См. ОписаниеМенеджера
-//  Менеджер - Произвольный - Менеджер
-//  ОписаниеТипа - см. ЮТМетаданные.СтруктураОписанияОбъектаМетаданных
-//  Данные - Структура
-// Возвращаемое значение:
-//  Произвольный - Создать объект
 Функция СоздатьОбъект(Менеджер, ОписаниеТипа, Данные)
 
 	Если ОписаниеТипа.Конструктор = "СоздатьЭлемент" Тогда
@@ -872,14 +863,6 @@
 
 КонецФункции
 
-// Записать объект.
-//
-// Параметры:
-//  Объект - Произвольный -  Объект
-//  ПараметрыЗаписи - см. ЮТОбщий.ПараметрыЗаписи
-//
-// Возвращаемое значение:
-//  ЛюбаяСсылка
 Функция ЗаписатьОбъект(Объект, ПараметрыЗаписи)
 
 	Если ПараметрыЗаписи.ОбменДаннымиЗагрузка Тогда

exts/yaxunit/src/CommonModules/ЮТУтверждения/Module.bsl

@@ -1850,7 +1850,7 @@
 	ПолноеОписаниеПроверки = ЮТСтроки.ДобавитьСтроку(Контекст.ПрефиксОшибки, ОписаниеПроверки, " ");
 
 	ПараметрыСообщенийОбОшибке = ЮТПредикатыСлужебныйКлиентСервер.ПараметрыСообщенийОбОшибке(ПолноеОписаниеПроверки,
-																							 ,
+																							 "проверяемое значение",
 																							 Контекст.ОбъектПроверки.Представление);
 	Результат = ЮТПредикатыСлужебныйКлиентСервер.ПроверитьПредикаты(Контекст.ОбъектПроверки.Значение,
 																	Предикат,

tests/src/CommonModules/ОМ_ЮТСравнениеСлужебныйКлиентСервер/Module.bsl

@@ -164,17 +164,38 @@
 
 Процедура ЗначенияРавны_ТабличныйДокумент() Экспорт
 
-	Макет1 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_МакетТестовыхДанных");
-	Макет2 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_МакетТестовыхДанных");
-
-	Макет3 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_МакетТестовыхДанных.R2C1:R5C10");
-	Макет4 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_МакетТестовыхДанных.R2C2:R5C11");
+	Равные = ЮТест.Варианты("Область1, Область2")
+		.Добавить("R2C1:R3C2", "R2C3:R3C4")
+		.Добавить("R2C1:R2C2", "R2C3:R2C4")
+		.Добавить("R4C2", "R4C6");
+
+	Для Каждого Вариант Из Равные.СписокВариантов() Цикл
+		Макет1 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_СравнениеТабличныхДокументов." + Вариант.Область1);
+		Макет2 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_СравнениеТабличныхДокументов." + Вариант.Область2);
+
+		Описание = СтрШаблон("Равные области: `%1` и `%2`", Вариант.Область1, Вариант.Область2);
+		ЮТест.ОжидаетЧто(Макет1, Описание)
+			.Равно(Макет2);
+	КонецЦикла;
 
-	ЮТест.ОжидаетЧто(Макет1, "Равные макеты")
-		.Равно(Макет2);
+	НеРавные = ЮТест.Варианты("Область1, Область2")
+		.Добавить("R1C1:R4C2", "R1C3:R4C4")
+		.Добавить("R1C1:R3C2", "R1C3:R3C4")
+		.Добавить("R2C1:R4C2", "R2C3:R4C4")
+		.Добавить("R1C2:R2C2", "R1C4:R2C4")
+		.Добавить("R1C2:R2C2", "R1C4:R2C4")
+		.Добавить("R1C2:R1C3", "R1C4:R1C5")
+		.Добавить("R4C2", "R4C4")
+	;
 
-	ЮТест.ОжидаетЧто(Макет3, "Разные макеты")
-		.НеРавно(Макет4);
+	Для Каждого Вариант Из НеРавные.СписокВариантов() Цикл
+		Макет1 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_СравнениеТабличныхДокументов." + Вариант.Область1);
+		Макет2 = ЮТОбщий.Макет("ОбщийМакет.ЮТ_СравнениеТабличныхДокументов." + Вариант.Область2);
+
+		Описание = СтрШаблон("Разные области: `%1` и `%2`", Вариант.Область1, Вариант.Область2);
+		ЮТест.ОжидаетЧто(Макет1, Описание)
+			.НеРавно(Макет2);
+	КонецЦикла;
 
 КонецПроцедуры