Skip to content

Commit 1388822

Browse files
committed
test(kb): update tests for enabledFilter and removed userId params
1 parent 3aedf41 commit 1388822

File tree

1 file changed

+8
-10
lines changed

1 file changed

+8
-10
lines changed

apps/sim/app/api/knowledge/[id]/documents/route.test.ts

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ describe('Knowledge Base Documents API Route', () => {
157157
expect(vi.mocked(getDocuments)).toHaveBeenCalledWith(
158158
'kb-123',
159159
{
160-
includeDisabled: false,
160+
enabledFilter: undefined,
161161
search: undefined,
162162
limit: 50,
163163
offset: 0,
@@ -166,7 +166,7 @@ describe('Knowledge Base Documents API Route', () => {
166166
)
167167
})
168168

169-
it('should filter disabled documents by default', async () => {
169+
it('should return documents with default filter', async () => {
170170
const { checkKnowledgeBaseAccess } = await import('@/app/api/knowledge/utils')
171171
const { getDocuments } = await import('@/lib/knowledge/documents/service')
172172

@@ -194,7 +194,7 @@ describe('Knowledge Base Documents API Route', () => {
194194
expect(vi.mocked(getDocuments)).toHaveBeenCalledWith(
195195
'kb-123',
196196
{
197-
includeDisabled: false,
197+
enabledFilter: undefined,
198198
search: undefined,
199199
limit: 50,
200200
offset: 0,
@@ -203,7 +203,7 @@ describe('Knowledge Base Documents API Route', () => {
203203
)
204204
})
205205

206-
it('should include disabled documents when requested', async () => {
206+
it('should filter documents by enabled status when requested', async () => {
207207
const { checkKnowledgeBaseAccess } = await import('@/app/api/knowledge/utils')
208208
const { getDocuments } = await import('@/lib/knowledge/documents/service')
209209

@@ -223,7 +223,7 @@ describe('Knowledge Base Documents API Route', () => {
223223
},
224224
})
225225

226-
const url = 'http://localhost:3000/api/knowledge/kb-123/documents?includeDisabled=true'
226+
const url = 'http://localhost:3000/api/knowledge/kb-123/documents?enabledFilter=disabled'
227227
const req = new Request(url, { method: 'GET' }) as any
228228

229229
const { GET } = await import('@/app/api/knowledge/[id]/documents/route')
@@ -233,7 +233,7 @@ describe('Knowledge Base Documents API Route', () => {
233233
expect(vi.mocked(getDocuments)).toHaveBeenCalledWith(
234234
'kb-123',
235235
{
236-
includeDisabled: true,
236+
enabledFilter: 'disabled',
237237
search: undefined,
238238
limit: 50,
239239
offset: 0,
@@ -361,8 +361,7 @@ describe('Knowledge Base Documents API Route', () => {
361361
expect(vi.mocked(createSingleDocument)).toHaveBeenCalledWith(
362362
validDocumentData,
363363
'kb-123',
364-
expect.any(String),
365-
'user-123'
364+
expect.any(String)
366365
)
367366
})
368367

@@ -470,8 +469,7 @@ describe('Knowledge Base Documents API Route', () => {
470469
expect(vi.mocked(createDocumentRecords)).toHaveBeenCalledWith(
471470
validBulkData.documents,
472471
'kb-123',
473-
expect.any(String),
474-
'user-123'
472+
expect.any(String)
475473
)
476474
expect(vi.mocked(processDocumentsWithQueue)).toHaveBeenCalled()
477475
})

0 commit comments

Comments
 (0)