Skip to content

Commit 82cae41

Browse files
authored
test(flaky): skip all performance tests. (#6459)
## Problem #6117 #6457 These issues keep popping up, regardless of the the thresholds. The performance tests are causing more problems then they are solving. Real performance regressions would be captured in telemetry. ## Solution skip all performance tests, revisit once there is less flakiness. --- - Treat all work as PUBLIC. Private `feature/x` branches will not be squash-merged at release time. - Your code changes must meet the guidelines in [CONTRIBUTING.md](https://github.com/aws/aws-toolkit-vscode/blob/master/CONTRIBUTING.md#guidelines). - License: I confirm that my contribution is made under the terms of the Apache 2.0 license.
1 parent 349fe04 commit 82cae41

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

packages/core/src/shared/performance/performance.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -102,8 +102,8 @@ export function performanceTest<TSetup, TExecute>(
102102
const testOption = options[process.platform as 'linux' | 'darwin' | 'win32']
103103

104104
const totalTestRuns = options.testRuns ?? 10
105-
106-
return describe(`${name} performance tests`, () => {
105+
// TODO: unskip this once flakiness is reduced.
106+
return describe.skip(`${name} performance tests`, () => {
107107
let performanceTracker: PerformanceTracker | undefined
108108
const testRunMetrics: PerformanceMetrics[] = []
109109

0 commit comments

Comments
 (0)