Skip to content

Commit 60ad58f

Browse files
committed
remove disableFileList test from controller
1 parent b5f548d commit 60ad58f

File tree

1 file changed

+0
-41
lines changed

1 file changed

+0
-41
lines changed

packages/core/src/test/amazonqFeatureDev/controllers/chat/controller.test.ts

Lines changed: 0 additions & 41 deletions
Original file line numberDiff line numberDiff line change
@@ -258,14 +258,6 @@ describe('Controller', () => {
258258

259259
assertTelemetry('amazonq_endChat', { amazonqConversationId: conversationID, result: 'Succeeded' })
260260
})
261-
262-
it('calls disableFileList', async () => {
263-
const disableFileListSpy = sinon.stub(session, 'disableFileList').resolves()
264-
265-
await newTaskClicked()
266-
267-
assert.ok(disableFileListSpy.calledOnce)
268-
})
269261
})
270262

271263
describe('fileClicked', () => {
@@ -409,14 +401,6 @@ describe('Controller', () => {
409401
}, {})
410402
}
411403

412-
it('calls disableFileList', async () => {
413-
const disableFileListSpy = sinon.stub(session, 'disableFileList').resolves()
414-
415-
await fireChatMessage()
416-
417-
assert.ok(disableFileListSpy.calledOnce)
418-
})
419-
420404
describe('processErrorChatMessage', function () {
421405
// TODO: fix disablePreviousFileList error
422406
const runs = [
@@ -453,13 +437,11 @@ describe('Controller', () => {
453437
const sendAnswerSpy = sinon.stub(controllerSetup.messenger, 'sendAnswer')
454438
const sendErrorMessageSpy = sinon.stub(controllerSetup.messenger, 'sendErrorMessage')
455439
const sendMonthlyLimitErrorSpy = sinon.stub(controllerSetup.messenger, 'sendMonthlyLimitError')
456-
const disableFileListSpy = sinon.stub(session, 'disableFileList').resolves()
457440

458441
await fireChatMessage()
459442

460443
switch (error.constructor.name) {
461444
case ContentLengthError.name:
462-
assert.ok(disableFileListSpy.calledOnce)
463445
assert.ok(
464446
sendAnswerSpy.calledWith({
465447
type: 'answer',
@@ -470,15 +452,13 @@ describe('Controller', () => {
470452
)
471453
break
472454
case MonthlyConversationLimitError.name:
473-
assert.ok(disableFileListSpy.calledOnce)
474455
assert.ok(sendMonthlyLimitErrorSpy.calledWith(tabID))
475456
break
476457
case FeatureDevServiceError.name:
477458
case UploadCodeError.name:
478459
case UserMessageNotFoundError.name:
479460
case TabIdNotFoundError.name:
480461
case PrepareRepoFailedError.name:
481-
assert.ok(disableFileListSpy.calledOnce)
482462
assert.ok(
483463
sendErrorMessageSpy.calledWith(
484464
createTestErrorMessage(error.message),
@@ -490,7 +470,6 @@ describe('Controller', () => {
490470
break
491471
case PromptRefusalException.name:
492472
case ZipFileError.name:
493-
assert.ok(disableFileListSpy.calledOnce)
494473
assert.ok(
495474
sendErrorMessageSpy.calledWith(
496475
createTestErrorMessage(error.message),
@@ -502,19 +481,8 @@ describe('Controller', () => {
502481
)
503482
break
504483
case NoChangeRequiredException.name:
505-
assert.ok(disableFileListSpy.calledTwice)
506-
assert.ok(
507-
sendAnswerSpy.calledWith({
508-
type: 'answer',
509-
tabID,
510-
message: error.message,
511-
canBeVoted: true,
512-
})
513-
)
514-
break
515484
case CodeIterationLimitError.name:
516485
case UploadURLExpired.name:
517-
assert.ok(disableFileListSpy.calledOnce)
518486
assert.ok(
519487
sendAnswerSpy.calledWith({
520488
type: 'answer',
@@ -525,7 +493,6 @@ describe('Controller', () => {
525493
)
526494
break
527495
default:
528-
assert.ok(disableFileListSpy.calledOnce)
529496
assert.ok(
530497
sendErrorMessageSpy.calledWith(
531498
i18n('AWS.amazonq.featureDev.error.codeGen.default'),
@@ -580,13 +547,5 @@ describe('Controller', () => {
580547

581548
assertTelemetry('amazonq_endChat', { amazonqConversationId: conversationID, result: 'Succeeded' })
582549
})
583-
584-
it('calls disableFileList', async () => {
585-
const disableFileListSpy = sinon.stub(session, 'disableFileList').resolves()
586-
587-
await closeSessionClicked()
588-
589-
assert.ok(disableFileListSpy.calledOnce)
590-
})
591550
})
592551
})

0 commit comments

Comments
 (0)