Minor regex change to avoid ':' being provided as hint

Author: njqdev

src/typeSearch.ts

@@ -161,7 +161,7 @@ export class TypeSearch {
      */
     public static hintOfSimilarParam(param: string, src: string): string | null {
 
-        const m = new RegExp(`^[ \t]*def [^(\\s]+\\([^)]*\\b${param}\\b: *([^),\\s=#]+)`, "m").exec(src);
+        const m = new RegExp(`^[ \t]*def [^(\\s]+\\([^)]*\\b${param}\\b: *([^),\\s=#:]+)`, "m").exec(src);
         if (m) {
             let hint = m[1].trim();
             return hint ? hint : null;

test/typeSearch/hintOfSimilarParam.test.ts

@@ -7,7 +7,7 @@ suite('TypeSearch.hintOfSimilarParam', () => {
     test("finds hint of lone param", () => {
         const expected = "str";
         const param = "test";
-        let src = `def func(${param}: str):\ndef test(self, ${param}:`;
+        let src = `def func(${param}: str):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
@@ -16,7 +16,7 @@ suite('TypeSearch.hintOfSimilarParam', () => {
         const expected = "str";
         const param = "test";
 
-        let src = `def func(self, p1: int,${param}: str):\ndef test(self, ${param}:`;
+        let src = `def func(self, p1: int,${param}: str):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
@@ -25,7 +25,7 @@ suite('TypeSearch.hintOfSimilarParam', () => {
         const expected = "str";
         const param = "test";
 
-        let src = `def func(self, ${param}: str,new: int):\ndef test(self, ${param}:`;
+        let src = `def func(self, ${param}: str,new: int):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
@@ -34,7 +34,7 @@ suite('TypeSearch.hintOfSimilarParam', () => {
         const expected = "str";
         const param = "test";
 
-        let src = `def func(\n\t${param}: str,new: int):\ndef test(self, ${param}:`;
+        let src = `def func(\n\t${param}: str,new: int):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
@@ -43,23 +43,23 @@ suite('TypeSearch.hintOfSimilarParam', () => {
         const expected = "str";
         const param = "test";
 
-        let src = `def func(${param}: str='exclude',new: int):\ndef test(self, text${param}:`;
+        let src = `def func(${param}: str='exclude',new: int):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
 
     test("finds non-ascii hint", () => {
         const expected = "蟒蛇";
         const param = "test";
-        let src = `def func(${param}: 蟒蛇):\ndef test(self, ${param}:`;
+        let src = `def func(${param}: 蟒蛇):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
 
     test("matches non-ascii function names", () => {
         const expected = "str";
         const param = "test";
-        let src = `def 蟒蛇(${param}: str):\ndef test(self, ${param}:`;
+        let src = `def 蟒蛇(${param}: str):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
@@ -68,15 +68,25 @@ suite('TypeSearch.hintOfSimilarParam', () => {
         const expected = null;
         const param = "test";
 
-        let src = `def func(text${param}: str,new: int):\ndef test(self, text${param}:`;
+        let src = `def func(text${param}: str,new: int):\ndef test(${param}:`;
+        let actual = TypeSearch.hintOfSimilarParam(param, src);
+        assert.equal(actual, expected);
+    });
+
+    test("doesn't provide items for ':' followed by ':'", () => {
+        
+        const expected = null;
+        const param = "test";
+
+        let src = `def func(${param}::):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected);
     });
 
     test("doesn't match comments", () => {
         const expected = null;
         const param = "test";
-        let src = `# def func(${param}: str):\ndef test(self, ${param}:`;
+        let src = `# def func(${param}: str):\ndef test(${param}:`;
         let actual = TypeSearch.hintOfSimilarParam(param, src);
         assert.equal(actual, expected, messageFor(src, expected, actual));