ci(api): fix coverage collection and enforce a ratchet coverage gate

.github/workflows/ci.yml

@@ -39,4 +39,4 @@ jobs:
         run: npm run build
 
       - name: Test
-        run: npm test
+        run: npm run test:cov

package.json

@@ -94,7 +94,11 @@
       "json",
       "ts"
     ],
-    "rootDir": "test",
+    "rootDir": ".",
+    "roots": [
+      "<rootDir>/test",
+      "<rootDir>/src"
+    ],
     "testRegex": ".*\\.spec\\.ts$",
     "transform": {
       "^.+\\.ts$": [
@@ -105,12 +109,20 @@
       ]
     },
     "collectCoverageFrom": [
-      "../src/**/*.{t,j}s"
+      "src/**/*.{t,j}s"
     ],
-    "coverageDirectory": "../coverage",
+    "coverageDirectory": "coverage",
+    "coverageThreshold": {
+      "global": {
+        "statements": 45,
+        "branches": 30,
+        "functions": 35,
+        "lines": 45
+      }
+    },
     "testEnvironment": "node",
     "moduleNameMapper": {
-      "^src/(.*)$": "<rootDir>/../src/$1"
+      "^src/(.*)$": "<rootDir>/src/$1"
     }
   },
   "overrides": {

src/modules/vendors/vendors.service.spec.ts

@@ -1,9 +1,11 @@
 import { Test, TestingModule } from '@nestjs/testing';
 import { ConflictException, NotFoundException, ForbiddenException, UnauthorizedException, ExecutionContext } from '@nestjs/common';
 import { ConfigService } from '@nestjs/config';
+import { CACHE_MANAGER } from '@nestjs/cache-manager';
 import { VendorsService } from './vendors.service';
 import { VendorsController } from './vendors.controller';
 import { SupabaseService } from '../../database/supabase.client';
+import { VendorsRepository } from '../../database/repositories/vendors.repository';
 import { VendorRegistryContractClient } from '../../stellar/contracts/clients/vendor-registry.client';
 import { VendorType, VendorStatus } from './dto/vendor.dto';
 import { AdminGuard } from '../../common/guards/admin.guard';
@@ -14,6 +16,8 @@ describe('VendorsModule', () => {
   let mockSupabaseService: any;
   let mockVendorRegistryClient: any;
   let mockConfigService: any;
+  let mockVendorsRepository: any;
+  let mockCacheManager: any;
   let adminGuard: AdminGuard;
 
   const mockAdminWallet = 'GA3D5342W...ADMIN_WALLET_ADDRESS';
@@ -79,14 +83,30 @@ describe('VendorsModule', () => {
       }),
     };
 
+    // The methods under test read through supabaseService and the registry
+    // client; the repository and cache are wired only to satisfy DI.
+    mockVendorsRepository = {
+      findById: jest.fn(),
+      findAll: jest.fn(),
+      updateStatus: jest.fn(),
+    };
+
+    mockCacheManager = {
+      get: jest.fn(),
+      set: jest.fn(),
+      del: jest.fn(),
+    };
+
     const module: TestingModule = await Test.createTestingModule({
       controllers: [VendorsController],
       providers: [
         VendorsService,
         AdminGuard,
         { provide: SupabaseService, useValue: mockSupabaseService },
+        { provide: VendorsRepository, useValue: mockVendorsRepository },
         { provide: VendorRegistryContractClient, useValue: mockVendorRegistryClient },
         { provide: ConfigService, useValue: mockConfigService },
+        { provide: CACHE_MANAGER, useValue: mockCacheManager },
       ],
     }).compile();