NodeKit
NodeKit copied to clipboard
SPT-1998 Мелкий рефакторинг + моки на билдер
Что сделано
- Добавлены структуры-обертки AnyAsyncNode и AnyAsyncStreamNode в связи с тем, что компилятор не может правильно зарезолвить типы при использование any AsyncNode в билдере. При этом использовать some AsyncNode мы не можем, так как some не поддерживается в протоколах и нельзя наследовать методы класса, которые возвращают some.
- Добавлены методы eraseToAnyNode, которые оборачивают ноду в AnyAsyncNode или AnyAsyncStreamNode.
- Не много изменен билдер.
- Написаны моки на билдеры. Теперь можно передавать в сервис протокол ChainBuilder, а в тестах менять на мок и стаббать ноды.
- Переименованы классы, методы и переменные с Url на URL.
- Удалена нода LoadIndicatableNode (по моему бесползеная).
- Удалены лишние параметры из моделей (serializationTimeout и metrics)
Codecov Report
Attention: Patch coverage is 65.88785% with 73 lines in your changes are missing coverage. Please review.
Project coverage is 91.50%. Comparing base (
da998b5) to head (a9b9219). Report is 1 commits behind head on 5.0.0.
Additional details and impacted files
@@ Coverage Diff @@
## 5.0.0 #133 +/- ##
==========================================
+ Coverage 91.39% 91.50% +0.11%
==========================================
Files 83 85 +2
Lines 1267 1260 -7
==========================================
- Hits 1158 1153 -5
+ Misses 109 107 -2
| Flag | Coverage Δ | |
|---|---|---|
| tests | 91.50% <65.88%> (+0.11%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Добавлен метод merge, для двух AsyncNode. Позволяет смержить две AsyncNode с одинаковыми Input и Output в AsyncStreamNode.