Prevent return type hints ending with typing hints from creating completion items

Author: njqdev

src/completionProvider.ts

@@ -141,7 +141,7 @@ export class ParamHintCompletionProvider extends CompletionProvider implements C
     private getParam(precedingText: string): string | null {
         const split = precedingText.split(/[,(*]/);
         let param = split.length > 1 ? split[split.length - 1].trim() : precedingText;
-        return !param || /[!:?/\\{}.+/=)'";@&£%¤|<>$^~¨ -]/.test(param) ? null : param;
+        return !param || /[!:\]\[?/\\{}.+/=)'";@&£%¤|<>$^~¨ -]/.test(param) ? null : param;
     }
 
     private pushEstimationsToItems(typeHints: string[], items: CompletionItem[]) {

test/suite/providers/completionProvider.test.ts

@@ -137,6 +137,16 @@ suite('ParamHintCompletionProvider', () => {
         assert.strictEqual(actual, null);
     });
 
+    test("does not provide items for end of function definition with typing hint", async () => {
+        let actual = await providerResult(provider, "def test(n: int) -> List[str]:");
+        assert.strictEqual(actual, null);
+    });
+
+    test("does not provide items for end of function definition with typing Dict hint", async () => {
+        let actual = await providerResult(provider, "def test(n: int) -> Dict[str, str]:");
+        assert.strictEqual(actual, null);
+    });
+
     test("does not include * in parameter name", async () => {
         let param = "*paramName:";
         let actual = await providerResult(provider, param, "\n\nparamName = 12");