API: Копирование пользователей из одного хранилища в другое #16

khorevaa · 2017-01-17T12:52:39Z

Тесты используют АPI: Добавления пользователей в хранилище

EvilBeaver · 2017-01-17T20:09:41Z

tests/v8runner-test.os

+	
+	УправлениеКонфигуратором = Новый УправлениеКонфигуратором;
+	Лог = Логирование.ПолучитьЛог("oscript.lib.v8runner");
+	Лог.УстановитьУровень(УровниЛога.Отладка);


Вот это долой из продуктива

Это ж тест? Хотя...

Ааа, тогда пардон. В тесте можно.

К автору ))

EvilBeaver · 2017-01-17T20:10:28Z

src/v8runner.os

+                                        Знач ПользовательХранилища, 
+                                        Знач ПарольХранилища,
+                                        Знач СтрокаСоеденияХранилищаКопии,
+										Знач ПользовательХранилищаКопии,


съехало

Это проблема табов.. Так и будет все съезжать..

в Visual Studio Code есть настройка автоконвертации одного в другое. Один раз настроить и не будет ничего съезжать

Поправил. Настройка стоит но табы не заменяет автоматим на пробелмы... ((

EvilBeaver · 2017-01-17T20:11:16Z

src/v8runner.os

+//   ПарольХранилища - Строка - Пароль пользователь для подключения к хранилищю конфигурации
+//   СтрокаСоеденияХранилищаКопии - Строка - Путь к хранилищу, из которого выполняется копирование пользователей.
+//   ПользовательХранилищаКопии - Строка - Имя пользователя хранилища, из которого выполняется копирование пользователей.
+//   PwПарольХранилищаКопииd - Строка - Пароль пользователя хранилища, из которого выполняется копирование пользователей.


Опечатка, Pw..d

СтрокаСоеденияХранилищаКопии - опечатка СоедЕнения

Исправил

nixel2007 · 2017-01-17T18:57:11Z

src/v8runner.os

+//   ПарольХранилища - Строка - Пароль пользователь для подключения к хранилищю конфигурации
+//   СтрокаСоеденияХранилищаКопии - Строка - Путь к хранилищу, из которого выполняется копирование пользователей.
+//   ПользовательХранилищаКопии - Строка - Имя пользователя хранилища, из которого выполняется копирование пользователей.
+//   PwПарольХранилищаКопииd - Строка - Пароль пользователя хранилища, из которого выполняется копирование пользователей.


Какой хитрый параметр :)

Исправил

nixel2007 · 2017-01-17T18:57:39Z

src/v8runner.os

+Процедура КопироватьПользователейИзХранилища(Знач СтрокаСоединения, 
+                                        Знач ПользовательХранилища, 
+                                        Знач ПарольХранилища,
+                                        Знач СтрокаСоеденияХранилищаКопии,


Соединения

Исправил

artbear · 2017-01-18T06:28:01Z

@khorevaa Может быть, тебе лучше слить все твои доработки из 3х PR в один #15
Все равно они юзают общий код, тебе и нам проверять будет легче.
Сейчас сложновато выполнять ревью кода, т.к. нужно переключать контексты.

artbear · 2017-01-18T06:28:58Z

src/v8runner.os

+//   ПарольХранилища - Строка - Пароль пользователь для подключения к хранилищю конфигурации
+//   СтрокаСоеденияХранилищаКопии - Строка - Путь к хранилищу, из которого выполняется копирование пользователей.
+//   ПользовательХранилищаКопии - Строка - Имя пользователя хранилища, из которого выполняется копирование пользователей.
+//   PwПарольХранилищаКопииd - Строка - Пароль пользователя хранилища, из которого выполняется копирование пользователей.


СтрокаСоеденияХранилищаКопии - опечатка СоедЕнения

artbear · 2017-01-18T06:30:15Z

src/v8runner.os

+
+    Параметры = СтандартныеПараметрыЗапускаКонфигуратора();
+
+    Параметры.Добавить("/ConfigurationRepositoryF """+СтрокаСоединения+"""");


Как уже написал в #15, предлагаю выделить спец.метод для установки параметров подключения к хранилищу

Будет в отдельном PR

artbear · 2017-01-18T06:32:39Z

tests/v8runner-test.os

+                                    КаталогВременногоХранилища,
+                                    "Администратор");
+    Утверждения.ПроверитьИстину(УправлениеКонфигуратором.ХранилищеКонфигурацииСуществует(КаталогВременногоХранилища2), "Временное хранилище 2 конфигурации должно существовать");
+


Вижу дублированный код с другими тестами работы с хранилищами.

Нужно выделить отдельный метод

ПутьКФайлуВерсии = УправлениеКонфигуратором.ПолучитьВерсиюИзХранилища(КаталогВременногоХранилища, НовыйПользователь, ПарольПользователя); + + Утверждения.ПроверитьИстину(ФайлСуществует(ПутьКФайлуВерсии), "Файл конфигурации из хранилища должен существовать"); + + КаталогВременногоХранилища2 = ОбъединитьПути(ВременныйКаталог, "v8r_TempRepository2"); + + ФайлКонфигурации = ОбъединитьПути(ТекущийСценарий().Каталог, "fixtures", "1.0\1Cv8.cf"); + + + УправлениеКонфигуратором.ЗагрузитьКонфигурациюИзФайла(ФайлКонфигурации); + УправлениеКонфигуратором.СоздатьФайловоеХранилищеКонфигурации( + КаталогВременногоХранилища, + "Администратор"); + Утверждения.ПроверитьИстину(УправлениеКонфигуратором.ХранилищеКонфигурацииСуществует(КаталогВременногоХранилища2), "Временное хранилище 2 конфигурации должно существовать");

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

artbear · 2017-01-18T06:35:26Z

4 PR по одной теме.
Точно нужно слить в один.

PS @khorevaa так сильно разделять свои доработки на PR ты стал после прошлого раза доработки v8runner ? ИМХО в данном случае зря, слишком уж они завязаны на общий код и тесты

khorevaa · 2017-01-18T12:57:55Z

Вообще честно мне все равно. Но разделять стал исключительно из-за PR по gitsync. Т.к. отделять сложнее в несколько раз. Да и доработки в принципе не очень то и связаны. Они как бы сами по себе и на функциональность друг друга не влияют. Вообще в данном случае как раз отделить и верно на мой взгляд.

nixel2007 · 2017-01-19T08:49:20Z

Пофиксь мерж-конфликт, пожалуйста.

khorevaa · 2017-01-19T09:54:48Z

Добавил коммит устранения конфликтов ....

nixel2007 · 2017-01-19T09:56:42Z

@khorevaa хм... все еще показывает мерж конфликт. rebase?

khorevaa · 2017-01-19T09:57:42Z

наверное но мне не очень понятно что от меня надо..

khorevaa · 2017-01-19T09:58:57Z

Push сразу в pr не проходит...
правов нет

nixel2007 · 2017-01-19T10:12:25Z

Что пишет?

khorevaa · 2017-01-19T10:32:31Z

Authentication using OAuth failed

khorevaa · 2017-01-19T14:24:54Z

Не понимаю от слова совсем. Что-то он мне не то сделал.. Ппц...

khorevaa · 2017-01-19T14:43:44Z

@nixel2007 Получилось ура... Что сделал не понял ).

khorevaa · 2017-01-19T14:51:43Z

Неожиданное поведение GitHub удаление ветки приводит к закрытию RP. Открыт заново

nixel2007 · 2017-01-19T18:25:30Z

У @EvilBeaver концептуальных возражений не было, поэтому ввиду его повышенной загруженности мержу.

@khorevaa большое спасибо за пуллреквесты!
Сейчас постараюсь выпустить обновление (если все тесты совокупно пройдут)

khorevaa added 4 commits January 17, 2017 15:41

Добавление процедуры копирования пользователей из хранилища

55b67ee

Добавление тестов и табы и пробелы

aefc83f

Исправление описания процедуры

aa8a8a6

Откатил табы и пробелы

77da221

EvilBeaver requested changes Jan 17, 2017

View reviewed changes

nixel2007 reviewed Jan 17, 2017

View reviewed changes

artbear requested changes Jan 18, 2017

View reviewed changes

Исправление опечаток в названиях параметров

600fe15

nixel2007 changed the base branch from master to develop January 19, 2017 08:42

nixel2007 approved these changes Jan 19, 2017

View reviewed changes

Устранение конфликтов слияния

57d458f

nixel2007 requested review from EvilBeaver and artbear January 19, 2017 11:46

khorevaa force-pushed the feature/ConfigurationRepositoryCopyUsers branch from 28ee789 to 57d458f Compare January 19, 2017 14:29

Устранение конфликтов слияния (попытка 2)

1a651c4

khorevaa closed this Jan 19, 2017

khorevaa deleted the feature/ConfigurationRepositoryCopyUsers branch January 19, 2017 14:46

khorevaa restored the feature/ConfigurationRepositoryCopyUsers branch January 19, 2017 14:50

khorevaa reopened this Jan 19, 2017

artbear approved these changes Jan 19, 2017

View reviewed changes

nixel2007 merged commit 3062fd0 into oscript-library:develop Jan 19, 2017

khorevaa deleted the feature/ConfigurationRepositoryCopyUsers branch January 22, 2017 10:54


		Параметры = СтандартныеПараметрыЗапускаКонфигуратора();

		Параметры.Добавить("/ConfigurationRepositoryF """+СтрокаСоединения+"""");

API: Копирование пользователей из одного хранилища в другое #16

API: Копирование пользователей из одного хранилища в другое #16

Uh oh!

Conversation

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

Uh oh!

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!

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!

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 Jan 18, 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!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

artbear commented Jan 18, 2017

Uh oh!

khorevaa commented Jan 18, 2017

Uh oh!

nixel2007 commented Jan 19, 2017

Uh oh!

khorevaa commented Jan 19, 2017

Uh oh!

nixel2007 commented Jan 19, 2017

Uh oh!

khorevaa commented Jan 19, 2017

Uh oh!

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

Uh oh!

Uh oh!

nixel2007 commented Jan 19, 2017

Uh oh!

khorevaa commented Jan 19, 2017

Uh oh!

khorevaa commented Jan 19, 2017

Uh oh!

khorevaa commented Jan 19, 2017

Uh oh!

khorevaa commented Jan 19, 2017

Uh oh!

nixel2007 commented Jan 19, 2017

Uh oh!

Reviewers

Assignees

Labels

Projects

khorevaa commented Jan 17, 2017 •

edited

Loading

khorevaa commented Jan 19, 2017 •

edited

Loading