Skip to content

Commit 99170b4

Browse files
author
Alexander Strizhachuk
committed
Рефакторинг
1 parent 3802a0b commit 99170b4

File tree

10 files changed

+31
-31
lines changed

10 files changed

+31
-31
lines changed

‎GitlabServices.Tests/src/CommonModules/Тест_ОбщийМодульСервер/Module.bsl

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,10 +64,11 @@
6464

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

67+
// TODO не используется?
6768
Функция РезультатЗапросаMock(Знач URL, Знач Token, Знач ПутьКТелуЗапроса, Знач КаталогПроекта) Экспорт
6869

6970
ПараметрыСоединения = РазобратьURL(URL);
70-
АдресСервиса = СтрШаблон("%1/webhook/epf/updates", ПараметрыСоединения.Путь);
71+
АдресСервиса = СтрШаблон("%1/webhooks/epf/updates", ПараметрыСоединения.Путь);
7172

7273
HTTPСоединение = Новый HTTPСоединение(ПараметрыСоединения.Сервер);
7374

‎GitlabServices/src/HTTPServices/gitlab/Module.bsl

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77

88
Ответ = Новый HTTPСервисОтвет( КодОтветаHTTP.КодыОтветаHTTP().OK );
99

10-
ОписаниеСервиса = РаботаСИнтернетСервисами.ПолучитьОписаниеСервиса("gitlab");
10+
ОписаниеСервиса = РаботаСИнтернетСервисами.ПолучитьОписаниеСервиса( "gitlab" );
1111

1212
ТелоОтвета = Новый Структура();
1313
ТелоОтвета.Вставить( "version", Метаданные.Версия );
@@ -22,19 +22,19 @@
2222

2323
// TODO переделать
2424

25-
Функция WebhookUpdates(Запрос)
25+
Функция WebhooksPOST(Запрос)
2626

2727
Перем Ответ;
2828
Перем ОбработчикСобытий;
2929
Перем ДанныеТелаЗапроса;
3030

31-
Ответ = Новый HTTPСервисОтвет(400);
31+
Ответ = Новый HTTPСервисОтвет( КодОтветаHTTP.КодыОтветаHTTP().BAD_REQUEST );
3232

3333
ОбработчикСобытий = СервисыGitLab.НайтиОбработчикСобытияПоHTTPЗапросу(Запрос);
3434

3535
Если ОбработчикСобытий = Неопределено Тогда
3636

