fix: remove tests for generateFix, as we are not generating fixes anymore, it is generated by maestro

Author: singhAws

packages/core/src/test/codewhisperer/commands/basicCommands.test.ts

@@ -789,174 +789,6 @@ def execute_input_compliant():
         })
     })
 
-    describe('generateFix', function () {
-        let sandbox: sinon.SinonSandbox
-        let mockClient: Stub<DefaultCodeWhispererClient>
-        let startCodeFixGenerationStub: sinon.SinonStub
-        let filePath: string
-        let codeScanIssue: CodeScanIssue
-        let issueItem: IssueItem
-        let updateSecurityIssueWebviewMock: sinon.SinonStub
-        let updateIssueMock: sinon.SinonStub
-        let refreshTreeViewMock: sinon.SinonStub
-        let mockExtContext: ExtContext
-
-        beforeEach(async function () {
-            sandbox = sinon.createSandbox()
-            mockClient = stub(DefaultCodeWhispererClient)
-            startCodeFixGenerationStub = sinon.stub(startCodeFixGeneration, 'startCodeFixGeneration')
-            filePath = 'dummy/file.py'
-            codeScanIssue = createCodeScanIssue({
-                findingId: randomUUID(),
-                ruleId: 'dummy-rule-id',
-            })
-            issueItem = new IssueItem(filePath, codeScanIssue)
-            updateSecurityIssueWebviewMock = sinon.stub(securityIssueWebview, 'updateSecurityIssueWebview')
-            updateIssueMock = sinon.stub(SecurityIssueProvider.instance, 'updateIssue')
-            refreshTreeViewMock = sinon.stub(SecurityIssueTreeViewProvider.instance, 'refresh')
-            mockExtContext = await FakeExtensionContext.getFakeExtContext()
-        })
-
-        afterEach(function () {
-            sandbox.restore()
-        })
-
-        it('should call generateFix command successfully', async function () {
-            startCodeFixGenerationStub.resolves({
-                suggestedFix: {
-                    codeDiff: 'codeDiff',
-                    description: 'description',
-                    references: [],
-                },
-                jobId: 'jobId',
-            })
-
-            targetCommand = testCommand(generateFix, mockClient, mockExtContext)
-            await targetCommand.execute(codeScanIssue, filePath, 'webview')
-
-            assert.ok(updateSecurityIssueWebviewMock.calledWith(sinon.match({ isGenerateFixLoading: true })))
-            assert.ok(
-                startCodeFixGenerationStub.calledWith(mockClient, codeScanIssue, filePath, codeScanIssue.findingId)
-            )
-
-            const expectedUpdatedIssue = {
-                ...codeScanIssue,
-                fixJobId: 'jobId',
-                suggestedFixes: [{ code: 'codeDiff', description: 'description', references: [] }],
-            }
-            assert.ok(
-                updateSecurityIssueWebviewMock.calledWith(
-                    sinon.match({
-                        issue: expectedUpdatedIssue,
-                        isGenerateFixLoading: false,
-                        filePath: filePath,
-                        shouldRefreshView: true,
-                    })
-                )
-            )
-            assert.ok(updateIssueMock.calledWith(expectedUpdatedIssue, filePath))
-            assert.ok(refreshTreeViewMock.calledOnce)
-
-            assertTelemetry('codewhisperer_codeScanIssueGenerateFix', {
-                detectorId: codeScanIssue.detectorId,
-                findingId: codeScanIssue.findingId,
-                ruleId: codeScanIssue.ruleId,
-                component: 'webview',
-                result: 'Succeeded',
-            })
-        })
-
-        it('should call generateFix from tree view item', async function () {
-            startCodeFixGenerationStub.resolves({
-                suggestedFix: {
-                    codeDiff: 'codeDiff',
-                    description: 'description',
-                    references: [],
-                },
-                jobId: 'jobId',
-            })
-
-            targetCommand = testCommand(generateFix, mockClient, mockExtContext)
-            await targetCommand.execute(issueItem, filePath, 'tree')
-
-            assertTelemetry('codewhisperer_codeScanIssueGenerateFix', {
-                detectorId: codeScanIssue.detectorId,
-                findingId: codeScanIssue.findingId,
-                ruleId: codeScanIssue.ruleId,
-                component: 'tree',
-                result: 'Succeeded',
-            })
-        })
-
-        it('should call generateFix with refresh=true to indicate fix regenerated', async function () {
-            startCodeFixGenerationStub.resolves({
-                suggestedFix: {
-                    codeDiff: 'codeDiff',
-                    description: 'description',
-                    references: [],
-                },
-                jobId: 'jobId',
-            })
-
-            targetCommand = testCommand(generateFix, mockClient, mockExtContext)
-            await targetCommand.execute(codeScanIssue, filePath, 'webview', true)
-
-            assertTelemetry('codewhisperer_codeScanIssueGenerateFix', {
-                detectorId: codeScanIssue.detectorId,
-                findingId: codeScanIssue.findingId,
-                ruleId: codeScanIssue.ruleId,
-                component: 'webview',
-                result: 'Succeeded',
-                variant: 'refresh',
-            })
-        })
-
-        it('should handle generateFix error', async function () {
-            startCodeFixGenerationStub.throws(new Error('Unexpected error'))
-
-            targetCommand = testCommand(generateFix, mockClient, mockExtContext)
-            await targetCommand.execute(codeScanIssue, filePath, 'webview')
-
-            assert.ok(updateSecurityIssueWebviewMock.calledWith(sinon.match({ isGenerateFixLoading: true })))
-            assert.ok(
-                updateSecurityIssueWebviewMock.calledWith(
-                    sinon.match({
-                        issue: codeScanIssue,
-                        isGenerateFixLoading: false,
-                        generateFixError: 'Unexpected error',
-                        shouldRefreshView: false,
-                    })
-                )
-            )
-            assert.ok(updateIssueMock.calledWith(codeScanIssue, filePath))
-            assert.ok(refreshTreeViewMock.calledOnce)
-
-            assertTelemetry('codewhisperer_codeScanIssueGenerateFix', {
-                detectorId: codeScanIssue.detectorId,
-                findingId: codeScanIssue.findingId,
-                ruleId: codeScanIssue.ruleId,
-                component: 'webview',
-                result: 'Failed',
-                reason: 'Error',
-                reasonDesc: 'Unexpected error',
-            })
-        })
-
-        it('exits early for SAS findings', async function () {
-            targetCommand = testCommand(generateFix, mockClient, mockExtContext)
-            codeScanIssue = createCodeScanIssue({
-                ruleId: CodeWhispererConstants.sasRuleId,
-            })
-            issueItem = new IssueItem(filePath, codeScanIssue)
-            await targetCommand.execute(codeScanIssue, filePath, 'webview')
-            assert.ok(updateSecurityIssueWebviewMock.notCalled)
-            assert.ok(startCodeFixGenerationStub.notCalled)
-            assert.ok(updateIssueMock.notCalled)
-            assert.ok(refreshTreeViewMock.notCalled)
-            assertNoTelemetryMatch('codewhisperer_codeScanIssueGenerateFix')
-        })
-    })
-
     describe('rejectFix', function () {
         let mockExtensionContext: vscode.ExtensionContext
         let sandbox: sinon.SinonSandbox