refactor: add factory method for http request builder

Co-authored-by: imdudu1 <cd80@kakao.com>

Author: jbl428

lib/builders/http-request.builder.ts

@@ -19,7 +19,7 @@ export class HttpRequestBuilder {
   private readonly requestHeaderBuilder?: RequestHeaderBuilder;
   private readonly payloadBuilder: PayloadBuilder;
 
-  constructor(
+  private constructor(
     readonly target: object,
     readonly propertyKey: string,
     readonly method: HttpMethod,
@@ -37,6 +37,40 @@ export class HttpRequestBuilder {
     );
   }
 
+  static forRest(
+    target: object,
+    propertyKey: string,
+    method: HttpMethod,
+    url: string,
+    options?: HttpClientOptions,
+  ): HttpRequestBuilder {
+    return new HttpRequestBuilder(
+      target,
+      propertyKey,
+      method,
+      url,
+      undefined,
+      options,
+    );
+  }
+
+  static forGraphQL(
+    target: object,
+    propertyKey: string,
+    query: string,
+    options?: HttpClientOptions,
+    url = '/graphql',
+  ): HttpRequestBuilder {
+    return new HttpRequestBuilder(
+      target,
+      propertyKey,
+      'POST',
+      url,
+      query,
+      options,
+    );
+  }
+
   setBaseUrl(baseUrl: string): void {
     this.baseUrl = baseUrl;
   }

lib/decorators/graphql-exchange.decorator.ts

@@ -13,7 +13,7 @@ export function GraphQLExchange(
   ) => {
     Reflect.defineMetadata(
       HTTP_EXCHANGE_METADATA,
-      new HttpRequestBuilder(target, propertyKey, 'POST', url, query, options),
+      HttpRequestBuilder.forGraphQL(target, propertyKey, query, options, url),
       target,
       propertyKey,
     );

lib/decorators/http-exchange.decorator.ts

@@ -11,14 +11,7 @@ export const HttpExchange =
   <P extends string>(target: Record<P, AsyncFunction>, propertyKey: P) => {
     Reflect.defineMetadata(
       HTTP_EXCHANGE_METADATA,
-      new HttpRequestBuilder(
-        target,
-        propertyKey,
-        method,
-        url,
-        undefined,
-        options,
-      ),
+      HttpRequestBuilder.forRest(target, propertyKey, method, url, options),
       target,
       propertyKey,
     );

lib/supports/fetch-http-client.spec.ts

@@ -12,7 +12,7 @@ describe('FetchHttpClient', () => {
   let fastify: FastifyInstance;
 
   beforeAll(async () => {
-    fastify = Fastify({ logger: false });
+    fastify = Fastify({ logger: false, forceCloseConnections: true });
 
     fastify.get('/', async (request, reply) => {
       await reply.send({ hello: 'world' });
@@ -115,13 +115,13 @@ describe('FetchHttpClient', () => {
     const address = fastify.server.address() as AddressInfo;
     const httpClient = new FetchHttpClient(60000);
     const request = new Request(`http://localhost:${address.port}/timeout`);
-    const httpClientOptions = { timeout: 300 };
+    const httpClientOptions = { timeout: 1000 };
 
     // when
     const doRequest = async (): Promise<Response> =>
       await httpClient.request(request, httpClientOptions);
 
     // then
-    await expect(doRequest).rejects.toThrowError('Request Timeout: 300ms');
+    await expect(doRequest).rejects.toThrowError('Request Timeout: 1000ms');
   });
 });