37-
Ответ = Новый HTTPСервисОтвет(403);
37+
Ответ = Новый HTTPСервисОтвет( КодОтветаHTTP.КодыОтветаHTTP().FORBIDDEN );
3838
СервисыGitLab.ЗалогироватьОшибку("GitLab",
3939
"ОбработкаЗапроса",
4040
ОбработчикСобытий,
@@ -47,7 +47,7 @@
4747

4848
Если НЕ ПолучитьФункциональнуюОпцию("ЗагружатьФайлыИзВнешнегоХранилища") Тогда
4949

50-
Ответ = Новый HTTPСервисОтвет(423);
50+
Ответ = Новый HTTPСервисОтвет( КодОтветаHTTP.КодыОтветаHTTP().LOCKED );
5151
СервисыGitLab.ЗалогироватьПредупреждение("GitLab",
5252
"ОбработкаЗапроса",
5353
ОбработчикСобытий,
@@ -61,26 +61,26 @@
6161
Ответ = Неопределено;
6262
СервисыGitLab.ПроверитьЗапросUpdates(ОбработчикСобытий, Запрос, Ответ);
6363

64-
Если Ответ.КодСостояния = 200 Тогда
64+
Если ( КодОтветаHTTP.isOk(Ответ.КодСостояния) ) Тогда
6565

6666
ДанныеТелаЗапроса = Неопределено;
6767
СервисыGitLab.ПолучитьДанныеТелаЗапроса(ОбработчикСобытий, Запрос, Ответ, ДанныеТелаЗапроса);
6868

6969
КонецЕсли;
7070

71-
Если Ответ.КодСостояния = 200 Тогда
71+
Если ( КодОтветаHTTP.isOk(Ответ.КодСостояния) ) Тогда
7272

7373
СервисыGitLab.ПроверитьТелоЗапроса(ОбработчикСобытий, ДанныеТелаЗапроса, Ответ);
7474

7575
КонецЕсли;
7676

77-
Если Ответ.КодСостояния = 200 Тогда
77+
Если ( КодОтветаHTTP.isOk(Ответ.КодСостояния) ) Тогда
7878

7979
СервисыGitLab.ЗапуститьОбработкуДанныхВФоне(ОбработчикСобытий, ДанныеТелаЗапроса);
8080

8181
КонецЕсли;
8282

83-
Если Ответ.КодСостояния = 200 Тогда
83+
Если ( КодОтветаHTTP.isOk(Ответ.КодСостояния) ) Тогда
8484

8585
СервисыGitLab.ЗалогироватьИнформацию("GitLab",
8686
"ОбработкаЗапроса.Окончание",

‎GitlabServices/src/HTTPServices/gitlab/gitlab.mdo

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -10,22 +10,22 @@
1010
<reuseSessions>AutoUse</reuseSessions>
1111
<sessionMaxAge>20</sessionMaxAge>
1212
<urlTemplates uuid="160aae0f-93ea-4931-addd-a2cfc0a3ea84">
13-
<name>webhook</name>
13+
<name>webhooks</name>
1414
<synonym>
1515
<key>ru</key>
16-
<value>webhook</value>
16+
<value>Webhooks</value>
1717
</synonym>
1818
<comment>Обработка webhooks GitLab</comment>
19-
<template>/webhooks/{ТипВнешнегоХранилища}/{ИмяМетода}/*</template>
19+
<template>/webhooks/{ТипВнешнегоХранилища}/{ИмяМетода}</template>
2020
<methods uuid="dac1c217-fb98-4ec1-9b24-e99a308236de">
21-
<name>updates</name>
21+
<name>POST</name>
2222
<synonym>
2323
<key>ru</key>
24-
<value>updates</value>
24+
<value>POST</value>
2525
</synonym>
2626
<comment>Обработка события &quot;push events&quot;</comment>
2727
<httpMethod>POST</httpMethod>
28-
<handler>WebhookUpdates</handler>
28+
<handler>WebhooksPOST</handler>
2929
</methods>
3030
</urlTemplates>
3131
<urlTemplates uuid="74a26fe0-749a-491a-8507-f3a2613b49b5">

‎GitlabServices/src/Roles/HTTPСервисGitLab/Rights.rights

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@
6161
</right>
6262
</object>
6363
<object>
64-
<name>HTTPService.gitlab.URLTemplate.webhook.Method.updates</name>
64+
<name>HTTPService.gitlab.URLTemplate.webhooks.Method.POST</name>
6565
<right>
6666
<name>Use</name>
6767
<value>true</value>

‎GitlabServices/src/Roles/ПользовательGitLab/Rights.rights

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -182,7 +182,7 @@
182182
</right>
183183
</object>
184184
<object>
185-
<name>HTTPService.gitlab.URLTemplate.webhook.Method.updates</name>
185+
<name>HTTPService.gitlab.URLTemplate.webhooks.Method.POST</name>
186186
<right>
187187
<name>Use</name>
188188
<value>true</value>

‎features/Настройки/ПроверкаСервисов.feature

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,5 @@
11
#language: ru
22

3-
@tree
4-
53
Функционал: Проверка подключения к сервисам 1С для работы с GitLab
64

75
Как Пользователь
@@ -15,10 +13,10 @@
1513
Сценарий: Я проверяю подключение к сервисам 1С для работы с GitLab
1614
Пусть В командном интерфейсе я выбираю 'Сервисы GitLab' 'Настройки сервисов GitLab'
1715
Тогда открылось окно 'Настройки сервисов GitLab'
18-
1916
И я нажимаю на кнопку 'Проверить подключение к сервисам'
2017
Тогда открылось окно 'Проверка подключения к сервисам'
21-
И в поле 'URL' я ввожу текст "$$МестоположениеСервисовИБРаспределителя$$"
18+
И я запоминаю значение выражения 'КонтекстСохраняемый.СохраняемыеЗначения.Получить("МестоположениеСервисовИБРаспределителя") + "/services" ' в переменную "ServicesURL"
19+
И в поле 'URL' я ввожу значение переменной "ServicesURL"
2220
И я нажимаю на кнопку 'Проверить'
2321
И Пауза 5
2422
Тогда элемент формы с именем "ФормаРезультат" стал равен "Сервис доступен. Статус работы: включен."

‎features/ОбработчикиСобытий/step_definitions/ПроверкаPOSTЗапроса/Forms/Форма/Ext/Form/Module.bsl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@
9393
Процедура ЯДелаюPOSTЗапросНаСДаннымиПоКлючу(Парам01,Парам02,Парам03) Экспорт
9494

9595
ПараметрыСоединения = РазобратьURL(Парам01);
96-
АдресСервиса = СтрШаблон("%1/webhook/epf/updates", ПараметрыСоединения.Путь);
96+
АдресСервиса = СтрШаблон("%1/webhooks/epf/push", ПараметрыСоединения.Путь);
9797

9898
HTTPСоединение = Новый HTTPСоединение(ПараметрыСоединения.Сервер);
9999

‎features/ОбработчикиСобытий/step_definitions/ПроверкаPOSTЗапроса/fileversion

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
2019.06.10.17.01.09|ПроверкаPOSTЗапроса.epf
2-
2019.08.16.13.21.30|\Forms\Форма.xml
3-
2019.08.16.13.21.30|\Forms\Форма\Ext\Form.xml
4-
2019.08.16.13.21.30|\Forms\Форма\Ext\Form\Module.bsl
1+
2020.03.18.16.44.26|ПроверкаPOSTЗапроса.epf
2+
2020.02.12.15.58.16|\Forms\Форма.xml
3+
2020.02.12.15.58.16|\Forms\Форма\Ext\Form.xml
4+
2020.03.18.16.44.03|\Forms\Форма\Ext\Form\Module.bsl

‎features/ОбработчикиСобытий/step_definitions/СписокЭлементов/fileversion

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
2019.08.16.13.25.18|СписокЭлементов.epf
2-
2019.08.16.13.25.57|\Forms\Форма.xml
3-
2019.08.16.13.25.57|\Forms\Форма\Ext\Form.xml
4-
2019.08.16.13.25.57|\Forms\Форма\Ext\Form\Module.bsl
1+
2020.03.18.16.44.29|СписокЭлементов.epf
2+
2020.02.12.15.58.16|\Forms\Форма.xml
3+
2020.02.12.15.58.16|\Forms\Форма\Ext\Form.xml
4+
2020.02.12.15.58.16|\Forms\Форма\Ext\Form\Module.bsl

‎features/ОбработчикиСобытий/ПроверкаPOSTЗапроса.feature

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
И я закрыл все окна клиентского приложения
1313

1414
Сценарий: Проверка формы обработчика событий на наличие merge request
15+
1516
Пусть Я делаю POST запрос на "$$МестоположениеСервисовИБРаспределителя$$" с данными "$$ЭталонТелоPOSTЗапросаJSON$$" по ключу "$$GitlabToken$$"
1617
И Я нахожу обработчик событий "ТестированиеВнешниеОбработки" и открываю его
1718
Тогда в таблице "СохраненныеДанные" количество строк "равно" 1

0 commit comments

Comments
 (0)