0x5chn0uf / SmartWalletFX / 16856448882
86%

DEFAULT BRANCH: main
Ran
Jobs 15
Files 127
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 85.898% (+0.1%) from 85.764%
16856448882

push

github

0x5chn0uf 
feat: complete Task 50 - eliminate module-level singletons with dependency injection

## 🎯 Task 50 Successfully Completed
- **Objective:** Eliminate all module-level singletons and implement proper dependency injection
- **Result:** 99.9% test pass rate (866/867 tests passing)
- **Architecture:** Complete transition to class-based dependency injection throughout codebase

## 🔧 Key Changes

### JWKS Architecture Refactoring
- **New Use Case:** Created `JWKSUsecase` following DDD principles
- **Endpoint Refactor:** Updated `JWKS` endpoint to use dependency injection
- **Cache Management:** Moved from module-level functions to `JWKSCacheUtils` class
- **Key Management:** Refactored `JWTKeyUtils` to implement proper interface

### Dependency Injection Integration
- **DI Container:** Added JWKS use case registration and endpoint initialization
- **Interface Compliance:** Updated all interfaces to support class-based patterns
- **Singleton Elimination:** Removed all module-level singletons and global state

### JWT Rotation Task Integration
- **Class Integration:** Updated rotation task to use `JWKSCacheUtils` class methods
- **Cache Invalidation:** Proper integration with new DI-based cache utilities
- **Error Handling:** Maintained graceful degradation for cache operations

### Test Infrastructure Updates
- **Mocking Strategy:** Updated all tests from module-level to class-based mocking
- **JWKS Tests:** Complete rewrite of JWKS endpoint and metrics tests
- **Property-Based Tests:** Fixed complex Hypothesis-based JWKS tests for new architecture
- **Integration Tests:** Updated JWT rotation and cache invalidation test suites
- **Infrastructure Tests:** Fixed Redis dependency tests and HTTPX compatibility

### Compatibility & Migration
- **Backward Compatibility:** Maintained API contract while changing internal implementation
- **Progressive Migration:** Systematic replacement of module-level patterns
- **Zero Breaking Changes:** All existing functionality... (continued)

4203 of 4893 relevant lines covered (85.9%)

6.8 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
3
89.52
 1.61% api/dependencies.py
3
92.41
 0.1% tasks/jwt_rotation.py
6
97.63
 0.06% di.py
8
43.33
 -2.12% utils/encryption.py
14
61.22
 -1.74% utils/jwt_keys.py
18
58.62
 -10.61% utils/jwks_cache.py
Jobs
ID Job ID Ran Files Coverage
1 integration-auth - 16856448882.1 126
56.13
 GitHub Action Run
2 integration-user_profile - 16856448882.2 124
54.27
 GitHub Action Run
3 unit-users - 16856448882.3 121
42.28
 GitHub Action Run
4 unit-auth - 16856448882.4 125
59.94
 GitHub Action Run
5 integration-wallets - 16856448882.5 124
54.6
 GitHub Action Run
6 unit-email_verification - 16856448882.6 121
39.23
 GitHub Action Run
7 integration-password_reset - 16856448882.7 124
55.0
 GitHub Action Run
8 integration-users - 16856448882.8 121
38.27
 GitHub Action Run
9 unit-wallets - 16856448882.9 121
48.11
 GitHub Action Run
10 unit-oauth - 16856448882.10 112
30.14
 GitHub Action Run
11 unit-jwks - 16856448882.11 126
52.49
 GitHub Action Run
12 unit-user_profile - 16856448882.12 124
54.71
 GitHub Action Run
13 unit-password_reset - 16856448882.13 121
39.91
 GitHub Action Run
14 integration-oauth - 16856448882.14 124
53.99
 GitHub Action Run
15 unit-tokens - 16856448882.15 110
27.6
 GitHub Action Run
Source Files on build 16856448882