замечания Сонара

Author: artbear

epf/bddRunner/bddRunner/Forms/ИсследовательФормы/Ext/Form/Module.bsl

@@ -283,6 +283,8 @@
 
 		ТекРодитель = ТекРодитель.ПолучитьРодителя();
 	КонецЦикла;
+
+	Возврат Неопределено;
 КонецФункции
 
 &НаКлиенте

epf/bddRunner/bddRunner/Forms/МодульАсинк/Ext/Form/Module.bsl

@@ -336,7 +336,7 @@
 КонецПроцедуры
 
 &НаКлиенте
-Процедура смв_ПоказатьЗначение(ОписаниеОповещенияОЗавершении = Неопределено, Значение) Экспорт
+Процедура смв_ПоказатьЗначение(ОписаниеОповещенияОЗавершении = Неопределено, Значение = Неопределено) Экспорт
 
 	#Если ВебКлиент Тогда
 

epf/bddRunner/bddRunner/Forms/УправляемаяФорма/Ext/Form/Module.bsl

@@ -2341,7 +2341,10 @@
 	ИначеЕсли Тип = Тип("ТестируемаяТаблицаФормы") Тогда
 		ТабДок = ПолучитьЗначениеТестируемаяТаблицаФормы(ПолеРеквизит);
 		Возврат ТабДок;
+	Иначе
+		Возврат "";
 	КонецЕсли;
+
 КонецФункции
 
 &НаКлиенте

epf/bddRunner/bddRunner/Forms/Форма/Ext/Form/Module.bsl

@@ -819,6 +819,7 @@
 
 // TODO почему пустое тело метода ?? ПолучитьТелоМодуляВМассивСервер(Стр)
 Функция ПолучитьТелоМодуляВМассивСервер(Стр)
+	Возврат Неопределено;
 КонецФункции
 
 // TODO Дубль кода в методах "УбратьСпецсимволыИзИмениОбработки" из УФ bddRunner, ОФ bddRunner и модуля объекта из vbFeatureReader
@@ -1054,6 +1055,7 @@
 
 // TODO почему пустое тело метода ?? НайтиФайлВМассивеФайлов(МассивФайлов,ИмяФайлаEpf)
 Функция НайтиФайлВМассивеФайлов(МассивФайлов,ИмяФайлаEpf)
+	Возврат Неопределено;
 КонецФункции
 
 Функция РаспаковатьEPF(СтруктураОписанияEpf)
@@ -1115,12 +1117,15 @@
 
 		ОчиститьСодержимоеКаталога(ВременныйКаталогБазы);
 
+		Возврат СтруктураОписанияEpf.ВременноеИмяМодуля;
+
 	Исключение
 		ИнформацияОбОшибке = ИнформацияОбОшибке();
 		СделатьСообщение("Не смог распаковать " + ИмяФайла);
 		СделатьСообщение(ПодробноеПредставлениеОшибки(ИнформацияОбОшибке));
-		Возврат Неопределено;
 	КонецПопытки;
+	
+	Возврат Неопределено;
 КонецФункции
 
 Функция СоздатьСтруктураОписанияEpf(ОтносительныйКаталогФичи,ИмяТекущейФичи,GenerateEpf,ДвДанныеvbFeatureReader,ДополнительныеПараметры)

epf/xddTestRunner/xddTestRunner/Forms/МодульАсинк/Ext/Form/Module.bsl

@@ -339,7 +339,7 @@
 КонецПроцедуры
 
 &НаКлиенте
-Процедура смв_ПоказатьЗначение(ОписаниеОповещенияОЗавершении = Неопределено, Значение) Экспорт
+Процедура смв_ПоказатьЗначение(ОписаниеОповещенияОЗавершении = Неопределено, Значение = Неопределено) Экспорт
 
 	#Если ВебКлиент Тогда
 

epf/xddTestRunner/xddTestRunner/Forms/УправляемаяФорма/Ext/Form/Module.bsl

@@ -77,25 +77,14 @@
 
 	ГлобальныйКонтекстИнициализировать();
 
-	//КэшироватьПеречисленияПлагинов();
-	//СброситьПодключенныеВнешниеОбработки();
-	//
-	//КаталогПроекта = КаталогВременныхФайлов();
-
-	//ЭтоПакетныйЗапуск = ЗначениеЗаполнено(ПараметрЗапуска);
-	//Если ЭтоПакетныйЗапуск Тогда
-	//	ВыполнитьПакетныйЗапуск(ПараметрЗапуска);
-	//Иначе
-	//	ПерезагрузитьПоследниеТестыПоИстории();
-	//КонецЕсли;
 КонецПроцедуры
 
 &НаКлиенте
 Процедура ПриОткрытииЗавершение()
 	КэшироватьПеречисленияПлагинов();
 	СброситьПодключенныеВнешниеОбработки();
 
-	КаталогПроекта = ПолучитьКаталогВременныхФайлов();
+	//КаталогПроекта = ПолучитьНовыйКаталогВременныхФайлов(); // TODO бесполезный код, т.к. Объект.КаталогПроекта
 
 	ЭтоПакетныйЗапуск = Найти(ПараметрЗапуска, "xdd") > 0;
 	Если ЭтоПакетныйЗапуск Тогда
