Port tests

Author: chrisreddington

.vscode/mcp.json

@@ -10,6 +10,17 @@
       "args": [
         "@playwright/mcp@latest"
       ]
+    },
+    "context7": {
+      "type": "http",
+      "url": "https://mcp.context7.com/mcp"
+    },
+    "sequentialthinking": {
+      "command": "npx",
+      "args": [
+        "-y",
+        "@modelcontextprotocol/server-sequential-thinking"
+      ]
     }
-  }
+  }  }
 }

mcp-server/jest.config.cjs

@@ -10,19 +10,19 @@
     "start": "node dist/index.js",
     "clean": "rm -rf dist",
     "type-check": "tsc --noEmit",
-    "test": "jest",
-    "test:watch": "jest --watch",
-    "test:coverage": "jest --coverage"
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "test:coverage": "vitest run --coverage",
+    "test:ui": "vitest --ui"
   },
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.17.0",
     "@turn-based-mcp/shared": "file:../shared"
   },
   "devDependencies": {
-    "@types/jest": "^30.0.0",
     "@types/node": "^24.1.0",
-    "jest": "^30.0.5",
-    "ts-jest": "^29.4.0",
-    "typescript": "^5.5.0"
+    "@vitest/ui": "^3.2.4",
+    "typescript": "^5.5.0",
+    "vitest": "^3.2.4"
   }
 }

mcp-server/package.json

@@ -1,46 +1,55 @@
+import { vi } from 'vitest'
 import { handleToolCall } from '../handlers/tool-handlers.js'
 import { listResources, readResource } from '../handlers/resource-handlers.js'
 import { listPrompts, getPrompt } from '../handlers/prompt-handlers.js'
+import * as httpClient from '../utils/http-client.js'
+
+// Import the real constants from shared package
+import { GAME_TYPES, DIFFICULTIES, isSupportedGameType, DEFAULT_PLAYER_NAME, DEFAULT_AI_DIFFICULTY } from '@turn-based-mcp/shared'
 
 // Mock the web API calls for testing
