Skip to content

fix #60: Исправление распознавания опций #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 23, 2023
Merged

fix #60: Исправление распознавания опций #61

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
8c3953b
Finish 0.9.7
khorevaa Jan 26, 2018
ecc8048
Finish 0.9.8
khorevaa Feb 9, 2018
d3e8377
Finish 0.9.9
khorevaa Jul 17, 2018
aa453d2
Merge branch 'develop'
khorevaa Jul 19, 2018
2907f8e
Finish 0.9.10
khorevaa Jul 19, 2018
9b6a2df
Исправлено ошибочное распознавание опций как аргументов
Morkhe Jan 25, 2020
b42b183
Исправлено формирование спека при наличии обязательных опций
Morkhe Jan 26, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 0 additions & 2 deletions src/core/Классы/internal/parser/Классы/ОпцииЗавершениеПарсера.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@
//
Функция Поиск(Аргументы, КонтекстПоиска) Экспорт

КонтекстПоиска.СбросОпций = Истина;

Результат = Новый Структура("РезультатПоиска, Аргументы", Истина, Аргументы);

Возврат Результат;
Expand Down
40 changes: 31 additions & 9 deletions src/core/Классы/КомандаПриложения.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -117,6 +117,17 @@
Возврат Описание;
КонецФункции

// Функция возвращает строку использования (спек) команды
//
// Возвращаемое значение:
// Cтрока - строка использования (спек) текущей команды
//
Функция ПолучитьСтрокуИспользования() Экспорт

Возврат ?(Не ПустаяСтрока(Спек), Спек, СформироватьСтандартнуюСтрокуИспользования());

КонецФункции

// Функция возвращает значение опции по переданному имени/синониму опции
//
// Параметры:
Expand Down Expand Up @@ -493,13 +504,7 @@

Лог.Отладка("Входящий спек: %1", Спек);

СтрокаИспользования = Спек;

Если ПустаяСтрока(СтрокаИспользования) Тогда

СтрокаИспользования = СформироватьСтандартнуюСтрокуИспользования();

КонецЕсли;
СтрокаИспользования = ПолучитьСтрокуИспользования();

Лог.Отладка("Разбираю строку использования с помощью лексера");

Expand Down Expand Up @@ -1101,7 +1106,24 @@
СтандартнаяСтрокаИспользования = "";

Лог.Отладка("Количество опций строки: %1", Опции.Количество());
Если Опции.Количество() > 0 Тогда
ЕстьОбязательнаяОпция = Ложь;
Для каждого Опция Из Опции Цикл

ИмяОпции = Опция.Ключ.НаименованияПараметров[0];
КлассОпции = Опция.Ключ;

Если КлассОпции.ПолучитьОбязательностьВвода() Тогда
ЕстьОбязательнаяОпция = Истина;
КонецЕсли;

ДополнитьИмяАргументаМассива(ИмяОпции, КлассОпции);
ДополнитьИмяАргументаНеобязательного(ИмяОпции, КлассОпции);

СтандартнаяСтрокаИспользования = СтандартнаяСтрокаИспользования + ИмяОпции + " ";

КонецЦикла;

Если Не ЕстьОбязательнаяОпция И Опции.Количество() > 0 Тогда
СтандартнаяСтрокаИспользования = "[ОПЦИИ] ";
КонецЕсли;

Expand All @@ -1118,7 +1140,7 @@
ДополнитьИмяАргументаМассива(ИмяАргумента, КлассАргумента);
ДополнитьИмяАргументаНеобязательного(ИмяАргумента, КлассАргумента);

Лог.Отладка("Добавляю аргумет <%1> в спек <%2>", ИмяАргумента, СтандартнаяСтрокаИспользования);
Лог.Отладка("Добавляю аргумент <%1> в спек <%2>", ИмяАргумента, СтандартнаяСтрокаИспользования);
СтандартнаяСтрокаИспользования = СтандартнаяСтрокаИспользования + ИмяАргумента + " ";

КонецЦикла;
Expand Down
34 changes: 33 additions & 1 deletion tests/КомандаПриложения_test.os
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -39,7 +39,7 @@
ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования");
ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования_МассивАргументов");
ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования_МассивАргументовНесколькоАргументов");

ИменаТестов.Добавить("ТестДолжен_СтандартнуюСтрокуИспользования_ОбязательнаяОпция");
// ИменаТестов.Добавить("ТестДолжен_СложныеСлучаиФормированияСтрокиИспользования");

ИменаТестов.Добавить("ТестДолжен_ПроверитьВыводСправки");
Expand Down Expand Up @@ -354,6 +354,37 @@

КонецПроцедуры

Процедура ТестДолжен_СтандартнуюСтрокуИспользования_ОбязательнаяОпция() Экспорт

Команда = ПодготовитьТестовуюКоманду();

ОбычныйОпция = Команда.Опция("z", , "Необязательная опция").ТСтрока();
ОпцияМассив = Команда.Опция("e env", Новый Массив, "Тестовый необязательный массив опций").ТМассивСтрок();
ARG2 = Команда.Аргумент("ARG2", "", "Тестовый простой необязательный аргумент").ТМассивСтрок();

СтрокаИспользования = СокрЛП(Команда.ПолучитьСтрокуИспользования());
Утверждения.ПроверитьРавенство(СтрокаИспользования, "[ОПЦИИ] -- ARG2...", "Результат <СтрокаИспользования> должны совпадать");


Команда = ПодготовитьТестовуюКоманду();

ОбычныйОпция = Команда.Опция("z", , "Необязательная опция").ТСтрока();
ОпцияМассив = Команда.Опция("e env", Новый Массив, "Тестовый обязательный массив опций").ТМассивСтрок().Обязательный(Истина);
ARG2 = Команда.Аргумент("ARG2", "", "Тестовый простой необязательный аргумент").ТМассивСтрок();

СтрокаИспользования = СокрЛП(Команда.ПолучитьСтрокуИспользования());
Утверждения.ПроверитьРавенство(СтрокаИспользования, "[-z] -e... -- ARG2...", "Результат <СтрокаИспользования> должны совпадать");


Команда = ПодготовитьТестовуюКоманду();

ARG2 = Команда.Аргумент("ARG2", "", "Тестовый простой необязательный аргумент").ТМассивСтрок();

СтрокаИспользования = СокрЛП(Команда.ПолучитьСтрокуИспользования());
Утверждения.ПроверитьРавенство(СтрокаИспользования, "-- ARG2...", "Результат <СтрокаИспользования> должны совпадать");

КонецПроцедуры

Процедура ТестДолжен_ПроверитьВыводСправки() Экспорт

Спек = "";
Expand Down Expand Up @@ -383,6 +414,7 @@
Аргументы.Добавить("-e=Знач1");
Аргументы.Добавить("-e=Знач2");
Аргументы.Добавить("-e=Знач3");
Аргументы.Добавить("--");
Аргументы.Добавить("-v");
Аргументы.Добавить("ARG2");
Аргументы.Добавить("run");
Expand Down