Получение исходников конфигурации через штатные механизмы платформы #97

khorevaa · 2017-11-16T14:29:40Z

Реализовано получение исходников через штатные механизмы, только для получения версии из хранилища конфигурации.
Тесты в стиле BDD

Частичная реализация 86

khorevaa · 2017-11-16T15:04:10Z

Все тесты проходят УРА!!

nixel2007 · 2017-11-16T15:17:57Z

У меня только вопросы к функциональности гитраннера в части add --all и init --bare. Вроде ж это все есть в самой библиотеке?

pumbaEO · 2017-11-16T15:14:46Z

src/core/Классы/КомандаExport.os

Если добавил параметр через Парсер.ДобавитьПараметрФлагКоманды - тот тут всегда будет булево, и по умолчанию ложь. Этого условия в принципе не надо.

Не во всех случая. Когда идет через пакетную синхронизацию там может быть неопределенно

Только команда sync вызывается из кода гитсинка, остальные не вызываются

nixel2007 · 2017-11-16T15:19:39Z

Для хранилища вроде бы есть уже устоявшиеся --storage-user и --storage-pwd (деплойка/враннер).

nixel2007 · 2017-11-16T15:21:03Z

Говорят, использовать "no" в флага - нехорошо :) может развернуть это? Или даже сделать не параметр флаг, а прям именованный с true/false? На обсуждение

artbear

Очень круто.
По коду у меня вопросов нет.
А вот по фичам немного есть :)

artbear · 2017-11-16T15:21:14Z

features/gitsync-sync.feature

+    И Я создаю временный каталог и сохраняю его в контекст
+    И Я сохраняю значение временного каталога в переменной "ВременнаяДиректория"
+    И Я добавляю параметр "-tempdir" для команды "gitsync" из переменной "ВременнаяДиректория"
+    И Я добавляю параметр "-userRep" для команды "gitsync" со значением "Администратор"


портянка шагов из
И Я добавляю параметр "-userRep" для команды "gitsync" со значением "Администратор"
странновато выглядит.

есть же простые шаги

Когда Я добавляю параметр "--version" для команды "git"

или

Когда Я добавляю параметры для команды "oscript" | -version | | -encoding=utf-8 |

т.е. можно
1 или юзать простой шаг Я добавляю параметр "-userRep Администратор" для команды "git" с передачей в одном параметре нужных настроек вместо И Я добавляю параметр "-userRep" для команды "gitsync" со значением "Администратор"

2 или еще проще и нагляднее

Когда Я добавляю параметры для команды "oscript" | -userRep Администратор | | -push-every-n-commits 5 |

и т.п.

Переделал.

artbear · 2017-11-16T15:27:52Z

Что-то я по коду пропустил замечания :( видимо, пора домой.

artbear · 2017-11-17T08:06:07Z

features/gitsync-sync.feature

+    И Код возврата команды "gitsync" равен 0
+
+Сценарий: Синхронизация хранилища с git-репозиторием без tool1CD
+    Допустим Я добавляю параметр "-dontUseTool1cd" для команды "gitsync"


Предлагаю имя флага -useVendorUnload вместо -dontUseTool1cd

Заменил.

khorevaa · 2017-11-17T09:27:05Z

@nixel2007 - смотрел библиотеку gitrunner и таких функций неть.

…с хранилищем

nixel2007 · 2017-11-17T09:32:39Z

Значит надо добавить. Повесь ишузы, пожалуйста. 2017-11-17 12:27 GMT+03:00 Aleksey Khorev <notifications@github.com>:

…

@nixel2007 <https://github.com/nixel2007> - смотрел библиотеку gitrunner и таких функций неть. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#97 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABFJKJEKci5N0WmcaT_9Ig7tiwXB8IAXks5s3VFpgaJpZM4QgkrA> .

artbear · 2017-11-17T09:50:29Z

features/gitsync-sync.feature

-    И Я добавляю параметр "-userRep" для команды "gitsync" со значением "Администратор"
-    И Я добавляю параметр "-push-every-n-commits" для команды "gitsync" со значением "5"
-    И Я добавляю параметр "-dontUseTool1cd" для команды "gitsync"
-    И Я добавляю параметр "-process-fatform-modules" для команды "gitsync"


"-process-fatform-modules" специально убрал или случайно потерялось?

Потерялось... ))) вернул

