fix unwanted spacing around equals in override completion (#161)

Author: jakebailey

src/Analysis/Engine/Test/LanguageServerTests.cs

@@ -443,6 +443,16 @@ public async Task CompletionForOverride() {
             await AssertCompletion(s, u, new[] { "__init__" }, new[] { "def" }, new SourceLocation(3, 9), cmpKey: ci => ci.label);
         }
 
+        [TestMethod, Priority(0)]
+        public async Task CompletionForOverrideArgs() {
+            using (var s = await CreateServer()) {
+                var u = await AddModule(s, "class A:\n    def bar(arg=None): pass\n\nclass B(A):\n    def b");
+
+                await AssertNoCompletion(s, u, new SourceLocation(2, 9));
+                await AssertCompletion(s, u, new[] { "bar(arg=None):\r\n    return super().bar()" }, new[] { "bar(arg = None):\r\n    return super().bar()" }, new SourceLocation(5, 10));
+            }
+        }
+
         [TestMethod, Priority(0)]
         public async Task CompletionInDecorator() {
             var s = await CreateServer();

src/LanguageServer/Impl/Implementation/CompletionAnalysis.cs

@@ -876,7 +876,7 @@ private static CompletionItemKind ToCompletionItemKind(PythonMemberType memberTy
 
         private static string GetSafeParameterName(ParameterResult result, int index) {
             if (!string.IsNullOrEmpty(result.DefaultValue)) {
-                return GetSafeArgumentName(result, index) + " = " + result.DefaultValue;
+                return GetSafeArgumentName(result, index) + "=" + result.DefaultValue;
             }
             return GetSafeArgumentName(result, index);
         }