Fix spacing issues with Callback completion

Abc-Arbitrage · Jun 11, 2022 · 78462f6 · 78462f6
1 parent 102cb18
commit 78462f6
Show file tree

Hide file tree

Showing 19 changed files with 28 additions and 28 deletions.
diff --git a/...MoqComplete/Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion.cs b/...MoqComplete/Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using Moq.AutoMock;
 using NUnit.Framework;

diff --git a/...mplete/Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion.cs.gold b/...mplete/Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using Moq.AutoMock;
 using NUnit.Framework;
@@ -18,7 +18,7 @@ namespace ConsoleApp1.Tests
         {
             var _mocker = new AutoMocker();
             _mocker.Setup<ITestInterface>(x => x.BuildSomething(It.IsAny<int>(),It.IsAny<string>(), It.IsAny<bool>()))
-                 .Callback<int,string,bool>((i,toto,ok) => {{caret}})
+                 .Callback<int, string, bool>((i, toto, ok) => {{caret}})
         }
     }
 }
diff --git a/...Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_afterReturn.cs b/...Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_afterReturn.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using Moq.AutoMock;
 using NUnit.Framework;

diff --git a/...oqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_afterReturn.cs.gold b/...oqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_afterReturn.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using Moq.AutoMock;
 using NUnit.Framework;
@@ -19,7 +19,7 @@ namespace ConsoleApp1.Tests
             var _mocker = new AutoMocker();
             _mocker.Setup<ITestInterface>(x => x.BuildSomething(It.IsAny<int>(),It.IsAny<string>(), It.IsAny<bool>()))
 				.Returns(0)
-                .Callback<int,string,bool>((i,toto,ok) => {{caret}})
+                .Callback<int, string, bool>((i, toto, ok) => {{caret}})
         }
     }
 }
diff --git a/...ete/Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_generic.cs b/...ete/Abc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_generic.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<ITest2,double,string>((test,test2,test3) => {\})}
+// ${COMPLETE_ITEM:Callback<ITest2, double, string>((test, test2, test3) => {\})}
 using NUnit.Framework;
 using Moq;
 using Moq.AutoMock;

diff --git a/...bc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_generic.cs.gold b/...bc.MoqComplete.Tests/data/AutoMockerCallbackCompletion/callbackCompletion_generic.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<ITest2,double,string>((test,test2,test3) => {\})}
+// ${COMPLETE_ITEM:Callback<ITest2, double, string>((test, test2, test3) => {\})}
 using NUnit.Framework;
 using Moq;
 using Moq.AutoMock;
@@ -24,7 +24,7 @@ namespace ConsoleApp1.Tests
         {
             var _mocker = new AutoMocker();
             _mocker.Setup<ITest2>(x=>x.Coco(It.IsAny<ITest2>(), It.IsAny<double>(), It.IsAny<string>()))
-             .Callback<ITest2,double,string>((test,test2,test3) => {{caret}})
+             .Callback<ITest2, double, string>((test, test2, test3) => {{caret}})
         }
     }
 }
diff --git a/Abc.MoqComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion.cs b/Abc.MoqComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 

diff --git a/Abc.MoqComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion.cs.gold b/Abc.MoqComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 
@@ -17,7 +17,7 @@ namespace ConsoleApp1.Tests
         {
             Mock<ITestInterface> temp = new Mock<ITestInterface>();
             temp.Setup(x => x.BuildSomething(It.IsAny<int>(),It.IsAny<string>(), It.IsAny<bool>()))
-                 .Callback<int,string,bool>((i,toto,ok) => {{caret}})
+                 .Callback<int, string, bool>((i, toto, ok) => {{caret}})
         }
     }
 }
diff --git a/...qComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturn.cs b/...qComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturn.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 

diff --git a/...lete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturn.cs.gold b/...lete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturn.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 
@@ -18,7 +18,7 @@ namespace ConsoleApp1.Tests
             Mock<ITestInterface> temp = new Mock<ITestInterface>();
             temp.Setup(x => x.BuildSomething(It.IsAny<int>(),It.IsAny<string>(), It.IsAny<bool>()))
 				.Returns(0)
-                .Callback<int,string,bool>((i,toto,ok) => {{caret}})
+                .Callback<int, string, bool>((i, toto, ok) => {{caret}})
         }
     }
 }
diff --git a/...lete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsync.cs b/...lete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsync.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 using System.Threading.Tasks;

