Skip to content

Commit 384725e

Browse files
authored
Use Operations instead of SyntaxNodes for analysis (#177)
1 parent 5a2a6cf commit 384725e

30 files changed

Lines changed: 85 additions & 154 deletions

System.IO.Abstractions.Analyzers.Tests/Analyzers/DirectoryInfoAnalyzerTests.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,6 @@ public void Analyzer_is_not_triggered(string filename)
2121

2222
[Theory]
2323
[InlineData("WithOutFileSystem.txt", 15, 28)]
24-
[InlineData("StaticInvocation.txt", 15, 40)]
2524
public void Analyzer_is_triggered(string filename, int diagnosticLine, int diagnosticColumn)
2625
{
2726
var source = ReadFile(filename);

System.IO.Abstractions.Analyzers.Tests/Analyzers/PathAnalyzerTests.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ public class PathAnalyzerTests : CSharpDiagnosticAnalyzerTest<PathAnalyzer>
1313
[InlineData("Valid.txt")]
1414
[InlineData("UsingStaticFalsePositive.txt")]
1515
[InlineData("UsingStaticLinqFalsePositive.txt")]
16+
[InlineData("NonRelatedFieldFalsePositive.txt")]
1617
public void Analyzer_is_not_triggered(string filename)
1718
{
1819
var source = ReadFile(filename);

System.IO.Abstractions.Analyzers.Tests/TestData/Analyzer/DirectoryAnalyzer/WithOutFileSystem.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ namespace SomeNameSpace
1212
{
1313
const string path = "C:\\temp";
1414

15-
Directory.Remove(path);
15+
Directory.Delete(path);
1616
}
1717
}
1818
}

System.IO.Abstractions.Analyzers.Tests/TestData/Analyzer/DirectoryInfoAnalyzer/StaticInvocation.txt

Lines changed: 0 additions & 18 deletions
This file was deleted.
Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,12 @@
1+
namespace SomeNameSpace
2+
{
3+
public class NonRelatedFieldFalsePositive
4+
{
5+
private const string Path = "Some/Path/To/Something";
6+
7+
public string SomeMethod()
8+
{
9+
return Path.Replace('/', '_');
10+
}
11+
}
12+
}

System.IO.Abstractions.Analyzers.Tests/TestData/CodeFix/DirectoryCodeFix/AfterFix.txt

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using System.IO;
12
using System.IO.Abstractions;
23

34
namespace SomeNameSpace
@@ -15,7 +16,9 @@ namespace SomeNameSpace
1516
{
1617
const string filePath = "C:\\temp";
1718

18-
new ConfigurationBuilder().SetBasePath(_fileSystem.Directory.GetCurrentDirectory());
19+
SetBasePath(_fileSystem.Directory.GetCurrentDirectory());
1920
}
21+
22+
private void SetBasePath(string path) {}
2023
}
2124
}

System.IO.Abstractions.Analyzers.Tests/TestData/CodeFix/DirectoryCodeFix/BeforeFix.txt

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using System.IO;
12
using System.IO.Abstractions;
23

34
namespace SomeNameSpace
@@ -15,7 +16,9 @@ namespace SomeNameSpace
1516
{
1617
const string filePath = "C:\\temp";
1718

18-
new ConfigurationBuilder().SetBasePath(Directory.GetCurrentDirectory());
19+
SetBasePath(Directory.GetCurrentDirectory());
1920
}
21+
22+
private void SetBasePath(string path) {}
2023
}
2124
}

System.IO.Abstractions.Analyzers.Tests/TestData/CodeFix/DirectoryInfoCodeFix/AfterFix.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using System.IO;
12
using System.IO.Abstractions;
23

34
namespace SomeNameSpace
@@ -15,7 +16,7 @@ namespace SomeNameSpace
1516
{
1617
const string filePath = "C:\\temp.txt";
1718

18-
DirectoryInfo dirInfo = _fileSystem.DirectoryInfo.FromDirectoryName(filePath);
19+
var dirInfo = _fileSystem.DirectoryInfo.New(filePath);
1920
}
2021
}
2122
}

System.IO.Abstractions.Analyzers.Tests/TestData/CodeFix/DirectoryInfoCodeFix/BeforeFix.txt

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using System.IO;
12
using System.IO.Abstractions;
23

34
namespace SomeNameSpace
@@ -15,7 +16,7 @@ namespace SomeNameSpace
1516
{
1617
const string filePath = "C:\\temp.txt";
1718

18-
DirectoryInfo dirInfo = new DirectoryInfo(filePath);
19+
var dirInfo = new DirectoryInfo(filePath);
1920
}
2021
}
2122
}

System.IO.Abstractions.Analyzers.Tests/TestData/CodeFix/FileCodeFix/AfterFix.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
using System.IO;
12
using System.IO.Abstractions;
23

34
namespace SomeNameSpace

0 commit comments

Comments
 (0)