Skip to content

feat: implement credentials refresh#72

Open
ns-vasilev wants to merge 3 commits into
mainfrom
feature/typhoon-v3
Open

feat: implement credentials refresh#72
ns-vasilev wants to merge 3 commits into
mainfrom
feature/typhoon-v3

Conversation

@ns-vasilev
Copy link
Copy Markdown
Member

No description provided.

@ns-vasilev ns-vasilev self-assigned this May 6, 2026
@ns-vasilev ns-vasilev requested a review from a team as a code owner May 6, 2026 11:31
@spacecode-github-bot
Copy link
Copy Markdown

spacecode-github-bot commented May 6, 2026

4 Warnings
⚠️ Big PR - consider breaking it into smaller chunks
⚠️

⚠️ Some source files are missing corresponding test files:

  • NetworkLayerAssembly → expected test file: NetworkLayerAssemblyTests.swift
  • Response → expected test file: ResponseTests.swift
⚠️ New dependency added but README not updated
⚠️ Public API changed but no documentation updates found
4 Messages
📖 📦 Package.swift was modified - please ensure:
📖
  • Dependencies are pinned to specific versions
📖
  • New dependencies are documented in README
📖
  • Platform requirements are correct

Generated by 🚫 Danger

@codecov
Copy link
Copy Markdown

codecov Bot commented May 6, 2026

Codecov Report

❌ Patch coverage is 97.31013% with 17 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.91%. Comparing base (400f26e) to head (586ea42).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...aces/Classes/Core/Services/IRequestProcessor.swift 0.00% 6 Missing ⚠️
...lasses/Tests/UnitTests/RequestProcessorTests.swift 97.06% 6 Missing ⚠️
...ionTests/RequestProcessorAuthenticationTests.swift 84.62% 2 Missing ⚠️
...ntegrationTests/RequestProcessorRequestTests.swift 80.00% 2 Missing ⚠️
...NetworkLayer/Classes/DI/NetworkLayerAssembly.swift 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #72      +/-   ##
==========================================
+ Coverage   90.19%   91.91%   +1.72%     
==========================================
  Files          43       44       +1     
  Lines        1773     2348     +575     
==========================================
+ Hits         1599     2158     +559     
- Misses        174      190      +16     
Files with missing lines Coverage Δ
...e/Services/RequestProcessor/RequestProcessor.swift 100.00% <100.00%> (ø)
...LayerInterfaces/Classes/Core/Models/Response.swift 100.00% <ø> (ø)
...rInterfaces/Classes/DI/INetworkLayerAssembly.swift 0.00% <ø> (ø)
...lasses/Helpers/Helpers/RequestProcessor+Mock.swift 100.00% <100.00%> (ø)
...Helpers/Mocks/AuthentificatorInterceptorMock.swift 100.00% <100.00%> (ø)
...Classes/Helpers/Mocks/DataRequestHandlerMock.swift 83.87% <100.00%> (+3.10%) ⬆️
...es/Tests/UnitTests/RequestProcessorAuthTests.swift 100.00% <100.00%> (ø)
...NetworkLayer/Classes/DI/NetworkLayerAssembly.swift 0.00% <0.00%> (ø)
...ionTests/RequestProcessorAuthenticationTests.swift 98.43% <84.62%> (-1.57%) ⬇️
...ntegrationTests/RequestProcessorRequestTests.swift 95.71% <80.00%> (-2.67%) ⬇️
... and 2 more

@ns-vasilev ns-vasilev force-pushed the feature/typhoon-v3 branch from 24e2e8f to 9a78793 Compare May 11, 2026 11:05
@ns-vasilev ns-vasilev force-pushed the feature/typhoon-v3 branch from 9a78793 to 586ea42 Compare May 11, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants