Skip to content

Commit 88573af

Browse files
committed
restructure test helpers, integration tests, clean up package.json
1 parent 2ff0cc5 commit 88573af

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

43 files changed

+191
-72
lines changed

examples/shared/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -41,6 +41,7 @@
4141
"@modelcontextprotocol/tsconfig": "workspace:^",
4242
"@modelcontextprotocol/eslint-config": "workspace:^",
4343
"@modelcontextprotocol/vitest-config": "workspace:^",
44+
"@modelcontextprotocol/test-helpers": "workspace:^",
4445
"@types/express": "catalog:devTools",
4546
"@typescript/native-preview": "catalog:devTools",
4647
"@eslint/js": "catalog:devTools",

examples/shared/test/demoInMemoryOAuthProvider.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
import type { OAuthClientInformationFull } from '@modelcontextprotocol/sdk-core';
22
import type { AuthorizationParams } from '@modelcontextprotocol/sdk-server';
33
import { InvalidRequestError } from '@modelcontextprotocol/sdk-server';
4+
import { createExpressResponseMock } from '@modelcontextprotocol/test-helpers';
45
import type { Response } from 'express';
56
import { beforeEach, describe, expect, it } from 'vitest';
67

7-
import { createExpressResponseMock } from '../../../packages/integration/test/helpers/http.js';
88
import { DemoInMemoryAuthProvider, DemoInMemoryClientsStore } from '../src/demoInMemoryOAuthProvider.js';
99

1010
describe('DemoInMemoryAuthProvider', () => {

examples/shared/tsconfig.json

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,9 @@
1111
"node_modules/@modelcontextprotocol/sdk-server/node_modules/@modelcontextprotocol/sdk-core/src/index.ts"
1212
],
1313
"@modelcontextprotocol/eslint-config": ["node_modules/@modelcontextprotocol/eslint-config/tsconfig.json"],
14-
"@modelcontextprotocol/vitest-config": ["node_modules/@modelcontextprotocol/vitest-config/tsconfig.json"]
14+
"@modelcontextprotocol/vitest-config": ["node_modules/@modelcontextprotocol/vitest-config/tsconfig.json"],
15+
"@modelcontextprotocol/test-helpers": ["node_modules/@modelcontextprotocol/test-helpers/src/index.ts"],
16+
"@modelcontextprotocol/sdk-client": ["node_modules/@modelcontextprotocol/test-helpers/node_modules/@modelcontextprotocol/sdk-client/src/index.ts"],
1517
}
1618
}
1719
}

packages/client/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,7 @@
6868
"@modelcontextprotocol/tsconfig": "workspace:^",
6969
"@modelcontextprotocol/vitest-config": "workspace:^",
7070
"@modelcontextprotocol/eslint-config": "workspace:^",
71+
"@modelcontextprotocol/test-helpers": "workspace:^",
7172
"@cfworker/json-schema": "catalog:runtimeShared",
7273
"@types/content-type": "catalog:devTools",
7374
"@types/cross-spawn": "catalog:devTools",

packages/client/test/client/auth-extensions.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1+
import { createMockOAuthFetch } from '@modelcontextprotocol/test-helpers';
12
import { describe, expect, it } from 'vitest';
23

3-
import { createMockOAuthFetch } from '../../../integration/test/helpers/oauth.js';
44
import { auth } from '../../src/client/auth.js';
55
import {
66
ClientCredentialsProvider,

packages/client/test/client/sse.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@ import type { AddressInfo } from 'node:net';
44

55
import type { JSONRPCMessage, OAuthTokens } from '@modelcontextprotocol/sdk-core';
66
import { InvalidClientError, InvalidGrantError, UnauthorizedClientError } from '@modelcontextprotocol/sdk-core';
7+
import { listenOnRandomPort } from '@modelcontextprotocol/test-helpers';
78
import type { Mock, Mocked, MockedFunction, MockInstance } from 'vitest';
89

9-
import { listenOnRandomPort } from '../../../integration/test/helpers/http.js';
1010
import type { OAuthClientProvider } from '../../src/client/auth.js';
1111
import { UnauthorizedError } from '../../src/client/auth.js';
1212
import { SSEClientTransport } from '../../src/client/sse.js';

packages/client/tsconfig.json

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,13 @@
77
"paths": {
88
"*": ["./*"],
99
"@modelcontextprotocol/sdk-core": ["node_modules/@modelcontextprotocol/sdk-core/src/index.ts"],
10-
"@modelcontextprotocol/sdk-client": ["node_modules/@modelcontextprotocol/sdk-client/src/index.ts"],
11-
"@modelcontextprotocol/vitest-config": ["node_modules/@modelcontextprotocol/vitest-config/tsconfig.json"],
12-
"@modelcontextprotocol/eslint-config": ["node_modules/@modelcontextprotocol/eslint-config/tsconfig.json"]
10+
"@modelcontextprotocol/test-helpers": ["node_modules/@modelcontextprotocol/test-helpers/src/index.ts"],
11+
"@modelcontextprotocol/sdk-client": [
12+
"node_modules/@modelcontextprotocol/test-helpers/node_modules/@modelcontextprotocol/sdk-client/src/index.ts"
13+
],
14+
"@modelcontextprotocol/sdk-server": [
15+
"node_modules/@modelcontextprotocol/test-helpers/node_modules/@modelcontextprotocol/sdk-server/src/index.ts"
16+
]
1317
}
1418
}
1519
}

packages/core/package.json

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@
2222
"mcp"
2323
],
2424
"scripts": {
25-
"fetch:spec-types": "tsx scripts/fetch-spec-types.ts",
2625
"typecheck": "tsc -p tsconfig.json --noEmit",
2726
"examples:simple-server:w": "tsx --watch src/examples/server/simpleStreamableHttp.ts --oauth",
2827
"lint": "eslint src/ && prettier --ignore-path ../../.prettierignore --check .",

packages/server/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
"dist"
3131
],
3232
"scripts": {
33-
"fetch:spec-types": "tsx scripts/fetch-spec-types.ts",
3433
"typecheck": "tsc -p tsconfig.json --noEmit",
3534
"build": "tsdown",
3635
"build:watch": "tsdown --watch",
@@ -73,6 +72,7 @@
7372
"@modelcontextprotocol/tsconfig": "workspace:^",
7473
"@modelcontextprotocol/vitest-config": "workspace:^",
7574
"@modelcontextprotocol/eslint-config": "workspace:^",
75+
"@modelcontextprotocol/test-helpers": "workspace:^",
7676
"@cfworker/json-schema": "catalog:runtimeShared",
7777
"@eslint/js": "catalog:devTools",
7878
"@types/content-type": "catalog:devTools",

packages/server/test/server/auth/middleware/bearerAuth.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
import type { AuthInfo } from '@modelcontextprotocol/sdk-core';
22
import { CustomOAuthError, InsufficientScopeError, InvalidTokenError, ServerError } from '@modelcontextprotocol/sdk-core';
3+
import { createExpressResponseMock } from '@modelcontextprotocol/test-helpers';
34
import type { Request, Response } from 'express';
45
import type { Mock } from 'vitest';
56

6-
import { createExpressResponseMock } from '../../../../../integration/test/helpers/http.js';
77
import { requireBearerAuth } from '../../../../src/server/auth/middleware/bearerAuth.js';
88
import type { OAuthTokenVerifier } from '../../../../src/server/auth/provider.js';
99

0 commit comments

Comments
 (0)