Skip to content

Conversation

@ns-vasilev
Copy link
Member

No description provided.

@ns-vasilev ns-vasilev self-assigned this Dec 7, 2025
@spacecode-github-bot
Copy link

spacecode-github-bot commented Dec 7, 2025

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

⚠️ Some source files are missing corresponding test files:

  • DataRequestHandler → expected test file: DataRequestHandlerTests.swift
  • Response → expected test file: ResponseTests.swift
⚠️ Public API changed but no documentation updates found
5 Messages
📖

ℹ️ Some source files have tests that weren't updated:

  • AuthenticationInterceptor → test file exists: AuthenticationInterceptorTests.swift (consider updating if logic changed)
📖 📦 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

@ns-vasilev ns-vasilev force-pushed the feature/mise branch 2 times, most recently from 6298052 to e9057f0 Compare December 7, 2025 18:06
@codecov
Copy link

codecov bot commented Dec 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.85%. Comparing base (1ca93b6) to head (3c02cb6).
⚠️ Report is 5 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   88.48%   91.85%   +3.36%     
==========================================
  Files          10       41      +31     
  Lines         330     1398    +1068     
==========================================
+ Hits          292     1284     +992     
- Misses         38      114      +76     
Files with missing lines Coverage Δ
...e/Authentification/AuthenticationInterceptor.swift 100.00% <100.00%> (ø)
...rvices/DataRequestHandler/DataRequestHandler.swift 96.94% <100.00%> (ø)
...e/Services/RequestProcessor/RequestProcessor.swift 98.80% <100.00%> (ø)
...LayerInterfaces/Classes/Core/Models/IRequest.swift 0.00% <ø> (ø)
...LayerInterfaces/Classes/Core/Models/Response.swift 100.00% <ø> (ø)
...aces/Classes/Core/Services/IRequestProcessor.swift 0.00% <ø> (ø)
...lasses/Helpers/Fakes/URLSessionDataTask+Fake.swift 100.00% <100.00%> (ø)
...ts/Classes/Helpers/Fakes/URLSessionTask+Fake.swift 100.00% <100.00%> (ø)
...erTests/Classes/Helpers/Helpers/DynamicStubs.swift 100.00% <ø> (ø)
...ests/Classes/Helpers/Mocks/AuthenticatorMock.swift 100.00% <ø> (ø)
... and 4 more

... and 20 files with indirect coverage changes

@ns-vasilev ns-vasilev merged commit da40782 into main Dec 7, 2025
24 checks passed
@ns-vasilev ns-vasilev deleted the feature/mise branch December 7, 2025 19:46
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.

3 participants