Добавил возможность запуска командных файлов на язке PowerShell и др. #2

Berckk · 2017-10-12T21:20:36Z

Привет.
Добавил указание расширения для командных файлов и возможность указания исполняемых файлов других сценарных языков.

artbear

Хорошее дополнение библиотеки.
Но есть небольшие замечания.
Исправишь?

artbear · 2017-10-14T11:20:02Z

features/step_definitions/КомандныеФайлы.os

-	КомандныйФайл = Новый КомандныйФайл; 
-	КомандныйФайл.Создать();
-	БДД.СохранитьВКонтекст("КомандныйФайл", КомандныйФайл);
+	СоздатьФайл("","");


Здесь проще и понятнее написать СоздатьФайл(), т.к. у этого метода оба параметр по умолчанию пустые строки.

artbear · 2017-10-14T11:22:26Z

features/КомандныеФайлы.feature

@@ -9,7 +9,16 @@
 Контекст: Отключение отладки в логах
    Допустим Я выключаю отладку лога с именем "oscript.lib.commands"

-Сценарий: Выполнение простого командного файла
+Сценарий: Выполнение командного файла PowerShell
+    Когда Я устанавливаю приложение запуска "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe"


На Линуксе шаг упадет.
Предлагаю в реализацию шага ЯУстанавливаюПриложениеЗапуска добавить проверку на Линукс
Если Линукс, то отмечать шаг как пропущенный.

Это лучше, чем падающий тест

Тут наверно для Линукса надо вообще весь сценарий проускать, но я не представляю как.

Внутри кода реализации вызови спец.метод БДД.ВыполнитьШаг("Пропускаю шаг в Linux")
Документация на этот метод

ИМХО за счет того, что у нас нет реализации шага Пропускаю шаг в Linux, твой шаг/сценарий будет отмечен как пропущенный.

Попробуй.

В консоле добавляется сообщение "Пропускаю шаг в Linux".
В отчете bdd-log.xml шаг отмечается как status="passed".

artbear · 2017-10-14T11:24:20Z

features/КомандныеФайлы.feature

+Сценарий: Выполнение командного файла PowerShell
+    Когда Я устанавливаю приложение запуска "C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe"
+    И Я создаю командный файл PowerShell ""  ".ps1"
+    И Я добавляю строку "echo PowerShell" в командный файл


Предлагаю добавить какую-нибудь уникальную для PowerShell-команду, а не псевдоним, который работает и для cmd.
Например, Write-Host PowerShell

artbear · 2017-10-14T11:26:59Z

readme.md

