refactoring

Author: ovcharenko-di

features/step_definitions/limit.os

@@ -1,11 +1,12 @@
-// Реализация шагов BDD-фич/сценариев c помощью фреймворка https://github.com/artbear/1bdd
+// BSLLS:MissingParameterDescription-off
+
 #Использовать tempfiles
 #Использовать asserts
 
 Перем БДД; //контекст фреймворка 1bdd
 
 // Метод выдает список шагов, реализованных в данном файле-шагов
-Функция ПолучитьСписокШагов(КонтекстФреймворкаBDD) Экспорт
+Функция ПолучитьСписокШагов(КонтекстФреймворкаBDD) Экспорт // BSLLS:MissingReturnedValueDescription-off
 	БДД = КонтекстФреймворкаBDD;
 
 	ВсеШаги = Новый Массив;
@@ -23,7 +24,7 @@
 КонецПроцедуры
 
 // Процедура выполняется после завершения каждого сценария
-Процедура ПослеЗапускаСценария(Знач Узел) Экспорт
+Процедура ПослеЗапускаСценария(Знач Узел) Экспорт // BSLLS:UnusedParameters-off
 	ВременныеФайлы.Удалить();
 КонецПроцедуры
 

features/step_definitions/shared.os

@@ -1,7 +1,5 @@
-// BSLLS:UnusedParameters-off
-// BSLLS:LatinAndCyrillicSymbolInWord-off
+// BSLLS:LatinAndCyrillicSymbolInWord-off
 // BSLLS:MissingParameterDescription-off
-// BSLLS:MissingReturnedValueDescription-off
 
 // Реализация шагов BDD-фич/сценариев c помощью фреймворка https://github.com/artbear/1bdd
 #Использовать gitrunner
@@ -12,7 +10,7 @@
 Перем ЭтоWindows;
 
 // Метод выдает список шагов, реализованных в данном файле-шагов
-Функция ПолучитьСписокШагов(КонтекстФреймворкаBDD) Экспорт
+Функция ПолучитьСписокШагов(КонтекстФреймворкаBDD) Экспорт // BSLLS:MissingReturnedValueDescription-off
 	БДД = КонтекстФреймворкаBDD;
 
 	ВсеШаги = Новый Массив;
@@ -45,7 +43,6 @@
 	ВсеШаги.Добавить("ЯСоздаюВременныйКаталогИСохраняюЕгоВПеременной");
 	ВсеШаги.Добавить("ЯСкопировалКаталогТестовогоХранилищаКонфигурацииВКаталогИзПеременной");
 
-	
 	Возврат ВсеШаги;
 
 КонецФункции
@@ -130,7 +127,6 @@
 	УстановитьТекущийКаталог(ОбъединитьПути(ТекущийКаталог(), ВременныйКаталог));
 КонецПроцедуры
 
-
 //Я выключаю все плагины
 Процедура ЯВыключаюВсеПлагины() Экспорт
 
@@ -150,15 +146,13 @@
 	Возврат "bdd.gitsync.feature";
 КонецФункции
 
-
 // Реализация шагов
 
 // Процедура выполняется перед запуском каждого сценария
 Процедура ПередЗапускомСценария(Знач Узел) Экспорт
 	// ЯСоздаюНовыйОбъектГитрепозиторий()
 КонецПроцедуры
 
-
 //Я инициализирую репозиторий в каталоге из переменной "ПутьКаталогаИсходников"
 Процедура ЯИнициализируюРепозиторийВКаталогеИзПеременной(Знач ПутьКаталогаИсходников) Экспорт
 	ГитРепозиторий = БДД.ПолучитьИзКонтекста("ГитРепозиторий");
@@ -172,10 +166,9 @@
 КонецПроцедуры
 
 //Я наполняю bare репозиторий из переменной "URLРепозитория" тестовыми данными
-Процедура ЯНаполняюBareРепозиторийИзПеременнойТестовымиДанными(Знач ПеременнаяURLРепозитория) Экспорт
-
+Процедура ЯНаполняюBareРепозиторийИзПеременнойТестовымиДанными(Знач URLРепозитория) Экспорт
 	ГитРепозиторий = БДД.ПолучитьИзКонтекста("ГитРепозиторий");
-	URLРепозитория = БДД.ПолучитьИзКонтекста(ПеременнаяURLРепозитория);
+	URLРепозитория = БДД.ПолучитьИзКонтекста(URLРепозитория);
 
 	ВременныйКаталог = ВременныеФайлы.СоздатьКаталог();
 
