test: add http interface module test case

Co-authored-by: imdudu1 <[email protected]>

Author: jbl428

lib/http-interface.module.spec.ts

@@ -0,0 +1,35 @@
+import { Test } from '@nestjs/testing';
+import { describe, expect, test } from 'vitest';
+import { GetExchange, HttpInterface } from './decorators';
+import { StubHttpClient } from './fixtures/stub-http-client';
+import { HttpInterfaceModule } from './http-interface.module';
+import { imitation } from './supports';
+
+describe('HttpInterfaceModule', () => {
+  @HttpInterface('http://localhost:3000')
+  class TestHttpService {
+    @GetExchange('/api/v1/test')
+    async request(): Promise<string> {
+      return imitation();
+    }
+  }
+
+  test('should request with given client', async () => {
+    // given
+    const httpClient = new StubHttpClient();
+    const module = await Test.createTestingModule({
+      imports: [HttpInterfaceModule.register({ httpClient })],
+      providers: [TestHttpService],
+    }).compile();
+    await module.init();
+    const app = module.createNestApplication();
+    const service = app.get(TestHttpService);
+    httpClient.addResponse({ data: 'ok' });
+
+    // when
+    const response = await service.request();
+
+    // then
+    expect(response).toBe('{"data":"ok"}');
+  });
+});

lib/http-interface.module.ts

@@ -14,7 +14,7 @@ export interface HttpInterfaceConfig {
 }
 
 export class HttpInterfaceModule {
-  static register(config: HttpInterfaceConfig): DynamicModule {
+  static register(config: HttpInterfaceConfig = {}): DynamicModule {
     return {
       imports: [DiscoveryModule],
       module: HttpInterfaceModule,

lib/supports/node-fetch.injector.ts

@@ -8,7 +8,7 @@ import {
   HTTP_INTERFACE_METADATA,
   RESPONSE_BODY_METADATA,
 } from '../decorators';
-import { HttpClient } from '../types/http-client.interface';
+import { HttpClient } from '../types';
 
 @Injectable()
 export class NodeFetchInjector implements OnModuleInit {
@@ -71,8 +71,11 @@ export class NodeFetchInjector implements OnModuleInit {
       .getProviders()
       .filter(
         (wrapper) =>
-          Reflect.getMetadata(HTTP_INTERFACE_METADATA, wrapper.metatype) ==
-          null,
+          wrapper.metatype?.prototype != null &&
+          Reflect.hasMetadata(
+            HTTP_INTERFACE_METADATA,
+            wrapper.metatype.prototype,
+          ),
       );
   }
 }

package.json

@@ -29,6 +29,8 @@
   "devDependencies": {
     "@nestjs/common": "^9.4.0",
     "@nestjs/core": "^9.4.0",
+    "@nestjs/platform-express": "^9.4.0",
+    "@nestjs/testing": "^9.4.0",
     "@vitest/coverage-c8": "^0.30.1",
     "class-transformer": "^0.5.1",
     "eslint": "^8.37.0",