Skip to content
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.

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

WIP: xUnitFor1C + Сильно упростил код + заготовка кода для #29 #31

Merged
merged 26 commits into from
Jan 25, 2018

Conversation

artbear
Copy link
Collaborator

@artbear artbear commented Nov 9, 2017

  • Сильно упростил и облегчил код
  • заготовка кода для WIP: Feature/same-dir #29 (отказ от дублирования файлов) при помощи управляющей переменной в коде скриптов
  • первоначальное вливание xUnitFor1C и его плагинов
    • обновил код плагинов из 4.2.0.2

Нужно для подготовки вливания xUnitFor1C

@artbear artbear requested review from pumbaEO and nixel2007 November 9, 2017 20:36
@artbear artbear added this to the 0.1 milestone Nov 9, 2017
tasks/depf.os Outdated
ЗаписьXML.УстановитьСтроку();

Процесс = СоздатьПроцесс(СтрокаЗапуска, "./", Истина, Истина);
Попытка
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

вот этого нет в 1comands - запуск на прямую процесса может и не пройти, т.к. не нашло программу в переменной PATH, а если через sh -c - то все находит и работает.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В 1коммандс уже давно запуск идет через cmd или sh.
Сделай поиск по sh -c в репо 1commands

tasks/cepf.os Outdated
Если ОчереднаяСтрокаОшибок <> "" Тогда
Если СтрНайти(ОчереднаяСтрокаОшибок, "No bp log location saved, using default.") = 0 Тогда
//Сообщить(ОчереднаяСтрокаОшибок);
Лог.Предупреждение("%2%1", ОчереднаяСтрокаОшибок, Символы.ПС);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

в 1commands никак не определить это поток ошибок или же стандартный вывод процесса. Плюч под линуксом No bp log location saved, using default - всегда выводится и захломляет вывод.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 Пока не было задач разделения потока ошибок или стандартного вывода.
Всегда!! хватает типовых возможностей

2 что за сообщение под линуксом No bp log location saved, using default ?
Впервые слышу об этом.
Виноват 1скрипт?
Может быть, передать проблему @EvilBeaver @dmpas для решения и не делать костыли/заглушки?

В крайнем случае, заглушку для линукса можно добавить в 1коммандс.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Виновата 1с, сама платформа пишет лишнего в ошибки no bp log

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pumbaEO Я доработал 1коммандс специально, чтобы разрулить эту проблему
Смотреть PR artbear/1commands#30

Теперь можно

  • использовать 1коммандс в полном объеме
  • передать в запускаемую команду спец.лог
  • для этого лога установить раскладку, в обработчике которого под линуксом будет скрываться лог No bp log location saved, using default

В итоге код станет очень простым и удобным.

Если по указанному PR для 1коммандс возражений и дополнений не будет, я переделаю текущий PR на использование отдельного лога.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Договорились с Никитой - спец.лог на No bp log location saved, using default я сделаю в этом PR
А Никита затем вольет мои правки в свой #29

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Выпустил релиз 1command 1.3.0 для продолжения работы по данному PR

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pumbaEO я внес исправления по пропуску ненужных строк в Линуксе.
Проверь, плиз.

tasks/init.os Outdated
Лог.Информация("runner compile");
Команда.УстановитьСтрокуЗапуска("runner compile");
Команда.Исполнить();
ИсполнитьКоманду("runner-old compile");
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ну зачем же так, там должно быть runner compile

@nixel2007
Copy link
Contributor

Можно кейс управления "управляющей переменной"?
Я помню про желание Жени иметь возможность собирать бинари в отдельный каталог (тем более, что это поможет со сборкой релиза). Ее можно как-то переустанавливать без правки исходников?

@artbear
Copy link
Collaborator Author

artbear commented Nov 10, 2017

По поводу управляющей переменной.
Я не знал про желание Жени иметь возможность собирать бинари в отдельный каталог (тем более, что это поможет со сборкой релиза)

Поэтому сейчас логика жестко зашита в исходники - для изменения логики нужно будет менять код, менять значение переменной в конце модуля кода.

Можно исправить на параметр ком.строки для внешнего управления

Или даже одновременно на параметр из json c помощью репо params

@pumbaEO
Copy link
Contributor

pumbaEO commented Nov 10, 2017

Давайте так, по факту нам необходим параметр коммандной строки по умолчанию. При єтом данній параметр затронет и запись файла настройки env.json , т.к. в зависимости от этого может быть путь к epf другой и workpath другой для запуска самотестирования - opm run vanessa...

Как минимум надо перезаписать https://github.com/silverbulleters/add/blob/master/tools/JSON/env.json.file#L11 или же создать еще один шаблон env.json и его копировать.

@artbear
Copy link
Collaborator Author

artbear commented Nov 10, 2017