khorevaa · 2017-11-17T10:03:01Z

@nixel2007 @artbear А что с сервером сборок почему то падает сборка.. До теста и не доходит

artbear

Отдохнул и накинул еще замечаний

artbear · 2017-11-17T09:52:49Z

features/step_definitions/gitsync-sync.os

+//Я добавляю параметры для команды "gitsync"
+//|--storage-user Администратор|
+//|-useVendorUnload|
+Процедура ЯДобавляюПараметрыДляКоманды(Знач ИмяКоманды, Знач ТаблицаПараметров) Экспорт


А зачем ты дублируешь шаг? ведь он уже есть в 1bdd/features/lib, который по умолчанию подключается при запуске.
Я специально давал ссылку на шаг, чтобы ты его использовал, а не писал с нуля :)

artbear · 2017-11-17T09:54:49Z

features/step_definitions/gitsync-sync.os

+КонецПроцедуры
+
+//я инициализирую связь "ПутьКаталогаИсходников" с внешним репозиторием "URLРепозитория"
+Процедура ЯИнициализируюСвязьСВнешнимРепозиторием(Знач ПарамСтрока1, Знач ПарамСтрока2) Экспорт


Знач ПарамСтрока1, Знач ПарамСтрока2 нужно переименовать для большей прозрачности кода

Сделал

artbear · 2017-11-17T09:56:40Z

features/step_definitions/gitsync-sync.os

+КонецПроцедуры
+
+//я инициализрую связь с внешним репозиторием
+Процедура ЯИнициализруюСвязьСВнешнимРепозиторием() Экспорт


Насколько я вижу, шаг ЯИнициализруюСвязьСВнешнимРепозиторием больше не юзается.
Удалишь?

Сделал

artbear · 2017-11-17T10:42:21Z

Запустил сборку по новой

khorevaa · 2017-11-17T10:45:20Z

@artbear - а что ты ставишь все версию 2.3.0 в мейлстоуне - если она уже выпущена?

artbear · 2017-11-17T10:47:31Z

Не знал :)

artbear · 2017-11-17T10:47:45Z

