2026-02-21 04:30:36 +00:00
|
|
|
import { describe, it, expect, vi, beforeEach } from 'vitest';
|
|
|
|
|
import { ProjectService } from '../src/services/project.service.js';
|
|
|
|
|
import { NotFoundError, ConflictError } from '../src/services/mcp-server.service.js';
|
|
|
|
|
import type { IProjectRepository } from '../src/repositories/project.repository.js';
|
feat: replace profiles with kubernetes-style secrets
Replace the confused Profile abstraction with a dedicated Secret resource
following Kubernetes conventions. Servers now have env entries with inline
values or secretRef references. Env vars are resolved and passed to
containers at startup (fixes existing gap).
- Add Secret CRUD (model, repo, service, routes, CLI commands)
- Server env: {name, value} or {name, valueFrom: {secretRef: {name, key}}}
- Add env-resolver utility shared by instance startup and config generation
- Remove all profile-related code (models, services, routes, CLI, tests)
- Update backup/restore for secrets instead of profiles
- describe secret masks values by default, --show-values to reveal
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-22 18:40:58 +00:00
|
|
|
import type { IMcpServerRepository } from '../src/repositories/interfaces.js';
|
2026-02-21 04:30:36 +00:00
|
|
|
|
|
|
|
|
function mockProjectRepo(): IProjectRepository {
|
|
|
|
|
return {
|
|
|
|
|
findAll: vi.fn(async () => []),
|
|
|
|
|
findById: vi.fn(async () => null),
|
|
|
|
|
findByName: vi.fn(async () => null),
|
|
|
|
|
create: vi.fn(async (data) => ({
|
|
|
|
|
id: 'proj-1',
|
|
|
|
|
name: data.name,
|
|
|
|
|
description: data.description ?? '',
|
|
|
|
|
ownerId: data.ownerId,
|
|
|
|
|
version: 1,
|
|
|
|
|
createdAt: new Date(),
|
|
|
|
|
updatedAt: new Date(),
|
|
|
|
|
})),
|
|
|
|
|
update: vi.fn(async (id) => ({
|
|
|
|
|
id, name: 'test', description: '', ownerId: 'u1', version: 2,
|
|
|
|
|
createdAt: new Date(), updatedAt: new Date(),
|
|
|
|
|
})),
|
|
|
|
|
delete: vi.fn(async () => {}),
|
|
|
|
|
};
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
function mockServerRepo(): IMcpServerRepository {
|
|
|
|
|
return {
|
|
|
|
|
findAll: vi.fn(async () => []),
|
|
|
|
|
findById: vi.fn(async () => null),
|
|
|
|
|
findByName: vi.fn(async () => null),
|
|
|
|
|
create: vi.fn(async () => ({} as never)),
|
|
|
|
|
update: vi.fn(async () => ({} as never)),
|
|
|
|
|
delete: vi.fn(async () => {}),
|
|
|
|
|
};
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
describe('ProjectService', () => {
|
|
|
|
|
let projectRepo: ReturnType<typeof mockProjectRepo>;
|
|
|
|
|
let serverRepo: ReturnType<typeof mockServerRepo>;
|
|
|
|
|
let service: ProjectService;
|
|
|
|
|
|
|
|
|
|
beforeEach(() => {
|
|
|
|
|
projectRepo = mockProjectRepo();
|
|
|
|
|
serverRepo = mockServerRepo();
|
feat: replace profiles with kubernetes-style secrets
Replace the confused Profile abstraction with a dedicated Secret resource
following Kubernetes conventions. Servers now have env entries with inline
values or secretRef references. Env vars are resolved and passed to
containers at startup (fixes existing gap).
- Add Secret CRUD (model, repo, service, routes, CLI commands)
- Server env: {name, value} or {name, valueFrom: {secretRef: {name, key}}}
- Add env-resolver utility shared by instance startup and config generation
- Remove all profile-related code (models, services, routes, CLI, tests)
- Update backup/restore for secrets instead of profiles
- describe secret masks values by default, --show-values to reveal
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-22 18:40:58 +00:00
|
|
|
service = new ProjectService(projectRepo, serverRepo);
|
2026-02-21 04:30:36 +00:00
|
|
|
});
|
|
|
|
|
|
|
|
|
|
describe('create', () => {
|
|
|
|
|
it('creates a project', async () => {
|
|
|
|
|
const result = await service.create({ name: 'my-project' }, 'user-1');
|
|
|
|
|
expect(result.name).toBe('my-project');
|
|
|
|
|
expect(result.ownerId).toBe('user-1');
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('throws ConflictError when name exists', async () => {
|
|
|
|
|
vi.mocked(projectRepo.findByName).mockResolvedValue({ id: '1' } as never);
|
|
|
|
|
await expect(service.create({ name: 'taken' }, 'u1')).rejects.toThrow(ConflictError);
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
it('validates input', async () => {
|
|
|
|
|
await expect(service.create({ name: '' }, 'u1')).rejects.toThrow();
|
|
|
|
|
});
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
describe('getById', () => {
|
|
|
|
|
it('throws NotFoundError when not found', async () => {
|
|
|
|
|
await expect(service.getById('missing')).rejects.toThrow(NotFoundError);
|
|
|
|
|
});
|
|
|
|
|
});
|
|
|
|
|
|
|
|
|
|
describe('delete', () => {
|
|
|
|
|
it('deletes project', async () => {
|
|
|
|
|
vi.mocked(projectRepo.findById).mockResolvedValue({ id: 'p1' } as never);
|
|
|
|
|
await service.delete('p1');
|
|
|
|
|
expect(projectRepo.delete).toHaveBeenCalledWith('p1');
|
|
|
|
|
});
|
|
|
|
|
});
|
|
|
|
|
});
|