@nocobase/app-template-default 1.0.0-beta.88 → 1.0.0-beta.90

package/tests/client/foundation-integration.test.ts

@@ -1,12 +1,17 @@
 import { describe, expect, it } from 'vitest';
+import {
+  createCapabilityCatalog,
+  type DependencyManifest,
+} from '@nocobase/app-plugin-foundation/capability-guide';
+import packageMetadata from '../../package.json' with { type: 'json' };
 import { resolveAppRuntime } from '@nocobase/app-client/runtime';
 import { defineAppRuntime } from '@nocobase/app-client/runtime';
 import appRuntime from '../../client/runtime.js';
 import { createApplicationRoutes } from '../../client/routes.js';
 import cliPlugins from '../../cli/plugins.js';
-import serverPlugins from '../../server/plugins.js';
 import settingsRoutes from '../../client/pages/settings/routes.js';
 import {
+  routeKey,
   buildRouteNavigation,
   buildSettingsCatalog,
 } from '../../client/routing/route-navigation.js';
@@ -19,38 +24,21 @@ describe('default Foundation integration', () => {
       ),
     ).toHaveLength(1);
   });
-  it('exposes Foundation in System settings for an unrestricted administrator and removes the entry when denied', async () => {
+  it('provides Features and extensions in System settings with its existing route', async () => {
     const runtime = await resolveAppRuntime(appRuntime);
-    const menu = buildRouteNavigation(runtime.routes, new Set());
-    // One System settings entry; Foundation is a card on its home, not a sidebar entry of its own.
-    expect(menu.map(({ route }) => route.name)).toEqual([
-      'home',
-      'settingsHome',
-    ]);
     const category = buildSettingsCatalog(runtime.routes, new Set()).find(
       ({ route }) => route.name === 'settingsSystem',
     );
-    const page = category?.modules.find(
-      ({ route }) => route.name === 'foundation',
-    )?.route;
-    expect(page).toMatchObject({
-      path: '/foundation',
-      auth: 'required',
-      authz: 'unrestricted',
-    });
-    if (!page) throw new Error('Foundation route missing');
-    const denied = new Set([`${page.packageName}:${page.path}:${page.id}`]);
-    expect(
-      buildRouteNavigation(runtime.routes, denied).map(
-        ({ route }) => route.name,
-      ),
-    ).toEqual(['home']);
-    expect(buildSettingsCatalog(runtime.routes, denied)).toEqual([]);
+    const guide = category?.modules.find(
+      ({ route }) => route.name === 'capabilityGuide',
+    );
+    expect(guide?.target.path).toBe('/capabilities');
+    expect(guide?.route.auth).toBe('required');
     expect(
-      serverPlugins.plugins.filter(
-        (plugin) => plugin.packageName === '@nocobase/app-plugin-foundation',
-      ),
-    ).toHaveLength(1);
+      buildSettingsCatalog(runtime.routes, new Set([routeKey(guide!.route)]))
+        .flatMap((item) => item.modules)
+        .some((item) => item.route.name === 'capabilityGuide'),
+    ).toBe(false);
   });
 
   it('keeps Foundation beside business settings without changing its protected URL', async () => {
@@ -77,7 +65,32 @@ describe('default Foundation integration', () => {
           ['permissionManagement', '/settings/roles'],
         ],
       ],
-      ['settingsSystem', [['foundation', '/foundation']]],
+      ['settingsSystem', [['capabilityGuide', '/capabilities']]],
     ]);
   });
