adding a todo and one more test

Author: aiday-mar

src/vs/editor/contrib/indentation/test/browser/indentation.test.ts

@@ -741,6 +741,8 @@ suite('`Full` Auto Indent On Type - TypeScript/JavaScript', () => {
 		// issue: Should indent after an equal sign is detected followed by whitespace characters.
 		// This should be outdented when a semi-colon is detected indicating the end of the assignment.
 
+		// TODO: requires exploring indent/outdent pairs instead
+
 		const model = createTextModel([
 			'const array ='
 		].join('\n'), languageId, {});
@@ -762,6 +764,8 @@ suite('`Full` Auto Indent On Type - TypeScript/JavaScript', () => {
 		// https://github.com/microsoft/vscode/issues/43244
 		// issue: When a dot is written, we should detect that this is a method call and indent accordingly
 
+		// TODO: requires exploring indent/outdent pairs instead
+
 		const model = createTextModel([
 			'const array = [1, 2, 3];',
 			'array.'
@@ -785,6 +789,8 @@ suite('`Full` Auto Indent On Type - TypeScript/JavaScript', () => {
 		// https://github.com/microsoft/vscode/issues/43244
 		// issue: When a dot is written, we should detect that this is a method call and indent accordingly
 
+		// TODO: requires exploring indent/outdent pairs instead
+
 		const model = createTextModel([
 			'const array = [1, 2, 3]',
 		].join('\n'), languageId, {});
@@ -807,6 +813,8 @@ suite('`Full` Auto Indent On Type - TypeScript/JavaScript', () => {
 		// https://github.com/microsoft/vscode/issues/43244
 		// Currently passes, but should pass with all the tests above too
 
+		// TODO: requires exploring indent/outdent pairs instead
+
 		const model = createTextModel([
 			'const array = [1, 2, 3]',
 			'    .filter(() => true)'
@@ -825,10 +833,38 @@ suite('`Full` Auto Indent On Type - TypeScript/JavaScript', () => {
 		});
 	});
 
+	test.skip('issue #43244: keep indentation when chained methods called on object/array', () => {
+
+		// https://github.com/microsoft/vscode/issues/43244
+		// When the call chain is not finished yet, and we type a dot, we do not want to change the indentation
+
+		// TODO: requires exploring indent/outdent pairs instead
+
+		const model = createTextModel([
+			'const array = [1, 2, 3]',
+			'    .filter(() => true)',
+			'    '
+		].join('\n'), languageId, {});
+		disposables.add(model);
+
+		withTestCodeEditor(model, { autoIndent: "full" }, (editor, viewModel, instantiationService) => {
+			registerLanguage(instantiationService, languageId, Language.TypeScript, disposables);
+			editor.setSelection(new Selection(3, 5, 3, 5));
+			viewModel.type(".");
+			assert.strictEqual(model.getValue(), [
+				'const array = [1, 2, 3]',
+				'    .filter(() => true)',
+				'    .' // here we don't want to increase the indentation because we have chained methods
+			].join('\n'));
+		});
+	});
+
 	test.skip('issue #43244: outdent when a semi-color is detected indicating the end of the assignment', () => {
 
 		// https://github.com/microsoft/vscode/issues/43244
 
+		// TODO: requires exploring indent/outdent pairs instead
+
 		const model = createTextModel([
 			'const array = [1, 2, 3]',
 			'    .filter(() => true);'

src/vs/workbench/contrib/codeEditor/test/node/autoindent.test.ts

@@ -282,10 +282,8 @@ suite('Auto-Reindentation - TypeScript/JavaScript', () => {
 		// related: https://github.com/microsoft/vscode/issues/43244
 		// explanation: When you have an arrow function, you don't have { or }, but you would expect indentation to still be done in that way
 
-		/*
-		Notes: Currently the reindent edit operations does not call the onEnter rules
-		The reindent should also call the onEnter rules to get the correct indentation
-		*/
+		// TODO: requires exploring indent/outdent pairs instead
+
 		const fileContents = [
 			'const add1 = (n) =>',
 			'	n + 1;',