-jest.mock('../utils/http-client.js', () => ({
-  httpGet: jest.fn(),
-  httpPost: jest.fn(),
-  getGameViaAPI: jest.fn(),
-  createGameViaAPI: jest.fn(),
-  submitMoveViaAPI: jest.fn(),
-  getGamesByType: jest.fn()
+vi.mock('../utils/http-client.js', () => ({
+  httpGet: vi.fn(),
+  httpPost: vi.fn(),
+  getGameViaAPI: vi.fn(),
+  createGameViaAPI: vi.fn(),
+  submitMoveViaAPI: vi.fn(),
+  getGamesByType: vi.fn()
 }))
 
-// Mock shared library
-jest.mock('@turn-based-mcp/shared', () => ({
-  TicTacToeGame: jest.fn(() => ({
-    getValidMoves: jest.fn(() => [{ row: 0, col: 0 }])
-  })),
-  RockPaperScissorsGame: jest.fn(() => ({}))
-}))
+// Mock only the game classes from shared library
+vi.mock('@turn-based-mcp/shared', async (importOriginal) => {
+  const actual = await importOriginal() as any
+  return {
+    ...actual,
+    TicTacToeGame: vi.fn(() => ({
+      getValidMoves: vi.fn(() => [{ row: 0, col: 0 }])
+    })),
+    RockPaperScissorsGame: vi.fn(() => ({}))
+  }
+})
 
 // Mock AI modules
-jest.mock('../ai/tic-tac-toe-ai.js', () => ({
-  TicTacToeAI: jest.fn(() => ({
-    makeMove: jest.fn(() => ({ row: 0, col: 0 }))
+vi.mock('../ai/tic-tac-toe-ai.js', () => ({
+  TicTacToeAI: vi.fn(() => ({
+    makeMove: vi.fn(() => ({ row: 0, col: 0 }))
   }))
 }))
 
-jest.mock('../ai/rock-paper-scissors-ai.js', () => ({
-  RockPaperScissorsAI: jest.fn(() => ({
-    makeChoice: jest.fn(() => 'rock')
+vi.mock('../ai/rock-paper-scissors-ai.js', () => ({
+  RockPaperScissorsAI: vi.fn(() => ({
+    makeChoice: vi.fn(() => 'rock')
   }))
 }))
 
 describe('MCP Server Integration', () => {
   beforeEach(() => {
-    jest.clearAllMocks()
+    vi.clearAllMocks()
   })
 
   describe('Resource Handlers', () => {
     it('should list resources correctly', async () => {
-      const mockGetGamesByType = require('../utils/http-client.js').getGamesByType
+      const mockGetGamesByType = vi.mocked(httpClient.getGamesByType)
       mockGetGamesByType.mockResolvedValue([
         {
           gameState: {
@@ -70,7 +79,7 @@ describe('MCP Server Integration', () => {
     })
 
     it('should read game type resource correctly', async () => {
-      const mockGetGamesByType = require('../utils/http-client.js').getGamesByType
+      const mockGetGamesByType = vi.mocked(httpClient.getGamesByType)
       mockGetGamesByType.mockResolvedValue([
         {
           gameState: {
@@ -95,7 +104,7 @@ describe('MCP Server Integration', () => {
     })
 
     it('should read individual game resource correctly', async () => {
-      const mockGetGameViaAPI = require('../utils/http-client.js').getGameViaAPI
+      const mockGetGameViaAPI = vi.mocked(httpClient.getGameViaAPI)
       mockGetGameViaAPI.mockResolvedValue({
         gameState: {
           id: 'test-game-1',
@@ -118,7 +127,7 @@ describe('MCP Server Integration', () => {
 
   describe('Tool Handlers', () => {
     it('should create tic-tac-toe game correctly', async () => {
-      const mockCreateGameViaAPI = require('../utils/http-client.js').createGameViaAPI
+      const mockCreateGameViaAPI = vi.mocked(httpClient.createGameViaAPI)
       mockCreateGameViaAPI.mockResolvedValue({
         gameState: {
           id: 'new-game-id',
@@ -136,8 +145,8 @@ describe('MCP Server Integration', () => {
     })
 
     it('should handle play moves correctly', async () => {
-      const mockGetGameViaAPI = require('../utils/http-client.js').getGameViaAPI
-      const mockSubmitMoveViaAPI = require('../utils/http-client.js').submitMoveViaAPI
+      const mockGetGameViaAPI = vi.mocked(httpClient.getGameViaAPI)
+      const mockSubmitMoveViaAPI = vi.mocked(httpClient.submitMoveViaAPI)
 
       mockGetGameViaAPI.mockResolvedValue({
         gameState: {

mcp-server/src/integration/integration.test.ts

@@ -1,14 +1,15 @@
+import { vi } from 'vitest'
 import { getRPSGameForMCP, makeRPSMove } from '@turn-based-mcp/shared'
 import type { GameSession } from '@turn-based-mcp/shared'
 import type { RPSGameState, RPSMove } from '@turn-based-mcp/shared'
 
 // Mock fetch for testing
-global.fetch = jest.fn()
-const mockFetch = global.fetch as jest.MockedFunction<typeof fetch>
+global.fetch = vi.fn()
+const mockFetch = global.fetch as vi.MockedFunction<typeof fetch>
 
 describe('MCP Server RPS Security Integration', () => {
   beforeEach(() => {
-    jest.clearAllMocks()
+    vi.clearAllMocks()
     process.env.WEB_API_BASE = 'http://localhost:3000'
   })
 

mcp-server/src/integration/mcp-rps-security.test.ts

@@ -0,0 +1,21 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({
+  test: {
+    environment: 'node',
+    globals: true,
+    setupFiles: ['./vitest.setup.ts'],
+    coverage: {
+      provider: 'v8',
+      reporter: ['text', 'lcov', 'html'],
+      thresholds: {
+        branches: 85,
+        functions: 85,
+        lines: 85,
+        statements: 85
+      },
+      include: ['src/**/*.{ts,tsx}'],
+      exclude: ['src/**/*.d.ts', 'src/**/index.ts']
+    }
+  },
+})

mcp-server/vitest.config.ts

@@ -1,10 +1,11 @@
 /**
- * Jest setup for mcp-server package tests
+ * Vitest setup for mcp-server package tests
  * 
  * Sets up test database before all tests and cleans up after
  */
 
-const { setupTestDatabase, teardownTestDatabase } = require('@turn-based-mcp/shared')
+import { setupTestDatabase, teardownTestDatabase } from '@turn-based-mcp/shared'
+import { beforeAll, afterAll } from 'vitest'
 
 // Setup test database before all tests
 beforeAll(async () => {