search

valitydev / hooker

Apache License 2.0
0 stars 0 forks source link

Ft/td 138/gh action #1

Closed inallyoung closed 2 years ago

codecov[bot] commented 2 years ago

Codecov Report

Merging #1 (6d55c62) into master (a7b866b) will increase coverage by 4.54%. The diff coverage is 75.51%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master       #1      +/-   ##
============================================
+ Coverage     59.07%   63.61%   +4.54%     
+ Complexity      397      307      -90     
============================================
  Files            90       68      -22     
  Lines          2082     1542     -540     
  Branches        174      124      -50     
============================================
- Hits           1230      981     -249     
+ Misses          781      494     -287     
+ Partials         71       67       -4
Impacted Files Coverage Δ
...ava/dev/vality/hooker/configuration/AppConfig.java 100.00% <ø> (ø)
...dev/vality/hooker/dao/WebhookAdditionalFilter.java 80.00% <ø> (ø)
...a/dev/vality/hooker/dao/impl/AllHookTablesRow.java 90.00% <ø> (ø)
...n/java/dev/vality/hooker/dao/impl/HookDaoImpl.java 71.87% <ø> (ø)
.../vality/hooker/endpoint/WebhookManagerServlet.java 18.18% <ø> (ø)
...rc/main/java/dev/vality/hooker/handler/Mapper.java 100.00% <ø> (ø)
.../handler/customer/NeedReadCustomerEventMapper.java 7.69% <0.00%> (ø)
...ndler/invoicing/AdjustmentStatusChangedMapper.java 33.33% <ø> (ø)
...r/invoicing/InvoicePaymentRefundStartedMapper.java 25.00% <ø> (ø)
...icing/InvoicePaymentRefundStatusChangedMapper.java 27.77% <ø> (ø)
... and 52 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a7b866b...6d55c62. Read the comment docs.

inallyoung commented 2 years ago

@strug Зависимостей между событиями нет, инвойсы не зависят от кастомеров. что касается графа - есть ли в нем необходимость? Хуки должны уходить в порядке их получения из кафки. Последовательность изменения статусов прежняя, то есть InvoiceCreated -> PaymentPending -> PaymentProcessed -> PaymentCaptured -> InvoicePaid Родительским для каждого события считается предыдущее событие, при условии, что для его типа есть хук, кроме InvoiceCreated (для него -1).

strug commented 2 years ago

@strug Зависимостей между событиями нет, инвойсы не зависят от кастомеров. что касается графа - есть ли в нем необходимость? Хуки должны уходить в порядке их получения из кафки. Последовательность изменения статусов прежняя, то есть InvoiceCreated -> PaymentPending -> PaymentProcessed -> PaymentCaptured -> InvoicePaid Родительским для каждого события считается предыдущее событие, при условии, что для его типа есть хук, кроме InvoiceCreated (для него -1).

да звучит норм, этого достаточно) просто хотел убедиться