inserting overrides inserts a new line fixes #106

OmniSharp.Tests/Overrides/TestOverrideContext.cs

@@ -1,14 +1,16 @@
 using System.Linq;
 using NUnit.Framework;
+using Should;
 using OmniSharp.Common;
 using OmniSharp.Overrides;
 using OmniSharp.Parser;
 using OmniSharp.Tests.Rename;
+using OmniSharp.Configuration;
 
 namespace OmniSharp.Tests.Overrides
 {
     [TestFixture]
-    public class TestOverrideContext
+    public class OverridesTest
     {
         [Test]
         public void Should_not_offer_already_overridden_method()
@@ -32,5 +34,36 @@ public override string ToString()
             overrides.ShouldNotContain("public virtual string ToString ();");
 
         }
+
+        [Test]
+        public void Should_insert_stub_method_override()
+        {
+            var buffer = 
+@"
+public class WeirdString : String
+{
+$
+}";
+            var location = TestHelpers.GetLineAndColumnFromDollar(buffer);
+            buffer = buffer.Replace("$","");
+            var request = new RunOverrideTargetRequest { Buffer = buffer, FileName = "myfile.cs", Line = location.Line, Column = location.Column };
+            var solution = new FakeSolutionBuilder().AddFile(buffer, "myfile.cs").Build();
+            var parser = new BufferParser(solution);
+            var handler = new OverrideHandler (parser, new OmniSharpConfiguration ());
+            request.OverrideTargetName = "public virtual string ToString ();";
+            var response = handler.RunOverrideTarget (request);
+            string expected = 
+@"
+public class WeirdString : String
+{
+
+    public override string ToString()
+    {
+        throw new System.NotImplementedException();
+    }
+}";
+            string result = response.Buffer.Replace ("\r\n", "\n");
+            result.ShouldEqual(expected);
+        }
     }
 }

OmniSharp/Configuration/OmniSharpConfiguration.cs

@@ -10,6 +10,7 @@ public OmniSharpConfiguration()
             PathReplacements = new List<PathReplacement>();
             IgnoredCodeIssues = new List<string>();
             TextEditorOptions = new TextEditorOptions ();
+            TextEditorOptions.TabsToSpaces = true;
             CSharpFormattingOptions = FormattingOptionsFactory.CreateAllman();
         }
 

OmniSharp/Overrides/OverrideHandler.cs

@@ -106,11 +106,11 @@ IDocument runOverrideTargetWorker
                 , n => n.NodeType == NodeType.TypeDeclaration);
 
             // Even empty classes have nodes, so this works
-            var memberBeforeClosingBraceNode =
-                typeDeclaration.Children.Last().GetPrevNode();
+            var lastNode =
+                typeDeclaration.Children.Last();
 
-            script.InsertAfter
-                ( node    : memberBeforeClosingBraceNode
+            script.InsertBefore
+                ( node    : lastNode
                 , newNode : memberDeclaration);
             script.FormatText(memberDeclaration);
 

OmniSharp/SyntaxErrors/SyntaxErrorsHandler.cs

@@ -0,0 +1 @@
+jason@Jasons-MacBook-Pro.local.18686

packages/repositories.config

@@ -1,5 +1,7 @@
-<?xml version="1.0" encoding="utf-8"?>
-<repositories>
-  <repository path="..\Omnisharp.Tests\packages.config" />
-  <repository path="..\OmniSharp\packages.config" />
+<?xml version="1.0" encoding="utf-8"?>
+<repositories>
+  <repository path="../OmniSharp.Tests/packages.config" />
+  <repository path="../OmniSharp/packages.config" />
+  <repository path="..\Omnisharp.Tests\packages.config" />
+  <repository path="..\OmniSharp\packages.config" />
 </repositories>