Merge pull request #439 from objectstack-ai/copilot/improve-test-coverage-i18n

Author: hotlong

packages/core/package.json

@@ -2,6 +2,7 @@
   "name": "@object-ui/core",
   "version": "0.5.0",
   "type": "module",
+  "sideEffects": false,
   "license": "MIT",
   "description": "Core logic, types, and validation for Object UI. Zero React dependencies.",
   "homepage": "https://www.objectui.org",

packages/i18n/package.json

@@ -2,6 +2,7 @@
   "name": "@object-ui/i18n",
   "version": "0.5.0",
   "type": "module",
+  "sideEffects": false,
   "license": "MIT",
   "description": "Internationalization (i18n) support for Object UI with 10+ language packs, RTL layout, and date/currency formatting.",
   "homepage": "https://www.objectui.org",

packages/i18n/src/__tests__/provider.test.tsx

@@ -0,0 +1,117 @@
+import { describe, it, expect, vi } from 'vitest';
+import { renderHook, waitFor, act } from '@testing-library/react';
+import React from 'react';
+import { I18nProvider, useObjectTranslation, useI18nContext } from '../provider';
+import { createI18n } from '../i18n';
+
+describe('I18nProvider', () => {
+  it('creates i18n instance from config', () => {
+    const wrapper = ({ children }: { children: React.ReactNode }) =>
+      React.createElement(I18nProvider, { config: { defaultLanguage: 'en', detectBrowserLanguage: false } }, children);
+
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper });
+
+    expect(result.current.i18n).toBeDefined();
+    expect(result.current.language).toBe('en');
+  });
+
+  it('accepts pre-created instance', () => {
+    const instance = createI18n({ defaultLanguage: 'fr', detectBrowserLanguage: false });
+
+    const wrapper = ({ children }: { children: React.ReactNode }) =>
+      React.createElement(I18nProvider, { instance }, children);
+
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper });
+
+    expect(result.current.i18n).toBeDefined();
+    expect(result.current.language).toBe('fr');
+  });
+});
+
+describe('useObjectTranslation', () => {
+  const wrapper = ({ children }: { children: React.ReactNode }) =>
+    React.createElement(I18nProvider, { config: { defaultLanguage: 'en', detectBrowserLanguage: false } }, children);
+
+  it('returns t, language, changeLanguage, direction, i18n', () => {
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper });
+
+    expect(result.current.t).toBeTypeOf('function');
+    expect(result.current.language).toBeTypeOf('string');
+    expect(result.current.changeLanguage).toBeTypeOf('function');
+    expect(result.current.direction).toBeTypeOf('string');
+    expect(result.current.i18n).toBeDefined();
+  });
+
+  it('translates keys correctly', () => {
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper });
+
+    expect(result.current.t('common.save')).toBe('Save');
+    expect(result.current.t('common.cancel')).toBe('Cancel');
+    expect(result.current.t('common.delete')).toBe('Delete');
+  });
+
+  it('returns en as default language', () => {
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper });
+
+    expect(result.current.language).toBe('en');
+  });
+
+  it('works with Chinese language', () => {
+    const zhWrapper = ({ children }: { children: React.ReactNode }) =>
+      React.createElement(I18nProvider, { config: { defaultLanguage: 'zh', detectBrowserLanguage: false } }, children);
+
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper: zhWrapper });
+
+    expect(result.current.language).toBe('zh');
+    expect(result.current.t('common.save')).toBe('保存');
+    expect(result.current.t('common.cancel')).toBe('取消');
+  });
+
+  it('changeLanguage updates language', async () => {
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper });
+
+    expect(result.current.language).toBe('en');
+
+    await act(async () => {
+      await result.current.changeLanguage('zh');
+    });
+
+    await waitFor(() => {
+      expect(result.current.language).toBe('zh');
+    });
+  });
+
+  it('returns RTL direction for Arabic', () => {
+    const arWrapper = ({ children }: { children: React.ReactNode }) =>
+      React.createElement(I18nProvider, { config: { defaultLanguage: 'ar', detectBrowserLanguage: false } }, children);
+
+    const { result } = renderHook(() => useObjectTranslation(), { wrapper: arWrapper });
+
+    expect(result.current.direction).toBe('rtl');
+  });
+});
+
+describe('useI18nContext', () => {
+  it('throws when used outside provider', () => {
+    // Suppress console.error from React for the expected error
+    const spy = vi.spyOn(console, 'error').mockImplementation(() => {});
+
+    expect(() => {
+      renderHook(() => useI18nContext());
+    }).toThrow('useI18nContext must be used within an I18nProvider');
+
+    spy.mockRestore();
+  });
+
+  it('returns context inside provider', () => {
+    const wrapper = ({ children }: { children: React.ReactNode }) =>
+      React.createElement(I18nProvider, { config: { defaultLanguage: 'en', detectBrowserLanguage: false } }, children);
+
+    const { result } = renderHook(() => useI18nContext(), { wrapper });
+
+    expect(result.current.language).toBe('en');
+    expect(result.current.changeLanguage).toBeTypeOf('function');
+    expect(result.current.direction).toBe('ltr');
+    expect(result.current.i18n).toBeDefined();
+  });
+});

packages/layout/package.json

@@ -2,6 +2,7 @@
   "name": "@object-ui/layout",
   "version": "0.5.0",
   "type": "module",
+  "sideEffects": false,
   "main": "dist/index.umd.cjs",
   "module": "dist/index.js",
   "types": "dist/index.d.ts",