@@ -974,7 +963,10 @@
 
 	Попытка
 
-		РезультатТестирования = ВыполнитьТестовыйМетодНаСервере(КонтекстВыполнения, ЭлементДереваТестов, МассивСообщений);
+		ОписаниеРезультатаТестирования = ВыполнитьТестовыйМетодНаСервере(КонтекстВыполнения, ЭлементДереваТестов);
+		
+		РезультатТестирования = ОписаниеРезультатаТестирования.РезультатТестирования;
+		МассивСообщений = ОписаниеРезультатаТестирования.МассивСообщений;
 
 		Если ТестУпалИлиНеРеализован(РезультатТестирования.Состояние) Тогда
 			ВывестиНакопленныеСообщенияОтСервера(МассивСообщений);
@@ -1022,7 +1014,10 @@
 КонецПроцедуры
 
 &НаСервере
-Функция ВыполнитьТестовыйМетодНаСервере(Знач ОписаниеКонтекста, Знач ЭлементДереваТестов, МассивСообщений)
+Функция ВыполнитьТестовыйМетодНаСервере(Знач ОписаниеКонтекста, Знач ЭлементДереваТестов)
+	
+	Результат = Новый Структура("РезультатТестирования, МассивСообщений", Неопределено, Новый Массив);
+	
 	ОбъектНаСервере = ЭтотОбъектНаСервере();
 
 	Если ОписаниеКонтекста.ЭтоОбщийМодуль Тогда
@@ -1049,11 +1044,11 @@
 
 	КонецЕсли;
 
-	РезультатТестирования = ОбъектНаСервере.ВыполнитьТестовыйМетод(КонтекстВыполнения, ЭлементДереваТестов);
+	Результат.РезультатТестирования = ОбъектНаСервере.ВыполнитьТестовыйМетод(КонтекстВыполнения, ЭлементДереваТестов);
 
-	МассивСообщений = ПолучитьСообщенияПользователю(Истина);
+	Результат.МассивСообщений = ПолучитьСообщенияПользователю(Истина);
 
-	Возврат РезультатТестирования;
+	Возврат Результат;
 КонецФункции
 
 &НаКлиенте
@@ -2826,13 +2821,6 @@
 	Возврат Версия1БольшеИлиРавно;
 КонецФункции
 
-&НаКлиентеНаСервереБезКонтекста
-Функция ПолучитьКаталогВременныхФайлов()
-	ИмяВременногоФайла = ПолучитьИмяВременногоФайла("123");
-	ВременныйФайл = Новый Файл(ИмяВременногоФайла);
-	Возврат ВременныйФайл.Путь;
-КонецФункции
-
 // Универсальная функция для проверки наличия свойств у значения любого типа данных
 //
 // Параметры:

examples/features/Buh30/step_definitions/Avance/Avance/Forms/Форма/Ext/Form/Module.bsl

@@ -125,7 +125,6 @@
 	ФайлПроверкаСуществования = Новый Файл(ПутьК1С);
 	Если НЕ ФайлПроверкаСуществования.Существует() Тогда
 		ВызватьИсключение "Не нашел путь к 1cv8c.exe: " + ПутьК1С;
-		Возврат;
 	КонецЕсли;	 
 
 	КаталогБазы = СтрокаСоединенияИнформационнойБазы();

features/libraries/UITestRunner/step_definitions/ОткрытьTestClient/ОткрытьTestClient/Forms/Форма/Ext/Form/Module.bsl

@@ -106,7 +106,6 @@
 
 	Если НЕ Ванесса.ФайлСуществуетКомандаСистемы(ПутьК1С) Тогда
 		ВызватьИсключение Ванесса.СтрШаблон_("Не нашел путь к %1: %2", ИмяФайлаПриложения1С, ПутьК1С);
-		Возврат;
 	КонецЕсли;
 
 	КаталогБазы = СтрокаСоединенияИнформационнойБазы();

features/libraries/Данные/step_definitions/Регламентные_задания/Регламентные_задания/Forms/Форма/Ext/Form/Module.bsl

@@ -350,7 +350,10 @@
 		ВызватьИсключение "Фоновое задание завершено аварийно. " + РасшифровкаОшибки;
 	ИначеЕсли ФЗ.Состояние = СостояниеФоновогоЗадания.Отменено Тогда
 		ВызватьИсключение "Фоновое задание завершено отменено. " + РасшифровкаОшибки;
+	Иначе
+		ВызватьИсключение "Попали в ветку в ФоновоеЗаданиеЗавершено, в которую не должны были попасть. " + РасшифровкаОшибки;
 	КонецЕсли;
+
 КонецФункции
 
 &НаСервереБезКонтекста

lib/cfe/ОткрытиеВнешнихФайлов/DataProcessors/ОткрытьВнешнийФайл/Forms/МодульАсинк/Ext/Form/Module.bsl

@@ -341,7 +341,7 @@
 КонецПроцедуры
 
 &НаКлиенте
-Процедура смв_ПоказатьЗначение(ОписаниеОповещенияОЗавершении = Неопределено, Значение) Экспорт
+Процедура смв_ПоказатьЗначение(ОписаниеОповещенияОЗавершении = Неопределено, Значение = Неопределено) Экспорт
 
 	#Если ВебКлиент Тогда