Merge remote-tracking branch 'khorevaa/develop' into fix/arrayparams

Author: ivanovEV

packagedef

@@ -8,7 +8,7 @@
         .Автор("Khorev Aleksey")
         .АдресАвтора("khorevaa@gmail.com")
         .Описание("Данный пакет облегчает создание консольных приложений на Oscript")
-        .ВерсияСреды("1.0.20")
+        .ВерсияСреды("1.8.3")
         .ВключитьФайл("src")
         .ВключитьФайл("docs")
         //.ВключитьФайл("tests")
@@ -17,13 +17,12 @@
         .ЗависитОт("logos", "1.1.1")
         .ЗависитОт("delegate", "0.2.0")
         .ЗависитОт("reflector", "0.5.1")
-        .ЗависитОт("fluent", "0.3.1")
+        .ЗависитОт("fluent", "0.6.0")
         .ЗависитОт("datetime", "0.1.0")
         //.ЗависитОт("asserts")
         //.ЗависитОт("1bdd") // Необходимо установил для тестирования
         //.ЗависитОт("1testrunner") // Необходимо установил для тестирования
         .ОпределяетКласс("КонсольноеПриложение", "src/core/Классы/КонсольноеПриложение.os")
         .ОпределяетКласс("КомандаПриложения", "src/core/Классы/КомандаПриложения.os")
         .ОпределяетКласс("ПараметрКоманды", "src/core/Классы/ПараметрКоманды.os")
-        
         ;

src/core/Классы/internal/parser/Классы/ОпцииЗавершениеПарсера.os

@@ -11,8 +11,6 @@
 //
 Функция Поиск(Аргументы, КонтекстПоиска) Экспорт
 
-	КонтекстПоиска.СбросОпций = Истина;
-
 	Результат  = Новый Структура("РезультатПоиска, Аргументы", Истина, Аргументы);
 
 	Возврат Результат;

src/core/Классы/КомандаПриложения.os

@@ -117,6 +117,17 @@
 	Возврат Описание;
 КонецФункции
 
+// Функция возвращает строку использования (спек) команды
+//
+// Возвращаемое значение:
+//   Cтрока - строка использования (спек) текущей команды
+//
+Функция ПолучитьСтрокуИспользования() Экспорт
+
+	Возврат ?(Не ПустаяСтрока(Спек), Спек, СформироватьСтандартнуюСтрокуИспользования());
+
+КонецФункции
+
 // Функция возвращает значение опции по переданному имени/синониму опции
 //
 // Параметры:
@@ -493,13 +504,7 @@
 
 	Лог.Отладка("Входящий спек: %1", Спек);
 
-	СтрокаИспользования = Спек;
-
-	Если ПустаяСтрока(СтрокаИспользования) Тогда
-
-		СтрокаИспользования = СформироватьСтандартнуюСтрокуИспользования();
-		
-	КонецЕсли;
+	СтрокаИспользования = ПолучитьСтрокуИспользования();
 
 	Лог.Отладка("Разбираю строку использования с помощью лексера");
 
@@ -935,7 +940,7 @@
 
 		Если Не ПустаяСтрока(СтрокаТаблицы.ПодробноеОписание) Тогда
 
-			СтрокаНаименования = СтрШаблон("  %2%1%", Символы.Таб, СтрокаТаблицы.Наименование);
+			СтрокаНаименования = СтрШаблон("  %2%1", Символы.Таб, СтрокаТаблицы.Наименование);
 
 			ДлинаДополнения = СтрДлина(СтрокаНаименования) + ДобавочнаяДлинаДополнения;
 			МассивСтрок = СтрРазделить(СтрокаТаблицы.ПодробноеОписание, Символы.ПС, Ложь);
@@ -1043,7 +1048,9 @@
 	ПроцессорКоллекций = Новый ПроцессорКоллекций;
 	ПроцессорКоллекций.УстановитьКоллекцию(НаименованияПараметров);
 	СортированныеНаименования = ПроцессорКоллекций
