Merge pull request #79

check invoked members data type for being from Abstractions

Author: inyutin-maxim

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

@@ -7,12 +7,13 @@
 
 namespace System.IO.Abstractions.Analyzers.Tests.Analyzers
 {
-	public class DirectoryAnalyzerTests: CSharpDiagnosticAnalyzerTest<DirectoryAnalyzer>
+	public class DirectoryAnalyzerTests : CSharpDiagnosticAnalyzerTest<DirectoryAnalyzer>
 	{
 		[Theory]
 		[InlineData("Valid.txt")]
 		[InlineData("FalsePositive.txt")]
 		[InlineData("ModelObjectFalsePositive.txt")]
+		[InlineData("UsingStaticFalsePositive.txt")]
 		public void Analyzer_is_not_triggered(string filename)
 		{
 			var source = ReadFile(filename);

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

@@ -7,11 +7,12 @@
 
 namespace System.IO.Abstractions.Analyzers.Tests.Analyzers
 {
-	public class DirectoryInfoAnalyzerTests: CSharpDiagnosticAnalyzerTest<DirectoryInfoAnalyzer>
+	public class DirectoryInfoAnalyzerTests : CSharpDiagnosticAnalyzerTest<DirectoryInfoAnalyzer>
 	{
 		[Theory]
 		[InlineData("Valid.txt")]
 		[InlineData("ModelObjectFalsePositive.txt")]
+		[InlineData("UsingStaticFalsePositive.txt")]
 		public void Analyzer_is_not_triggered(string filename)
 		{
 			var source = ReadFile(filename);

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

@@ -11,6 +11,7 @@ public class FileAnalyzerTests : CSharpDiagnosticAnalyzerTest<FileAnalyzer>
 	{
 		[Theory]
 		[InlineData("Valid.txt")]
+		[InlineData("UsingStaticFalsePositive.txt")]
 		public void Analyzer_is_not_triggered(string filename)
 		{
 			var source = ReadFile(filename);

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

@@ -7,10 +7,11 @@
 
 namespace System.IO.Abstractions.Analyzers.Tests.Analyzers
 {
-	public class FileInfoAnalyzerTests: CSharpDiagnosticAnalyzerTest<FileInfoAnalyzer>
+	public class FileInfoAnalyzerTests : CSharpDiagnosticAnalyzerTest<FileInfoAnalyzer>
 	{
 		[Theory]
 		[InlineData("Valid.txt")]
+		[InlineData("UsingStaticFalsePositive.txt")]
 		public void Analyzer_is_not_triggered(string filename)
 		{
 			var source = ReadFile(filename);

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

@@ -7,10 +7,11 @@
 
 namespace System.IO.Abstractions.Analyzers.Tests.Analyzers
 {
-	public class FileStreamAnalyzerTests: CSharpDiagnosticAnalyzerTest<FileStreamAnalyzer>
+	public class FileStreamAnalyzerTests : CSharpDiagnosticAnalyzerTest<FileStreamAnalyzer>
 	{
 		[Theory]
 		[InlineData("Valid.txt")]
+		[InlineData("UsingStaticFalsePositive.txt")]
 		public void Analyzer_is_not_triggered(string filename)
 		{
 			var source = ReadFile(filename);

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

@@ -11,6 +11,7 @@ public class PathAnalyzerTests : CSharpDiagnosticAnalyzerTest<PathAnalyzer>
 	{
 		[Theory]
 		[InlineData("Valid.txt")]
+		[InlineData("UsingStaticFalsePositive.txt")]
 		public void Analyzer_is_not_triggered(string filename)
 		{
 			var source = ReadFile(filename);

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

@@ -0,0 +1,26 @@
+namespace SomeNameSpace.Mocking
+{
+	using System.IO.Abstractions;
+	public static class FileSystemProxy
+	{
+		internal static IDirectory Directory { get; }
+	}
+}
+
+namespace SomeNameSpace
+{
+	using static SomeNameSpace.Mocking.FileSystemProxy;
+	public class UsingStaticFalsePositive
+	{
+		public UsingStaticFalsePositive()
+		{
+		}
+
+		public void SomeMethod()
+		{
+			const string path = "C:\\temp";
+
+			Directory.Delete(path);
+		}
+	}
+}

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

@@ -0,0 +1,26 @@
+namespace SomeNameSpace.Mocking
+{
+	using System.IO.Abstractions;
+	public static class FileSystemProxy
+	{
+		internal static IDirectoryInfo DirectoryInfo { get; }
+	}
+}
+
+namespace SomeNameSpace
+{
+	using System.IO.Abstractions;
+	using static SomeNameSpace.Mocking.FileSystemProxy;
+	public class UsingStaticFalsePositive
+	{
+		public UsingStaticFalsePositive()
+		{
+		}
+
+		public void SomeMethod()
+		{
+			const string path = "C:\\temp.txt";
+			IDirectoryInfo dirInfo = DirectoryInfo.FromDirectoryName(path);
+		}
+	}
+}

System.IO.Abstractions.Analyzers.Tests/TestData/Analyzer/FileAnalyzer/UsingStaticFalsePositive.txt

@@ -0,0 +1,26 @@
+namespace SomeNameSpace.Mocking
+{
+    using System.IO.Abstractions;
+    public static class FileSystemProxy
+    {
+        internal static IFile File { get; }
+    }
+}
+
+namespace SomeNameSpace
+{
+    using static SomeNameSpace.Mocking.FileSystemProxy;
+    public class UsingStaticFalsePositive
+    {
+        public UsingStaticFalsePositive()
+        {
+        }
+
+        public void SomeMethod()
+        {
+            const string filePath = "C:\\temp.txt";
+
+            File.Delete(filePath);
+        }
+    }
+}

System.IO.Abstractions.Analyzers.Tests/TestData/Analyzer/FileInfoAnalyzer/UsingStaticFalsePositive.txt

@@ -0,0 +1,27 @@
+namespace SomeNameSpace.Mocking
+{
+	using System.IO.Abstractions;
+	public static class FileSystemProxy
+	{
+		internal static IFileInfo FileInfo { get; }
+	}
+}
+
+namespace SomeNameSpace
+{
+	using System.IO.Abstractions;
+	using static SomeNameSpace.Mocking.FileSystemProxy;
+	public class UsingStaticFalsePositive
+	{
+		public UsingStaticFalsePositive()
+		{
+		}
+
+		public void SomeMethod()
+		{
+			const string path = "C:\\temp.txt";
+
+			IFileInfo fileInf = FileInfo.FromFileName(path);
+		}
+	}
+}