@@ -208,7 +201,7 @@
 КонецПроцедуры
 
 // Процедура выполняется после завершения каждого сценария
-Процедура ПослеЗапускаСценария(Знач Узел) Экспорт
+Процедура ПослеЗапускаСценария(Знач Узел) Экспорт // BSLLS:UnusedParameters-off
 	ВременныеФайлы.Удалить();
 КонецПроцедуры
 
@@ -287,11 +280,11 @@
 КонецПроцедуры
 
 //я инициализирую связь "ПутьКаталогаИсходников" с внешним репозиторием "URLРепозитория"
-Процедура ЯИнициализируюСвязьСВнешнимРепозиторием(Знач ПеременнаяПутьКаталогаИсходников, Знач ПеременнаяURLРепозитория) Экспорт
+Процедура ЯИнициализируюСвязьСВнешнимРепозиторием(Знач ПутьКаталогаИсходников, Знач URLРепозитория) Экспорт
 
 	ГитРепозиторий = БДД.ПолучитьИзКонтекста("ГитРепозиторий");
-	URLРепозитория = БДД.ПолучитьИзКонтекста(ПеременнаяURLРепозитория);
-	ПутьКаталогаИсходников = БДД.ПолучитьИзКонтекста(ПеременнаяПутьКаталогаИсходников);
+	URLРепозитория = БДД.ПолучитьИзКонтекста(URLРепозитория);
+	ПутьКаталогаИсходников = БДД.ПолучитьИзКонтекста(ПутьКаталогаИсходников);
 
 	ГитРепозиторий.УстановитьРабочийКаталог(ПутьКаталогаИсходников);
 	ГитРепозиторий.Инициализировать();
@@ -458,4 +451,4 @@
 
 ЭтоWindows = ЭтоWindows();
 
-Лог = Логирование.ПолучитьЛог(ИмяЛога());
+Лог = Логирование.ПолучитьЛог(ИмяЛога()); // BSLLS:UnusedLocalVariable-off

features/step_definitions/smart-tags.os

@@ -1,11 +1,13 @@
-#Использовать asserts
+// BSLLS:MissingParameterDescription-off
+
+#Использовать asserts
 
 // Реализация шагов BDD-фич/сценариев c помощью фреймворка https://github.com/artbear/1bdd
 
 Перем БДД; //контекст фреймворка 1bdd
 
 // Метод выдает список шагов, реализованных в данном файле-шагов
-Функция ПолучитьСписокШагов(КонтекстФреймворкаBDD) Экспорт
+Функция ПолучитьСписокШагов(КонтекстФреймворкаBDD) Экспорт // BSLLS:MissingReturnedValueDescription-off
 	БДД = КонтекстФреймворкаBDD;
 
 	ВсеШаги = Новый Массив;
@@ -29,7 +31,6 @@
 
 //Тег "1.0" должен присутствовать в репозитории
 Процедура ТегДолженПрисутствоватьВРепозитории(Знач ОжидаемыйВыводКоманды) Экспорт
-	
 
 	ГитРепозиторий = БДД.ПолучитьИзКонтекста("ГитРепозиторий");
 	ПутьКаталогаИсходников = БДД.ПолучитьИзКонтекста("ПутьКаталогаИсходников");
@@ -44,4 +45,3 @@
 	Ожидаем.Что(Вывод).Содержит(ОжидаемыйВыводКоманды);
 
 КонецПроцедуры
-

src/Классы/disableSupport.os

@@ -1,11 +1,6 @@
 
 #Использовать logos
 
-Перем Лог;
-Перем КомандыПлагина;
-Перем Описание;
-Перем Обработчик;
-
 #Область Интерфейс_плагина
 
 // Возвращает версию плагина
@@ -64,25 +59,19 @@
 
 #КонецОбласти
 
-Процедура ПриАктивизации(СтандартныйОбработчик) Экспорт
-
-	Обработчик = СтандартныйОбработчик;
-
-КонецПроцедуры
-
 Процедура ПередВыгрузкойКонфигурациюВИсходники(Конфигуратор,
-												КаталогРабочейКопии,
-												КаталогВыгрузки,
-												ПутьКХранилищу,
-												НомерВерсииФормат) Экспорт
+												КаталогРабочейКопии,        // BSLLS:UnusedParameters-off
+												КаталогВыгрузки,            // BSLLS:UnusedParameters-off
+												ПутьКХранилищу,             // BSLLS:UnusedParameters-off
+												НомерВерсииФормат) Экспорт  // BSLLS:UnusedParameters-off
 
 	Конфигуратор.СнятьКонфигурациюСПоддержки(Истина);
 
 КонецПроцедуры
 
 Процедура Инициализация()
 