-		.Сортировать("Результат = СтрДлина(Элемент1) >  СтрДлина(Элемент2)")
+		.Сортировать("(Первый, Второй) ->
+			|	Возврат ПроцессорыКоллекций.СтандартнаяФункцияСравненияПрямойПорядок()
+			|		.Выполнить(СтрДлина(Первый), СтрДлина(Второй))")
 		.Получить(Тип("Массив"));
 
 	НаименованиеПараметра = СтрСоединить(СортированныеНаименования, ", ");
@@ -1101,7 +1108,24 @@
 	СтандартнаяСтрокаИспользования = "";
 
 	Лог.Отладка("Количество опций строки: %1", Опции.Количество());
-	Если Опции.Количество() > 0 Тогда
+	ЕстьОбязательнаяОпция = Ложь;
+	Для каждого Опция Из Опции Цикл
+		
+		ИмяОпции = Опция.Ключ.НаименованияПараметров[0];
+		КлассОпции = Опция.Ключ;
+
+		Если КлассОпции.ПолучитьОбязательностьВвода() Тогда
+			ЕстьОбязательнаяОпция = Истина;
+		КонецЕсли;
+
+		ДополнитьИмяАргументаМассива(ИмяОпции, КлассОпции);
+		ДополнитьИмяАргументаНеобязательного(ИмяОпции, КлассОпции);
+
+		СтандартнаяСтрокаИспользования = СтандартнаяСтрокаИспользования + ИмяОпции + " ";
+
+	КонецЦикла;
+
+	Если Не ЕстьОбязательнаяОпция И Опции.Количество() > 0 Тогда
 		СтандартнаяСтрокаИспользования = "[ОПЦИИ] ";
 	КонецЕсли;
 
@@ -1118,7 +1142,7 @@
 		ДополнитьИмяАргументаМассива(ИмяАргумента, КлассАргумента);
 		ДополнитьИмяАргументаНеобязательного(ИмяАргумента, КлассАргумента);
 
-		Лог.Отладка("Добавляю аргумет <%1> в спек <%2>", ИмяАргумента, СтандартнаяСтрокаИспользования);
+		Лог.Отладка("Добавляю аргумент <%1> в спек <%2>", ИмяАргумента, СтандартнаяСтрокаИспользования);
 		СтандартнаяСтрокаИспользования = СтандартнаяСтрокаИспользования + ИмяАргумента + " ";
 
 	КонецЦикла;
@@ -1290,4 +1314,4 @@
 
 КонецПроцедуры
 
-Лог = Логирование.ПолучитьЛог("oscript.lib.cli_command");
+Лог = Логирование.ПолучитьЛог("oscript.lib.cli_command");

tests/КомандаПриложения_test.os

@@ -39,7 +39,7 @@
 	ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования");
 	ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования_МассивАргументов");
 	ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования_МассивАргументовНесколькоАргументов");
-
+	ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования_ОбязательнаяОпция");
 	// ИменаТестов.Добавить("ТестДолжен_СложныеСлучаиФормированияСтрокиИспользования");
 
 	ИменаТестов.Добавить("ТестДолжен_ПроверитьВыводСправки");
@@ -354,6 +354,37 @@
 
 КонецПроцедуры
 
+Процедура ТестДолжен_СтандартнуюСтрокуИспользования_ОбязательнаяОпция() Экспорт
+	
+	Команда = ПодготовитьТестовуюКоманду();
+
+	ОбычныйОпция = Команда.Опция("z", , "Необязательная опция").ТСтрока();
+	ОпцияМассив = Команда.Опция("e env", Новый Массив, "Тестовый необязательный массив опций").ТМассивСтрок();
+	ARG2 = Команда.Аргумент("ARG2", "", "Тестовый простой необязательный аргумент").ТМассивСтрок();
+	
+	СтрокаИспользования = СокрЛП(Команда.ПолучитьСтрокуИспользования());
+	Утверждения.ПроверитьРавенство(СтрокаИспользования, "[ОПЦИИ] -- ARG2...", "Результат <СтрокаИспользования> должны совпадать");
+
+
+	Команда = ПодготовитьТестовуюКоманду();
+
+	ОбычныйОпция = Команда.Опция("z", , "Необязательная опция").ТСтрока();
+	ОпцияМассив = Команда.Опция("e env", Новый Массив, "Тестовый обязательный массив опций").ТМассивСтрок().Обязательный(Истина);
+	ARG2 = Команда.Аргумент("ARG2", "", "Тестовый простой необязательный аргумент").ТМассивСтрок();
+	
+	СтрокаИспользования = СокрЛП(Команда.ПолучитьСтрокуИспользования());
+	Утверждения.ПроверитьРавенство(СтрокаИспользования, "[-z] -e... -- ARG2...", "Результат <СтрокаИспользования> должны совпадать");
+
+	
+	Команда = ПодготовитьТестовуюКоманду();
+
+	ARG2 = Команда.Аргумент("ARG2", "", "Тестовый простой необязательный аргумент").ТМассивСтрок();
+	
+	СтрокаИспользования = СокрЛП(Команда.ПолучитьСтрокуИспользования());
+	Утверждения.ПроверитьРавенство(СтрокаИспользования, "-- ARG2...", "Результат <СтрокаИспользования> должны совпадать");
+
+КонецПроцедуры
+
 Процедура ТестДолжен_ПроверитьВыводСправки() Экспорт
 
 	Спек = "";
@@ -383,6 +414,7 @@
 	Аргументы.Добавить("-e=Знач1");
 	Аргументы.Добавить("-e=Знач2");
 	Аргументы.Добавить("-e=Знач3");
+	Аргументы.Добавить("--");
 	Аргументы.Добавить("-v");
 	Аргументы.Добавить("ARG2");
 	Аргументы.Добавить("run");