+
+  it('lists every plugin this application declares in the capability guide, with the six core modules introduced on their own', () => {
+    const manifest = packageMetadata as DependencyManifest;
+    const { entries } = createCapabilityCatalog(manifest);
+    const declared = Object.keys({
+      ...manifest.dependencies,
+      ...manifest.devDependencies,
+    })
+      .filter((name) => name.includes('app-plugin-'))
+      .sort();
+    expect(
+      entries
+        .filter((entry) => entry.source === 'plugin')
+        .map((entry) => entry.packageName)
+        .sort(),
+    ).toEqual(declared);
+    expect(
+      entries
+        .filter((entry) => entry.source === 'kernel')
+        .map((entry) => entry.name),
+    ).toEqual(['db', 'jobs', 'lifecycle', 'queue', 'realtime']);
+    // Every entry has its own wording; none falls back to the generic text.
+    expect(entries.filter((entry) => !entry.known)).toEqual([]);
+    expect(entries.filter((entry) => entry.role === 'example').length).toBe(0);
+  });
 });

package/tests/client/home.test.tsx

@@ -1,13 +1,23 @@
 import { renderWithApp } from '@nocobase/app-testing/client';
-import { screen, waitFor, within } from '@testing-library/react';
+import { screen } from '@testing-library/react';
 import userEvent from '@testing-library/user-event';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 
 import packageMetadata from '../../package.json' with { type: 'json' };
 import enUS from '../../client/locales/en-US.js';
-import zhCN from '../../client/locales/zh-CN.js';
 import HomePage from '../../client/pages/home/index.js';
