@qlover/create-app 0.7.15 → 0.8.0

package/dist/templates/react-app/__tests__/setup/index.ts

@@ -1,51 +1 @@
-import { createMockGlobals } from '../__mocks__/createMockGlobals';
-
-// 设置测试环境
-beforeEach(() => {
-  // 清理DOM
-  document.body.innerHTML = '';
-
-  // 创建root元素（如果不存在）
-  if (!document.getElementById('root')) {
-    const rootElement = document.createElement('div');
-    rootElement.id = 'root';
-    document.body.appendChild(rootElement);
-  }
-});
-
-afterEach(() => {
-  // 清理所有mock
-  vi.clearAllMocks();
-});
-
-// 全局测试配置
-global.ResizeObserver = vi.fn().mockImplementation(() => ({
-  observe: vi.fn(),
-  unobserve: vi.fn(),
-  disconnect: vi.fn()
-}));
-
-// Mock window.matchMedia
-Object.defineProperty(window, 'matchMedia', {
-  writable: true,
-  value: vi.fn().mockImplementation((query) => ({
-    matches: false,
-    media: query,
-    onchange: null,
-    addListener: vi.fn(), // deprecated
-    removeListener: vi.fn(), // deprecated
-    addEventListener: vi.fn(),
-    removeEventListener: vi.fn(),
-    dispatchEvent: vi.fn()
-  }))
-});
-
-// Mock IntersectionObserver
-global.IntersectionObserver = vi.fn().mockImplementation(() => ({
-  observe: vi.fn(),
-  unobserve: vi.fn(),
-  disconnect: vi.fn()
-}));
-
-// Mock globals
-vi.mock('@/core/globals', () => createMockGlobals());
+import './setupGlobal';

package/dist/templates/react-app/__tests__/setup/setupGlobal.ts

@@ -0,0 +1,51 @@
+import { createMockGlobals } from '__tests__/__mocks__/createMockGlobals';
+
+// 设置测试环境
+beforeEach(() => {
+  // 清理DOM
+  document.body.innerHTML = '';
+
+  // 创建root元素（如果不存在）
+  if (!document.getElementById('root')) {
+    const rootElement = document.createElement('div');
+    rootElement.id = 'root';
+    document.body.appendChild(rootElement);
+  }
+});
+
+afterEach(() => {
+  // 清理所有mock
+  vi.clearAllMocks();
+});
+
+// 全局测试配置
+global.ResizeObserver = vi.fn().mockImplementation(() => ({
+  observe: vi.fn(),
+  unobserve: vi.fn(),
+  disconnect: vi.fn()
+}));
+
+// Mock window.matchMedia
+Object.defineProperty(window, 'matchMedia', {
+  writable: true,
+  value: vi.fn().mockImplementation((query) => ({
+    matches: false,
+    media: query,
+    onchange: null,
+    addListener: vi.fn(), // deprecated
+    removeListener: vi.fn(), // deprecated
+    addEventListener: vi.fn(),
+    removeEventListener: vi.fn(),
+    dispatchEvent: vi.fn()
+  }))
+});
+
+// Mock IntersectionObserver
+global.IntersectionObserver = vi.fn().mockImplementation(() => ({
+  observe: vi.fn(),
+  unobserve: vi.fn(),
+  disconnect: vi.fn()
+}));
+
+// Mock globals
+vi.mock('@/core/globals', () => createMockGlobals());

package/dist/templates/react-app/__tests__/src/App.structure.test.tsx