-	Лог = Логирование.ПолучитьЛог(ИмяЛога());
+	Лог = Логирование.ПолучитьЛог(ИмяЛога()); // BSLLS:UnusedLocalVariable-off
 
 КонецПроцедуры
 

src/Классы/dropConfigDump.os

@@ -1,10 +1,12 @@
 #Использовать logos
 #Использовать gitsync
 
+// BSLLS:UnusedLocalVariable-off
 Перем Лог;              //   Лог                 - объект протоколирования (logger)
 Перем Обработчик;       //   Команда             - обработчик текущей команды gitsync
 Перем КомандыПлагина;   //   Массив из Строка    - список команд к которым подключается текущий плагин
 Перем ТекущаяКоманда;   //   Строка              - имя выполняемой команды gitsync
+// BSLLS:UnusedLocalVariable-on
 
 Перем ЗакоммититьФайлИгнорируемыхИзменений;   //   Булево    - Истина - закоммитить файл .gitignore.
 
@@ -95,7 +97,7 @@
 
 КонецПроцедуры
 
-Процедура ПриРегистрацииКомандыПриложения(ИмяКоманды, КлассРеализации) Экспорт
+Процедура ПриРегистрацииКомандыПриложения(ИмяКоманды, КлассРеализации) Экспорт // BSLLS:UnusedParameters-off
 
 	ТекущаяКоманда = ИмяКоманды;
 
@@ -110,7 +112,9 @@
 
 КонецПроцедуры
 
