adopt ensureNoDisposablesLeaked in code action model tests

.eslintrc.json

@@ -151,7 +151,6 @@
 							"src/vs/base/test/browser/browser.test.ts",
 							"src/vs/base/test/browser/ui/scrollbar/scrollableElement.test.ts",
 							"src/vs/base/test/browser/ui/scrollbar/scrollbarState.test.ts",
-							"src/vs/editor/contrib/codeAction/test/browser/codeActionModel.test.ts",
 							"src/vs/editor/test/common/services/languageService.test.ts",
 							"src/vs/editor/test/node/classification/typescript.test.ts",
 							"src/vs/platform/configuration/test/common/configuration.test.ts",

src/vs/editor/contrib/codeAction/browser/codeAction.ts

@@ -155,12 +155,14 @@ export async function getCodeActions(
 
 	try {
 		const actions = await Promise.all(promises);
-		const allActions = actions.map(x => x.actions).flat();
+		const allActions = actions.flatMap(x => x.actions);
 		const allDocumentation = [
 			...coalesce(actions.map(x => x.documentation)),
 			...getAdditionalDocumentationForShowingActions(registry, model, trigger, allActions)
 		];
-		return new ManagedCodeActionSet(allActions, allDocumentation, disposables);
+		const result = new ManagedCodeActionSet(allActions, allDocumentation, disposables);
+		result.dispose();
+		return result;
 	} finally {
 		listener.dispose();
 		cts.dispose();

src/vs/editor/contrib/codeAction/test/browser/codeActionModel.test.ts

@@ -9,6 +9,7 @@ import { DisposableStore } from 'vs/base/common/lifecycle';
 import { assertType } from 'vs/base/common/types';
 import { URI } from 'vs/base/common/uri';
 import { runWithFakedTimers } from 'vs/base/test/common/timeTravelScheduler';
+import { ensureNoDisposablesAreLeakedInTestSuite } from 'vs/base/test/common/utils';
 import { ICodeEditor } from 'vs/editor/browser/editorBrowser';
 import { LanguageFeatureRegistry } from 'vs/editor/common/languageFeatureRegistry';
 import * as languages from 'vs/editor/common/languages';
@@ -38,19 +39,18 @@ suite('CodeActionModel', () => {
 	let markerService: MarkerService;
 	let editor: ICodeEditor;
 	let registry: LanguageFeatureRegistry<languages.CodeActionProvider>;
-	const disposables = new DisposableStore();
 
 	setup(() => {
-		disposables.clear();
 		markerService = new MarkerService();
 		model = createTextModel('foobar  foo bar\nfarboo far boo', languageId, undefined, uri);
 		editor = createTestCodeEditor(model);
 		editor.setPosition({ lineNumber: 1, column: 1 });
 		registry = new LanguageFeatureRegistry();
 	});
 
+	const store = ensureNoDisposablesAreLeakedInTestSuite();
+
 	teardown(() => {
-		disposables.clear();
 		editor.dispose();
 		model.dispose();
 		markerService.dispose();
@@ -61,11 +61,11 @@ suite('CodeActionModel', () => {
 
 		await runWithFakedTimers({ useFakeTimers: true }, () => {
 			const reg = registry.register(languageId, testProvider);
-			disposables.add(reg);
+			store.add(reg);
 
 			const contextKeys = new MockContextKeyService();
-			const model = disposables.add(new CodeActionModel(editor, registry, markerService, contextKeys, undefined));
-			disposables.add(model.onDidChangeState((e: CodeActionsState.State) => {
+			const model = store.add(new CodeActionModel(editor, registry, markerService, contextKeys, undefined));
+			store.add(model.onDidChangeState((e: CodeActionsState.State) => {
 				assertType(e.type === CodeActionsState.Type.Triggered);
 
 				assert.strictEqual(e.trigger.type, languages.CodeActionTriggerType.Auto);
@@ -93,7 +93,7 @@ suite('CodeActionModel', () => {
 	test('Oracle -> position changed', async () => {
 		await runWithFakedTimers({ useFakeTimers: true }, () => {
 			const reg = registry.register(languageId, testProvider);
-			disposables.add(reg);
+			store.add(reg);
 
 			markerService.changeOne('fake', uri, [{
 				startLineNumber: 1, startColumn: 1, endLineNumber: 1, endColumn: 6,
@@ -107,8 +107,8 @@ suite('CodeActionModel', () => {
 
 			return new Promise((resolve, reject) => {
 				const contextKeys = new MockContextKeyService();
-				const model = disposables.add(new CodeActionModel(editor, registry, markerService, contextKeys, undefined));
-				disposables.add(model.onDidChangeState((e: CodeActionsState.State) => {
+				const model = store.add(new CodeActionModel(editor, registry, markerService, contextKeys, undefined));
+				store.add(model.onDidChangeState((e: CodeActionsState.State) => {
 					assertType(e.type === CodeActionsState.Type.Triggered);
 
 					assert.strictEqual(e.trigger.type, languages.CodeActionTriggerType.Auto);
@@ -129,12 +129,12 @@ suite('CodeActionModel', () => {
 		const { promise: donePromise, resolve: done } = promiseWithResolvers<void>();
 		await runWithFakedTimers({ useFakeTimers: true }, () => {
 			const reg = registry.register(languageId, testProvider);
-			disposables.add(reg);
+			store.add(reg);
 
 			let triggerCount = 0;
 			const contextKeys = new MockContextKeyService();
-			const model = disposables.add(new CodeActionModel(editor, registry, markerService, contextKeys, undefined));
-			disposables.add(model.onDidChangeState((e: CodeActionsState.State) => {
+			const model = store.add(new CodeActionModel(editor, registry, markerService, contextKeys, undefined));
+			store.add(model.onDidChangeState((e: CodeActionsState.State) => {
 				assertType(e.type === CodeActionsState.Type.Triggered);
 
 				assert.strictEqual(e.trigger.type, languages.CodeActionTriggerType.Auto);