From dd446a730e9fbf9c7cb6d4ee86c3d314d6648112 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 21 Jan 2026 08:25:08 +0000 Subject: [PATCH] fix: update test files with new doc links Co-authored-by: hyoban <38493346+hyoban@users.noreply.github.com> --- .../app/create-app-modal/index.spec.tsx | 2 +- .../datasets/create/step-three/index.spec.tsx | 2 +- .../workflow-onboarding-modal/index.spec.tsx | 39 +------------------ 3 files changed, 4 insertions(+), 39 deletions(-) diff --git a/web/app/components/app/create-app-modal/index.spec.tsx b/web/app/components/app/create-app-modal/index.spec.tsx index cb8f4db67f..730bc93448 100644 --- a/web/app/components/app/create-app-modal/index.spec.tsx +++ b/web/app/components/app/create-app-modal/index.spec.tsx @@ -41,7 +41,7 @@ vi.mock('@/context/app-context', () => ({ useAppContext: vi.fn(), })) vi.mock('@/context/i18n', () => ({ - useDocLink: () => () => '/guides', + useDocLink: () => () => 'https://docs.dify.ai/en', })) vi.mock('@/hooks/use-theme', () => ({ default: () => ({ theme: 'light' }), diff --git a/web/app/components/datasets/create/step-three/index.spec.tsx b/web/app/components/datasets/create/step-three/index.spec.tsx index 43b4916778..74c5912a1b 100644 --- a/web/app/components/datasets/create/step-three/index.spec.tsx +++ b/web/app/components/datasets/create/step-three/index.spec.tsx @@ -190,7 +190,7 @@ describe('StepThree', () => { // Assert const link = screen.getByText('datasetPipeline.addDocuments.stepThree.learnMore') - expect(link).toHaveAttribute('href', 'https://docs.dify.ai/en-US/guides/knowledge-base/integrate-knowledge-within-application') + expect(link).toHaveAttribute('href', 'https://docs.dify.ai/en-US/use-dify/knowledge/integrate-knowledge-within-application') expect(link).toHaveAttribute('target', '_blank') expect(link).toHaveAttribute('rel', 'noreferrer noopener') }) diff --git a/web/app/components/workflow-app/components/workflow-onboarding-modal/index.spec.tsx b/web/app/components/workflow-app/components/workflow-onboarding-modal/index.spec.tsx index 6dac82a642..fd43b547ec 100644 --- a/web/app/components/workflow-app/components/workflow-onboarding-modal/index.spec.tsx +++ b/web/app/components/workflow-app/components/workflow-onboarding-modal/index.spec.tsx @@ -30,7 +30,7 @@ vi.mock('@/app/components/base/modal', () => ({ // Mock useDocLink hook vi.mock('@/context/i18n', () => ({ - useDocLink: () => (path: string) => `https://docs.example.com${path}`, + useDocLink: () => (path: string) => `https://docs.dify.ai/en${path}`, })) // Mock StartNodeSelectionPanel (using real component would be better for integration, @@ -119,25 +119,13 @@ describe('WorkflowOnboardingModal', () => { // Arrange & Act const { container } = renderComponent() - // Assert - Check both parts of description (separated by link) + // Assert - Check both parts of description const descriptionDiv = container.querySelector('.body-xs-regular.leading-4') expect(descriptionDiv).toBeInTheDocument() expect(descriptionDiv).toHaveTextContent('workflow.onboarding.description') expect(descriptionDiv).toHaveTextContent('workflow.onboarding.aboutStartNode') }) - it('should render learn more link', () => { - // Arrange & Act - renderComponent() - - // Assert - const learnMoreLink = screen.getByText('workflow.onboarding.learnMore') - expect(learnMoreLink).toBeInTheDocument() - expect(learnMoreLink.closest('a')).toHaveAttribute('href', 'https://docs.example.com/guides/workflow/node/start') - expect(learnMoreLink.closest('a')).toHaveAttribute('target', '_blank') - expect(learnMoreLink.closest('a')).toHaveAttribute('rel', 'noopener noreferrer') - }) - it('should render StartNodeSelectionPanel', () => { // Arrange & Act renderComponent() @@ -547,16 +535,6 @@ describe('WorkflowOnboardingModal', () => { expect(heading).toHaveTextContent('workflow.onboarding.title') }) - it('should have external link with proper attributes', () => { - // Arrange & Act - renderComponent() - - // Assert - const link = screen.getByText('workflow.onboarding.learnMore').closest('a') - expect(link).toHaveAttribute('target', '_blank') - expect(link).toHaveAttribute('rel', 'noopener noreferrer') - }) - it('should have keyboard navigation support via ESC key', () => { // Arrange renderComponent({ isShow: true }) @@ -595,16 +573,6 @@ describe('WorkflowOnboardingModal', () => { const title = screen.getByText('workflow.onboarding.title') expect(title).toHaveClass('text-text-primary') }) - - it('should have underlined learn more link', () => { - // Arrange & Act - renderComponent() - - // Assert - const link = screen.getByText('workflow.onboarding.learnMore').closest('a') - expect(link).toHaveClass('underline') - expect(link).toHaveClass('cursor-pointer') - }) }) // Integration Tests @@ -654,9 +622,6 @@ describe('WorkflowOnboardingModal', () => { const heading = container.querySelector('h3') expect(heading).toBeInTheDocument() - // Assert - Description with link - expect(screen.getByText('workflow.onboarding.learnMore').closest('a')).toBeInTheDocument() - // Assert - Selection panel expect(screen.getByTestId('start-node-selection-panel')).toBeInTheDocument()