fix: add unique names and fix isPublic check in integration tests

- MCP HTTP test: use unique collection name
- Public collections test: use unique collection name
- Remove isPublic assertion since public API doesn't return that field
This commit is contained in:
Ajax Davis 2026-01-14 02:01:46 +10:00
parent 56c4df74b7
commit 0fc39da6c9
2 changed files with 5 additions and 10 deletions

View file

@ -24,9 +24,9 @@ describe('MCP HTTP Endpoint', () => {
beforeAll(async () => {
ctx = getTestContext();
// Create a test collection for MCP tests
// Create a test collection for MCP tests with unique name
testCollection = await ctx.factories.collection.create({
name: 'MCP Test Collection',
name: `MCP Test Collection ${Date.now()}`,
description: 'Collection for testing MCP endpoints',
isPublic: true,
});

View file

@ -12,7 +12,6 @@ interface PublicCollection {
slug: string;
name: string;
description: string | null;
isPublic: boolean;
likeCount: number;
toolCount: number;
createdBy: {
@ -28,9 +27,9 @@ describe('Public Collections Endpoints', () => {
beforeAll(async () => {
ctx = getTestContext();
// Create a public test collection
// Create a public test collection with unique name
await ctx.factories.collection.create({
name: 'Public Test Collection',
name: `Public Test Collection ${Date.now()}`,
description: 'A public collection for testing',
isPublic: true,
});
@ -57,11 +56,7 @@ describe('Public Collections Endpoints', () => {
expect(result.data.success).toBe(true);
expect(Array.isArray(result.data.data)).toBe(true);
expect(result.data.pagination).toBeDefined();
// All returned collections should be public
for (const collection of result.data.data) {
expect(collection.isPublic).toBe(true);
}
// Note: isPublic is not in the response - by definition, only public collections are returned
}
});