@@ -0,0 +1,115 @@
+import { routerPrefix } from '@config/common';
+import { render, screen } from '@testing-library/react';
+import { testIOC } from '__tests__/__mocks__/testIOC/TestIOC';
+import { describe, it, expect, vi, beforeAll } from 'vitest';
+import App from '@/App';
+import { BootstrapClient } from '@/core/bootstraps/BootstrapClient';
+
+// Mock CSS imports
+vi.mock('@/styles/css/index.css', () => ({}));
+
+// Mock RouterProvider
+vi.mock('react-router-dom', () => ({
+  createBrowserRouter: vi.fn((routes, options) => ({
+    routes,
+    basename: options?.basename
+  })),
+  RouterProvider: ({ router }: any) => (
+    <div data-testid="router-provider">
+      <div data-testid="router-basename">{router?.basename}</div>
+      <div data-testid="router-routes-count">{router?.routes?.length || 0}</div>
+    </div>
+  ),
+  useNavigate: () => vi.fn()
+}));
+
+// Mock AntdThemeProvider
+vi.mock('@brain-toolkit/antd-theme-override/react', () => ({
+  AntdThemeProvider: ({ children, theme }: any) => (
+    <div data-testid="antd-theme-provider">
+      <div data-testid="theme-key">{theme?.cssVar?.key}</div>
+      <div data-testid="theme-prefix">{theme?.cssVar?.prefix}</div>
+      {children}
+    </div>
+  )
+}));
+
+// Mock react-kit
+vi.mock('@brain-toolkit/react-kit', () => ({
+  useMountedClient: () => true,
+  useStore: (_service: any, selector?: any) => {
+    if (selector) {
+      return [
+        { path: '/', component: 'HomePage' },
+        { path: '/about', component: 'AboutPage' }
+      ];
+    }
+    return [];
+  }
+}));
+
+// Mock ComboProvider - force children to render
+vi.mock('@/uikit/components/ComboProvider', () => ({
+  ComboProvider: ({ children }: any) => children
+}));
+
+// Mock AppRouterProvider - return our mocked router structure
+vi.mock('@/uikit/components/AppRouterProvider', () => ({
+  AppRouterProvider: () => (
+    <div data-testid="router-provider">
+      <div data-testid="router-basename">{routerPrefix}</div>
+      <div data-testid="router-routes-count">0</div>
+    </div>
+  )
+}));
+
+// Import App after all mocks
+import { ClientIOCRegister } from '@/core/clientIoc/ClientIOCRegister';
+import { appConfig } from '@/core/globals';
+
+describe('App Component with IOC Initialization', () => {
+  beforeAll(async () => {
+    /**
+     * Initialize IOC container with BootstrapClient.main()
+     * This is essential to register all services including DialogHandler
+     *
+     * Without this initialization, you'll get:
+     * "No bindings found for service: DialogHandler"
+     */
+    await BootstrapClient.main({
+      root: globalThis,
+      bootHref: 'http://localhost:3000',
+      ioc: testIOC,
+      iocRegister: new ClientIOCRegister({
+        pathname: '/',
+        appConfig: appConfig
+      })
+    });
+  });
+
+  it('should render without crashing', () => {
+    const { container } = render(<App />);
+    expect(container).toBeDefined();
+  });
+
+  it('should initialize and render the app structure', () => {
+    render(<App />);
+
+    // App should render successfully after IOC initialization
+    const routerProvider = screen.getByTestId('router-provider');
+    expect(routerProvider).toBeDefined();
+  });
+
+  it('should configure router with correct basename', () => {
+    render(<App />);
+
+    const routerBasename = screen.getByTestId('router-basename');
+    expect(routerBasename.textContent).toBe(routerPrefix);
+  });
+
+  it('should render without DialogHandler binding errors', () => {
+    // This test verifies that BootstrapClient.main() correctly initialized the IOC container
+    // If DialogHandler was not registered, this would throw an error
+    expect(() => render(<App />)).not.toThrow();
+  });
+});

package/dist/templates/react-app/__tests__/src/base/cases/AppConfig.test.ts

@@ -12,10 +12,10 @@
  * 8. Configuration validation
  */
 
+import { envBlackList, envPrefix } from '@config/common';
+import { InjectEnv } from '@qlover/corekit-bridge';
 import { describe, it, expect, beforeEach, afterEach } from 'vitest';
 import { AppConfig } from '@/base/cases/AppConfig';
