Skip to content

refactor(tests): move existing performance tests to the integ folder. #5735

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 16 commits into from
Oct 14, 2024
Merged

refactor(tests): move existing performance tests to the integ folder. #5735

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
16 commits
Select commit Hold shift + click to select a range
3f765c5
move performance test for prepareRepo to integ
Hweinstock Oct 7, 2024
5a0357a
move security scan test
Hweinstock Oct 7, 2024
e07593f
split up perf and non perf tests
Hweinstock Oct 8, 2024
3c1f909
remove shared code to utils
Hweinstock Oct 8, 2024
f5d6c60
move out more shared code
Hweinstock Oct 8, 2024
2897aee
Merge branch 'master' into pTests/moveToInteg
Hweinstock Oct 8, 2024
caf2be8
move some files around
Hweinstock Oct 8, 2024
658ed5a
update imports
Hweinstock Oct 8, 2024
366b038
fix test changes
Hweinstock Oct 8, 2024
1786ff4
delete duplicate test file
Hweinstock Oct 8, 2024
3b987b7
fix tests again
Hweinstock Oct 8, 2024
56819a5
fix tests again
Hweinstock Oct 8, 2024
a13a6c7
resolve conflicts
Hweinstock Oct 10, 2024
dbefbf0
Merge branch 'master' into pTests/moveToInteg
Hweinstock Oct 14, 2024
685924c
move tests into testPerf
Hweinstock Oct 14, 2024
0aa5546
rename folder
Hweinstock Oct 14, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion ...amazonqFeatureDev/prepareRepoData.test.ts → ...amazonqFeatureDev/prepareRepoData.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
import assert from 'assert'
import { WorkspaceFolder } from 'vscode'
import { performanceTest } from '../../shared/performance/performance'
import { createTestWorkspace } from '../testUtil'
import { createTestWorkspace } from '../../test/testUtil'
import { prepareRepoData, TelemetryHelper } from '../../amazonqFeatureDev'
import { AmazonqCreateUpload, getRandomString } from '../../shared'
import { Span } from '../../shared/telemetry'
Expand Down
34 changes: 17 additions & 17 deletions ...sperer/commands/startSecurityScan.test.ts → ...g/codewhisperer/startSecurityScan.test.ts
View file
Open in desktop
Copy link
Contributor

@justinmk3 justinmk3 Oct 7, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this file has non-perf tests in it. only the perf tests should be moved

Copy link
Contributor

@justinmk3 justinmk3 Oct 8, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's put this in testInteg/amazon/...test.ts. "codewhisperer" is not a name that means anything anymore.

Original file line number Diff line number Diff line change
Expand Up @@ -7,36 +7,36 @@ import assert from 'assert'
import * as vscode from 'vscode'
import * as sinon from 'sinon'
import * as semver from 'semver'
import { DefaultCodeWhispererClient } from '../../../codewhisperer/client/codewhisperer'
import * as startSecurityScan from '../../../codewhisperer/commands/startSecurityScan'
import { SecurityPanelViewProvider } from '../../../codewhisperer/views/securityPanelViewProvider'
import { FakeExtensionContext } from '../../fakeExtensionContext'
import * as diagnosticsProvider from '../../../codewhisperer/service/diagnosticsProvider'
import { getTestWorkspaceFolder } from '../../../testInteg/integrationTestsUtilities'
import { DefaultCodeWhispererClient } from '../../codewhisperer'
import * as startSecurityScan from '../../codewhisperer/commands/startSecurityScan'
import { SecurityPanelViewProvider } from '../../codewhisperer/views/securityPanelViewProvider'
import { FakeExtensionContext } from '../../test/fakeExtensionContext'
import * as diagnosticsProvider from '../../codewhisperer/service/diagnosticsProvider'
import { getTestWorkspaceFolder } from '../../testInteg/integrationTestsUtilities'
import { join } from 'path'
import {
assertTelemetry,
closeAllEditors,
createTestWorkspaceFolder,
getFetchStubWithResponse,
toFile,
} from '../../testUtil'
import { stub } from '../../utilities/stubber'
} from '../../test/testUtil'
import { stub } from '../../test/utilities/stubber'
import { AWSError, HttpResponse } from 'aws-sdk'
import { getTestWindow } from '../../shared/vscode/window'
import { SeverityLevel } from '../../shared/vscode/message'
import { cancel } from '../../../shared/localizedText'
import { getTestWindow } from '../../test/shared/vscode/window'
import { SeverityLevel } from '../../test/shared/vscode/message'
import { cancel } from '../../shared'
import {
showScannedFilesMessage,
stopScanMessage,
CodeAnalysisScope,
projectScansLimitReached,
} from '../../../codewhisperer/models/constants'
import * as model from '../../../codewhisperer/models/model'
import { CodewhispererSecurityScan } from '../../../shared/telemetry/telemetry.gen'
import * as errors from '../../../shared/errors'
import * as timeoutUtils from '../../../shared/utilities/timeoutUtils'
import { performanceTest } from '../../../shared/performance/performance'
} from '../../codewhisperer'
import * as model from '../../codewhisperer/models/model'
import { CodewhispererSecurityScan } from '../../shared/telemetry/telemetry.gen'
import * as errors from '../../shared/errors'
import * as timeoutUtils from '../../shared/utilities/timeoutUtils'
import { performanceTest } from '../../shared/performance/performance'

const mockCreateCodeScanResponse = {
$response: {
Expand Down
Loading