You can not select more than 25 topics
Topics must start with a letter or number, can include dashes ('-') and can be up to 35 characters long.
233 lines
5.6 KiB
233 lines
5.6 KiB
// |
|
// TestFormattingBugs.cs |
|
// |
|
// Author: |
|
// Mike Krüger <mkrueger@novell.com> |
|
// |
|
// Copyright (c) 2010 Novell, Inc (http://www.novell.com) |
|
// |
|
// Permission is hereby granted, free of charge, to any person obtaining a copy |
|
// of this software and associated documentation files (the "Software"), to deal |
|
// in the Software without restriction, including without limitation the rights |
|
// to use, copy, modify, merge, publish, distribute, sublicense, and/or sell |
|
// copies of the Software, and to permit persons to whom the Software is |
|
// furnished to do so, subject to the following conditions: |
|
// |
|
// The above copyright notice and this permission notice shall be included in |
|
// all copies or substantial portions of the Software. |
|
// |
|
// THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR |
|
// IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY, |
|
// FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE |
|
// AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER |
|
// LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, |
|
// OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN |
|
// THE SOFTWARE. |
|
|
|
using System; |
|
using System.IO; |
|
using NUnit.Framework; |
|
using ICSharpCode.NRefactory.CSharp; |
|
|
|
namespace ICSharpCode.NRefactory.CSharp.FormattingTests |
|
{ |
|
[TestFixture()] |
|
public class TestFormattingBugs : TestBase |
|
{ |
|
/// <summary> |
|
/// Bug 325187 - Bug in smart indent |
|
/// </summary> |
|
[Test()] |
|
public void TestBug325187 () |
|
{ |
|
CSharpFormattingOptions policy = FormattingOptionsFactory.CreateMono (); |
|
policy.PlaceElseOnNewLine = true; |
|
|
|
TestStatementFormatting (policy, |
|
@"foreach (int i in myints) |
|
if (i == 6) |
|
Console.WriteLine (""Yeah""); |
|
else |
|
Console.WriteLine (""Bad indent"");", |
|
@"foreach (int i in myints) |
|
if (i == 6) |
|
Console.WriteLine (""Yeah""); |
|
else |
|
Console.WriteLine (""Bad indent"");"); |
|
} |
|
|
|
/// <summary> |
|
/// Bug 415469 - return ternary in a switch is not tabbed properly |
|
/// </summary> |
|
[Test()] |
|
public void TestBug415469 () |
|
{ |
|
CSharpFormattingOptions policy = FormattingOptionsFactory.CreateMono (); |
|
|
|
TestStatementFormatting (policy, |
|
@"switch (condition) { |
|
case CONDITION1: |
|
return foo != null ? foo.Bar : null; |
|
case CONDITION2: |
|
string goo = foo != null ? foo.Bar : null; |
|
return ""Should be indented like this""; |
|
}", @"switch (condition) { |
|
case CONDITION1: |
|
return foo != null ? foo.Bar : null; |
|
case CONDITION2: |
|
string goo = foo != null ? foo.Bar : null; |
|
return ""Should be indented like this""; |
|
}"); |
|
} |
|
|
|
/// <summary> |
|
/// Bug 540043 - Format option for alignment of using-statements |
|
/// </summary> |
|
[Test()] |
|
public void TestBug540043 () |
|
{ |
|
CSharpFormattingOptions policy = FormattingOptionsFactory.CreateMono (); |
|
|
|
TestStatementFormatting (policy, |
|
@"using (IDisposable a = null) |
|
using (IDisposable b = null) { |
|
int c; |
|
} |
|
", @"using (IDisposable a = null) |
|
using (IDisposable b = null) { |
|
int c; |
|
}"); |
|
} |
|
|
|
/// <summary> |
|
/// Bug 655635 - Auto format document doesn't indent comments as well |
|
/// </summary> |
|
[Test()] |
|
public void TestBug655635 () |
|
{ |
|
CSharpFormattingOptions policy = FormattingOptionsFactory.CreateMono (); |
|
|
|
TestStatementFormatting (policy, |
|
@"try { |
|
// Comment 1 |
|
myObject.x = Run (); |
|
} catch (InvalidOperationException e) { |
|
Console.WriteLine (e.Message); |
|
}", @"try { |
|
// Comment 1 |
|
myObject.x = Run (); |
|
} catch (InvalidOperationException e) { |
|
Console.WriteLine (e.Message); |
|
}"); |
|
} |
|
|
|
void TestStatementFormatting (CSharpFormattingOptions policy, string input, string expectedOutput) |
|
{ |
|
var result = GetResult (policy, @"class Test |
|
{ |
|
MyType TestMethod () |
|
{ |
|
" + input + @" |
|
} |
|
}"); |
|
int start = result.GetOffset (5, 1); |
|
int end = result.GetOffset (result.LineCount - 1, 1); |
|
string text = result.GetText (start, end - start).Trim (); |
|
expectedOutput = NormalizeNewlines(expectedOutput).Replace ("\n", "\n\t\t"); |
|
if (expectedOutput != text) |
|
Console.WriteLine (text); |
|
Assert.AreEqual (expectedOutput, text); |
|
} |
|
|
|
/// <summary> |
|
/// Bug 659675 - on-the-fly code formatting breaks @-prefixed identifiers |
|
/// </summary> |
|
[Test()] |
|
public void TestBug659675 () |
|
{ |
|
CSharpFormattingOptions policy = FormattingOptionsFactory.CreateMono (); |
|
TestStatementFormatting (policy, "@string=@int;", "@string = @int;"); |
|
} |
|
|
|
/// <summary> |
|
/// Bug 670213 - Document formatter deletes valid text! |
|
/// </summary> |
|
[Test()] |
|
public void TestBug670213 () |
|
{ |
|
CSharpFormattingOptions policy = FormattingOptionsFactory.CreateMono (); |
|
policy.MethodBraceStyle = BraceStyle.EndOfLine; |
|
|
|
Test (policy, @"class Test |
|
{ |
|
Test MyMethod() // Comment |
|
{ |
|
} |
|
}", |
|
@"class Test |
|
{ |
|
Test MyMethod () { // Comment |
|
} |
|
}"); |
|
} |
|
|
|
|
|
/// <summary> |
|
/// Bug 677261 - Format Document with constructor with over-indented opening curly brace |
|
/// </summary> |
|
[Test()] |
|
public void TestBug677261 () |
|
{ |
|
CSharpFormattingOptions policy = FormattingOptionsFactory.CreateMono (); |
|
policy.ConstructorBraceStyle = BraceStyle.EndOfLine; |
|
|
|
Test (policy, @"class Test |
|
{ |
|
Test () |
|
{ |
|
} |
|
}", |
|
@"class Test |
|
{ |
|
Test () { |
|
} |
|
}"); |
|
} |
|
|
|
/// <summary> |
|
/// Bug 3586 -Format code is removing try { code blocks |
|
/// </summary> |
|
[Test()] |
|
public void TestBug3586 () |
|
{ |
|
var policy = FormattingOptionsFactory.CreateMono (); |
|
policy.ConstructorBraceStyle = BraceStyle.EndOfLine; |
|
|
|
Test (policy, @"public class A |
|
{ |
|
public object GetValue () |
|
{ |
|
try |
|
{ |
|
foo (); |
|
} |
|
catch |
|
{ |
|
} |
|
} |
|
}", |
|
@"public class A |
|
{ |
|
public object GetValue () |
|
{ |
|
try { |
|
foo (); |
|
} catch { |
|
} |
|
} |
|
}"); |
|
} |
|
|
|
} |
|
} |
|
|
|
|