refactor: enhance hypercert retrieval in SalesResolver

- Updated the hypercert retrieval process to include fetching metadata alongside the hypercert data.
- Improved the return structure to include both hypercert and its associated metadata.

Author: Jipperism

src/services/graphql/resolvers/salesResolver.ts

@@ -100,13 +100,25 @@ class SalesResolver {
     }
 
     try {
-      return await this.hypercertsService.getHypercert({
-        where: {
-          hypercert_id: {
-            eq: sale.hypercert_id,
+      const [hypercert, metadata] = await Promise.all([
+        this.hypercertsService.getHypercert({
+          where: {
+            hypercert_id: { eq: sale.hypercert_id },
           },
-        },
-      });
+        }),
+        this.hypercertsService.getHypercertMetadata({
+          hypercert_id: sale.hypercert_id,
+        }),
+      ]);
+
+      if (!hypercert) {
+        return null;
+      }
+
+      return {
+        ...hypercert,
+        metadata: metadata || null,
+      };
     } catch (e) {
       console.error(
         `[SalesResolver::hypercert] Error fetching hypercert: ${(e as Error).message}`,

test/services/graphql/resolvers/salesResolver.test.ts

@@ -16,6 +16,7 @@ describe("SalesResolver", () => {
   };
   let mockHypercertsService: {
     getHypercert: Mock;
+    getHypercertMetadata: Mock;
   };
   let mockSale: Sale;
 
@@ -31,6 +32,7 @@ describe("SalesResolver", () => {
 
     mockHypercertsService = {
       getHypercert: vi.fn(),
+      getHypercertMetadata: vi.fn(),
     };
 
     // Register mocks with the DI container
@@ -103,6 +105,7 @@ describe("SalesResolver", () => {
       const expectedHypercert = {
         id: faker.string.uuid(),
         hypercert_id: mockSale.hypercert_id,
+        metadata: null,
       };
       mockHypercertsService.getHypercert.mockResolvedValue(expectedHypercert);