Merge pull request #3347 from Microsoft/irregularExpressionCompletion

Block completion when in the flags of a regular expression

tests/cases/fourslash/completionListAfterRegularExpressionLiteral01.ts

@@ -0,0 +1,8 @@
+/// <reference path="fourslash.ts" />
+
+////let v = 100;
+/////a/./**/
+
+goTo.marker();
+verify.not.memberListContains('v');
+verify.memberListContains('compile');

tests/cases/fourslash/completionListAfterRegularExpressionLiteral02.ts

@@ -0,0 +1,10 @@
+/// <reference path="fourslash.ts" />
+
+////let v = 100;
+////let x = /absidey//**/
+
+// Should get nothing at the marker since it's
+// going to be considered part of the regex flags.
+
+goTo.marker();
+verify.completionListIsEmpty();

tests/cases/fourslash/completionListAfterRegularExpressionLiteral03.ts

@@ -0,0 +1,11 @@
+/// <reference path="fourslash.ts" />
+
+////let v = 100;
+////let x = /absidey/
+/////**/
+
+// Should not be blocked since there is a
+// newline separating us from the regex flags.
+
+goTo.marker();
+verify.completionListContains("v");

tests/cases/fourslash/completionListAfterRegularExpressionLiteral04.ts

@@ -0,0 +1,10 @@
+/// <reference path="fourslash.ts" />
+
+////let v = 100;
+////let x = /absidey/ /**/
+
+// Should not be blocked since there is a
+// space separating us from the regex flags.
+
+goTo.marker();
+verify.completionListContains("v");

tests/cases/fourslash/completionListAfterRegularExpressionLiteral05.ts

@@ -0,0 +1,10 @@
+/// <reference path="fourslash.ts" />
+
+////let v = 100;
+////let x = /absidey/g/**/
+
+// Should get nothing at the marker since it's
+// going to be considered part of the regex flags.
+
+goTo.marker();
+verify.completionListIsEmpty()

tests/cases/fourslash/completionListAfterRegularExpressionLiteral1.ts

@@ -3,5 +3,5 @@
 /////a/./**/
 
 goTo.marker();
-//verify.not.memberListContains('alert');
-//verify.memberListContains('compile');
+verify.not.memberListContains('alert');
+verify.memberListContains('compile');