test: make coverage 100%

Author: jbl428

lib/http-interface.module.spec.ts

@@ -4,6 +4,9 @@ import { GetExchange, HttpInterface } from './decorators';
 import { StubHttpClient } from './fixtures/stub-http-client';
 import { HttpInterfaceModule } from './http-interface.module';
 import { imitation } from './supports';
+import { FetchHttpClient } from './supports/fetch-http-client';
+import { NodeFetchInjector } from './supports/node-fetch.injector';
+import { type HttpClient } from './types';
 
 describe('HttpInterfaceModule', () => {
   @HttpInterface('http://localhost:3000')
@@ -14,6 +17,21 @@ describe('HttpInterfaceModule', () => {
     }
   }
 
+  test('should use default http client if not given', async () => {
+    // given
+    const module = await Test.createTestingModule({
+      imports: [HttpInterfaceModule.register()],
+    }).compile();
+    await module.init();
+    const app = module.createNestApplication();
+
+    // when
+    const injector = app.get<{ httpClient: HttpClient }>(NodeFetchInjector);
+
+    // then
+    expect(injector.httpClient).toBeInstanceOf(FetchHttpClient);
+  });
+
   test('should request with given client', async () => {
     // given
     const httpClient = new StubHttpClient();

lib/supports/node-fetch.injector.ts

@@ -22,12 +22,12 @@ export class NodeFetchInjector implements OnModuleInit {
     const httpProviders = this.getHttpProviders();
 
     httpProviders.forEach((wrapper) => {
-      const baseUrl: string | undefined = Reflect.getMetadata(
+      const baseUrl: string = Reflect.getMetadata(
         HTTP_INTERFACE_METADATA,
         wrapper.metatype.prototype,
       );
 
-      this.wrapMethods(wrapper, baseUrl ?? '');
+      this.wrapMethods(wrapper, baseUrl);
     });
   }