337 / Оптимизация клиент-серверного взаимодействия при выполнении тестов #361
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
| @@ -103,4 +106,24 @@ | |||
|
|
|||
| КонецФункции | |||
|
|
|||
| Процедура ПриоритезироватьОбработчикиСобытий(ГруппаОбработчиков, Модули) | |||
There was a problem hiding this comment.
Возможная опечатка в "Приоритезировать" Why is this an issue?
| КонецФункции | ||
|
|
||
| #Если Сервер Тогда | ||
| Функция СерверныйВызовОбработкиСобытияИсполненияТестов(ИмяСобытия, ИдентификаторМодуля = Неопределено, ИдентификаторНабора = Неопределено, ИдентификаторТеста = Неопределено) Экспорт |
There was a problem hiding this comment.
Длина строки 181 превышает максимально допустимую 150 Why is this an issue?
| Возврат СтрШаблон("%1.%2", Дата, Формат(УниверсальнаяДатаВМиллисекундах % 1000, "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;")); | ||
| ФорматДата = ?(ТолькоВремя, "ДФ='HH:mm:ss';", "ДФ='dd.MM.yyyy HH:mm:ss';"); | ||
| ФорматМиллисекунды = "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;"; | ||
| Возврат СтрШаблон("%1.%2", Формат(Дата, ФорматДата), Формат(УниверсальнаяДатаВМиллисекундах % 1000, ФорматМиллисекунды)); |
There was a problem hiding this comment.
Создайте константу с понятным названием, присвойте ей значение "1000" и используйте эту константу вместо магического числа. Why is this an issue?
| // | ||
| // Возвращаемое значение: | ||
| // Массив из см. ЮТФабрикаСлужебный.ОписаниеИсполняемогоНабораТестов - Результат прогона наборов тестов, структура набора см. ЮТФабрикаСлужебный.ОписаниеИсполняемогоНабораТестов | ||
| Функция ВыполнитьГруппуНаборовТестов(Знач Наборы, Знач ТестовыйМодуль) Экспорт | ||
| Функция ВыполнитьГруппуНаборовТестов(Знач ИдентификаторыТестовыхНаборов, Знач ИдентификаторТестовогоМодуля) Экспорт |
There was a problem hiding this comment.
Необходимо добавить описание параметра "ИдентификаторыТестовыхНаборов" Why is this an issue?
| ПредставлениеРежима = СтрШаблон(" [%1]", РезультатТеста.Режим); | ||
| ПолныйИдентификаторТеста = СтрШаблон("%1_%2_%3", РезультатТеста.НомерВНаборе, РезультатТеста.ПолноеИмяМетода, РезультатТеста.Режим); | ||
| ПолныйИдентификаторТеста = СтрШаблон("%1_%2.3_%4", РезультатТеста.НомерВНаборе, ПолноеИмяМетода, РезультатТеста.Режим); |
There was a problem hiding this comment.
Исправьте передачу параметров при вызове метода "СтрШаблон" Why is this an issue?
|
|
||
| #Область СлужебныйПрограммныйИнтерфейс | ||
|
|
||
| Функция НовыйИсполняемый(ОписаниеБазовогоОбъекта) |
There was a problem hiding this comment.
Переместите неэкспортный метод "НовыйИсполняемый" из области "СлужебныйПрограммныйИнтерфейс" Why is this an issue?
|
|
||
| #Область СлужебныйПрограммныйИнтерфейс | ||
|
|
||
| Функция НовыйИсполняемый(ОписаниеБазовогоОбъекта) |
There was a problem hiding this comment.
Метод "НовыйИсполняемый" не вызывается в теле модуля Why is this an issue?
|
|
||
| #Область СлужебныйПрограммныйИнтерфейс | ||
|
|
||
| Функция НовыйИсполняемый(ОписаниеБазовогоОбъекта) |
There was a problem hiding this comment.
Функция не содержит "Возврат" Why is this an issue?
|
|
||
| КонецФункции | ||
|
|
||
| Функция РезультатТесирования(ДанныеИсполняемого) Экспорт |
There was a problem hiding this comment.
Возможная опечатка в "Тесирования" Why is this an issue?
|
|
||
| КонецФункции | ||
|
|
||
| Функция РезультатТесирования(ДанныеИсполняемого) Экспорт |
There was a problem hiding this comment.
Функция не содержит "Возврат" Why is this an issue?
SonarQube Code AnalyticsQuality Gate passedAdditional informationThe following metrics might not affect the Quality Gate status but improving them will improve your project code quality. IssuesCoverage and DuplicationsNo data Coverage |
| @@ -103,4 +106,24 @@ | |||
|
|
|||
| КонецФункции | |||
|
|
|||
| Процедура ПриоритезироватьОбработчикиСобытий(ГруппаОбработчиков, Модули) | |||
There was a problem hiding this comment.
Возможная опечатка в "Приоритезировать" Why is this an issue?
| КонецФункции | ||
|
|
||
| #Если Сервер Тогда | ||
| Функция СерверныйВызовОбработкиСобытияИсполненияТестов(ИмяСобытия, ИдентификаторМодуля = Неопределено, ИдентификаторНабора = Неопределено, ИдентификаторТеста = Неопределено) Экспорт |
There was a problem hiding this comment.
Длина строки 181 превышает максимально допустимую 150 Why is this an issue?
| Возврат СтрШаблон("%1.%2", Дата, Формат(УниверсальнаяДатаВМиллисекундах % 1000, "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;")); | ||
| ФорматДата = ?(ТолькоВремя, "ДФ='HH:mm:ss';", "ДФ='dd.MM.yyyy HH:mm:ss';"); | ||
| ФорматМиллисекунды = "ЧЦ=3; ЧН=000; ЧВН=; ЧГ=0;"; | ||
| Возврат СтрШаблон("%1.%2", Формат(Дата, ФорматДата), Формат(УниверсальнаяДатаВМиллисекундах % 1000, ФорматМиллисекунды)); |
There was a problem hiding this comment.
Создайте константу с понятным названием, присвойте ей значение "1000" и используйте эту константу вместо магического числа. Why is this an issue?
| // | ||
| // Возвращаемое значение: | ||
| // Массив из см. ЮТФабрикаСлужебный.ОписаниеИсполняемогоНабораТестов - Результат прогона наборов тестов, структура набора см. ЮТФабрикаСлужебный.ОписаниеИсполняемогоНабораТестов | ||
| Функция ВыполнитьГруппуНаборовТестов(Знач Наборы, Знач ТестовыйМодуль) Экспорт | ||
| Функция ВыполнитьГруппуНаборовТестов(Знач ИдентификаторыТестовыхНаборов, Знач ИдентификаторТестовогоМодуля) Экспорт |
There was a problem hiding this comment.
Необходимо добавить описание параметра "ИдентификаторыТестовыхНаборов" Why is this an issue?
| ПредставлениеРежима = СтрШаблон(" [%1]", РезультатТеста.Режим); | ||
| ПолныйИдентификаторТеста = СтрШаблон("%1_%2_%3", РезультатТеста.НомерВНаборе, РезультатТеста.ПолноеИмяМетода, РезультатТеста.Режим); | ||
| ПолныйИдентификаторТеста = СтрШаблон("%1_%2.3_%4", РезультатТеста.НомерВНаборе, ПолноеИмяМетода, РезультатТеста.Режим); |
There was a problem hiding this comment.
Исправьте передачу параметров при вызове метода "СтрШаблон" Why is this an issue?
|
|
||
| #Область СлужебныйПрограммныйИнтерфейс | ||
|
|
||
| Функция НовыйИсполняемый(ОписаниеБазовогоОбъекта) |
There was a problem hiding this comment.
Переместите неэкспортный метод "НовыйИсполняемый" из области "СлужебныйПрограммныйИнтерфейс" Why is this an issue?
|
|
||
| #Область СлужебныйПрограммныйИнтерфейс | ||
|
|
||
| Функция НовыйИсполняемый(ОписаниеБазовогоОбъекта) |
There was a problem hiding this comment.
Метод "НовыйИсполняемый" не вызывается в теле модуля Why is this an issue?
|
|
||
| #Область СлужебныйПрограммныйИнтерфейс | ||
|
|
||
| Функция НовыйИсполняемый(ОписаниеБазовогоОбъекта) |
There was a problem hiding this comment.
Функция не содержит "Возврат" Why is this an issue?
|
|
||
| КонецФункции | ||
|
|
||
| Функция РезультатТесирования(ДанныеИсполняемого) Экспорт |
There was a problem hiding this comment.
Возможная опечатка в "Тесирования" Why is this an issue?
|
|
||
| КонецФункции | ||
|
|
||
| Функция РезультатТесирования(ДанныеИсполняемого) Экспорт |
There was a problem hiding this comment.
Функция не содержит "Возврат" Why is this an issue?
| // | ||
| // Возвращаемое значение: | ||
| // Массив из см. ЮТФабрикаСлужебный.ОписаниеИсполняемогоНабораТестов - Результат прогона наборов тестов, структура набора см. ЮТФабрикаСлужебный.ОписаниеИсполняемогоНабораТестов | ||
| Функция ВыполнитьГруппуНаборовТестов(Знач Наборы, Знач ТестовыйМодуль) Экспорт | ||
| Функция ВыполнитьГруппуНаборовТестов(Знач ИдентификаторыТестовыхНаборов, Знач ИдентификаторТестовогоМодуля) Экспорт |
There was a problem hiding this comment.
Необходимо добавить описание всех параметров метода Why is this an issue?
Описание изменений
Close #337