-Процедура ПередВыгрузкойКонфигурациюВИсходники(Конфигуратор,
+// BSLLS:UnusedParameters-off
+
+Процедура ПередВыгрузкойКонфигурациюВИсходники(Конфигуратор, // BSLLS:MissingParameterDescription-off
                                                КаталогРабочейКопии,
                                                КаталогВыгрузки,
                                                ПутьКХранилищу,
@@ -119,14 +123,15 @@
 	УдалитьФайлДампаИзменений(КаталогРабочейКопии);
 
 КонецПроцедуры
+// BSLLS:UnusedParameters-on
 
-Процедура ПослеВыгрузкиКонфигурациюВИсходники(Конфигуратор, КаталогВыгрузки) Экспорт
+Процедура ПослеВыгрузкиКонфигурациюВИсходники(Конфигуратор, КаталогВыгрузки) Экспорт // BSLLS:UnusedParameters-off
 
 	УдалитьФайлДампаИзменений(КаталогВыгрузки);
 
 КонецПроцедуры
 
-Процедура ПередКоммитом(КаталогРабочейКопии, Комментарий, Автор, Дата) Экспорт
+Процедура ПередКоммитом(КаталогРабочейКопии, Комментарий, Автор, Дата) Экспорт // BSLLS:UnusedParameters-off
 
 	КорневойКаталогГит = КорневойКаталогГит(КаталогРабочейКопии);
 
@@ -163,14 +168,18 @@
 
 КонецПроцедуры
 
-Процедура ПриКоммите(ГитРепозиторий,
+// BSLLS:NumberOfParams-off
+// BSLLS:UnusedParameters-off
+Процедура ПриКоммите(ГитРепозиторий, // BSLLS:MissingParameterDescription-off
                      Комментарий,
                      ПроиндексироватьОтслеживаемыеФайлы,
                      ИмяФайлаКомментария,
                      АвторДляГит,
                      ДатаДляГит,
                      Коммитер,
                      ДатаКоммитера) Экспорт
+// BSLLS:NumberOfParams-on
+// BSLLS:UnusedParameters-on
 
 	Если НЕ ЗакоммититьФайлИгнорируемыхИзменений Тогда
 		Возврат;
@@ -212,11 +221,12 @@
 
 	ТекущийКаталог = Новый Файл(КаталогРабочейКопии);
 
+	// BSLLS:MagicNumber-off
 	ЭтоКорневойКаталог = (ТекущийКаталог.ПолноеИмя = "/"
 	                 ИЛИ Прав(ТекущийКаталог.ПолноеИмя, 1) = ":"
 	                 ИЛИ Прав(ТекущийКаталог.ПолноеИмя, 2) = ":/"
 	                 ИЛИ Прав(ТекущийКаталог.ПолноеИмя, 2) = ":\");
-
+	// BSLLS:MagicNumber-on
 	Если ЭтоКорневойКаталог Тогда
 		Возврат "";
 	КонецЕсли;

src/Классы/dropSupport.os

@@ -1,10 +1,10 @@
+// BSLLS:MissingParameterDescription-off
+
 #Использовать logos
 #Использовать gitsync
 
 Перем Лог;              //   Лог                 - объект протоколирования (logger)
-Перем Обработчик;       //   Команда             - обработчик текущей команды gitsync
 Перем КомандыПлагина;   //   Массив из Строка    - список команд к которым подключается текущий плагин
-Перем ТекущаяКоманда;   //   Строка              - имя выполняемой команды gitsync
 
 Перем ЗакоммититьФайлИгнорируемыхИзменений;   //   Булево    - Истина - закоммитить файл .gitignore.
 
@@ -70,15 +70,7 @@
 
 #Область Подписки_на_события
 
-Процедура ПриАктивизации(СтандартныйОбработчик) Экспорт
-
-	Обработчик = СтандартныйОбработчик;
-
-КонецПроцедуры
-
-Процедура ПриРегистрацииКомандыПриложения(ИмяКоманды, КлассРеализации) Экспорт
-
-	ТекущаяКоманда = ИмяКоманды;
+Процедура ПриРегистрацииКомандыПриложения(ИмяКоманды, КлассРеализации) Экспорт // BSLLS:UnusedParameters-off
 
 	Лог.Отладка("Ищу команду <%1> в списке поддерживаемых", ИмяКоманды);
 	Если КомандыПлагина.Найти(ИмяКоманды) = Неопределено Тогда
@@ -91,23 +83,25 @@
 
 КонецПроцедуры
 
+// BSLLS:UnusedParameters-off
 Процедура ПередВыгрузкойКонфигурациюВИсходники(Конфигуратор,
                                                КаталогРабочейКопии,
                                                КаталогВыгрузки,
                                                ПутьКХранилищу,
                                                НомерВерсии) Экспорт
+// BSLLS:UnusedParameters-on
 
 	УдалитьИнформациюОПоддержке(КаталогРабочейКопии);
 
 КонецПроцедуры
 
-Процедура ПослеВыгрузкиКонфигурациюВИсходники(Конфигуратор, КаталогВыгрузки) Экспорт
+Процедура ПослеВыгрузкиКонфигурациюВИсходники(Конфигуратор, КаталогВыгрузки) Экспорт // BSLLS:UnusedParameters-off
 
 	УдалитьИнформациюОПоддержке(КаталогВыгрузки);
 
 КонецПроцедуры
 
-Процедура ПередКоммитом(КаталогРабочейКопии, Комментарий, Автор, Дата) Экспорт
+Процедура ПередКоммитом(КаталогРабочейКопии, Комментарий, Автор, Дата) Экспорт // BSLLS:UnusedParameters-off
 
 	КорневойКаталогГит = КорневойКаталогГит(КаталогРабочейКопии);
 
@@ -142,6 +136,8 @@
 
 КонецПроцедуры
 
+// BSLLS:UnusedParameters-off
+// BSLLS:NumberOfParams-off
 Процедура ПриКоммите(ГитРепозиторий,
                      Комментарий,
                      ПроиндексироватьОтслеживаемыеФайлы,
@@ -150,6 +146,8 @@
                      ДатаДляГит,
                      Коммитер,
                      ДатаКоммитера) Экспорт
+// BSLLS:UnusedParameters-on
+// BSLLS:NumberOfParams-on
 
 	Если НЕ ЗакоммититьФайлИгнорируемыхИзменений Тогда
 		Возврат;
@@ -191,10 +189,12 @@
 
 	ТекущийКаталог = Новый Файл(КаталогРабочейКопии);
 
+	// BSLLS:MagicNumber-off
 	ЭтоКорневойКаталог = (ТекущийКаталог.ПолноеИмя = "/"
 	                 ИЛИ Прав(ТекущийКаталог.ПолноеИмя, 1) = ":"
 	                 ИЛИ Прав(ТекущийКаталог.ПолноеИмя, 2) = ":/"
 	                 ИЛИ Прав(ТекущийКаталог.ПолноеИмя, 2) = ":\");
+	// BSLLS:MagicNumber-on
 
 	Если ЭтоКорневойКаталог Тогда
 		Возврат "";