Все понятно, кроме настройки в json-файле.
Женя, можешь сценарий описать?

@pumbaEO
Copy link
Contributor

pumbaEO commented Nov 10, 2017

сценарий прост, при opm run init копируется базовый файл в env.json - этот файл загигнорен и там прописываютса параметры подключения к базе, путь к ванессе и к workspace. Соотвественно можно запускать как opm run vanessa, так и просто vrunner vanessa - автоматом все настройки подтянутса из этого файла. Там сейчас явно указано, что надо искать все в ./build а по факту у нас будет 2 варианта.

@artbear artbear changed the title Сильно упростил код + заготовка кода для #29 WIP: Сильно упростил код + заготовка кода для #29 Nov 14, 2017
@artbear artbear self-assigned this Nov 14, 2017
@artbear
Copy link
Collaborator Author

artbear commented Nov 14, 2017

@pumbaEO Мой PR не мержим, @nixel2007 после моих доработок сольет его в свой PR #29

@artbear
Copy link
Collaborator Author

artbear commented Nov 14, 2017

Последние коммиты пока не мержим, т.к. они пока еще не рабочие.

Почему-то после добавления лога к команде скрываются все последующие сообщения команды на винде, а хотелось бы только скрытие только одного сообщения, да и того на линуксе.

А сил продолжить на сегодня уже не осталось, пойду спать :(

… xUnitFor1C

+ Доработал разбор файлов. Теперь в командной строке можно указать как каталоги, так и отдельные файлы
+ Добавил возможность разбора xddTestRunner.epf
+ Добавлена возможность сбора одной обработки из каталога epf
+ Упростил и ускорил этап инициализации, также избавился от использования deployka
+ Добавлена зависимость от свежей версии fs
@artbear artbear changed the title WIP: Сильно упростил код + заготовка кода для #29 WIP: xUnitFor1C + Сильно упростил код + заготовка кода для #29 Nov 26, 2017
@artbear
Copy link
Collaborator Author

artbear commented Nov 26, 2017

  • первоначальное вливание xUnitFor1C и его плагинов
    • обновил код плагинов из 4.2.0.2

# Conflicts:
#	plugins/СериализаторMXL/СериализаторMXL/Forms/Форма/Ext/Form.bin
#	plugins/СериализаторMXL/СериализаторMXL/Forms/Форма/Ext/Form/form
… через ключ.командной строки "--useBuildFolderForBinaries"

Под внешнее управление доработаны скрипты инициализации, сборки  из исходников и разборки на исходники
Добавлен разный набор json-файлов настройки для реализации этого механизма
@artbear
Copy link
Collaborator Author

artbear commented Dec 1, 2017

сценарий прост, при opm run init копируется базовый файл в env.json - этот файл загигнорен и там прописываютса параметры подключения к базе, путь к ванессе и к workspace. Соотвественно можно запускать как opm run vanessa, так и просто vrunner vanessa - автоматом все настройки подтянутса из этого файла. Там сейчас явно указано, что надо искать все в ./build а по факту у нас будет 2 варианта.

@pumbaEO Проверь, плиз, мой коммит eb9a18d
Там доработано на внешнее управление через ключ командной строки

@nixel2007 @pumbaEO Вроде бы это было последнее по новой схеме с исходниками?
Или у @nixel2007 что-то еще осталось?

УФ: Реализован экспорт отчета о тестировании. Closes #761.

Формат отчета определяется по расширению файла, в который нужно экспортировать.
Поддерживаются форматы: MXL, ODS, DOCX, HTML, XLS, XLSX, TXT.
@artbear
Copy link
Collaborator Author

artbear commented Dec 3, 2017

@nixel2007 @pumbaEO Я исправил существенные ошибки сборки/разборки файлов.

Например, злосчастный неожиданный Возврат :( или удаление существующих текстовых файлов при сборке/разборке и т.п.

Скоро замержу в девелоп

Исправлена неверная разборка внешних обработок из корня рабочего каталога не в каталог epf, а в корень своего же каталога
@artbear
Copy link
Collaborator Author

artbear commented Dec 3, 2017

Все страньше и страньше :(
разборка не работает :(
нужно еще разбираться.

artbear and others added 5 commits December 7, 2017 09:13
Только для получения контекстов обработок надо искать по нрег, а файлы подключать по честному пути.
@artbear
Copy link
Collaborator Author

artbear commented Jan 25, 2018

Ура, практически закончил

image

1 исходники и бинарники обработок находятся рядом
2 исходники и бинарники обработок находятся в разных местах

Настройки путей собираемых файлов вынесены в json-файл настройки
@artbear artbear merged commit 4a66398 into develop Jan 25, 2018
@artbear artbear deleted the feature/tasks-simplification branch January 25, 2018 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants