@softeria/ms-365-mcp-server 0.3.5 → 0.4.1

package/test/mappings.test.js

@@ -1,29 +0,0 @@
-import { describe, expect, it } from 'vitest';
-import { validateEndpoints } from '../src/dynamic-tools.mjs';
-
-/**
- * This test file ensures that all the mappings in TARGET_ENDPOINTS actually match
- * the endpoints in the OpenAPI spec. It helps catch issues where:
- *
- * 1. An endpoint in TARGET_ENDPOINTS doesn't exist in the OpenAPI spec
- * 2. The method for an endpoint doesn't match what's in the OpenAPI spec
- *
- * This is a more automated approach than manually running the app and tailing logs.
- */
-
-describe('Mappings Validation', () => {
-  it('should verify all TARGET_ENDPOINTS exist in the OpenAPI spec', () => {
-    const missingEndpoints = validateEndpoints();
-
-    if (missingEndpoints.length > 0) {
-      console.error('The following endpoints are missing from the OpenAPI spec:');
-      missingEndpoints.forEach((endpoint) => {
-        console.error(
-          `- Tool: ${endpoint.toolName}, Path: ${endpoint.pathPattern}, Method: ${endpoint.method}`
-        );
-      });
-    }
-
-    expect(missingEndpoints).toEqual([]);
-  });
-});

package/test/mcp-server.test.js

