@nextclaw/ui 0.12.9 → 0.12.11

package/src/components/config/ChannelsList.test.tsx

@@ -1,7 +1,7 @@
-import { fireEvent, render, screen, waitFor } from '@testing-library/react';
-import userEvent from '@testing-library/user-event';
-import type * as ReactQueryModule from '@tanstack/react-query';
-import { ChannelsList } from '@/components/config/ChannelsList';
+import { fireEvent, render, screen, waitFor } from "@testing-library/react";
+import userEvent from "@testing-library/user-event";
+import type * as ReactQueryModule from "@tanstack/react-query";
+import { ChannelsList } from "@/components/config/ChannelsList";
 
 const mocks = vi.hoisted(() => ({
   updateChannelMutate: vi.fn(),
@@ -13,88 +13,90 @@ const mocks = vi.hoisted(() => ({
       channels: {
         weixin: {
           enabled: false,
-          defaultAccountId: '1344b2b24720@im.bot',
-          baseUrl: 'https://ilinkai.weixin.qq.com',
+          defaultAccountId: "1344b2b24720@im.bot",
+          baseUrl: "https://ilinkai.weixin.qq.com",
           pollTimeoutMs: 35000,
-          allowFrom: ['o9cq804svxfyCCTIqzddDqRBeMC0@im.wechat'],
+          allowFrom: ["o9cq804svxfyCCTIqzddDqRBeMC0@im.wechat"],
           accounts: {
-            '1344b2b24720@im.bot': {
-              enabled: true
-            }
-          }
-        }
-      }
+            "1344b2b24720@im.bot": {
+              enabled: true,
+            },
+          },
+        },
+      },
     },
-    isLoading: false
+    isLoading: false,
   },
   metaQuery: {
     data: {
       channels: [
         {
-          name: 'weixin',
-          displayName: 'Weixin',
-          enabled: false
-        }
-      ]
-    }
+          name: "weixin",
+          displayName: "Weixin",
+          enabled: false,
+        },
+      ],
+    },
   },
   schemaQuery: {
     data: {
       uiHints: {
-        'channels.weixin': {
-          label: 'Weixin',
-          help: 'Weixin QR login + getupdates long-poll channel'
+        "channels.weixin": {
+          label: "Weixin",
+          help: "Weixin QR login + getupdates long-poll channel",
+        },
+        "channels.weixin.baseUrl": {
+          label: "API Base URL",
         },
-        'channels.weixin.baseUrl': {
-          label: 'API Base URL'
-        }
       },
-      actions: []
-    }
-  }
+      actions: [],
+    },
+  },
 }));
 
