feat: Update tests for MixDatabaseRestPortalService and related services, enhance error handling and response structure

Author: Smilefounder

packages/database/tests/mix-database-rest-portal-service.test.ts

@@ -14,8 +14,12 @@ describe('MixDatabaseRestPortalService', () => {
     expect(dbService).toBeInstanceOf(MixDatabaseRestPortalService);
   });
 
-  it('should throw if no id provided', async () => {
-    await expect(dbService.migrate({} as any)).rejects.toThrow('Missing id for migration');
+  it('should return error if no id provided', async () => {
+    const result = await dbService.migrate({} as any);
+    expect(result).toEqual({
+      errors: ['Missing id for migration'],
+      isSucceed: false
+    });
   });
 
   it('should call migrate', async () => {

packages/database/tests/related-attribute-data-rest-portal-service.test.ts

@@ -1,4 +1,5 @@
-import { RelatedAttributeDataRestPortalService } from './related-attribute-data-rest-portal-service';
+import { RelatedAttributeDataRestPortalService } from '../src/related-attribute-data-rest-portal-service';
+import { ApiService } from '@mixcore/api';
 
 describe('RelatedAttributeDataRestPortalService', () => {
   it('should instantiate with correct endpoint', () => {

packages/database/tests/related-attribute-set-rest-portal-service.test.ts

@@ -1,8 +1,10 @@
-import { RelatedAttributeSetRestPortalService } from './related-attribute-set-rest-portal-service';
+import { RelatedAttributeSetRestPortalService } from '../src/related-attribute-set-rest-portal-service';
+import { ApiService } from '@mixcore/api';
 
 describe('RelatedAttributeSetRestPortalService', () => {
   it('should instantiate with correct endpoint', () => {
-    const service = new RelatedAttributeSetRestPortalService();
+    const api = new ApiService({ apiBaseUrl: 'http://localhost/api/' });
+    const service = new RelatedAttributeSetRestPortalService(api);
     // @ts-expect-error endpoint is protected/private in base, this is a placeholder for real test
     expect(service.endpoint).toBe('related-mix-database/portal');
   });

packages/template/src/module-article-services.ts

@@ -26,7 +26,7 @@ export class ModuleArticleService {
    * @param params - Optional query params
    */
   async fetchArticles(moduleId: string, params?: Record<string, any>): Promise<any[]> {
-    const url = new URL('/api/v2/rest/mixcore/module-article/get-module-article', this.config.apiBaseUrl);
+    const url = new URL('/api/v2/rest/mix-portal/module-article', this.config.apiBaseUrl);
     url.searchParams.append('moduleId', String(moduleId));
     if (params) {
       Object.entries(params).forEach(([k, v]) => url.searchParams.append(k, String(v)));

packages/template/src/module-gallery-services.ts

@@ -26,7 +26,7 @@ export class ModuleGalleryService {
    * @param params - Optional query params
    */
   async fetchGalleryItems(moduleId: string, params?: Record<string, any>): Promise<any[]> {
-    const url = new URL('/api/v2/rest/mixcore/module-gallery/get-module-gallery', this.config.apiBaseUrl);
+    const url = new URL('/api/v2/rest/mix-portal/module-gallery', this.config.apiBaseUrl);
     url.searchParams.append('moduleId', String(moduleId));
     if (params) {
       Object.entries(params).forEach(([k, v]) => url.searchParams.append(k, String(v)));

packages/template/tests/module-article-services.test.ts

@@ -1,4 +1,8 @@
-import { ModuleArticleService } from './module-article-services';
+import { ModuleArticleService } from '../src/module-article-services';
+
+// Mock fetch implementation
+const mockFetch = jest.fn();
+global.fetch = mockFetch;
 
 const config = {
   apiBaseUrl: 'https://mixcore.net',
@@ -9,8 +13,16 @@ describe('ModuleArticleService', () => {
   const service = new ModuleArticleService(config);
 
   it('fetchArticles should return an array', async () => {
+    mockFetch.mockResolvedValueOnce({
+      ok: true,
+      json: async () => ([{ id: 1, title: 'test-article' }])
+    });
     const articles = await service.fetchArticles('1');
-    expect(Array.isArray(articles)).toBe(true);
+    expect(articles).toEqual([{ id: 1, title: 'test-article' }]);
+    expect(mockFetch).toHaveBeenCalledWith(
+      'https://mixcore.net/api/v2/rest/mix-portal/module-article?moduleId=1',
+      { headers: undefined }
+    );
   });
 
   // Note: createArticle and deleteArticle require valid credentials and real data

packages/template/tests/module-gallery-services.test.ts

@@ -1,4 +1,8 @@
-import { ModuleGalleryService } from './module-gallery-services';
+import { ModuleGalleryService } from '../src/module-gallery-services';
+
+// Mock fetch implementation
+const mockFetch = jest.fn();
+global.fetch = mockFetch;
 
 const config = {
   apiBaseUrl: 'https://mixcore.net',
@@ -9,8 +13,16 @@ describe('ModuleGalleryService', () => {
   const service = new ModuleGalleryService(config);
 
   it('fetchGalleryItems should return an array', async () => {
+    mockFetch.mockResolvedValueOnce({
+      ok: true,
+      json: async () => ([{ id: 1, name: 'test-item' }])
+    });
     const items = await service.fetchGalleryItems('1');
-    expect(Array.isArray(items)).toBe(true);
+    expect(items).toEqual([{ id: 1, name: 'test-item' }]);
+    expect(mockFetch).toHaveBeenCalledWith(
+      'https://mixcore.net/api/v2/rest/mix-portal/module-gallery?moduleId=1',
+      { headers: undefined }
+    );
   });
 
   // Note: uploadGalleryItem and deleteGalleryItem require valid credentials and real data

packages/template/tests/template-services.test.ts

@@ -17,6 +17,10 @@ describe('TemplateService', () => {
   it('should call copy', async () => {
     globalThis.fetch = jest.fn().mockResolvedValue({ ok: true, json: async () => ({ id: 'copy' }) });
     const result = await templateService.copy('123');
-    expect(result).toEqual({ id: 'copy' });
+    expect(result).toEqual({
+      data: { id: 'copy' },
+      isSucceed: true,
+      status: undefined
+    });
   });
 });