-import { InjectEnv } from '@qlover/corekit-bridge';
-import { envBlackList, envPrefix } from '@config/common';
 
 // Mock import.meta.env
 const mockImportMetaEnv = {

package/dist/templates/react-app/__tests__/src/base/cases/AppError.test.ts

@@ -7,9 +7,9 @@
  * 3. error properties - Verify error properties are set correctly
  */
 
+import { ExecutorError } from '@qlover/fe-corekit';
 import { describe, it, expect } from 'vitest';
 import { AppError } from '@/base/cases/AppError';
-import { ExecutorError } from '@qlover/fe-corekit';
 
 describe('AppError', () => {
   describe('constructor', () => {

package/dist/templates/react-app/__tests__/src/base/cases/DialogHandler.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
 /**
  * DialogHandler test suite
  *
@@ -12,13 +11,12 @@
  */
 
 import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import { DialogHandler } from '../../../../src/base/cases/DialogHandler';
+import { DialogHandler } from '@/base/cases/DialogHandler';
 import type {
   MessageApi,
   ModalApi,
   NotificationApi
 } from '@brain-toolkit/antd-theme-override/react';
-import type { ConfirmOptions } from '../../../../src/base/port/InteractionHubInterface';
 
 describe('DialogHandler', () => {
   let dialogHandler: DialogHandler;
@@ -192,7 +190,7 @@ describe('DialogHandler', () => {
     });
 
     it('should call modal.confirm with options', () => {
-      const options: ConfirmOptions = {
+      const options = {
         title: 'Confirm',
         content: 'Are you sure?',
         onOk: vi.fn(),
@@ -212,7 +210,7 @@ describe('DialogHandler', () => {
     });
 
     it('should handle undefined modal api', () => {
-      const options: ConfirmOptions = {
+      const options = {
         title: 'test',
         content: 'test content'
       };

package/dist/templates/react-app/__tests__/src/base/cases/I18nKeyErrorPlugin.test.ts

@@ -8,12 +8,23 @@
  * 4. edge cases       - Various error scenarios
  */
 
+import { MockLogger } from '@__mocks__/MockLogger';
 import { describe, it, expect, beforeEach } from 'vitest';
 import { I18nKeyErrorPlugin } from '@/base/cases/I18nKeyErrorPlugin';
-import { MockLogger } from '../../../__mocks__/MockLogger';
-import { MockI18nService } from '../../../__mocks__/I18nService';
+import { I18nService } from '@/base/services/I18nService';
 import type { ExecutorContext } from '@qlover/fe-corekit';
 
+class MockI18nService extends I18nService {
+  constructor() {
+    super('/');
+  }
+
+  t = vi.fn((key: string) => key);
+  changeLanguage = vi.fn();
+  changeLoading = vi.fn();
+  onBefore = vi.fn();
+}
+
 describe('I18nKeyErrorPlugin', () => {
   let plugin: I18nKeyErrorPlugin;
   let mockLogger: MockLogger;

package/dist/templates/react-app/__tests__/src/base/cases/InversifyContainer.test.ts

@@ -10,8 +10,8 @@
  * 6. error handling  - Error cases and boundary tests
  */
 
-import { InversifyContainer } from '@/base/cases/InversifyContainer';
 import { injectable } from 'inversify';
+import { InversifyContainer } from '@/base/cases/InversifyContainer';
 
 describe('InversifyContainer', () => {
   let container: InversifyContainer;

package/dist/templates/react-app/__tests__/src/base/cases/PublicAssetsPath.test.ts

@@ -6,9 +6,9 @@
  * 2. getPath         - Simple path concatenation with prefix
  */
 
+import { routerPrefix } from '@config/common';
 import { describe, it, expect } from 'vitest';
 import { PublicAssetsPath } from '@/base/cases/PublicAssetsPath';
-import { routerPrefix } from '@config/common';
 
 describe('PublicAssetsPath', () => {
   describe('constructor', () => {

package/dist/templates/react-app/__tests__/src/base/cases/RequestLogger.test.ts

@@ -9,18 +9,18 @@
  * 5. loggerError     - Error formatting and logging
  */
 
+import { MockLogger } from '@__mocks__/MockLogger';
 import { describe, it, expect, beforeEach, vi } from 'vitest';
 import { RequestLogger } from '@/base/cases/RequestLogger';
-import { MockLogger } from '../../../__mocks__/MockLogger';
+import type {
+  ApiCatchPluginConfig,
+  ApiCatchPluginResponse
+} from '@qlover/corekit-bridge';
 import type {
   ExecutorContext,
   RequestAdapterFetchConfig,
   RequestAdapterResponse
 } from '@qlover/fe-corekit';
-import type {
-  ApiCatchPluginConfig,
-  ApiCatchPluginResponse
-} from '@qlover/corekit-bridge';
 
 describe('RequestLogger', () => {
   let logger: MockLogger;

package/dist/templates/react-app/__tests__/src/base/cases/RequestStatusCatcher.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
 /**
  * RequestStatusCatcher test suite
  *
@@ -9,9 +8,9 @@
  * 4. case200         - Success case handling
  */
 
+import { MockLogger } from '@__mocks__/MockLogger';
 import { describe, it, expect, beforeEach } from 'vitest';
 import { RequestStatusCatcher } from '@/base/cases/RequestStatusCatcher';
-import { MockLogger } from '../../../__mocks__/MockLogger';
 import type { RequestAdapterResponse } from '@qlover/fe-corekit';
 
 describe('RequestStatusCatcher', () => {

package/dist/templates/react-app/__tests__/src/base/cases/RouterLoader.test.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
 /**
  * RouterLoader test suite
  *
@@ -10,13 +9,13 @@
  * 5. error handling  - Invalid configurations
  */
 
+import { createElement } from 'react';
 import { describe, it, expect, beforeEach, vi } from 'vitest';
 import { RouterLoader } from '@/base/cases/RouterLoader';
 import type {
   RouteConfigValue,
   RouterLoaderOptions
 } from '@/base/cases/RouterLoader';
-import { createElement } from 'react';
 
 describe('RouterLoader', () => {
   // Mock components and render function
@@ -29,9 +28,19 @@ describe('RouterLoader', () => {
     createElement('div', null, route.element())
   );
 
+  let mockLogger: any;
   let defaultOptions: RouterLoaderOptions;
 
   beforeEach(() => {
+    vi.clearAllMocks();
+
+    mockLogger = {
+      warn: vi.fn(),
+      error: vi.fn(),
+      info: vi.fn(),
+      debug: vi.fn()
+    };
+
     defaultOptions = {
       routes: [
         {
@@ -50,10 +59,9 @@ describe('RouterLoader', () => {
         About: mockAboutComponent,
         '404': mockNotFoundComponent
       },
-      render: mockRender
+      render: mockRender,
+      logger: mockLogger
     };
-
-    vi.clearAllMocks();
   });
 
   describe('constructor', () => {
@@ -68,6 +76,13 @@ describe('RouterLoader', () => {
         'RouterLoader render is required'
       );
     });
+
+    it('should accept logger in options', () => {
+      const loader = new RouterLoader(defaultOptions);
+      expect(loader).toBeInstanceOf(RouterLoader);
+      // Logger should be accessible through protected options
+      expect((loader as any).options.logger).toBe(mockLogger);
+    });
   });
 
   describe('getComponentMaps', () => {
@@ -110,15 +125,14 @@ describe('RouterLoader', () => {
 
   describe('toRoute', () => {
     let loader: RouterLoader;
-    let consoleWarnSpy: any;
 
     beforeEach(() => {
       loader = new RouterLoader(defaultOptions);
-      consoleWarnSpy = vi.spyOn(console, 'warn').mockImplementation(() => {});
     });
 
-    afterEach(() => {
-      consoleWarnSpy.mockRestore();
+    it('should have logger configured', () => {
+      expect(mockLogger).toBeDefined();
+      expect(mockLogger.warn).toBeDefined();
     });
 
     it('should transform route configuration correctly', () => {
@@ -161,7 +175,7 @@ describe('RouterLoader', () => {
 
       const result = loader.toRoute(route);
 
-      expect(consoleWarnSpy).toHaveBeenCalledWith(
+      expect(mockLogger.warn).toHaveBeenCalledWith(
         'Invalid route, path is: /invalid, element is: undefined'
       );
       expect(result.element).toBeTruthy();
@@ -221,15 +235,11 @@ describe('RouterLoader', () => {
         path: '/invalid'
       } as RouteConfigValue;
 
-      const consoleWarnSpy = vi
-        .spyOn(console, 'warn')
-        .mockImplementation(() => {});
       loader.toRoute(invalidRoute);
 
-      expect(consoleWarnSpy).toHaveBeenCalledWith(
+      expect(mockLogger.warn).toHaveBeenCalledWith(
         'Invalid route, path is: /invalid, element is: undefined'
       );
-      consoleWarnSpy.mockRestore();
     });
   });
 });

package/dist/templates/react-app/__tests__/src/base/services/I18nService.test.ts

@@ -1,19 +1,20 @@
+/* eslint-disable import/no-named-as-default-member */
 /**
  * I18nService test suite
  *
  * Coverage:
- * 1. constructor      - State initialization
- * 2. onBefore        - Plugin initialization
- * 3. changeLanguage  - Language switching
- * 4. changeLoading   - Loading state management
- * 5. static methods  - Language validation and retrieval
- * 6. translation     - Key translation with parameters
+ * 1. constructor       - State initialization
+ * 2. onBefore         - Plugin initialization
+ * 3. changeLanguage   - Language switching
+ * 4. changeLoading    - Loading state management
+ * 5. language methods - Language validation and retrieval
+ * 6. translation      - Key translation with parameters
  */
 
+import { i18nConfig } from '@config/i18n/i18nConfig';
+import i18n from 'i18next';
 import { describe, it, expect, beforeEach, vi, afterEach } from 'vitest';
 import { I18nService, I18nServiceState } from '@/base/services/I18nService';
-import i18n from 'i18next';
-import i18nConfig from '@config/i18n';
 
 const { supportedLngs, fallbackLng } = i18nConfig;
 
@@ -29,7 +30,8 @@ vi.mock('i18next', () => ({
       languageDetector: {
         addDetector: vi.fn()
       }
-    }
+    },
+    on: vi.fn()
   }
 }));
 
@@ -100,8 +102,7 @@ describe('I18nService', () => {
       expect(i18n.services.languageDetector.addDetector).toHaveBeenCalledWith(
         expect.objectContaining({
           name: 'pathLanguageDetector',
-          lookup: expect.any(Function),
-          cacheUserLanguage: expect.any(Function)
+          lookup: expect.any(Function)
         })
       );
     });
@@ -160,22 +161,35 @@ describe('I18nService', () => {
     });
   });
 
-  describe('static methods', () => {
+  describe('language methods', () => {
     describe('getCurrentLanguage', () => {
       it('should return current i18n language', () => {
-        expect(I18nService.getCurrentLanguage()).toBe('en');
+        expect(service.getCurrentLanguage()).toBe('en');
       });
     });
 
     describe('isValidLanguage', () => {
       it('should validate supported languages', () => {
         for (const lang of supportedLngs) {
-          expect(I18nService.isValidLanguage(lang)).toBe(true);
+          expect(service.isValidLanguage(lang)).toBe(true);
         }
       });
 
       it('should reject unsupported languages', () => {
-        expect(I18nService.isValidLanguage('invalid')).toBe(false);
+        expect(service.isValidLanguage('invalid')).toBe(false);
+      });
+    });
+
+    describe('getSupportedLanguages', () => {
+      it('should return all supported languages', () => {
+        const languages = service.getSupportedLanguages();
+        expect(languages).toEqual(supportedLngs);
+      });
+
+      it('should return a copy of supported languages array', () => {
+        const languages1 = service.getSupportedLanguages();
+        const languages2 = service.getSupportedLanguages();
+        expect(languages1).not.toBe(languages2);
       });
     });
   });

package/dist/templates/react-app/__tests__/src/core/IOC.test.ts

@@ -1,11 +1,11 @@
-import { IOC, type IOCContainer } from '@/core/IOC';
-import { InversifyContainer } from '@/base/cases/InversifyContainer';
-import { IocRegisterImpl } from '@/core/IocRegisterImpl';
 import { IOCIdentifier } from '@config/IOCIdentifier';
 import type { AppConfig } from '@/base/cases/AppConfig';
+import { InversifyContainer } from '@/base/cases/InversifyContainer';
+import { ClientIOCRegister } from '@/core/clientIoc/ClientIOCRegister';
+import { IOC } from '@/core/IOC';
 
 describe('IOC Container Tests', () => {
-  let container: IOCContainer;
+  let container: InversifyContainer;
   let mockAppConfig: AppConfig;
 
   beforeEach(() => {
@@ -73,10 +73,20 @@ describe('IOC Container Tests', () => {
       // This test verifies that IOC function can be called with string identifiers
       // The function should accept the identifier, but will throw if service is not bound
       expect(() => {
-        IOC(IOCIdentifier.Logger);
+        IOC('NonExistentService' as any);
       }).toThrow();
     });
 
+    it('should get registered services successfully', () => {
+      // Logger is registered during IOC initialization
+      expect(() => {
+        IOC(IOCIdentifier.Logger);
+      }).not.toThrow();
+
+      const logger = IOC(IOCIdentifier.Logger);
+      expect(logger).toBeDefined();
+    });
+
     it('should accept class constructors', () => {
       // Mock class for testing
       class TestService {
@@ -90,17 +100,17 @@ describe('IOC Container Tests', () => {
   });
 
   describe('IocRegisterImpl', () => {
-    let registerImpl: IocRegisterImpl;
+    let registerImpl: ClientIOCRegister;
 
     beforeEach(() => {
-      registerImpl = new IocRegisterImpl({
+      registerImpl = new ClientIOCRegister({
         pathname: '/test',
         appConfig: mockAppConfig
       });
     });
 
     it('should create register implementation correctly', () => {
-      expect(registerImpl).toBeInstanceOf(IocRegisterImpl);
+      expect(registerImpl).toBeInstanceOf(ClientIOCRegister);
     });
 
     it('should register services without throwing', () => {
@@ -132,7 +142,7 @@ describe('IOC Container Tests', () => {
 
   describe('Service Registration Integration', () => {
     it('should register global services correctly', async () => {
-      const registerImpl = new IocRegisterImpl({
+      const registerImpl = new ClientIOCRegister({
         pathname: '/test',
         appConfig: mockAppConfig
       });