chore: Adjust jest for esm and monorepo (#57)

* add jest node optino

* move test-util to src

* fix: Changes from lint

* fix

* fix

* fix path

* tests fix

* fix

* lock file

---------

Co-authored-by: cloud-sdk-js <[email protected]>

package.json

@@ -14,7 +14,7 @@
   "scripts": {
     "postinstall": "pnpm compile",
     "compile": "pnpm -r -w=false run compile",
-    "test": "pnpm -r run test",
+    "test": "NODE_OPTIONS=--experimental-vm-modules pnpm -r run test",
     "test:type": "tsd",
     "lint": "pnpm -r run lint",
     "lint:fix": "pnpm -r run lint:fix"
@@ -26,6 +26,7 @@
     "@sap-cloud-sdk/http-client": "^3.18.0",
     "@types/jest": "^29.5.12",
     "@types/node": "^20.14.14",
+    "@jest/globals": "^29.5.12",
     "eslint": "^9.8.0",
     "jest": "^30.0.0-alpha.3",
     "nock": "^13.5.4",

packages/ai-core/package.json

@@ -18,7 +18,7 @@
   ],
   "scripts": {
     "compile": "tsc",
-    "test": "jest",
+    "test": "NODE_OPTIONS=--experimental-vm-modules jest",
     "lint": "eslint . && prettier . --config ../../.prettierrc --ignore-path ../../.prettierignore -c",
     "lint:fix": "eslint . --fix && prettier . --config ../../.prettierrc --ignore-path ../../.prettierignore -w --log-level error"
   },

packages/gen-ai-hub/jest.config.mjs

@@ -1,7 +1,5 @@
 import config from '../../jest.config.mjs';
-const aiCoreConfig = {
+export default {
   ...config,
   displayName: 'gen-ai-hub',
 };
-
-export default aiCoreConfig;

packages/gen-ai-hub/package.json

@@ -20,7 +20,7 @@
   ],
   "scripts": {
     "compile": "tsc",
-    "test": "jest",
+    "test": "NODE_OPTIONS=--experimental-vm-modules jest",
     "lint": "eslint . && prettier . --config ../../.prettierrc --ignore-path ../../.prettierignore -c",
     "lint:fix": "eslint . --fix && prettier . --config ../../.prettierrc --ignore-path ../../.prettierignore -w --log-level error",
     "generate": "openapi-generator -i ./src/orchestration/spec/ -o ./src/orchestration/client"

packages/gen-ai-hub/src/client/openai/openai-client.test.ts

@@ -1,22 +1,25 @@
 import nock from 'nock';
+import { jest } from '@jest/globals';
 import { HttpDestination } from '@sap-cloud-sdk/connectivity';
-import { mockGetAiCoreDestination } from '../../../test-util/mock-context.js';
+import { mockGetAiCoreDestination } from '../../test-util/mock-context.js';
 import {
   BaseLlmParametersWithDeploymentId,
   EndpointOptions
 } from '../../core/http-client.js';
-import {
-  mockInference,
-  parseMockResponse
-} from '../../../test-util/mock-http.js';
-import { OpenAiClient } from './openai-client.js';
+import { mockInference, parseMockResponse } from '../../test-util/mock-http.js';
 import {
   OpenAiChatCompletionOutput,
   OpenAiChatCompletionParameters,
   OpenAiChatMessage,
   OpenAiEmbeddingOutput,
   OpenAiEmbeddingParameters
 } from './openai-types.js';
+jest.unstable_mockModule('../../core/context.js', () => ({
+  getAiCoreDestination: jest.fn(() =>
+    Promise.resolve(mockGetAiCoreDestination())
+  )
+}));
+const { OpenAiClient } = await import('./openai-client.js');
 
 describe('openai client', () => {
   let destination: HttpDestination;
@@ -44,6 +47,10 @@ describe('openai client', () => {
     nock.cleanAll();
   });
 
+  afterAll(() => {
+    jest.restoreAllMocks();
+  });
+
   describe('chatCompletion', () => {
     it('parses a successful response', async () => {
       const prompt = {

packages/gen-ai-hub/src/core/context.test.ts

@@ -1,7 +1,7 @@
 import {
   mockAiCoreEnvVariable,
   mockClientCredentialsGrantCall
-} from '../../test-util/mock-context.js';
+} from '../test-util/mock-context.js';
 import { getAiCoreDestination } from './context.js';
 
 describe('context', () => {

packages/gen-ai-hub/src/core/http-client.test.ts

@@ -1,7 +1,14 @@
+import { jest } from '@jest/globals';
 import { HttpDestination } from '@sap-cloud-sdk/connectivity';
-import { mockGetAiCoreDestination } from '../../test-util/mock-context.js';
-import { mockInference } from '../../test-util/mock-http.js';
-import { executeRequest } from './http-client.js';
+import { mockGetAiCoreDestination } from '../test-util/mock-context.js';
+import { mockInference } from '../test-util/mock-http.js';
+
+jest.unstable_mockModule('./context.js', () => ({
+  getAiCoreDestination: jest.fn(() =>
+    Promise.resolve(mockGetAiCoreDestination())
+  )
+}));
+const { executeRequest } = await import('./http-client.js');
 
 describe('http-client', () => {
   let destination: HttpDestination;
@@ -10,6 +17,10 @@ describe('http-client', () => {
     destination = mockGetAiCoreDestination();
   });
 
+  afterAll(() => {
+    jest.restoreAllMocks();
+  });
+
   it('should execute a request to the AI Core service', async () => {
     const mockPrompt = { prompt: 'some test prompt' };
     const mockPromptResponse = { completion: 'some test completion' };

packages/gen-ai-hub/src/orchestration/orchestration-client.test.ts

@@ -1,31 +1,39 @@
 import nock from 'nock';
+import { jest } from '@jest/globals';
 import { HttpDestination } from '@sap-cloud-sdk/connectivity';
-import { mockGetAiCoreDestination } from '../../test-util/mock-context.js';
-import { mockInference, parseMockResponse } from '../../test-util/mock-http.js';
+import { mockGetAiCoreDestination } from '../test-util/mock-context.js';
+import { mockInference, parseMockResponse } from '../test-util/mock-http.js';
 import { BaseLlmParametersWithDeploymentId } from '../core/index.js';
-import {
-  GenAiHubClient,
-  constructCompletionPostRequest
-} from './orchestration-client.js';
 import { CompletionPostResponse } from './api/index.js';
 import { GenAiHubCompletionParameters } from './orchestration-types.js';
+jest.unstable_mockModule('../core/context.js', () => ({
+  getAiCoreDestination: jest.fn(() =>
+    Promise.resolve(mockGetAiCoreDestination())
+  )
+}));
 
+const { GenAiHubClient, constructCompletionPostRequest } = await import(
+  './orchestration-client.js'
+);
 describe('GenAiHubClient', () => {
   let destination: HttpDestination;
-  let client: GenAiHubClient;
+  const client = new GenAiHubClient();
   const deploymentConfiguration: BaseLlmParametersWithDeploymentId = {
     deploymentId: 'deployment-id'
   };
 
   beforeAll(() => {
     destination = mockGetAiCoreDestination();
-    client = new GenAiHubClient();
   });
 
   afterEach(() => {
     nock.cleanAll();
   });
 
+  afterAll(() => {
+    jest.restoreAllMocks();
+  });
+
   it('calls chatCompletion with minimum configuration and parses response', async () => {
     const request: GenAiHubCompletionParameters = {
       deploymentConfiguration,

packages/gen-ai-hub/test-util/mock-context.ts → packages/gen-ai-hub/src/test-util/mock-context.ts

@@ -4,7 +4,6 @@ import {
   ServiceCredentials
 } from '@sap-cloud-sdk/connectivity';
 import nock from 'nock';
-import * as context from '../src/core/context.js';
 
 export const aiCoreServiceBinding = {
   label: 'aicore',
@@ -54,11 +53,6 @@ export function mockGetAiCoreDestination(
     authentication: 'OAuth2ClientCredentials',
     ...createDestinationTokens()
   };
-
-  jest
-    .spyOn(context, 'getAiCoreDestination')
-    .mockResolvedValue(mockDestination);
-
   return mockDestination;
 }
 

packages/gen-ai-hub/test-util/mock-http.ts → packages/gen-ai-hub/src/test-util/mock-http.ts

@@ -6,7 +6,7 @@ import {
   BaseLlmParameters,
   CustomRequestConfig,
   EndpointOptions
-} from '../src/core/http-client.js';
+} from '../core/http-client.js';
 
 const mockEndpoint: EndpointOptions = {
   url: 'mock-endpoint',
@@ -44,7 +44,7 @@ export function mockInference<D extends BaseLlmParameters>(
 
 export function parseMockResponse<T>(client: string, fileName: string): T {
   const fileContent = fs.readFileSync(
-    path.join('test-util', 'mock-data', client, fileName),
+    path.join('src', 'test-util', 'mock-data', client, fileName),
     'utf-8'
   );