Fix formatting between for and await (microsoft#21254)

esakkiraj · mhegazy · commit 92bde084c1e5 · 2018-01-21T09:45:58.000-08:00
* Fix issue 21084

* Removed unwanted rules.
diff --git a/src/services/formatting/rules.ts b/src/services/formatting/rules.ts
@@ -321,6 +321,9 @@ namespace ts.formatting {
             rule("NoSpaceAfterCloseBracket", SyntaxKind.CloseBracketToken, anyToken, [isNonJsxSameLineTokenContext, isNotBeforeBlockInFunctionDeclarationContext], RuleAction.Delete),
             rule("SpaceAfterSemicolon", SyntaxKind.SemicolonToken, anyToken, [isNonJsxSameLineTokenContext], RuleAction.Space),
 
+            // Remove extra space between for and await
+            rule("SpaceBetweenForAndAwaitKeyword", SyntaxKind.ForKeyword, SyntaxKind.AwaitKeyword, [isNonJsxSameLineTokenContext], RuleAction.Space),
+
             // Add a space between statements. All keywords except (do,else,case) has open/close parens after them.
             // So, we have a rule to add a space for [),Any], [do,Any], [else,Any], and [case,Any]
             rule(
diff --git a/tests/cases/fourslash/formattingAwait.ts b/tests/cases/fourslash/formattingAwait.ts
@@ -0,0 +1,18 @@
+/// <reference path='fourslash.ts'/>
+
+////async function f() {
+////    for          await (const x of g()) {
+////        console.log(x);
+////    }
+////}
+
+
+format.document();
+
+verify.currentFileContentIs(
+`async function f() {
+    for await (const x of g()) {
+        console.log(x);
+    }
+}`
+);