@@ -1,36 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { z } from 'zod';
-import { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-
-vi.mock('@modelcontextprotocol/sdk/server/mcp.js', () => ({
-  McpServer: vi.fn(() => ({
-    tool: vi.fn(),
-    connect: vi.fn().mockResolvedValue(undefined),
-    name: 'TestServer',
-    version: '0.1.1',
-  })),
-}));
-vi.spyOn(process.stderr, 'write').mockImplementation(() => true);
-
-describe('MCP Server', () => {
-  let server;
-
-  beforeEach(() => {
-    vi.clearAllMocks();
-    server = new McpServer();
-  });
-
-  it('should be created with proper configuration', () => {
-    expect(McpServer).toHaveBeenCalled();
-    expect(server).toBeDefined();
-  });
-
-  it('should be able to register tools', () => {
-    server.tool('test-tool', { param: z.string() }, async () => {});
-    expect(server.tool).toHaveBeenCalledWith(
-      'test-tool',
-      { param: expect.any(Object) },
-      expect.any(Function)
-    );
-  });
-});

package/test/openapi-helpers.test.js

@@ -1,210 +0,0 @@
-import { describe, expect, it, vi, beforeEach } from 'vitest';
-import { z } from 'zod';
-
-vi.mock('fs', () => {
-  return {
-    default: { readFileSync: vi.fn().mockReturnValue('mock yaml content') },
-    readFileSync: vi.fn().mockReturnValue('mock yaml content'),
-  };
-});
-
-vi.mock('js-yaml', () => {
-  const mockSpec = {
-    paths: {
-      '/test/path': {
-        get: {
-          parameters: [
-            { name: 'filter', in: 'query', required: false, schema: { type: 'string' } },
-          ],
-        },
-      },
-      '/test/{param}/path': {
-        get: {
-          parameters: [
-            { name: 'filter', in: 'query', required: false, schema: { type: 'string' } },
-          ],
-        },
-      },
-      '/excel/test': {
-        post: {
-          requestBody: {
-            content: {
-              'application/json': {
-                schema: { type: 'object' },
-              },
-            },
-          },
-        },
-      },
-    },
-  };
-
-  return {
-    default: { load: vi.fn().mockReturnValue(mockSpec) },
-    load: vi.fn().mockReturnValue(mockSpec),
-  };
-});
-
-vi.mock('../src/logger.mjs', () => ({
-  default: {
-    info: vi.fn(),
-    warn: vi.fn(),
-    error: vi.fn(),
-  },
-}));
-
-// Mock param-mapper module
-vi.mock('../src/param-mapper.mjs', () => ({
-  createFriendlyParamName: (name) => (name.startsWith('$') ? name.substring(1) : name),
-  registerParamMapping: vi.fn(),
-  getOriginalParamName: vi.fn(),
-}));
-
-import * as fs from 'fs';
-import * as yaml from 'js-yaml';
-import { mapToZodType, buildRequestUrl } from '../src/openapi-helpers.mjs';
-
-function mockProcessParameter(parameter) {
-  const schema = parameter.schema?.type === 'string' ? z.string() : z.any();
-  return parameter.required ? schema : schema.optional();
-}
-
-describe('OpenAPI Helpers', () => {
-  describe('mapToZodType', () => {
-    it('should map string schema to z.string()', () => {
-      const schema = { type: 'string' };
-      const result = mapToZodType(schema);
-
-      expect(result).toBeInstanceOf(z.ZodString);
-    });
-
-    it('should map number schema to z.number()', () => {
-      const schema = { type: 'number' };
-      const result = mapToZodType(schema);
-
-      expect(result).toBeInstanceOf(z.ZodNumber);
-    });
-
-    it('should map integer schema to z.number()', () => {
-      const schema = { type: 'integer' };
-      const result = mapToZodType(schema);
-
-      expect(result).toBeInstanceOf(z.ZodNumber);
-    });
-
-    it('should map boolean schema to z.boolean()', () => {
-      const schema = { type: 'boolean' };
-      const result = mapToZodType(schema);
-
-      expect(result).toBeInstanceOf(z.ZodBoolean);
-    });
-
-    it('should map array schema to z.array()', () => {
-      const schema = { type: 'array', items: { type: 'string' } };
-      const result = mapToZodType(schema);
-
-      expect(result).toBeInstanceOf(z.ZodArray);
-    });
-
-    it('should map object schema to z.object()', () => {
-      const schema = {
-        type: 'object',
-        properties: {
-          name: { type: 'string' },
-          age: { type: 'integer' },
-        },
-      };
-      const result = mapToZodType(schema);
-
-      expect(result).toBeInstanceOf(z.ZodObject);
-    });
-
-    it('should handle $ref schema', () => {
-      const schema = { $ref: '#/components/schemas/StringType' };
-      const result = mapToZodType(schema);
-
-      expect(result).toBeInstanceOf(z.ZodString);
-    });
-  });
-
-  describe('buildRequestUrl', () => {
-    it('should build a URL without parameters', () => {
-      const baseUrl = '/test/path';
-      const params = {};
-      const pathParams = [];
-      const queryParamDefs = [];
-
-      const result = buildRequestUrl(baseUrl, params, pathParams, queryParamDefs);
-
-      expect(result).toBe('/test/path');
-    });
-
-    it('should replace path parameters', () => {
-      const baseUrl = '/test/{id}/path';
-      const params = { id: '123' };
-      const pathParams = ['{id}'];
-      const queryParamDefs = [];
-
-      const result = buildRequestUrl(baseUrl, params, pathParams, queryParamDefs);
-
-      expect(result).toBe('/test/123/path');
-    });
-
-    it('should add query parameters', () => {
-      const baseUrl = '/test/path';
-      const params = { filter: 'test' };
-      const pathParams = [];
-      const queryParamDefs = [{ name: 'filter', in: 'query' }];
-
-      const result = buildRequestUrl(baseUrl, params, pathParams, queryParamDefs);
-
-      expect(result).toBe('/test/path?filter=test');
-    });
-
-    it('should handle Excel range addresses', () => {
-      const baseUrl = "/workbook/worksheets/{id}/range(address='{address}')";
-      const params = { id: 'Sheet1', address: 'A1:C10' };
-      const pathParams = ['{id}'];
-      const queryParamDefs = [];
-
-      const result = buildRequestUrl(baseUrl, params, pathParams, queryParamDefs);
-
-      expect(result).toBe("/workbook/worksheets/Sheet1/range(address='A1%3AC10')");
-    });
-
-    it('should handle array parameters', () => {
-      const baseUrl = '/test/path';
-      const params = { select: ['name', 'email'] };
-      const pathParams = [];
-      const queryParamDefs = [{ name: '$select', in: 'query' }];
-      const toolName = 'test-tool';
-
-      const result = buildRequestUrl(baseUrl, params, pathParams, queryParamDefs);
-
-      expect(result).toBe('/test/path?$select=name,email');
-    });
-
-    it('should map friendly parameter names to original names with $ prefix', () => {
-      const baseUrl = '/test/path';
-      const params = {
-        select: 'name,email',
-        filter: "contains(displayName, 'test')",
-        orderby: 'displayName',
-      };
-      const pathParams = [];
-      const queryParamDefs = [
-        { name: '$select', in: 'query' },
-        { name: '$filter', in: 'query' },
-        { name: '$orderby', in: 'query' },
-      ];
-      const toolName = 'test-tool';
-
-      const result = buildRequestUrl(baseUrl, params, pathParams, queryParamDefs);
-
-      // URL should contain original parameter names with $ prefix
-      expect(result).toContain('$select=');
-      expect(result).toContain('$filter=');
-      expect(result).toContain('$orderby=');
-    });
-  });
-});

package/test/param-mapper.test.js

@@ -1,56 +0,0 @@
-import { expect, test, describe } from 'vitest';
-import { 
-  createFriendlyParamName,
-  registerParamMapping,
-  getOriginalParamName,
-  transformParamsToOriginal 
-} from '../src/param-mapper.mjs';
-
-describe('Parameter Mapper', () => {
-  test('createFriendlyParamName removes $ prefix', () => {
-    expect(createFriendlyParamName('$orderby')).toBe('orderby');
-    expect(createFriendlyParamName('$select')).toBe('select');
-    expect(createFriendlyParamName('$expand')).toBe('expand');
-  });
-
-  test('createFriendlyParamName leaves regular params unchanged', () => {
-    expect(createFriendlyParamName('subject')).toBe('subject');
-    expect(createFriendlyParamName('contentType')).toBe('contentType');
-  });
-
-  test('registerParamMapping and getOriginalParamName work together', () => {
-    // Register a mapping
-    registerParamMapping('list-mail-messages', 'orderby', '$orderby');
-    
-    // Retrieve the original name
-    expect(getOriginalParamName('list-mail-messages', 'orderby')).toBe('$orderby');
-    
-    // Unknown mappings return the friendly name
-    expect(getOriginalParamName('list-mail-messages', 'unknown')).toBe('unknown');
-    expect(getOriginalParamName('unknown-tool', 'orderby')).toBe('orderby');
-  });
-
-  test('transformParamsToOriginal converts all params', () => {
-    // Register mappings
-    registerParamMapping('list-mail-messages', 'orderby', '$orderby');
-    registerParamMapping('list-mail-messages', 'select', '$select');
-    registerParamMapping('list-mail-messages', 'top', '$top');
-    
-    // Transform params
-    const params = {
-      orderby: 'receivedDateTime desc',
-      select: 'subject,from,receivedDateTime',
-      top: 10,
-      normalParam: 'value'
-    };
-    
-    const transformed = transformParamsToOriginal('list-mail-messages', params);
-    
-    expect(transformed).toEqual({
-      '$orderby': 'receivedDateTime desc',
-      '$select': 'subject,from,receivedDateTime',
-      '$top': 10,
-      'normalParam': 'value'
-    });
-  });
-});