mirrors/mcphub
1
0
Fork 0
mirror of https://github.com/samanhappy/mcphub.git synced 2025-12-23 18:29:21 -05:00
Code Issues Packages Projects Releases Wiki Activity

Address code review comments: remove redundant cleanup and clarify test intent

Browse Source 
Co-authored-by: samanhappy <2755122+samanhappy@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2025-11-05 11:15:19 +00:00
parent 8da0323326
commit 017e405c41
1 changed files with 8 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
tests/integration/base-path-routes.test.ts
View File

@@ -77,9 +77,6 @@ describe('AppServer with BASE_PATH configuration', () => {
// Test that /mcphub/public-config endpoint exists
const publicConfigResponse = await request(app).get('/mcphub/public-config');
expect(publicConfigResponse.status).not.toBe(404);
// Clean up
delete process.env.BASE_PATH;
});
it('should serve auth routes without BASE_PATH (default)', async () => {
@@ -109,7 +106,7 @@ describe('AppServer with BASE_PATH configuration', () => {
expect(publicConfigResponse.status).not.toBe(404);
});
it('should correctly mount API routes with BASE_PATH', async () => {
it('should serve global endpoints without BASE_PATH prefix', async () => {
process.env.BASE_PATH = '/test-base/';
jest.resetModules();
@@ -121,11 +118,13 @@ describe('AppServer with BASE_PATH configuration', () => {
const app = appServer.getApp();
// Test that API routes are accessible with base path
const apiHealthResponse = await request(app).get('/health');
expect(apiHealthResponse.status).not.toBe(404);
// Test that /health endpoint is accessible globally (no BASE_PATH prefix)
// The /health endpoint is intentionally mounted without BASE_PATH
const healthResponse = await request(app).get('/health');
expect(healthResponse.status).not.toBe(404);
// Clean up
delete process.env.BASE_PATH;
// Also verify that BASE_PATH prefixed routes exist
const configResponse = await request(app).get('/test-base/config');
expect(configResponse.status).not.toBe(404);
});
});