Merge pull request #146 from blockfrost/chore/dont-close-me-mneow

chore: don't spin up token registry server, mock func instead

CHANGELOG.md

@@ -21,6 +21,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - node_modules
 - yarn
+- updated vitest
+- removed token registry mock server, using function mocking instead
 
 ### Fixed
 

package.json

@@ -67,7 +67,7 @@
     "supertest": "^6.3.3",
     "ts-node": "^10.9.1",
     "typescript": "^5.2.2",
-    "vitest": "^0.27"
+    "vitest": "^1.2.2"
   },
   "packageManager": "[email protected]",
   "type": "module",

src/routes/assets/asset/index.ts

@@ -92,14 +92,11 @@ async function route(fastify: FastifyInstance) {
 
         // retrieve off-chain metadata
         const metadata = await fetchAssetMetadata(request.params.asset);
-        // @ts-expect-error due to using ESM with vitest and some weirdness with cip14 lib we need to call .default
-        // otherwise unit test fails on  ""...is not a function".
-        const fingerprint = AssetFingerprint.default
-          .fromParts(
-            Uint8Array.from(Buffer.from(rows[0].policy_id, 'hex')),
-            Uint8Array.from(Buffer.from(rows[0].asset_name ?? '', 'hex')),
-          )
-          .fingerprint();
+
+        const fingerprint = AssetFingerprint.fromParts(
+          Uint8Array.from(Buffer.from(rows[0].policy_id, 'hex')),
+          Uint8Array.from(Buffer.from(rows[0].asset_name ?? '', 'hex')),
+        ).fingerprint();
 
         return reply.send({
           ...rows[0],

test/unit/tests/routes/addresses.unit.ts

@@ -1,13 +1,20 @@
 import * as databaseUtils from '../../../../src/utils/database.js';
+import * as tokenRegistryUtils from '../../../../src/utils/token-registry.js';
 import sinon from 'sinon';
 import supertest from 'supertest';
 import fixtures from '../../fixtures/addresses.fixtures.js';
 import buildFastify from '../../../../src/app.js';
-import { describe, expect, test, vi } from 'vitest';
+import { beforeEach, describe, expect, test, vi } from 'vitest';
 
 import * as config from '../../../../src/config.js';
 
 describe('address service', () => {
+  beforeEach(() => {
+    vi.spyOn(tokenRegistryUtils, 'fetchAssetMetadata').mockImplementation(() => {
+      return Promise.resolve(null);
+    });
+  });
+
   fixtures.map(fixture => {
     test(fixture.name, async () => {
       const fastify = buildFastify({ maxParamLength: 32_768 });

vitest.config.ts

@@ -3,8 +3,9 @@ import { defineConfig, defaultInclude } from 'vitest/config';
 
 export default defineConfig({
   test: {
+    // https://vitest.dev/guide/common-errors.html#failed-to-terminate-worker
+    pool: 'forks',
     reporters: ['verbose'],
-    globalSetup: ['./test/unit/server-token-registry.ts'],
     testTimeout: 20_000,
     root: '.',
     mockReset: true,