-import { CAPABILITIES } from '../../client/pages/home/capabilities.js';
+import {
+  CAPABILITY_GUIDE_PATH,
+  capabilityPath,
+  createCapabilityCatalog,
+  entryContent,
+  format,
+  guideContent,
+} from '@nocobase/app-plugin-foundation/capability-guide';
+
+const catalog = createCapabilityCatalog(packageMetadata);
+const escape = (text: string): string =>
+  text.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
 
 function renderHome() {
   return renderWithApp(<HomePage />, {
@@ -48,79 +58,28 @@ describe('home page', () => {
     expect(screen.getByText(enUS.home.steps.describe.title)).toBeVisible();
   });
 
-  it('shows one card per built-in capability', async () => {
+  it('features capabilities from the shared guide catalog, each linking to its entry', async () => {
     await renderHome();
+    const content = guideContent('en-US');
 
-    for (const { id } of CAPABILITIES) {
-      const title =
-        enUS.home.capabilities[id as keyof typeof enUS.home.capabilities].title;
+    expect(catalog.featured.length).toBeGreaterThan(0);
+    for (const entry of catalog.featured) {
+      const title = entryContent(content, entry).title;
       expect(
-        screen.getByRole('button', { name: new RegExp(title) }),
-      ).toBeInTheDocument();
+        screen.getByRole('link', { name: new RegExp(escape(title)) }),
+      ).toHaveAttribute('href', capabilityPath(entry.id));
     }
   });
 
-  it('opens a capability, switches prompts, and copies the edited text', async () => {
-    const user = setup();
-    const view = await renderHome();
-    const scheduler = enUS.home.capabilities.scheduler;
-
-    await user.click(
-      screen.getByRole('button', { name: new RegExp(scheduler.title) }),
-    );
-    const dialog = await screen.findByRole('dialog');
-    const prompt = within(dialog).getByRole('textbox', {
-      name: enUS.home.prompt.label,
-    });
-    expect(prompt).toHaveValue(scheduler.prompts.reminder.text);
-
-    await user.click(
-      within(dialog).getByRole('tab', {
-        name: scheduler.prompts.weeklyReport.label,
-      }),
-    );
-    expect(prompt).toHaveValue(scheduler.prompts.weeklyReport.text);
-
-    await user.clear(prompt);
-    await user.type(prompt, 'Every Friday');
-    await user.click(
-      within(dialog).getByRole('button', { name: enUS.home.prompt.copy }),
-    );
-
-    expect(writeText).toHaveBeenCalledWith('Every Friday');
-    expect(view.toasts()).toEqual([
-      expect.objectContaining({
-        type: 'success',
-        title: enUS.home.prompt.copied,
-      }),
-    ]);
-
-    await user.click(
-      within(dialog).getByRole('button', { name: enUS.home.prompt.reset }),
-    );
-    expect(prompt).toHaveValue(scheduler.prompts.weeklyReport.text);
-  });
-
-  it('starts from the default prompt each time the dialog opens', async () => {
-    const user = setup();
+  it('leads to the full catalog, counted from the dependencies this application declares', async () => {
     await renderHome();
-    const theme = enUS.home.capabilities.theme;
-    const card = screen.getByRole('button', { name: new RegExp(theme.title) });
+    const content = guideContent('en-US');
 
-    await user.click(card);
-    const prompt = within(await screen.findByRole('dialog')).getByRole(
-      'textbox',
-    );
-    await user.type(prompt, ' Edited.');
-    await user.keyboard('{Escape}');
-    await waitFor(() =>
-      expect(screen.queryByRole('dialog')).not.toBeInTheDocument(),
-    );
-
-    await user.click(card);
     expect(
-      within(await screen.findByRole('dialog')).getByRole('textbox'),
-    ).toHaveValue(theme.prompts.changeTheme.text);
+      screen.getByRole('link', {
+        name: format(content.home.viewAll, { count: catalog.entries.length }),
+      }),
+    ).toHaveAttribute('href', CAPABILITY_GUIDE_PATH);
   });
 
   it('reports a copy the browser refuses', async () => {
@@ -140,21 +99,3 @@ describe('home page', () => {
     ]);
   });
 });
-
-describe('home page copy', () => {
-  it('translates every capability prompt into Chinese', () => {
-    for (const { id, prompts } of CAPABILITIES) {
-      const key = id as keyof typeof zhCN.home.capabilities;
-      for (const prompt of prompts) {
-        const zh = (
-          zhCN.home.capabilities[key].prompts as Record<
-            string,
-            { label: string; text: string }
-          >
-        )[prompt];
-        expect(zh?.text, `${id}.${prompt}`).toBeTruthy();
-        expect(zh?.label, `${id}.${prompt}`).toBeTruthy();
-      }
-    }
-  });
-});

package/tests/client/layout-i18n.test.tsx

@@ -1,3 +1,4 @@
+import foundation from '@nocobase/app-plugin-foundation/client/plugin';
 import {
   ClientApplicationContext,
   NavigationGuardProvider,
@@ -7,10 +8,17 @@ import {
 import type { AppClientRegisteredRoute } from '@nocobase/app-client/plugins';
 import { I18nRuntime } from '@nocobase/i18n';
 import { I18nProvider } from '@nocobase/i18n/client';
-import { act, render, screen } from '@testing-library/react';
+import {
+  act,
+  fireEvent,
+  render,
+  screen,
+  waitFor,
+  within,
+} from '@testing-library/react';
 import type { ReactNode } from 'react';
 import { MemoryRouter } from 'react-router';
-import { describe, expect, it, vi } from 'vitest';
+import { afterEach, describe, expect, it, vi } from 'vitest';
 
 import locales from '../../client/locales/index.js';
 import { AppLayout } from '../../client/layouts/app-layout.js';
@@ -37,6 +45,8 @@ vi.mock('../../client/routing/route-navigation.js', async (importOriginal) => ({
   }),
 }));
 
+afterEach(() => vi.unstubAllGlobals());
+
 /** The page the shell opens on: the header names it. */
 const overview: AppClientRegisteredRoute = {
   id: 'overview',
@@ -62,6 +72,10 @@ async function setup(children: ReactNode) {
     applicationNamespace: 'test-app',
   });
   runtime.registerApplicationNamespace('test-app', locales);
+  runtime.registerNamespace(
+    '@nocobase/app-plugin-foundation',
+    foundation().locales!,
+  );
   await runtime.init('en-US');
   const app = {
     config: createAppClientConfig({ rawConfig: {} }),
@@ -110,3 +124,70 @@ describe('shell translations', () => {
     ).toBeVisible();
   });
 });
+
+it('translates the settings navigation without a separate resources footer', async () => {
+  const settings: AppClientRegisteredRoute = {
+    ...overview,
+    id: 'settings',
+    name: 'settings',
+    path: '/settings',
+    packageName: 'test',
+    source: 'application',
+    auth: 'required',
+    authz: 'skip',
+    navigation: { title: 'settings.title' },
+  };
+  const runtime = await setup(<AppLayout routes={[overview, settings]} />);
+  const navigation = screen.getByRole('navigation', {
+    name: 'Application navigation',
+  });
+  const link = within(navigation).getByRole('link', {
+    name: 'System settings',
+  });
+  expect(link).toHaveAttribute('href', '/settings');
+  expect(link.closest('[data-slot="sidebar-content"]')).not.toBeNull();
+  expect(
+    screen.queryByRole('navigation', { name: 'Resources' }),
+  ).not.toBeInTheDocument();
+  fireEvent.click(link);
+  await waitFor(() =>
+    expect(
+      within(navigation).getByRole('link', { name: 'System settings' }),
+    ).toHaveAttribute('aria-current', 'page'),
+  );
+  await act(() => runtime.changeLanguage('zh-CN'));
+  expect(
+    within(screen.getByRole('navigation', { name: '应用导航' })).getByRole(
+      'link',
+      { name: '系统设置' },
+    ),
+  ).toHaveAttribute('href', '/settings');
+});
+
+it('closes the mobile sidebar after opening System settings', async () => {
+  vi.stubGlobal('innerWidth', 375);
+  const settings: AppClientRegisteredRoute = {
+    ...overview,
+    id: 'settings',
+    name: 'settings',
+    path: '/settings',
+    packageName: 'test',
+    source: 'application',
+    auth: 'required',
+    authz: 'skip',
+    navigation: { title: 'settings.title' },
+  };
+  await setup(<AppLayout routes={[overview, settings]} />);
+  fireEvent.click(screen.getByRole('button', { name: 'Open navigation' }));
+  const navigation = await screen.findByRole('navigation', {
+    name: 'Application navigation',
+  });
+  fireEvent.click(
+    within(navigation).getByRole('link', { name: 'System settings' }),
+  );
+  await waitFor(() =>
+    expect(
+      screen.queryByRole('navigation', { name: 'Application navigation' }),
+    ).not.toBeInTheDocument(),
+  );
+});

package/tests/client/list-feedback.test.tsx

@@ -97,7 +97,7 @@ describe('shared table and card feedback', () => {
       />,
       options,
     );
-    expect(screen.getByRole('status').textContent).toBe(
+    expect((await screen.findByRole('status')).textContent).toBe(
       enUS.listFeedback.loading,
     );
     expect(screen.getByRole('columnheader', { name: 'ID' })).toBeVisible();
@@ -159,7 +159,7 @@ describe('shared table and card feedback', () => {
       expect(
         screen.getByRole('button', { name: 'Search tools' }),
       ).toBeVisible();
-      expect(screen.getByRole(role).closest('td')).not.toBeNull();
+      expect((await screen.findByRole(role)).closest('td')).not.toBeNull();
     }
 
     view.rerender(
@@ -178,9 +178,8 @@ describe('shared table and card feedback', () => {
   });
 
   it('renders each built-in state as one icon and one short line without inline actions', async () => {
-    const view = await renderWithApp(<ListFeedback state='loading' />, options);
+    const view = await renderWithApp(<ListFeedback state='empty' />, options);
     const states = [
-      ['loading', 'status'],
       ['empty', 'status'],
       ['no-results', 'status'],
       ['forbidden', 'alert'],
@@ -273,8 +272,9 @@ describe('shared table and card feedback', () => {
     expect(screen.getByRole('alert').textContent).toBe(
       enUS.listFeedback.failed,
     );
+    expect(screen.getByRole('button')).toBeDisabled();
     expect(
-      screen.getByRole('button', { name: enUS.listFeedback.retrying }),
+      await screen.findByRole('button', { name: enUS.listFeedback.retrying }),
     ).toBeDisabled();
   });