- {formatNumber(value.value[1])}
+ {formatNumber(value.value[1]!)}
{t('weightedScore.keyword', { ns: 'dataset' })}
diff --git a/web/app/components/app/configuration/dataset-config/settings-modal/__tests__/retrieval-section.spec.tsx b/web/app/components/app/configuration/dataset-config/settings-modal/__tests__/retrieval-section.spec.tsx
index e100ed9490..e4e3f5ab71 100644
--- a/web/app/components/app/configuration/dataset-config/settings-modal/__tests__/retrieval-section.spec.tsx
+++ b/web/app/components/app/configuration/dataset-config/settings-modal/__tests__/retrieval-section.spec.tsx
@@ -152,7 +152,8 @@ describe('RetrievalChangeTip', () => {
await userEvent.click(screen.getByRole('button', { name: 'close-retrieval-change-tip' }))
// Assert
- expect(screen.getByText('Test message')).toBeInTheDocument()
+ // Assert
+ expect(screen.getByText('Test message'))!.toBeInTheDocument()
expect(onDismiss).toHaveBeenCalledTimes(1)
})
@@ -160,6 +161,37 @@ describe('RetrievalChangeTip', () => {
// Arrange & Act
render(
)
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
// Assert
expect(screen.queryByText('Test message')).not.toBeInTheDocument()
})
@@ -213,12 +245,13 @@ describe('RetrievalSection', () => {
/>,
)
const [topKIncrement] = screen.getAllByRole('button', { name: /increment/i })
- await userEvent.click(topKIncrement)
+ await userEvent.click(topKIncrement!)
// Assert
- expect(screen.getByText('External API')).toBeInTheDocument()
- expect(screen.getByText('https://api.external.com')).toBeInTheDocument()
- expect(screen.getByText('ext-id-999')).toBeInTheDocument()
+ // Assert
+ expect(screen.getByText('External API'))!.toBeInTheDocument()
+ expect(screen.getByText('https://api.external.com'))!.toBeInTheDocument()
+ expect(screen.getByText('ext-id-999'))!.toBeInTheDocument()
expect(handleExternalChange).toHaveBeenCalledWith(expect.objectContaining({ top_k: 4 }))
})
@@ -243,9 +276,10 @@ describe('RetrievalSection', () => {
)
// Assert
- expect(screen.getByText('dataset.retrieval.semantic_search.title')).toBeInTheDocument()
+ // Assert
+ expect(screen.getByText('dataset.retrieval.semantic_search.title'))!.toBeInTheDocument()
const learnMoreLink = screen.getByRole('link', { name: 'datasetSettings.form.retrievalSetting.learnMore' })
- expect(learnMoreLink).toHaveAttribute('href', 'https://docs.example/use-dify/knowledge/create-knowledge/setting-indexing-methods')
+ expect(learnMoreLink)!.toHaveAttribute('href', 'https://docs.example/use-dify/knowledge/create-knowledge/setting-indexing-methods')
expect(docLink).toHaveBeenCalledWith('/use-dify/knowledge/create-knowledge/setting-indexing-methods')
})
@@ -268,10 +302,11 @@ describe('RetrievalSection', () => {
/>,
)
const [topKIncrement] = screen.getAllByRole('button', { name: /increment/i })
- await userEvent.click(topKIncrement)
+ await userEvent.click(topKIncrement!)
// Assert
- expect(screen.getByText('dataset.retrieval.keyword_search.title')).toBeInTheDocument()
+ // Assert
+ expect(screen.getByText('dataset.retrieval.keyword_search.title'))!.toBeInTheDocument()
expect(handleRetrievalChange).toHaveBeenCalledWith(expect.objectContaining({
top_k: 3,
}))
diff --git a/web/app/components/app/configuration/debug/__tests__/index.spec.tsx b/web/app/components/app/configuration/debug/__tests__/index.spec.tsx
index 071307303b..4a50ecc626 100644
--- a/web/app/components/app/configuration/debug/__tests__/index.spec.tsx
+++ b/web/app/components/app/configuration/debug/__tests__/index.spec.tsx
@@ -479,8 +479,8 @@ describe('Debug', () => {
},
})
- expect(screen.getByText('appDebug.noModelProviderConfigured')).toBeInTheDocument()
- expect(screen.getByText('appDebug.noModelProviderConfiguredTip')).toBeInTheDocument()
+ expect(screen.getByText('appDebug.noModelProviderConfigured'))!.toBeInTheDocument()
+ expect(screen.getByText('appDebug.noModelProviderConfiguredTip'))!.toBeInTheDocument()
fireEvent.click(screen.getByRole('button', { name: 'appDebug.manageModels' }))
expect(onSetting).toHaveBeenCalledTimes(1)
@@ -500,8 +500,8 @@ describe('Debug', () => {
},
})
- expect(screen.getByText('appDebug.noModelSelected')).toBeInTheDocument()
- expect(screen.getByText('appDebug.noModelSelectedTip')).toBeInTheDocument()
+ expect(screen.getByText('appDebug.noModelSelected'))!.toBeInTheDocument()
+ expect(screen.getByText('appDebug.noModelSelectedTip'))!.toBeInTheDocument()
expect(screen.queryByText('appDebug.noModelProviderConfigured')).not.toBeInTheDocument()
})
})
@@ -510,9 +510,9 @@ describe('Debug', () => {
it('should render single-model panel and refresh conversation', () => {
renderDebug()
- expect(screen.getByTestId('debug-with-single-model')).toBeInTheDocument()
+ expect(screen.getByTestId('debug-with-single-model'))!.toBeInTheDocument()
- fireEvent.click(screen.getAllByTestId('action-button')[0])
+ fireEvent.click(screen.getAllByTestId('action-button')[0]!)
expect(mockState.mockHandleRestart).toHaveBeenCalledTimes(1)
})
@@ -535,8 +535,8 @@ describe('Debug', () => {
},
})
- expect(screen.getByTestId('chat-user-input')).toBeInTheDocument()
- fireEvent.click(screen.getAllByTestId('action-button')[1])
+ expect(screen.getByTestId('chat-user-input'))!.toBeInTheDocument()
+ fireEvent.click(screen.getAllByTestId('action-button')[1]!)
expect(screen.queryByTestId('chat-user-input')).not.toBeInTheDocument()
})
@@ -560,7 +560,7 @@ describe('Debug', () => {
},
})
- expect(screen.getByTestId('formatting-changed')).toBeInTheDocument()
+ expect(screen.getByTestId('formatting-changed'))!.toBeInTheDocument()
fireEvent.click(screen.getByTestId('formatting-cancel'))
expect(setFormattingChanged).toHaveBeenCalledWith(false)
})
@@ -623,8 +623,8 @@ describe('Debug', () => {
},
})
- expect(screen.getByTestId('prompt-value-panel')).toBeInTheDocument()
- expect(screen.getByText('appDebug.noResult')).toBeInTheDocument()
+ expect(screen.getByTestId('prompt-value-panel'))!.toBeInTheDocument()
+ expect(screen.getByText('appDebug.noResult'))!.toBeInTheDocument()
})
it('should notify when required input is missing', () => {
@@ -696,7 +696,7 @@ describe('Debug', () => {
})
fireEvent.click(screen.getByTestId('panel-send'))
- expect(screen.getByTestId('cannot-query-dataset')).toBeInTheDocument()
+ expect(screen.getByTestId('cannot-query-dataset'))!.toBeInTheDocument()
fireEvent.click(screen.getByTestId('cannot-query-confirm'))
expect(screen.queryByTestId('cannot-query-dataset')).not.toBeInTheDocument()
@@ -752,7 +752,7 @@ describe('Debug', () => {
fireEvent.click(screen.getByTestId('panel-send'))
await waitFor(() => expect(mockState.mockSendCompletionMessage).toHaveBeenCalledTimes(1))
- const [, requestData] = mockState.mockSendCompletionMessage.mock.calls[0]
+ const [, requestData] = (mockState.mockSendCompletionMessage.mock.calls[0] ?? []) as [unknown, any]
expect(requestData).toMatchObject({
inputs: { question: 'hello' },
model_config: {
@@ -763,9 +763,9 @@ describe('Debug', () => {
dataset_query_variable: 'question',
},
})
- expect(screen.getByTestId('text-generation')).toHaveTextContent('final answer')
- expect(screen.getByTestId('text-generation')).toHaveAttribute('data-message-id', 'msg-1')
- expect(screen.getByTestId('text-generation')).toHaveAttribute('data-tts', 'true')
+ expect(screen.getByTestId('text-generation'))!.toHaveTextContent('final answer')
+ expect(screen.getByTestId('text-generation'))!.toHaveAttribute('data-message-id', 'msg-1')
+ expect(screen.getByTestId('text-generation'))!.toHaveAttribute('data-tts', 'true')
})
it('should notify when sending again while a response is in progress', async () => {
@@ -830,7 +830,7 @@ describe('Debug', () => {
fireEvent.click(screen.getByTestId('panel-send'))
await waitFor(() => expect(mockState.mockSendCompletionMessage).toHaveBeenCalledTimes(1))
- expect(screen.getByText('appDebug.noResult')).toBeInTheDocument()
+ expect(screen.getByText('appDebug.noResult'))!.toBeInTheDocument()
})
it('should render prompt log modal in completion mode when store flag is enabled', () => {
@@ -845,7 +845,7 @@ describe('Debug', () => {
},
})
- expect(screen.getByTestId('prompt-log-modal')).toBeInTheDocument()
+ expect(screen.getByTestId('prompt-log-modal'))!.toBeInTheDocument()
})
it('should close prompt log modal in completion mode', () => {
@@ -904,7 +904,7 @@ describe('Debug', () => {
},
})
- expect(screen.getByRole('button', { name: 'common.modelProvider.addModel(4/4)' })).toBeDisabled()
+ expect(screen.getByRole('button', { name: 'common.modelProvider.addModel(4/4)' }))!.toBeDisabled()
})
it('should emit completion event in multiple-model completion mode', () => {
@@ -945,7 +945,7 @@ describe('Debug', () => {
},
})
- fireEvent.click(screen.getAllByTestId('action-button')[0])
+ fireEvent.click(screen.getAllByTestId('action-button')[0]!)
expect(mockState.mockEventEmitterEmit).toHaveBeenCalledWith({
type: APP_CHAT_WITH_MULTIPLE_MODEL_RESTART,
})
@@ -1012,8 +1012,8 @@ describe('Debug', () => {
},
})
- expect(screen.getByTestId('prompt-log-modal')).toBeInTheDocument()
- expect(screen.getByTestId('agent-log-modal')).toBeInTheDocument()
+ expect(screen.getByTestId('prompt-log-modal'))!.toBeInTheDocument()
+ expect(screen.getByTestId('agent-log-modal'))!.toBeInTheDocument()
})
it('should close prompt and agent log modals in multiple-model mode', () => {
diff --git a/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/context-provider.spec.tsx b/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/context-provider.spec.tsx
index 5608f4c5a2..6ed016ecc7 100644
--- a/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/context-provider.spec.tsx
+++ b/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/context-provider.spec.tsx
@@ -8,7 +8,7 @@ const ContextConsumer = () => {
{value.multipleModelConfigs.length}
-
+
{String(value.checkCanSend?.())}
)
@@ -32,7 +32,7 @@ describe('DebugWithMultipleModelContextProvider', () => {
,
)
- expect(screen.getByText('1')).toBeInTheDocument()
- expect(screen.getByText('true')).toBeInTheDocument()
+ expect(screen.getByText('1'))!.toBeInTheDocument()
+ expect(screen.getByText('true'))!.toBeInTheDocument()
})
})
diff --git a/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/debug-item.spec.tsx b/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/debug-item.spec.tsx
index 19c5fbf3fa..747ea4efcb 100644
--- a/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/debug-item.spec.tsx
+++ b/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/debug-item.spec.tsx
@@ -141,8 +141,8 @@ describe('DebugItem', () => {
it('should render with basic props', () => {
renderComponent()
- expect(screen.getByTestId('model-parameter-trigger')).toBeInTheDocument()
- expect(screen.getByTestId('dropdown')).toBeInTheDocument()
+ expect(screen.getByTestId('model-parameter-trigger'))!.toBeInTheDocument()
+ expect(screen.getByTestId('dropdown'))!.toBeInTheDocument()
})
it('should display correct index number', () => {
@@ -170,7 +170,7 @@ describe('DebugItem', () => {
})
const wrapper = container.firstChild as HTMLElement
- expect(wrapper).toHaveClass('custom-class')
+ expect(wrapper)!.toHaveClass('custom-class')
expect(wrapper.style.backgroundColor).toBe('red')
})
@@ -193,7 +193,7 @@ describe('DebugItem', () => {
renderComponent()
- expect(screen.getByTestId('chat-item')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-item'))!.toBeInTheDocument()
expect(screen.queryByTestId('text-generation-item')).not.toBeInTheDocument()
})
@@ -207,7 +207,7 @@ describe('DebugItem', () => {
renderComponent()
- expect(screen.getByTestId('chat-item')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-item'))!.toBeInTheDocument()
})
it('should not render ChatItem when model is not active', () => {
@@ -261,7 +261,7 @@ describe('DebugItem', () => {
renderComponent()
- expect(screen.getByTestId('text-generation-item')).toBeInTheDocument()
+ expect(screen.getByTestId('text-generation-item'))!.toBeInTheDocument()
expect(screen.queryByTestId('chat-item')).not.toBeInTheDocument()
})
@@ -502,7 +502,7 @@ describe('DebugItem', () => {
expect.arrayContaining([
models[0],
models[1],
- expect.objectContaining({ model: models[1].model }),
+ expect.objectContaining({ model: models[1]!.model }),
models[2],
]),
)
@@ -545,6 +545,37 @@ describe('DebugItem', () => {
renderComponent()
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
+ // When provider/model doesn't match, ChatItem won't render
// When provider/model doesn't match, ChatItem won't render
expect(screen.queryByTestId('chat-item')).not.toBeInTheDocument()
})
diff --git a/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/index.spec.tsx b/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/index.spec.tsx
index f1ee03ebbc..bd12ff2c14 100644
--- a/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/index.spec.tsx
+++ b/web/app/components/app/configuration/debug/debug-with-multiple-model/__tests__/index.spec.tsx
@@ -194,13 +194,13 @@ describe('DebugWithMultipleModel', () => {
it('should handle empty multipleModelConfigs array', () => {
renderComponent({ multipleModelConfigs: [] })
expect(screen.queryByTestId('debug-item')).not.toBeInTheDocument()
- expect(screen.getByTestId('chat-input-area')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-input-area'))!.toBeInTheDocument()
})
it('should handle model config with missing required fields', () => {
const incompleteConfig = { id: 'incomplete' } as ModelAndParameter
renderComponent({ multipleModelConfigs: [incompleteConfig] })
- expect(screen.getByTestId('debug-item')).toBeInTheDocument()
+ expect(screen.getByTestId('debug-item'))!.toBeInTheDocument()
})
it('should handle more than 4 model configs', () => {
@@ -257,7 +257,8 @@ describe('DebugWithMultipleModel', () => {
renderComponent()
// Should still render but handle gracefully
- expect(screen.getByTestId('chat-input-area')).toBeInTheDocument()
+ // Should still render but handle gracefully
+ expect(screen.getByTestId('chat-input-area'))!.toBeInTheDocument()
expect(capturedChatInputProps?.inputsForm).toHaveLength(3)
})
})
@@ -287,7 +288,7 @@ describe('DebugWithMultipleModel', () => {
rerender(
)
const items = screen.getAllByTestId('debug-item')
- expect(items[0]).toHaveAttribute('data-model-id', 'model-2')
+ expect(items[0])!.toHaveAttribute('data-model-id', 'model-2')
})
})
@@ -296,14 +297,14 @@ describe('DebugWithMultipleModel', () => {
renderComponent()
const chatInput = screen.getByTestId('chat-input-area')
- expect(chatInput).toBeInTheDocument()
+ expect(chatInput)!.toBeInTheDocument()
// Check for button accessibility
const sendButton = screen.getByRole('button', { name: /send/i })
- expect(sendButton).toBeInTheDocument()
+ expect(sendButton)!.toBeInTheDocument()
const featureButton = screen.getByRole('button', { name: /feature/i })
- expect(featureButton).toBeInTheDocument()
+ expect(featureButton)!.toBeInTheDocument()
})
it('should apply ARIA attributes correctly', () => {
@@ -312,8 +313,8 @@ describe('DebugWithMultipleModel', () => {
// Debug items should be identifiable
const debugItem = screen.getByTestId('debug-item')
- expect(debugItem).toBeInTheDocument()
- expect(debugItem).toHaveAttribute('data-model-id')
+ expect(debugItem)!.toBeInTheDocument()
+ expect(debugItem)!.toHaveAttribute('data-model-id')
})
})
@@ -422,7 +423,8 @@ describe('DebugWithMultipleModel', () => {
fireEvent.click(screen.getByRole('button', { name: /feature/i }))
// Assert
- expect(screen.getByTestId('chat-input-area')).toBeInTheDocument()
+ // Assert
+ expect(screen.getByTestId('chat-input-area'))!.toBeInTheDocument()
expect(capturedChatInputProps?.inputs).toEqual({ audience: 'engineers' })
expect(capturedChatInputProps?.inputsForm).toEqual([
expect.objectContaining({ label: 'City', variable: 'city', hide: false, required: true }),
@@ -446,7 +448,8 @@ describe('DebugWithMultipleModel', () => {
renderComponent()
// Assert
- expect(screen.getByTestId('chat-input-area')).toBeInTheDocument()
+ // Assert
+ expect(screen.getByTestId('chat-input-area'))!.toBeInTheDocument()
})
it('should hide chat input when not in chat mode', () => {
@@ -459,6 +462,37 @@ describe('DebugWithMultipleModel', () => {
// Act
renderComponent({ multipleModelConfigs })
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
+ // Assert
// Assert
expect(screen.queryByTestId('chat-input-area')).not.toBeInTheDocument()
expect(screen.getAllByTestId('debug-item')).toHaveLength(1)
@@ -538,7 +572,8 @@ describe('DebugWithMultipleModel', () => {
expect(secondItems).toHaveLength(1)
// Check that the element still renders the same content
- expect(firstItems[0]).toHaveTextContent(secondItems[0].textContent || '')
+ // Check that the element still renders the same content
+ expect(firstItems[0])!.toHaveTextContent(secondItems[0]!.textContent || '')
})
it('should recalculate size and position when number of models changes', () => {
@@ -557,8 +592,8 @@ describe('DebugWithMultipleModel', () => {
)
const twoItems = screen.getAllByTestId('debug-item')
- expect(twoItems[0].style.width).toBe('calc(50% - 4px - 24px)')
- expect(twoItems[1].style.width).toBe('calc(50% - 4px - 24px)')
+ expect(twoItems[0]!.style.width).toBe('calc(50% - 4px - 24px)')
+ expect(twoItems[1]!.style.width).toBe('calc(50% - 4px - 24px)')
})
})
@@ -583,7 +618,7 @@ describe('DebugWithMultipleModel', () => {
expect(element.style.height).toBe('')
expect(element.style.transform).toBe(expectation.transform)
- expectation.classes?.forEach(cls => expect(element).toHaveClass(cls))
+ expectation.classes?.forEach(cls => expect(element)!.toHaveClass(cls))
}
it('should arrange items in two-column layout for two models', () => {
@@ -596,13 +631,13 @@ describe('DebugWithMultipleModel', () => {
// Assert
expect(items).toHaveLength(2)
- expectItemLayout(items[0], {
+ expectItemLayout(items[0]!, {
width: 'calc(50% - 4px - 24px)',
height: '100%',
transform: 'translateX(0) translateY(0)',
classes: ['mr-2'],
})
- expectItemLayout(items[1], {
+ expectItemLayout(items[1]!, {
width: 'calc(50% - 4px - 24px)',
height: '100%',
transform: 'translateX(calc(100% + 8px)) translateY(0)',
@@ -620,19 +655,19 @@ describe('DebugWithMultipleModel', () => {
// Assert
expect(items).toHaveLength(3)
- expectItemLayout(items[0], {
+ expectItemLayout(items[0]!, {
width: 'calc(33.3% - 5.33px - 16px)',
height: '100%',
transform: 'translateX(0) translateY(0)',
classes: ['mr-2'],
})
- expectItemLayout(items[1], {
+ expectItemLayout(items[1]!, {
width: 'calc(33.3% - 5.33px - 16px)',
height: '100%',
transform: 'translateX(calc(100% + 8px)) translateY(0)',
classes: ['mr-2'],
})
- expectItemLayout(items[2], {
+ expectItemLayout(items[2]!, {
width: 'calc(33.3% - 5.33px - 16px)',
height: '100%',
transform: 'translateX(calc(200% + 16px)) translateY(0)',
@@ -655,25 +690,25 @@ describe('DebugWithMultipleModel', () => {
// Assert
expect(items).toHaveLength(4)
- expectItemLayout(items[0], {
+ expectItemLayout(items[0]!, {
width: 'calc(50% - 4px - 24px)',
height: 'calc(50% - 4px)',
transform: 'translateX(0) translateY(0)',
classes: ['mr-2', 'mb-2'],
})
- expectItemLayout(items[1], {
+ expectItemLayout(items[1]!, {
width: 'calc(50% - 4px - 24px)',
height: 'calc(50% - 4px)',
transform: 'translateX(calc(100% + 8px)) translateY(0)',
classes: ['mb-2'],
})
- expectItemLayout(items[2], {
+ expectItemLayout(items[2]!, {
width: 'calc(50% - 4px - 24px)',
height: 'calc(50% - 4px)',
transform: 'translateX(0) translateY(calc(100% + 8px))',
classes: ['mr-2'],
})
- expectItemLayout(items[3], {
+ expectItemLayout(items[3]!, {
width: 'calc(50% - 4px - 24px)',
height: 'calc(50% - 4px)',
transform: 'translateX(calc(100% + 8px)) translateY(calc(100% + 8px))',
@@ -699,7 +734,7 @@ describe('DebugWithMultipleModel', () => {
it('should set scroll area height for chat modes', () => {
const { container } = renderComponent()
const scrollArea = container.querySelector('.relative.mb-3.grow.overflow-auto.px-6') as HTMLElement
- expect(scrollArea).toBeInTheDocument()
+ expect(scrollArea)!.toBeInTheDocument()
expect(scrollArea.style.height).toBe('calc(100% - 60px)')
})
diff --git a/web/app/components/app/configuration/debug/debug-with-multiple-model/model-parameter-trigger.tsx b/web/app/components/app/configuration/debug/debug-with-multiple-model/model-parameter-trigger.tsx
index 65e6067e1a..a870538edc 100644
--- a/web/app/components/app/configuration/debug/debug-with-multiple-model/model-parameter-trigger.tsx
+++ b/web/app/components/app/configuration/debug/debug-with-multiple-model/model-parameter-trigger.tsx
@@ -40,7 +40,7 @@ const ModelParameterTrigger: FC
= ({
const handleSelectModel = ({ modelId, provider }: { modelId: string, provider: string }) => {
const newModelConfigs = [...multipleModelConfigs]
newModelConfigs[index] = {
- ...newModelConfigs[index],
+ ...newModelConfigs[index]!,
model: modelId,
provider,
}
@@ -49,7 +49,7 @@ const ModelParameterTrigger: FC = ({
const handleParamsChange = (params: FormValue) => {
const newModelConfigs = [...multipleModelConfigs]
newModelConfigs[index] = {
- ...newModelConfigs[index],
+ ...newModelConfigs[index]!,
parameters: params,
}
onMultipleModelConfigsChange(true, newModelConfigs)
diff --git a/web/app/components/app/configuration/debug/debug-with-single-model/__tests__/index.spec.tsx b/web/app/components/app/configuration/debug/debug-with-single-model/__tests__/index.spec.tsx
index 0ab6e86c17..634dd6217f 100644
--- a/web/app/components/app/configuration/debug/debug-with-single-model/__tests__/index.spec.tsx
+++ b/web/app/components/app/configuration/debug/debug-with-single-model/__tests__/index.spec.tsx
@@ -591,9 +591,10 @@ describe('DebugWithSingleModel', () => {
render(} />)
// Verify Chat component is rendered
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
- expect(screen.getByTestId('chat-input')).toBeInTheDocument()
- expect(screen.getByTestId('send-button')).toBeInTheDocument()
+ // Verify Chat component is rendered
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
+ expect(screen.getByTestId('chat-input'))!.toBeInTheDocument()
+ expect(screen.getByTestId('send-button'))!.toBeInTheDocument()
})
it('should render with custom checkCanSend prop', () => {
@@ -601,7 +602,7 @@ describe('DebugWithSingleModel', () => {
render(} checkCanSend={checkCanSend} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
})
@@ -620,7 +621,7 @@ describe('DebugWithSingleModel', () => {
expect(mockSsePost).toHaveBeenCalled()
})
- expect(mockSsePost.mock.calls[0][0]).toBe('apps/test-app-id/chat-messages')
+ expect(mockSsePost.mock.calls[0]![0]).toBe('apps/test-app-id/chat-messages')
})
it('should prevent send when checkCanSend returns false', async () => {
@@ -666,7 +667,7 @@ describe('DebugWithSingleModel', () => {
expect(mockSsePost).toHaveBeenCalled()
})
- const body = mockSsePost.mock.calls[0][1].body
+ const body = mockSsePost.mock.calls[0]![1].body
expect(body.model_config.opening_statement).toBe('Hello!')
expect(body.model_config.suggested_questions).toEqual(['Q1'])
})
@@ -685,7 +686,7 @@ describe('DebugWithSingleModel', () => {
expect(mockSsePost).toHaveBeenCalled()
})
- const body = mockSsePost.mock.calls[0][1].body
+ const body = mockSsePost.mock.calls[0]![1].body
expect(body.model_config.opening_statement).toBe('')
expect(body.model_config.suggested_questions).toEqual([])
})
@@ -717,7 +718,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
it('should handle missing model in provider list', () => {
@@ -735,7 +736,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
})
@@ -759,7 +760,8 @@ describe('DebugWithSingleModel', () => {
render(} />)
// Component should render successfully with filtered variables
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ // Component should render successfully with filtered variables
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
it('should handle empty prompt variables', () => {
@@ -775,7 +777,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
})
@@ -784,7 +786,7 @@ describe('DebugWithSingleModel', () => {
it('should map tool icons from collection list', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
it('should handle empty tools list', () => {
@@ -802,7 +804,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
it('should handle missing collection for tool', () => {
@@ -829,7 +831,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
})
@@ -843,7 +845,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
it('should handle missing user profile', () => {
@@ -859,7 +861,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
it('should handle null completion params', () => {
@@ -870,7 +872,7 @@ describe('DebugWithSingleModel', () => {
render(} />)
- expect(screen.getByTestId('chat-component')).toBeInTheDocument()
+ expect(screen.getByTestId('chat-component'))!.toBeInTheDocument()
})
})
@@ -940,7 +942,7 @@ describe('DebugWithSingleModel', () => {
expect(mockSsePost).toHaveBeenCalled()
})
- const body = mockSsePost.mock.calls[0][1].body
+ const body = mockSsePost.mock.calls[0]![1].body
expect(body.files).toEqual([])
})
@@ -989,7 +991,7 @@ describe('DebugWithSingleModel', () => {
expect(mockSsePost).toHaveBeenCalled()
})
- const body = mockSsePost.mock.calls[0][1].body
+ const body = mockSsePost.mock.calls[0]![1].body
expect(body.files).toHaveLength(1)
})
})
diff --git a/web/app/components/app/configuration/hooks/__tests__/use-configuration-utils.spec.ts b/web/app/components/app/configuration/hooks/__tests__/use-configuration-utils.spec.ts
index b5d00bd573..c0353c233d 100644
--- a/web/app/components/app/configuration/hooks/__tests__/use-configuration-utils.spec.ts
+++ b/web/app/components/app/configuration/hooks/__tests__/use-configuration-utils.spec.ts
@@ -345,7 +345,7 @@ describe('useConfiguration utils', () => {
},
url: '/datasets',
})
- expect(state.collectionList[0].icon).toBe('/console/tool.svg')
+ expect(state.collectionList[0]!.icon).toBe('/console/tool.svg')
expect(state.promptMode).toBe('advanced')
expect(state.nextDataSets).toEqual([{ id: 'dataset-1', name: 'Dataset One' }])
expect(state.annotationConfig).toEqual(expect.objectContaining({
diff --git a/web/app/components/app/configuration/tools/__tests__/external-data-tool-modal-utils.spec.ts b/web/app/components/app/configuration/tools/__tests__/external-data-tool-modal-utils.spec.ts
index 5d55c303c5..03f77671d9 100644
--- a/web/app/components/app/configuration/tools/__tests__/external-data-tool-modal-utils.spec.ts
+++ b/web/app/components/app/configuration/tools/__tests__/external-data-tool-modal-utils.spec.ts
@@ -42,7 +42,7 @@ describe('external-data-tool-modal-utils', () => {
it('should build localized providers and default configs for code-based tools', () => {
const providers = buildProviders({
codeBasedExtensionList,
- locale: LanguagesSupported[1],
+ locale: LanguagesSupported[1]!,
t,
})
diff --git a/web/app/components/app/configuration/tools/external-data-tool-modal-utils.ts b/web/app/components/app/configuration/tools/external-data-tool-modal-utils.ts
index 702ebdb5f4..0a9d0724e4 100644
--- a/web/app/components/app/configuration/tools/external-data-tool-modal-utils.ts
+++ b/web/app/components/app/configuration/tools/external-data-tool-modal-utils.ts
@@ -114,8 +114,8 @@ export const getValidationError = ({
if (!systemTypes.includes(localeData.type as typeof systemTypes[number]) && currentProvider?.form_schema) {
for (let i = 0; i < currentProvider.form_schema.length; i++) {
const form = currentProvider.form_schema[i]
- if (!localeData.config?.[form.variable] && form.required) {
- return t('errorMessage.valueOfVarRequired', { ns: 'appDebug', key: locale === LanguagesSupported[1] ? form.label['zh-Hans'] : form.label['en-US'] })
+ if (!localeData.config?.[form!.variable] && form!.required) {
+ return t('errorMessage.valueOfVarRequired', { ns: 'appDebug', key: locale === LanguagesSupported[1] ? form!.label['zh-Hans'] : form!.label['en-US'] })
}
}
}
diff --git a/web/app/components/app/configuration/utils.ts b/web/app/components/app/configuration/utils.ts
index 3f06c863a4..18bc144b5e 100644
--- a/web/app/components/app/configuration/utils.ts
+++ b/web/app/components/app/configuration/utils.ts
@@ -40,7 +40,7 @@ export const buildConfigurationFeaturesData = (
},
enabled: !!(modelConfig.file_upload?.enabled || modelConfig.file_upload?.image?.enabled),
allowed_file_types: modelConfig.file_upload?.allowed_file_types || [],
- allowed_file_extensions: modelConfig.file_upload?.allowed_file_extensions || [...FILE_EXTS[SupportUploadFileTypes.image], ...FILE_EXTS[SupportUploadFileTypes.video]].map(ext => `.${ext}`),
+ allowed_file_extensions: modelConfig.file_upload?.allowed_file_extensions || [...(FILE_EXTS[SupportUploadFileTypes.image] ?? []), ...(FILE_EXTS[SupportUploadFileTypes.video] ?? [])].map(ext => `.${ext}`),
allowed_file_upload_methods: modelConfig.file_upload?.allowed_file_upload_methods || modelConfig.file_upload?.image?.transfer_methods || ['local_file', 'remote_url'],
number_limits: modelConfig.file_upload?.number_limits || modelConfig.file_upload?.image?.number_limits || 3,
fileUploadConfig: fileUploadConfigResponse,
diff --git a/web/app/components/app/create-app-dialog/__tests__/index.spec.tsx b/web/app/components/app/create-app-dialog/__tests__/index.spec.tsx
index 6fa30ded24..cc59ce7456 100644
--- a/web/app/components/app/create-app-dialog/__tests__/index.spec.tsx
+++ b/web/app/components/app/create-app-dialog/__tests__/index.spec.tsx
@@ -53,6 +53,37 @@ describe('CreateAppTemplateDialog', () => {
it('should not render when show is false', () => {
render()
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
+ // FullScreenModal should not render any content when open is false
// FullScreenModal should not render any content when open is false
expect(screen.queryByRole('dialog')).not.toBeInTheDocument()
})
@@ -61,14 +92,15 @@ describe('CreateAppTemplateDialog', () => {
render()
// FullScreenModal renders with role="dialog"
- expect(screen.getByRole('dialog')).toBeInTheDocument()
- expect(screen.getByTestId('app-list')).toBeInTheDocument()
+ // FullScreenModal renders with role="dialog"
+ expect(screen.getByRole('dialog'))!.toBeInTheDocument()
+ expect(screen.getByTestId('app-list'))!.toBeInTheDocument()
})
it('should render create from blank button when onCreateFromBlank is provided', () => {
render()
- expect(screen.getByTestId('create-from-blank')).toBeInTheDocument()
+ expect(screen.getByTestId('create-from-blank'))!.toBeInTheDocument()
})
it('should not render create from blank button when onCreateFromBlank is not provided', () => {
@@ -87,7 +119,7 @@ describe('CreateAppTemplateDialog', () => {
expect(screen.queryByRole('dialog')).not.toBeInTheDocument()
rerender()
- expect(screen.getByRole('dialog')).toBeInTheDocument()
+ expect(screen.getByRole('dialog'))!.toBeInTheDocument()
})
it('should pass closable prop to FullScreenModal', () => {
@@ -97,8 +129,8 @@ describe('CreateAppTemplateDialog', () => {
// Verify that the modal has the proper dialog structure
const dialog = screen.getByRole('dialog')
- expect(dialog).toBeInTheDocument()
- expect(dialog).toHaveAttribute('aria-modal', 'true')
+ expect(dialog)!.toBeInTheDocument()
+ expect(dialog)!.toHaveAttribute('aria-modal', 'true')
})
})
@@ -109,11 +141,12 @@ describe('CreateAppTemplateDialog', () => {
// Test that the modal is rendered
const dialog = screen.getByRole('dialog')
- expect(dialog).toBeInTheDocument()
+ expect(dialog)!.toBeInTheDocument()
// Test that AppList component renders (child component interactions)
- expect(screen.getByTestId('app-list')).toBeInTheDocument()
- expect(screen.getByTestId('app-list-success')).toBeInTheDocument()
+ // Test that AppList component renders (child component interactions)
+ expect(screen.getByTestId('app-list'))!.toBeInTheDocument()
+ expect(screen.getByTestId('app-list-success'))!.toBeInTheDocument()
})
it('should call both onSuccess and onClose when app list success is triggered', () => {
@@ -204,8 +237,8 @@ describe('CreateAppTemplateDialog', () => {
// Verify that useKeyPress was called with a function
const calls = mockUseKeyPress.mock.calls
expect(calls.length).toBeGreaterThan(0)
- expect(calls[0][0]).toBe('esc')
- expect(typeof calls[0][1]).toBe('function')
+ expect(calls[0]![0]).toBe('esc')
+ expect(typeof calls[0]![1]).toBe('function')
})
})
@@ -243,7 +276,7 @@ describe('CreateAppTemplateDialog', () => {
// Test show state
render()
- expect(screen.getByRole('dialog')).toBeInTheDocument()
+ expect(screen.getByRole('dialog'))!.toBeInTheDocument()
// Test hide state
render()
@@ -258,7 +291,7 @@ describe('CreateAppTemplateDialog', () => {
render()
}).not.toThrow()
- expect(screen.getByTestId('app-list')).toBeInTheDocument()
+ expect(screen.getByTestId('app-list'))!.toBeInTheDocument()
expect(screen.queryByTestId('create-from-blank')).not.toBeInTheDocument()
})
@@ -273,8 +306,8 @@ describe('CreateAppTemplateDialog', () => {
render()
}).not.toThrow()
- expect(screen.getByRole('dialog')).toBeInTheDocument()
- expect(screen.getByTestId('app-list')).toBeInTheDocument()
+ expect(screen.getByRole('dialog'))!.toBeInTheDocument()
+ expect(screen.getByTestId('app-list'))!.toBeInTheDocument()
})
})
})
diff --git a/web/app/components/app/create-app-dialog/app-list/__tests__/index.spec.tsx b/web/app/components/app/create-app-dialog/app-list/__tests__/index.spec.tsx
index a319bb58f7..34024da54c 100644
--- a/web/app/components/app/create-app-dialog/app-list/__tests__/index.spec.tsx
+++ b/web/app/components/app/create-app-dialog/app-list/__tests__/index.spec.tsx
@@ -183,15 +183,15 @@ describe('Apps', () => {
render()
expect(screen.getAllByTestId('app-card')).toHaveLength(6)
- expect(screen.getByText('Alpha')).toBeInTheDocument()
- expect(screen.getByText('Bravo')).toBeInTheDocument()
+ expect(screen.getByText('Alpha'))!.toBeInTheDocument()
+ expect(screen.getByText('Bravo'))!.toBeInTheDocument()
})
it('opens create modal when a template card is clicked', () => {
render()
- fireEvent.click(screen.getAllByTestId('app-card')[0])
- expect(screen.getByTestId('create-from-template-modal')).toBeInTheDocument()
+ fireEvent.click(screen.getAllByTestId('app-card')[0]!)
+ expect(screen.getByTestId('create-from-template-modal'))!.toBeInTheDocument()
})
it('shows no template message when list is empty', () => {
@@ -202,8 +202,8 @@ describe('Apps', () => {
render()
- expect(screen.getByText('app.newApp.noTemplateFound')).toBeInTheDocument()
- expect(screen.getByText('app.newApp.noTemplateFoundTip')).toBeInTheDocument()
+ expect(screen.getByText('app.newApp.noTemplateFound'))!.toBeInTheDocument()
+ expect(screen.getByText('app.newApp.noTemplateFoundTip'))!.toBeInTheDocument()
})
it('filters templates by keyword and selected app type', async () => {
@@ -214,7 +214,7 @@ describe('Apps', () => {
})
await waitFor(() => {
- expect(screen.getByText('Bravo')).toBeInTheDocument()
+ expect(screen.getByText('Bravo'))!.toBeInTheDocument()
expect(screen.queryByText('Alpha')).not.toBeInTheDocument()
})
@@ -224,8 +224,8 @@ describe('Apps', () => {
fireEvent.click(screen.getByTestId('type-selector-chat'))
await waitFor(() => {
- expect(screen.getByText('Alpha')).toBeInTheDocument()
- expect(screen.getByText('Bravo')).toBeInTheDocument()
+ expect(screen.getByText('Alpha'))!.toBeInTheDocument()
+ expect(screen.getByText('Bravo'))!.toBeInTheDocument()
expect(screen.queryByText('Charlie')).not.toBeInTheDocument()
})
})
@@ -235,7 +235,7 @@ describe('Apps', () => {
render()
- fireEvent.click(screen.getAllByTestId('app-card')[0])
+ fireEvent.click(screen.getAllByTestId('app-card')[0]!)
fireEvent.click(screen.getByTestId('confirm-create'))
await waitFor(() => {
@@ -264,7 +264,7 @@ describe('Apps', () => {
render()
- fireEvent.click(screen.getAllByTestId('app-card')[0])
+ fireEvent.click(screen.getAllByTestId('app-card')[0]!)
fireEvent.click(screen.getByTestId('confirm-create'))
await waitFor(() => {
@@ -290,7 +290,7 @@ describe('Apps', () => {
render()
- expect(screen.getByRole('status')).toBeInTheDocument()
+ expect(screen.getByRole('status'))!.toBeInTheDocument()
})
it('should handle an undefined template payload by falling back to the empty state', () => {
@@ -301,7 +301,7 @@ describe('Apps', () => {
render()
- expect(screen.getByText('app.newApp.noTemplateFound')).toBeInTheDocument()
+ expect(screen.getByText('app.newApp.noTemplateFound'))!.toBeInTheDocument()
})
it('should filter templates by category and the remaining app modes', async () => {
@@ -309,8 +309,8 @@ describe('Apps', () => {
fireEvent.click(screen.getByText('Cat C'))
expect(screen.queryByText('Alpha')).not.toBeInTheDocument()
- expect(screen.getByText('Echo')).toBeInTheDocument()
- expect(screen.getByText('Foxtrot')).toBeInTheDocument()
+ expect(screen.getByText('Echo'))!.toBeInTheDocument()
+ expect(screen.getByText('Foxtrot'))!.toBeInTheDocument()
fireEvent.click(screen.getByTestId('type-selector-advanced'))
await waitFor(() => {
@@ -322,13 +322,13 @@ describe('Apps', () => {
fireEvent.click(screen.getByText('Cat C'))
fireEvent.click(screen.getByTestId('type-selector-agent'))
await waitFor(() => {
- expect(screen.getByText('Echo')).toBeInTheDocument()
+ expect(screen.getByText('Echo'))!.toBeInTheDocument()
expect(screen.queryByText('Foxtrot')).not.toBeInTheDocument()
})
fireEvent.click(screen.getByTestId('type-selector-workflow'))
await waitFor(() => {
- expect(screen.getByText('Foxtrot')).toBeInTheDocument()
+ expect(screen.getByText('Foxtrot'))!.toBeInTheDocument()
expect(screen.queryByText('Echo')).not.toBeInTheDocument()
})
@@ -356,11 +356,11 @@ describe('Apps', () => {
})
await waitFor(() => {
- expect(screen.getByText('Cat A')).toBeInTheDocument()
+ expect(screen.getByText('Cat A'))!.toBeInTheDocument()
})
- fireEvent.click(screen.getAllByTestId('app-card')[0])
- expect(screen.getByTestId('create-from-template-modal')).toBeInTheDocument()
+ fireEvent.click(screen.getAllByTestId('app-card')[0]!)
+ expect(screen.getByTestId('create-from-template-modal'))!.toBeInTheDocument()
fireEvent.click(screen.getByTestId('hide-create-modal'))
diff --git a/web/app/components/app/create-from-dsl-modal/__tests__/index.spec.tsx b/web/app/components/app/create-from-dsl-modal/__tests__/index.spec.tsx
index e106cc7eb3..40c32b7dd6 100644
--- a/web/app/components/app/create-from-dsl-modal/__tests__/index.spec.tsx
+++ b/web/app/components/app/create-from-dsl-modal/__tests__/index.spec.tsx
@@ -137,10 +137,10 @@ describe('CreateFromDSLModal', () => {
/>,
)
- expect(screen.getByText('importFromDSL')).toBeInTheDocument()
+ expect(screen.getByText('importFromDSL'))!.toBeInTheDocument()
await waitFor(() => {
- expect(screen.getByText('demo.yml')).toBeInTheDocument()
+ expect(screen.getByText('demo.yml'))!.toBeInTheDocument()
})
})
@@ -159,7 +159,7 @@ describe('CreateFromDSLModal', () => {
await act(async () => {
fireEvent.click(screen.getByText('importFromDSLUrl'))
})
- expect(screen.getByPlaceholderText('importFromDSLUrlPlaceholder')).toBeInTheDocument()
+ expect(screen.getByPlaceholderText('importFromDSLUrlPlaceholder'))!.toBeInTheDocument()
const closeTrigger = screen.getByText('importFromDSL').parentElement?.querySelector('.cursor-pointer.items-center') as HTMLElement
fireEvent.click(closeTrigger)
@@ -222,7 +222,7 @@ describe('CreateFromDSLModal', () => {
)
await waitFor(() => {
- expect(screen.getByText('demo.yml')).toBeInTheDocument()
+ expect(screen.getByText('demo.yml'))!.toBeInTheDocument()
})
await act(async () => {
@@ -245,7 +245,7 @@ describe('CreateFromDSLModal', () => {
)
await waitFor(() => {
- expect(screen.getByText('demo.yml')).toBeInTheDocument()
+ expect(screen.getByText('demo.yml'))!.toBeInTheDocument()
})
const removeButton = screen.getByText('demo.yml').closest('.group')?.querySelector('button') as HTMLButtonElement
@@ -255,7 +255,7 @@ describe('CreateFromDSLModal', () => {
await waitFor(() => {
expect(screen.queryByText('demo.yml')).not.toBeInTheDocument()
- expect(getCreateButton()).toBeDisabled()
+ expect(getCreateButton())!.toBeDisabled()
})
ahooksMocks.handlers.findLast(item => Array.isArray(item.keys))?.handler()
@@ -294,10 +294,10 @@ describe('CreateFromDSLModal', () => {
vi.advanceTimersByTime(300)
})
- expect(screen.getAllByText('newApp.appCreateDSLErrorTitle')[0]).toBeInTheDocument()
+ expect(screen.getAllByText('newApp.appCreateDSLErrorTitle')[0])!.toBeInTheDocument()
await act(async () => {
- fireEvent.click(screen.getAllByRole('button', { name: 'newApp.Confirm' })[0])
+ fireEvent.click(screen.getAllByRole('button', { name: 'newApp.Confirm' })[0]!)
})
expect(mockImportDSLConfirm).toHaveBeenCalledWith({
@@ -386,7 +386,7 @@ describe('CreateFromDSLModal', () => {
/>,
)
- expect(screen.getByText('apps-full')).toBeInTheDocument()
+ expect(screen.getByText('apps-full'))!.toBeInTheDocument()
ahooksMocks.handlers.findLast(item => Array.isArray(item.keys))?.handler()
expect(mockImportDSL).toHaveBeenCalledTimes(1)
})
@@ -461,12 +461,12 @@ describe('CreateFromDSLModal', () => {
vi.advanceTimersByTime(300)
})
await act(async () => {
- fireEvent.click(screen.getAllByRole('button', { name: 'newApp.Confirm' })[0])
+ fireEvent.click(screen.getAllByRole('button', { name: 'newApp.Confirm' })[0]!)
})
expect(toastMocks.error).toHaveBeenCalledWith('newApp.appCreateFailed')
await act(async () => {
- fireEvent.click(screen.getAllByRole('button', { name: 'newApp.Confirm' })[0])
+ fireEvent.click(screen.getAllByRole('button', { name: 'newApp.Confirm' })[0]!)
})
expect(toastMocks.error).toHaveBeenCalledTimes(2)
})
diff --git a/web/app/components/app/log/__tests__/filter.spec.tsx b/web/app/components/app/log/__tests__/filter.spec.tsx
index 4c377ecde7..ec1a0183be 100644
--- a/web/app/components/app/log/__tests__/filter.spec.tsx
+++ b/web/app/components/app/log/__tests__/filter.spec.tsx
@@ -77,7 +77,7 @@ describe('Filter', () => {
it('should render filter components', () => {
render()
- expect(screen.getByPlaceholderText('operation.search')).toBeInTheDocument()
+ expect(screen.getByPlaceholderText('operation.search'))!.toBeInTheDocument()
})
it('should return null when loading', () => {
@@ -89,13 +89,13 @@ describe('Filter', () => {
it('should render sort component in chat mode', () => {
render()
- expect(screen.getByPlaceholderText('operation.search')).toBeInTheDocument()
+ expect(screen.getByPlaceholderText('operation.search'))!.toBeInTheDocument()
})
it('should not render sort component when not in chat mode', () => {
render()
- expect(screen.getByPlaceholderText('operation.search')).toBeInTheDocument()
+ expect(screen.getByPlaceholderText('operation.search'))!.toBeInTheDocument()
})
})
@@ -105,23 +105,23 @@ describe('Filter', () => {
})
it('should have today period with value 0', () => {
- expect(TIME_PERIOD_MAPPING['1'].value).toBe(0)
- expect(TIME_PERIOD_MAPPING['1'].name).toBe('today')
+ expect(TIME_PERIOD_MAPPING['1']!.value).toBe(0)
+ expect(TIME_PERIOD_MAPPING['1']!.name).toBe('today')
})
it('should have last7days period with value 7', () => {
- expect(TIME_PERIOD_MAPPING['2'].value).toBe(7)
- expect(TIME_PERIOD_MAPPING['2'].name).toBe('last7days')
+ expect(TIME_PERIOD_MAPPING['2']!.value).toBe(7)
+ expect(TIME_PERIOD_MAPPING['2']!.name).toBe('last7days')
})
it('should have last4weeks period with value 28', () => {
- expect(TIME_PERIOD_MAPPING['3'].value).toBe(28)
- expect(TIME_PERIOD_MAPPING['3'].name).toBe('last4weeks')
+ expect(TIME_PERIOD_MAPPING['3']!.value).toBe(28)
+ expect(TIME_PERIOD_MAPPING['3']!.name).toBe('last4weeks')
})
it('should have allTime period with value -1', () => {
- expect(TIME_PERIOD_MAPPING['9'].value).toBe(-1)
- expect(TIME_PERIOD_MAPPING['9'].name).toBe('allTime')
+ expect(TIME_PERIOD_MAPPING['9']!.value).toBe(-1)
+ expect(TIME_PERIOD_MAPPING['9']!.name).toBe('allTime')
})
})
@@ -158,25 +158,25 @@ describe('Filter', () => {
it('should update and clear period, annotation, and sort filters', () => {
render()
- fireEvent.click(screen.getAllByText('select-9')[0])
+ fireEvent.click(screen.getAllByText('select-9')[0]!)
expect(mockSetQueryParams).toHaveBeenCalledWith({
...defaultQueryParams,
period: '9',
})
- fireEvent.click(screen.getAllByText('clear-chip')[0])
+ fireEvent.click(screen.getAllByText('clear-chip')[0]!)
expect(mockSetQueryParams).toHaveBeenCalledWith({
...defaultQueryParams,
period: '9',
})
- fireEvent.click(screen.getAllByText('select-not_annotated')[0])
+ fireEvent.click(screen.getAllByText('select-not_annotated')[0]!)
expect(mockSetQueryParams).toHaveBeenCalledWith({
...defaultQueryParams,
annotation_status: 'not_annotated',
})
- fireEvent.click(screen.getAllByText('clear-chip')[1])
+ fireEvent.click(screen.getAllByText('clear-chip')[1]!)
expect(mockSetQueryParams).toHaveBeenCalledWith({
...defaultQueryParams,
annotation_status: 'all',
@@ -200,7 +200,8 @@ describe('Filter', () => {
render()
// Period '1' maps to 'today' in TIME_PERIOD_MAPPING
- expect(screen.getByText('filter.period.today')).toBeInTheDocument()
+ // Period '1' maps to 'today' in TIME_PERIOD_MAPPING
+ expect(screen.getByText('filter.period.today'))!.toBeInTheDocument()
})
it('should display "last7days" when period is set to 2', () => {
@@ -211,14 +212,15 @@ describe('Filter', () => {
render()
- expect(screen.getByText('filter.period.last7days')).toBeInTheDocument()
+ expect(screen.getByText('filter.period.last7days'))!.toBeInTheDocument()
})
it('should display "allTime" when period is set to 9', () => {
render()
// Default period is '9' which maps to 'allTime'
- expect(screen.getByText('filter.period.allTime')).toBeInTheDocument()
+ // Default period is '9' which maps to 'allTime'
+ expect(screen.getByText('filter.period.allTime'))!.toBeInTheDocument()
})
it('should display annotated status with count when annotation_status is annotated', () => {
@@ -230,7 +232,8 @@ describe('Filter', () => {
render()
// The mock returns count: 10, so the text should include the count
- expect(screen.getByText('filter.annotation.annotated (10)')).toBeInTheDocument()
+ // The mock returns count: 10, so the text should include the count
+ expect(screen.getByText('filter.annotation.annotated (10)'))!.toBeInTheDocument()
})
it('should display not_annotated status when annotation_status is not_annotated', () => {
@@ -241,14 +244,15 @@ describe('Filter', () => {
render()
- expect(screen.getByText('filter.annotation.not_annotated')).toBeInTheDocument()
+ expect(screen.getByText('filter.annotation.not_annotated'))!.toBeInTheDocument()
})
it('should display all annotation status when annotation_status is all', () => {
render()
// Default annotation_status is 'all'
- expect(screen.getByText('filter.annotation.all')).toBeInTheDocument()
+ // Default annotation_status is 'all'
+ expect(screen.getByText('filter.annotation.all'))!.toBeInTheDocument()
})
it('should return null when annotation count data is unavailable', () => {
@@ -269,7 +273,7 @@ describe('Filter', () => {
render()
- expect(screen.getByPlaceholderText('operation.search')).toBeInTheDocument()
+ expect(screen.getByPlaceholderText('operation.search'))!.toBeInTheDocument()
})
it('should handle descending sort order', () => {
@@ -281,7 +285,7 @@ describe('Filter', () => {
render()
- expect(screen.getByPlaceholderText('operation.search')).toBeInTheDocument()
+ expect(screen.getByPlaceholderText('operation.search'))!.toBeInTheDocument()
})
})
})
diff --git a/web/app/components/app/log/__tests__/list-utils.spec.ts b/web/app/components/app/log/__tests__/list-utils.spec.ts
index 245ff0567e..deaaa01591 100644
--- a/web/app/components/app/log/__tests__/list-utils.spec.ts
+++ b/web/app/components/app/log/__tests__/list-utils.spec.ts
@@ -150,12 +150,12 @@ describe('log list utils', () => {
it('should update annotation state helpers', () => {
const items = createChatItems()
- expect(applyAnnotationEdited(items, 'updated question', 'updated answer', 1)[0].content).toBe('updated question')
- expect(applyAnnotationAdded(items, 'annotation-1', 'Dify', 'question', 'answer', 1)[1].annotation).toEqual(expect.objectContaining({
+ expect(applyAnnotationEdited(items, 'updated question', 'updated answer', 1)[0]!.content).toBe('updated question')
+ expect(applyAnnotationAdded(items, 'annotation-1', 'Dify', 'question', 'answer', 1)[1]!.annotation).toEqual(expect.objectContaining({
id: 'annotation-1',
authorName: 'Dify',
}))
- expect(applyAnnotationRemoved(items, 1)[1].annotation).toBeUndefined()
+ expect(applyAnnotationRemoved(items, 1)[1]!.annotation).toBeUndefined()
})
it('should derive urls, scroll thresholds, row values, and detail metadata', () => {
diff --git a/web/app/components/app/log/index.tsx b/web/app/components/app/log/index.tsx
index c30cf6cf51..411ec8694d 100644
--- a/web/app/components/app/log/index.tsx
+++ b/web/app/components/app/log/index.tsx
@@ -78,7 +78,7 @@ const Logs: FC = ({ appDetail }) => {
limit,
...((debouncedQueryParams.period !== '9')
? {
- start: dayjs().subtract(TIME_PERIOD_MAPPING[debouncedQueryParams.period].value, 'day').startOf('day').format('YYYY-MM-DD HH:mm'),
+ start: dayjs().subtract(TIME_PERIOD_MAPPING[debouncedQueryParams.period]!.value, 'day').startOf('day').format('YYYY-MM-DD HH:mm'),
end: dayjs().endOf('day').format('YYYY-MM-DD HH:mm'),
}
: {}),
diff --git a/web/app/components/app/overview/__tests__/app-chart-utils.spec.ts b/web/app/components/app/overview/__tests__/app-chart-utils.spec.ts
index a087df241a..5064af5f36 100644
--- a/web/app/components/app/overview/__tests__/app-chart-utils.spec.ts
+++ b/web/app/components/app/overview/__tests__/app-chart-utils.spec.ts
@@ -97,7 +97,7 @@ describe('app-chart-utils', () => {
expect(dataset.dimensions).toEqual(['date', 'count'])
expect(dataset.source).toHaveLength(2)
expect(yAxis.max).toBe(100)
- expect(series[0].lineStyle.color).toBe('rgba(6, 148, 162, 1)')
+ expect(series[0]!.lineStyle.color).toBe('rgba(6, 148, 162, 1)')
})
it('should build token-aware tooltip content and split-line intervals for cost charts', () => {
@@ -112,11 +112,11 @@ describe('app-chart-utils', () => {
})
const xAxis = options.xAxis as Array>
- const formatter = xAxis[0].axisLabel.formatter as (value: string) => string
- const outerInterval = xAxis[0].splitLine.interval as (index: number) => boolean
- const innerInterval = xAxis[1].splitLine.interval as (_index: number, value: string) => boolean
+ const formatter = xAxis[0]!.axisLabel.formatter as (value: string) => string
+ const outerInterval = xAxis[0]!.splitLine.interval as (index: number) => boolean
+ const innerInterval = xAxis[1]!.splitLine.interval as (_index: number, value: string) => boolean
const series = options.series as Array>
- const tooltipFormatter = series[0].tooltip.formatter as (params: { name: string, data: { total_cost: number, total_price: string } }) => string
+ const tooltipFormatter = series[0]!.tooltip.formatter as (params: { name: string, data: { total_cost: number, total_price: string } }) => string
expect(formatter('Jan 2, 2024')).toBe('Jan 2, 2024')
expect(outerInterval(0)).toBe(true)
diff --git a/web/app/components/app/overview/__tests__/app-chart.spec.tsx b/web/app/components/app/overview/__tests__/app-chart.spec.tsx
index ccd59a8f4a..d6aa241405 100644
--- a/web/app/components/app/overview/__tests__/app-chart.spec.tsx
+++ b/web/app/components/app/overview/__tests__/app-chart.spec.tsx
@@ -58,10 +58,10 @@ describe('app-chart', () => {
/>,
)
- expect(screen.getByText('Cost title')).toBeInTheDocument()
- expect(screen.getByText('300')).toBeInTheDocument()
- expect(screen.getByText(/\$3\.7500/)).toBeInTheDocument()
- expect(screen.getByTestId('echarts')).toBeInTheDocument()
+ expect(screen.getByText('Cost title'))!.toBeInTheDocument()
+ expect(screen.getByText('300'))!.toBeInTheDocument()
+ expect(screen.getByText(/\$3\.7500/))!.toBeInTheDocument()
+ expect(screen.getByTestId('echarts'))!.toBeInTheDocument()
})
})
@@ -85,10 +85,10 @@ describe('app-chart', () => {
/>,
)
- expect(screen.getByText('analysis.totalMessages.title')).toBeInTheDocument()
- expect(screen.getByTestId('echarts')).toBeInTheDocument()
+ expect(screen.getByText('analysis.totalMessages.title'))!.toBeInTheDocument()
+ expect(screen.getByTestId('echarts'))!.toBeInTheDocument()
- const options = reactEChartsMock.mock.calls[0][0] as {
+ const options = reactEChartsMock.mock.calls[0]![0] as {
dataset: { source: Array> }
yAxis: { max: number }
}
diff --git a/web/app/components/app/overview/embedded/__tests__/index.spec.tsx b/web/app/components/app/overview/embedded/__tests__/index.spec.tsx
index 9b8a2ad8c4..0a843c26fd 100644
--- a/web/app/components/app/overview/embedded/__tests__/index.spec.tsx
+++ b/web/app/components/app/overview/embedded/__tests__/index.spec.tsx
@@ -102,12 +102,12 @@ describe('Embedded', () => {
const optionButtons = document.body.querySelectorAll('[class*="option"]')
expect(optionButtons.length).toBeGreaterThanOrEqual(3)
act(() => {
- fireEvent.click(optionButtons[2])
+ fireEvent.click(optionButtons[2]!)
})
const [chromeText] = screen.getAllByText('appOverview.overview.appInfo.embedded.chromePlugin')
act(() => {
- fireEvent.click(chromeText)
+ fireEvent.click(chromeText!)
})
expect(mockWindowOpen).toHaveBeenCalledWith(
diff --git a/web/app/components/app/overview/embedded/index.tsx b/web/app/components/app/overview/embedded/index.tsx
index 547d80f0fe..029566587e 100644
--- a/web/app/components/app/overview/embedded/index.tsx
+++ b/web/app/components/app/overview/embedded/index.tsx
@@ -105,7 +105,7 @@ const Embedded = ({ siteInfo, isShow, onClose, appBaseUrl, accessToken, classNam
if (option === 'chromePlugin') {
const splitUrl = OPTION_MAP[option].getContent(appBaseUrl, accessToken).split(': ')
if (splitUrl.length > 1)
- copy(splitUrl[1])
+ copy(splitUrl[1]!)
}
else {
copy(OPTION_MAP[option].getContent(appBaseUrl, accessToken, themeBuilder.theme?.primaryColor ?? '#1C64F2', isTestEnv))
diff --git a/web/app/components/app/text-generate/saved-items/__tests__/index.spec.tsx b/web/app/components/app/text-generate/saved-items/__tests__/index.spec.tsx
index ddfea01951..3989817a71 100644
--- a/web/app/components/app/text-generate/saved-items/__tests__/index.spec.tsx
+++ b/web/app/components/app/text-generate/saved-items/__tests__/index.spec.tsx
@@ -37,8 +37,8 @@ describe('SavedItems', () => {
const { container } = render()
const markdownElement = container.querySelector('.markdown-body')
- expect(markdownElement).toBeInTheDocument()
- expect(screen.getByText('11 common.unit.char')).toBeInTheDocument()
+ expect(markdownElement)!.toBeInTheDocument()
+ expect(screen.getByText('11 common.unit.char'))!.toBeInTheDocument()
const actionArea = container.querySelector('[class*="bg-components-actionbar-bg"]')
const actionButtons = actionArea?.querySelectorAll('button') ?? []
@@ -56,11 +56,11 @@ describe('SavedItems', () => {
const copyButton = actionButtons[1]
const deleteButton = actionButtons[2]
- fireEvent.click(copyButton)
+ fireEvent.click(copyButton!)
expect(mockCopy).toHaveBeenCalledWith('hello world')
expect(toastSuccessSpy).toHaveBeenCalledWith('common.actionMsg.copySuccessfully')
- fireEvent.click(deleteButton)
+ fireEvent.click(deleteButton!)
expect(handleRemove).toHaveBeenCalledWith('1')
})
})
diff --git a/web/app/components/app/type-selector/index.tsx b/web/app/components/app/type-selector/index.tsx
index 6b8f8985c2..873593a600 100644
--- a/web/app/components/app/type-selector/index.tsx
+++ b/web/app/components/app/type-selector/index.tsx
@@ -148,9 +148,9 @@ function AppTypeSelectTrigger({ values }: { readonly values: AppSelectorProps['v
'flex h-8 flex-nowrap items-center justify-between gap-1',
)}
>
-
+