diff --git a/...Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsync.cs.gold b/...Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsync.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 using System.Threading.Tasks;
@@ -19,7 +19,7 @@ namespace ConsoleApp1.Tests
             Mock<ITestInterface> temp = new Mock<ITestInterface>();
             temp.Setup(x => x.BuildSomething(It.IsAny<int>(),It.IsAny<string>(), It.IsAny<bool>()))
 				.ReturnsAsync(0)
-                .Callback<int,string,bool>((i,toto,ok) => {{caret}})
+                .Callback<int, string, bool>((i, toto, ok) => {{caret}})
         }
     }
 }
diff --git a/....MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple.cs b/....MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 using System.Threading.Tasks;

diff --git a/...omplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple.cs.gold b/...omplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 using System.Threading.Tasks;
@@ -19,7 +19,7 @@ namespace ConsoleApp1.Tests
             Mock<ITestInterface> temp = new Mock<ITestInterface>();
             temp.Setup(x => x.BuildSomething(It.IsAny<int>(),It.IsAny<string>(), It.IsAny<bool>()))
 				.ReturnsAsync<int, string, bool, ITestInterface, int>((i, j, k) => 42)
-                .Callback<int,string,bool>((i,toto,ok) => {{caret}})
+                .Callback<int, string, bool>((i, toto, ok) => {{caret}})
         }
     }
 }
diff --git a/...MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple2.cs b/...MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple2.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 using System.Threading.Tasks;

diff --git a/...mplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple2.cs.gold b/...mplete.Tests/data/CallbackCompletion/callbackCompletion_afterReturnAsyncMultiple2.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<int,string,bool>((i,toto,ok) => {\})}
+// ${COMPLETE_ITEM:Callback<int, string, bool>((i, toto, ok) => {\})}
 using Moq;
 using NUnit.Framework;
 using System.Threading.Tasks;
@@ -19,7 +19,7 @@ namespace ConsoleApp1.Tests
             Mock<ITestInterface> temp = new Mock<ITestInterface>();
             temp.Setup(x => x.BuildSomething(It.IsAny<int>(),It.IsAny<string>(), It.IsAny<bool>()))
 				.ReturnsAsync((int i, string j, bool k) => 42)
-                .Callback<int,string,bool>((i,toto,ok) => {{caret}})
+                .Callback<int, string, bool>((i, toto, ok) => {{caret}})
         }
     }
 }
diff --git a/Abc.MoqComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_generic.cs b/Abc.MoqComplete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_generic.cs
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<ITest2,double,string>((test,test2,test3) => {\})}
+// ${COMPLETE_ITEM:Callback<ITest2, double, string>((test, test2, test3) => {\})}
 using NUnit.Framework;
 using Moq;
 

diff --git a/...Complete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_generic.cs.gold b/...Complete/Abc.MoqComplete.Tests/data/CallbackCompletion/callbackCompletion_generic.cs.gold
@@ -1,4 +1,4 @@
-// ${COMPLETE_ITEM:Callback<ITest2,double,string>((test,test2,test3) => {\})}
+// ${COMPLETE_ITEM:Callback<ITest2, double, string>((test, test2, test3) => {\})}
 using NUnit.Framework;
 using Moq;
 
@@ -23,7 +23,7 @@ namespace ConsoleApp1.Tests
         {
             var m = new Mock<ITest2>();
             m.Setup(x=>x.Coco(It.IsAny<ITest2>(), It.IsAny<double>(), It.IsAny<string>()))
-             .Callback<ITest2,double,string>((test,test2,test3) => {{caret}})
+             .Callback<ITest2, double, string>((test, test2, test3) => {{caret}})
 
         }
     }

diff --git a/Abc.MoqComplete/Abc.MoqComplete/CompletionProvider/Callback/BaseCallbackMethodProvider.cs b/Abc.MoqComplete/Abc.MoqComplete/CompletionProvider/Callback/BaseCallbackMethodProvider.cs
@@ -70,7 +70,7 @@ protected override bool AddLookupItems(CSharpCodeCompletionContext context, IIte
 
 			var types = GetMockedMethodParameterTypes(solution, invocation);
 			var variablesName = mockedMethod.Parameters.Select(p => p.ShortName);
-			var proposedCallback = $"Callback<{string.Join(",", types)}>(({string.Join(",", variablesName)}) => {{}})";
+			var proposedCallback = $"Callback<{string.Join(", ", types)}>(({string.Join(", ", variablesName)}) => {{}})";
 
 			var item = CSharpLookupItemFactory.Instance.CreateKeywordLookupItem(context,
 				proposedCallback,