Browse Source

[CodeIssues] Added failing test case for variable declared in wide

scope issue.
newNRvisualizers
Mike Krüger 13 years ago
parent
commit
2f91ba1df8
  1. 39
      ICSharpCode.NRefactory.Tests/CSharp/CodeIssues/VariableDeclaredInWideScopeTests.cs

39
ICSharpCode.NRefactory.Tests/CSharp/CodeIssues/VariableDeclaredInWideScopeTests.cs

@ -236,7 +236,7 @@ class A
var issues = GetIssues(new VariableDeclaredInWideScopeIssue(), input, out context); var issues = GetIssues(new VariableDeclaredInWideScopeIssue(), input, out context);
Assert.AreEqual(0, issues.Count); Assert.AreEqual(0, issues.Count);
} }
[Test] [Test]
public void DoesNotSuggestMovingIntoClosure () public void DoesNotSuggestMovingIntoClosure ()
{ {
@ -418,6 +418,43 @@ class A
return; return;
", 0); ", 0);
} }
[Ignore("FIXME")]
[Test]
public void DoesNotSuggestMovingIntoBodyAfterMethodCall()
{
var input = @"
using System.IO;
class FooBar
{
public int foo = 5;
public void ChangeFoo ()
{
foo = 10;
}
}
class A
{
FooBar foo = new FooBar();
public void F()
{
int length = foo.foo;
foo.ChangeFoo ();
if (true) {
System.Console.WriteLine (length);
}
}
}";
TestRefactoringContext context;
var issues = GetIssues(new VariableDeclaredInWideScopeIssue(), input, out context);
Assert.AreEqual(0, issues.Count);
}
} }
} }

Loading…
Cancel
Save