-vi.mock('qrcode', () => ({
-  toDataURL: vi.fn().mockResolvedValue('data:image/png;base64,weixin-qr')
+vi.mock("qrcode", () => ({
+  toDataURL: vi.fn().mockResolvedValue("data:image/png;base64,weixin-qr"),
 }));
 
-vi.mock('@tanstack/react-query', async () => {
-  const actual = await vi.importActual<typeof ReactQueryModule>('@tanstack/react-query');
+vi.mock("@tanstack/react-query", async () => {
+  const actual = await vi.importActual<typeof ReactQueryModule>(
+    "@tanstack/react-query",
+  );
   return {
     ...actual,
     useQueryClient: () => ({
-      invalidateQueries: vi.fn().mockResolvedValue(undefined)
-    })
+      invalidateQueries: vi.fn().mockResolvedValue(undefined),
+    }),
   };
 });
 
-vi.mock('@/hooks/useConfig', () => ({
+vi.mock("@/hooks/useConfig", () => ({
   useConfig: () => mocks.configQuery,
   useConfigMeta: () => mocks.metaQuery,
   useConfigSchema: () => mocks.schemaQuery,
   useUpdateChannel: () => ({
     mutate: mocks.updateChannelMutate,
     mutateAsync: mocks.updateChannelMutateAsync,
-    isPending: false
+    isPending: false,
   }),
   useExecuteConfigAction: () => ({
     mutateAsync: vi.fn(),
-    isPending: false
-  })
+    isPending: false,
+  }),
 }));
 
-vi.mock('@/hooks/use-channel-auth', () => ({
+vi.mock("@/hooks/use-channel-auth", () => ({
   useStartChannelAuth: () => ({
     mutateAsync: mocks.startChannelAuthMutateAsync,
-    isPending: false
+    isPending: false,
   }),
   usePollChannelAuth: () => ({
     mutateAsync: mocks.pollChannelAuthMutateAsync,
-    isPending: false
-  })
+    isPending: false,
+  }),
 }));
 
-describe('ChannelsList', () => {
+describe("ChannelsList", () => {
   beforeEach(() => {
     mocks.updateChannelMutate.mockReset();
     mocks.updateChannelMutateAsync.mockReset();
@@ -104,68 +106,76 @@ describe('ChannelsList', () => {
       channels: {
         weixin: {
           enabled: false,
-          defaultAccountId: '1344b2b24720@im.bot',
-          baseUrl: 'https://ilinkai.weixin.qq.com',
+          defaultAccountId: "1344b2b24720@im.bot",
+          baseUrl: "https://ilinkai.weixin.qq.com",
           pollTimeoutMs: 35000,
-          allowFrom: ['o9cq804svxfyCCTIqzddDqRBeMC0@im.wechat'],
+          allowFrom: ["o9cq804svxfyCCTIqzddDqRBeMC0@im.wechat"],
           accounts: {
-            '1344b2b24720@im.bot': {
-              enabled: true
-            }
-          }
-        }
-      }
+            "1344b2b24720@im.bot": {
+              enabled: true,
+            },
+          },
+        },
+      },
     };
     mocks.metaQuery.data = {
       channels: [
         {
-          name: 'weixin',
-          displayName: 'Weixin',
-          enabled: false
-        }
-      ]
+          name: "weixin",
+          displayName: "Weixin",
+          enabled: false,
+        },
+      ],
     };
   });
 
-  it('renders weixin qr auth card and starts channel auth', async () => {
+  it("renders weixin qr auth card and starts channel auth", async () => {
     const user = userEvent.setup();
     mocks.startChannelAuthMutateAsync.mockResolvedValue({
-      channel: 'weixin',
-      kind: 'qr_code',
-      sessionId: 'session-1',
-      qrCode: 'qr-token',
-      qrCodeUrl: 'https://example.com/weixin-qr.png',
-      expiresAt: '2026-03-23T10:00:00.000Z',
+      channel: "weixin",
+      kind: "qr_code",
+      sessionId: "session-1",
+      qrCode: "qr-token",
+      qrCodeUrl: "https://example.com/weixin-qr.png",
+      expiresAt: "2026-03-23T10:00:00.000Z",
       intervalMs: 60_000,
-      note: '请扫码'
+      note: "请扫码",
     });
 
     render(<ChannelsList />);
 
-    await user.click(await screen.findByRole('button', { name: /All Channels/i }));
+    await user.click(
+      await screen.findByRole("button", { name: /All Channels/i }),
+    );
 
-    expect((await screen.findAllByText('Weixin')).length).toBeGreaterThan(0);
-    expect(await screen.findByRole('button', { name: 'Reconnect with QR' })).toBeTruthy();
-    expect(screen.getByText('Weixin now uses QR login as the primary setup flow.')).toBeTruthy();
+    expect((await screen.findAllByText("Weixin")).length).toBeGreaterThan(0);
+    expect(
+      await screen.findByRole("button", { name: "Reconnect with QR" }),
+    ).toBeTruthy();
+    expect(
+      screen.getByText("Weixin now uses QR login as the primary setup flow."),
+    ).toBeTruthy();
 
-    await user.click(screen.getByRole('button', { name: 'Reconnect with QR' }));
+    await user.click(screen.getByRole("button", { name: "Reconnect with QR" }));
 
     await waitFor(() => {
       expect(mocks.startChannelAuthMutateAsync).toHaveBeenCalledWith({
-        channel: 'weixin',
+        channel: "weixin",
         data: expect.objectContaining({
-          accountId: '1344b2b24720@im.bot',
-          baseUrl: 'https://ilinkai.weixin.qq.com'
-        })
+          accountId: "1344b2b24720@im.bot",
+          baseUrl: "https://ilinkai.weixin.qq.com",
+        }),
       });
     });
 
     await waitFor(() => {
-      expect(screen.getByAltText('Weixin login QR code').getAttribute('src')).toBe('data:image/png;base64,weixin-qr');
+      expect(
+        screen.getByAltText("Weixin login QR code").getAttribute("src"),
+      ).toBe("data:image/png;base64,weixin-qr");
     });
   });
 
-  it('keeps Weixin, Feishu, Discord, and QQ at the front of the channel list', async () => {
+  it("keeps Weixin, Feishu, Discord, and QQ at the front of the channel list", async () => {
     const user = userEvent.setup();
     mocks.configQuery.data = {
       channels: {
@@ -173,92 +183,126 @@ describe('ChannelsList', () => {
         qq: { enabled: false },
         discord: { enabled: false },
         weixin: { enabled: false },
-        feishu: { enabled: false }
-      }
+        feishu: { enabled: false },
+      },
     } as unknown as typeof mocks.configQuery.data;
     mocks.metaQuery.data = {
       channels: [
-        { name: 'telegram', displayName: 'Telegram', enabled: false },
-        { name: 'qq', displayName: 'QQ', enabled: false },
-        { name: 'discord', displayName: 'Discord', enabled: false },
-        { name: 'weixin', displayName: 'Weixin', enabled: false },
-        { name: 'feishu', displayName: 'Feishu', enabled: false }
-      ]
+        { name: "telegram", displayName: "Telegram", enabled: false },
+        { name: "qq", displayName: "QQ", enabled: false },
+        { name: "discord", displayName: "Discord", enabled: false },
+        { name: "weixin", displayName: "Weixin", enabled: false },
+        { name: "feishu", displayName: "Feishu", enabled: false },
+      ],
     } as typeof mocks.metaQuery.data;
 
     const { container } = render(<ChannelsList />);
 
-    await user.click(await screen.findByRole('button', { name: /All Channels/i }));
+    await user.click(
+      await screen.findByRole("button", { name: /All Channels/i }),
+    );
 
-    const sidebarSection = container.querySelector('section');
+    const sidebarSection = container.querySelector("section");
     if (!(sidebarSection instanceof HTMLElement)) {
-      throw new Error('channel sidebar not found');
+      throw new Error("channel sidebar not found");
     }
 
-    const channelButtons = Array.from(sidebarSection.querySelectorAll('button[type="button"]')).filter((button) => (
-      ['Weixin', 'Feishu', 'Discord', 'QQ', 'Telegram'].some((label) => button.textContent?.includes(label))
-    ));
+    const channelButtons = Array.from(
+      sidebarSection.querySelectorAll('button[type="button"]'),
+    ).filter((button) =>
+      ["Weixin", "Feishu", "Discord", "QQ", "Telegram"].some((label) =>
+        button.textContent?.includes(label),
+      ),
+    );
 
     expect(channelButtons.map((button) => button.textContent)).toEqual([
-      expect.stringContaining('Weixin'),
-      expect.stringContaining('Feishu'),
-      expect.stringContaining('Discord'),
-      expect.stringContaining('QQ'),
-      expect.stringContaining('Telegram')
+      expect.stringContaining("Weixin"),
+      expect.stringContaining("Feishu"),
+      expect.stringContaining("Discord"),
+      expect.stringContaining("QQ"),
+      expect.stringContaining("Telegram"),
     ]);
   });
 
-  it('saves weixin advanced settings from the advanced section', async () => {
+  it("only enables internal scroll chaining on desktop-sized panes", async () => {
+    const user = userEvent.setup();
+    const { container } = render(<ChannelsList />);
+
+    await user.click(
+      await screen.findByRole("button", { name: /All Channels/i }),
+    );
+
+    const scrollRegions = Array.from(container.querySelectorAll("div")).filter(
+      (node) => {
+        const classTokens = node.className.split(/\s+/).filter(Boolean);
+        return classTokens.includes("xl:overflow-y-auto");
+      },
+    );
+
+    expect(scrollRegions.length).toBeGreaterThanOrEqual(2);
+    scrollRegions.forEach((region) => {
+      const classTokens = region.className.split(/\s+/).filter(Boolean);
+      expect(classTokens).toContain("xl:overscroll-contain");
+      expect(classTokens).not.toContain("overflow-y-auto");
+      expect(classTokens).not.toContain("overscroll-contain");
+    });
+  });
+
+  it("saves weixin advanced settings from the advanced section", async () => {
     const user = userEvent.setup();
 
     const { container } = render(<ChannelsList />);
 
-    await user.click(await screen.findByRole('button', { name: /All Channels/i }));
-    await user.click(await screen.findByText('Advanced settings'));
+    await user.click(
+      await screen.findByRole("button", { name: /All Channels/i }),
+    );
+    await user.click(await screen.findByText("Advanced settings"));
 
-    const timeoutInput = await screen.findByLabelText('Long Poll Timeout (ms)');
+    const timeoutInput = await screen.findByLabelText("Long Poll Timeout (ms)");
     await user.clear(timeoutInput);
-    await user.type(timeoutInput, '45000');
+    await user.type(timeoutInput, "45000");
 
-    const accountsJson = container.querySelector('textarea#accounts') as HTMLTextAreaElement | null;
+    const accountsJson = container.querySelector(
+      "textarea#accounts",
+    ) as HTMLTextAreaElement | null;
     expect(accountsJson).toBeTruthy();
     if (!accountsJson) {
-      throw new Error('accounts textarea not found');
+      throw new Error("accounts textarea not found");
     }
     await user.clear(accountsJson);
     fireEvent.change(accountsJson, {
       target: {
         value: JSON.stringify(
           {
-            '1344b2b24720@im.bot': {
+            "1344b2b24720@im.bot": {
               enabled: true,
-              baseUrl: 'https://ilinkai.weixin.qq.com'
-            }
+              baseUrl: "https://ilinkai.weixin.qq.com",
+            },
           },
           null,
-          2
-        )
-      }
+          2,
+        ),
+      },
     });
 
-    await user.click(screen.getByRole('button', { name: /save/i }));
+    await user.click(screen.getByRole("button", { name: /save/i }));
 
     await waitFor(() => {
       expect(mocks.updateChannelMutate).toHaveBeenCalledWith({
-        channel: 'weixin',
+        channel: "weixin",
         data: expect.objectContaining({
           enabled: false,
-          defaultAccountId: '1344b2b24720@im.bot',
-          baseUrl: 'https://ilinkai.weixin.qq.com',
+          defaultAccountId: "1344b2b24720@im.bot",
+          baseUrl: "https://ilinkai.weixin.qq.com",
           pollTimeoutMs: 45000,
-          allowFrom: ['o9cq804svxfyCCTIqzddDqRBeMC0@im.wechat'],
+          allowFrom: ["o9cq804svxfyCCTIqzddDqRBeMC0@im.wechat"],
           accounts: {
-            '1344b2b24720@im.bot': {
+            "1344b2b24720@im.bot": {
               enabled: true,
-              baseUrl: 'https://ilinkai.weixin.qq.com'
-            }
-          }
-        })
+              baseUrl: "https://ilinkai.weixin.qq.com",
+            },
+          },
+        }),
       });
     });
   });