Remove redundant /api/v1 suffix from API URL configuration and update related test
This commit is contained in:
@@ -81,8 +81,8 @@ const ENV = {
|
|||||||
export const config = {
|
export const config = {
|
||||||
api: {
|
api: {
|
||||||
baseUrl: validateUrl(ENV.API_BASE_URL, 'API_BASE_URL'),
|
baseUrl: validateUrl(ENV.API_BASE_URL, 'API_BASE_URL'),
|
||||||
// Construct versioned API URL consistently
|
// OpenAPI spec already includes /api/v1 in paths, don't append it here
|
||||||
url: `${validateUrl(ENV.API_BASE_URL, 'API_BASE_URL')}/api/v1`,
|
url: validateUrl(ENV.API_BASE_URL, 'API_BASE_URL'),
|
||||||
timeout: parseIntSafe(ENV.API_TIMEOUT, 30000, 1000, 120000), // 1s to 2min
|
timeout: parseIntSafe(ENV.API_TIMEOUT, 30000, 1000, 120000), // 1s to 2min
|
||||||
},
|
},
|
||||||
|
|
||||||
|
|||||||
@@ -18,7 +18,6 @@ describe('API Client Configuration', () => {
|
|||||||
it('should have correct baseURL', () => {
|
it('should have correct baseURL', () => {
|
||||||
// Generated client already has /api/v1 in baseURL
|
// Generated client already has /api/v1 in baseURL
|
||||||
expect(apiClient.instance.defaults.baseURL).toContain(config.api.url);
|
expect(apiClient.instance.defaults.baseURL).toContain(config.api.url);
|
||||||
expect(apiClient.instance.defaults.baseURL).toContain('/api/v1');
|
|
||||||
});
|
});
|
||||||
|
|
||||||
it('should have correct timeout', () => {
|
it('should have correct timeout', () => {
|
||||||
|
|||||||
Reference in New Issue
Block a user