+```bsl
+КомандныйФайл = Новый КомандныйФайл;
+КомандныйФайл.УстановитьПриложение("C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe");
+КомандныйФайл.Создать(,".ps1");


Предлагаю указать явное пустое имя файла через
КомандныйФайл.Создать("", ".ps1");

PS пропуск параметра выглядит очень плохо

artbear · 2017-10-14T11:28:05Z

src/КомандныйФайл.os

 //
 //  Возвращаемое значение:
 //   Строка - Путь созданного файла
 //
-Функция Создать(Знач Путь = "") Экспорт
+Функция Создать(Знач Путь = "",Знач Расширение="") Экспорт


Не хватает пробела перед вторым Знач

artbear · 2017-10-14T11:30:21Z

src/КомандныйФайл.os

+// Установить путь к приложению.
+//
+//  Параметры:
+//   ПутьПриложения - Путь приложения (Строка)


Предлагаю добавить пример задания приложения в описание метода
"C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe"

artbear · 2017-10-14T11:31:56Z

src/КомандныйФайл.os

@@ -125,7 +140,9 @@
 	Команда.УстановитьКодировкуВывода(КодировкаВывода);
 	Команда.ПоказыватьВыводНемедленно(НемедленнныйВывод);

-	Если ЭтоWindows Тогда 
+	Если НЕ ПустаяСтрока(Приложение) Тогда


ИМХО здесь точнее Если ЗначениеЗаполнено(Приложение) Тогда
т.к. по умолчанию, без явной инициализации, Приложение равно Неопределено
И проверка на пустую строку сработает неверно.

artbear · 2017-10-14T11:38:24Z

И последний вопрос - фичи прогонял?

Berckk · 2017-10-15T07:54:26Z

Фичи прогонял, но только на Windows.
То что при прогоне идет обращение к "C:\Program Files (x86)\OneScript\lib\1commands\src", а не к текущему каталогу проекта. Это как исправить?

artbear · 2017-10-15T11:42:52Z

То что при прогоне идет обращение к "C:\Program Files (x86)\OneScript\lib\1commands\src", а не к текущему каталогу проекта. Это как исправить?

@Berckk А откуда такая уверенность?
фичи настроены на прогон классов из исходников

artbear · 2017-10-16T05:23:13Z

После последних своих правок тесты прогонял. Шаги пропускаются?

artbear · 2017-10-16T05:26:53Z

features/step_definitions/КомандныеФайлы.os

+	БДД.СохранитьВКонтекст("КомандныйФайл", КомандныйФайл);	
+КонецПроцедуры
+
+Функция ПропускаюШаг() Экспорт


1 Экспорт для этого метода не нужен.
2 Название метода предлагать поменять на более точное ПрпускаюШагНаLinux

С последними правками шаги пропускаются, но помечаются как успешные.

artbear · 2017-10-16T08:53:30Z

features/step_definitions/КомандныеФайлы.os

+КонецПроцедуры
+
+Функция ПропускаюШагВЛинукс()
+	Если НЕ ЗначениеЗаполнено(ЭтоWindows) Тогда


Проверка Если НЕ ЗначениеЗаполнено(ЭтоWindows) Тогда в Линуксе всегда будет срабатывать и выполнять код внутри блока Если.
Т.к. значение ЭтоWindows будет Ложь, а ЗначениеЗаполнено(Ложь) всегда выдает Истину :(

Предлагаю исправить на Если ЭтоWindows = Неопределено Тогда

Сделай это последнее исправления и я принимаю пулл-реквест!

artbear · 2017-10-16T09:52:49Z

features/КомандныеФайлы.feature

+    Тогда Вывод командного файла содержит "PowerShell"
+    И Код возврата командного файла равен 0
+
+Контекст: Отключение отладки в логах


Оказывается, я пропустил неверную правку в фиче.

Контекст в фиче должен быть один, их не должно быть несколько.
Поэтому твое добавление контекста не валидно.

Предлагаю

оставить один контекст.

Контекст: Отключение отладки в логах Допустим Я выключаю отладку лога с именем "oscript.lib.commands"

Добавить шаг Я пропускаю этот сценарий в Линукс как первый шаг сценария Выполнение командного файла PowerShell

artbear · 2017-10-16T09:53:40Z

features/step_definitions/КомандныеФайлы.os

@@ -75,27 +123,66 @@

 //Код возврата командного файла равен 0
 Процедура КодВозвратаКомандногоФайлаРавен(Знач ОжидаемыйКодВозврата) Экспорт
+	Если ПропускаюШагВЛинукс() Тогда


Шаг Код возврата командного файла равен 0 является универсальным.
В него не нужно вставлять проверку на линукс, т.к. этот шаг должен работать в линукс

Да, но в сценарии с PowerShell в линуксе никого файла на этом шаге нет.
И тесты валятся.
Я поэтому и добавил проверку во все шаги.

В линуксе можно создать файл с расширением ps1, т.к. расширение ни на что не влияет.
Но тесты выпадут с ошибкой на шаге Я выполняю командный файл, т.к. Write-Host PowerShell в линукс не отработает.
Поэтому я изначально спрашивал, как пропустить весь сценарий под линукс.

artbear · 2017-10-16T09:54:00Z

features/step_definitions/КомандныеФайлы.os

@@ -67,6 +110,11 @@

 //Вывод командного файла содержит "командный файл"
 Процедура ВыводКомандногоФайлаСодержит(Знач ОжидаемыйВыводКоманды) Экспорт
+	Если ПропускаюШагВЛинукс() Тогда


Шаг Вывод командного файла содержит "командный файл" является универсальным.
В него не нужно вставлять проверку на линукс, т.к. этот шаг должен работать в линукс

artbear · 2017-10-16T09:54:17Z

features/step_definitions/КомандныеФайлы.os

 	ВывестиТекстФайла(ИмяФайла);
 КонецПроцедуры

 //Я сообщаю содержимое командного файла
 Процедура ЯСообщаюСодержимоеКомандногоФайла() Экспорт
+	Если ПропускаюШагВЛинукс() Тогда


Шаг Я сообщаю содержимое командного файла является универсальным.
В него не нужно вставлять проверку на линукс, т.к. этот шаг должен работать в линукс

artbear · 2017-10-16T09:54:38Z

features/step_definitions/КомандныеФайлы.os

 	КомандныйФайл = БДД.ПолучитьИзКонтекста("КомандныйФайл");

 	КомандныйФайл.Исполнить();
 КонецПроцедуры

 //Я сообщаю содержимое файла "ИмяФайла"
 Процедура ЯСообщаюСодержимоеФайла(Знач ИмяФайла) Экспорт
+	Если ПропускаюШагВЛинукс() Тогда


Шаг Я сообщаю содержимое файла "ИмяФайла" является универсальным.
В него не нужно вставлять проверку на линукс, т.к. этот шаг должен работать в линукс

artbear · 2017-10-16T09:55:35Z

features/step_definitions/КомандныеФайлы.os

+КонецПроцедуры
+
+//Я не пропускаю этот сценарий в Линукс
+Процедура ЯНЕПропускаюЭтотСценарийВЛинукс() Экспорт


Шаг ЯНЕПропускаюЭтотСценарийВЛинукс соответственно не нужен.
Его нужно удалить.

artbear · 2017-10-16T09:55:59Z

Извини за придирчивость :)

artbear · 2017-10-16T10:25:23Z

Возможно, мы друг друга не поняли.
Что происходит с прогоном шагов/фич, когда ты вставил БДД.ВыполнитьШаг("Пропустить шаг в Линукс");
Шаги падают, или отмечаются как пропущенные,
или успешно выполняются и далее выполняются следующие шаги??

Berckk · 2017-10-16T11:17:57Z

Рассказываю все.
Запускаю в Windows, но в функции
ПропускаюШагВЛинукс() добавляю ЭтоWindows = Ложь;
В каталоге проекта запускаю командную строку и выполняю opm test
в ответ выводится...
комментарий (серым цветом) Пропускаю шаг в Linux и зеленым шаги.
Т.е. шаги успешно выполняются.

artbear · 2017-10-16T13:16:42Z

Тут важно - именно последующие шаги из этого же сценария выполняются?
или сценарий останавливается и запускаются шаги другого сценария?

можешь полный лог от opm test скинуть ?

Berckk · 2017-10-16T13:29:49Z

Шаги выполняются, остановка не происходит.

artbear · 2017-10-16T14:07:41Z

Все понятно, наша добавка с пропускаемым шагом на самом деле шаг не пропускает :(
Пока оставим так.
Я в 1бдд (прогонялку фич) добавлю ишуз для фиксации проблемы.

Сейчас сделаю последний код-ревью, если не увижу явных проблем, наконец-то замержу пулл-реквест

artbear · 2017-10-16T14:11:17Z

@Berckk Большое спасибо!

Добавил возможность запуска командных файлов на язке PowerShell.

87d4736

artbear requested changes Oct 14, 2017

View reviewed changes

Рефакторинг

fa9ceb7

artbear self-assigned this Oct 15, 2017

artbear added the enhancement label Oct 15, 2017

Добавил пропуск шагов по Линукс.

fcc8fb1

artbear requested changes Oct 16, 2017

View reviewed changes

Рефакторинг пропуска шага в Линукс.

8f7cc82

artbear reviewed Oct 16, 2017

View reviewed changes

Добавил установку параметра пропуска шагов сценария в Линукс из фичи.

4617e7a

artbear reviewed Oct 16, 2017

View reviewed changes

Убрал не валидный контекст из фичи.

da6b8fa

artbear mentioned this pull request Oct 16, 2017

Добавить возможность отметки текущего шага как пропускаемого из кода реализации шага artbear/1bdd#121

Closed

artbear approved these changes Oct 16, 2017

View reviewed changes

artbear merged commit 3aff8f0 into oscript-library:develop Oct 16, 2017

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Добавил возможность запуска командных файлов на язке PowerShell и др. #2

Добавил возможность запуска командных файлов на язке PowerShell и др. #2

Berckk commented Oct 12, 2017

artbear left a comment

artbear Oct 14, 2017

artbear Oct 14, 2017

Berckk Oct 15, 2017

artbear Oct 15, 2017

Berckk Oct 16, 2017

artbear Oct 14, 2017

artbear Oct 14, 2017

artbear Oct 14, 2017

artbear Oct 14, 2017

artbear Oct 14, 2017

artbear commented Oct 14, 2017

Berckk commented Oct 15, 2017

artbear commented Oct 15, 2017

artbear commented Oct 16, 2017

artbear Oct 16, 2017

Berckk Oct 16, 2017

artbear Oct 16, 2017

artbear Oct 16, 2017

artbear Oct 16, 2017

Berckk Oct 16, 2017

Berckk Oct 16, 2017

artbear Oct 16, 2017

artbear Oct 16, 2017

artbear Oct 16, 2017

artbear Oct 16, 2017

artbear commented Oct 16, 2017

artbear commented Oct 16, 2017

Berckk commented Oct 16, 2017

artbear commented Oct 16, 2017 •

edited

Loading

Berckk commented Oct 16, 2017

artbear commented Oct 16, 2017

artbear commented Oct 16, 2017

Добавил возможность запуска командных файлов на язке PowerShell и др. #2

Добавил возможность запуска командных файлов на язке PowerShell и др. #2

Conversation

Berckk commented Oct 12, 2017

artbear left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

artbear commented Oct 14, 2017

Berckk commented Oct 15, 2017

artbear commented Oct 15, 2017

artbear commented Oct 16, 2017

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

artbear commented Oct 16, 2017

artbear commented Oct 16, 2017

Berckk commented Oct 16, 2017

artbear commented Oct 16, 2017 • edited Loading

Berckk commented Oct 16, 2017

artbear commented Oct 16, 2017

artbear commented Oct 16, 2017

artbear commented Oct 16, 2017 •

edited

Loading