А если серьезно, забыл майлстоун поправить :(

khorevaa · 2017-11-17T10:51:22Z

Надо в develop все слить, а то я хочу сделать в ближайшее время систему плагинов для gitsync. Что позволен убрать кучу параметров вынеся их в плагины.

artbear

Последнее замечание исправляй.
И замержу

artbear · 2017-11-17T11:06:53Z

src/core/Классы/КомандаExport.os

Код нужно убрать, т.к. параметр флаг всегда прилетает правильно.

Если НеИспользоватьTool1CD = Неопределено Тогда + + НеИспользоватьTool1CD = Ложь; // по умолчанию используем tool1cd + + КонецЕсли;

А вот и нет. Когда идет пакетная синхронизация может прилететь Неопределено.

Поясни, что это значит.
По коду команда Export и ее метод ВыполнитьКоманду никак не вызывается из других команд.
Значит, только штатная работа через cmdline, который вернет флаг правильно.

Вопрос в рабоче через класс "ПакетнаяСинхронизация", там

ЗначенияПараметров = Новый Соответствие; ЗначенияПараметров.Вставить("ПутьКХранилищу", Репо.КаталогХранилища1С); ЗначенияПараметров.Вставить("URLРепозитория", Репо.GitURL); ЗначенияПараметров.Вставить("ЛокальныйКаталогГит", Репо.КаталогВыгрузки); ЗначенияПараметров.Вставить("-email", Репо.ДоменПочтыДляGit); ЗначенияПараметров.Вставить("-v8version", Репо.ПутьКПлатформе83); ЗначенияПараметров.Вставить("-push-every-n-commits", Репо.КоличествоКоммитовДоPush); ЗначенияПараметров.Вставить("-check-authors", ПреобразоватьЗначениеКБулево(Репо.ПроверитьАвторовХранилища)); ЗначенияПараметров.Вставить("-stop-if-empty-comment", ПреобразоватьЗначениеКБулево(Репо.ПрерватьВыполнениеБезКомментарияКВерсии)); ЗначенияПараметров.Вставить("-auto-set-tags", ПреобразоватьЗначениеКБулево(Репо.АвтоматическаяУстановкаТэговПоВерсиям)); ЗначенияПараметров.Вставить("-process-fatform-modules", ПреобразоватьЗначениеКБулево(Репо.ПереименовыватьФайлМодуляОбычнойФормы)); // дополним настройками из параметров команды Для Каждого ПараметрКоманды Из мПараметрыКоманды Цикл ЗначенияПараметров.Вставить(ПараметрКоманды.Ключ, ПараметрКоманды.Значение); КонецЦикла; МенеджерКомандПриложения.ВыполнитьКоманду("sync", ЗначенияПараметров);

Если флага в настройке нет, тогда будет подставлено неопредленно

Посмотрел код sync, увидел проблему.
Значит, это API sync плохое.
Вообще, зачем его вызывать через ВыполнитьКоманду?
Лучше выделить отдельный класс/метод, которому можно будет передать чистые параметры, без лишних/странных преобразований с магическими именами ключей

Создал #99

artbear · 2017-11-17T11:07:05Z

src/core/Классы/КомандаSync.os

Код нужно убрать, т.к. параметр флаг всегда прилетает правильно.

Если НеИспользоватьTool1CD = Неопределено Тогда + + НеИспользоватьTool1CD = Ложь; // по умолчанию используем tool1cd + + КонецЕсли;

khorevaa added 3 commits November 16, 2017 14:07

fix for linux

de099ee

Основная функциоанльность работы через 1С Предприятие.

f02c7b7

Тесты в стиле BDD

21d04f7

artbear self-assigned this Nov 16, 2017

Исправил установку команды выполнения gitsync

dcd551b

pumbaEO suggested changes Nov 16, 2017

View reviewed changes

artbear requested changes Nov 16, 2017

View reviewed changes

artbear reviewed Nov 17, 2017

View reviewed changes

Мелкие доработки по замечаниям. Переименование параметров соединения …

6a2f459

…с хранилищем

artbear reviewed Nov 17, 2017

View reviewed changes

Вернул потерянный параметр -process-fatform-modules

ccf7ea2

artbear requested changes Nov 17, 2017

View reviewed changes

artbear added this to the 2.3.0 milestone Nov 17, 2017

Переминование параметров процедуры в наиболее адекватные.

3111765

artbear modified the milestones: 2.3.0, 2.4.0 Nov 17, 2017

artbear requested changes Nov 17, 2017

View reviewed changes

artbear mentioned this pull request Nov 17, 2017

Отрефакторить сильно дублирующий код #98

Open

artbear approved these changes Nov 17, 2017

View reviewed changes

artbear merged commit 1387bc3 into oscript-library:develop Nov 17, 2017

khorevaa deleted the feature/add-v8storage branch May 29, 2018 07:38

Получение исходников конфигурации через штатные механизмы платформы #97

Получение исходников конфигурации через штатные механизмы платформы #97

Uh oh!

Conversation

khorevaa commented Nov 16, 2017

Uh oh!

khorevaa commented Nov 16, 2017

Uh oh!

nixel2007 commented Nov 16, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

nixel2007 commented Nov 16, 2017

Uh oh!

nixel2007 commented Nov 16, 2017

Uh oh!

artbear left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

artbear commented Nov 16, 2017

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

khorevaa commented Nov 17, 2017

Uh oh!

nixel2007 commented Nov 17, 2017 via email

Uh oh!

Choose a reason for hiding this comment

Uh oh!

khorevaa Nov 17, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

khorevaa commented Nov 17, 2017

Uh oh!

artbear left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

artbear commented Nov 17, 2017

Uh oh!

khorevaa commented Nov 17, 2017

Uh oh!

artbear commented Nov 17, 2017

Uh oh!

artbear commented Nov 17, 2017

Uh oh!

khorevaa commented Nov 17, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

artbear left a comment

Choose a reason for hiding this comment

Uh oh!

artbear Nov 17, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

khorevaa Nov 17, 2017 •

edited

Loading

khorevaa commented Nov 17, 2017 •

edited

Loading

artbear Nov 17, 2017 •

edited

Loading

khorevaa Nov 17, 2017 •

edited

Loading

artbear Nov 17, 2017 •

edited

Loading