Add simple json file export, do not crash on bad projects

README.md

@@ -118,6 +118,19 @@ _Examine a specific project or solution to make sure there are no pre-release pa
 > snitch MyProject.csproj --no-prerelease
 ```
 
+_Examine a specific project or solution and export the result to a json file
+
+```
+> snitch MyProject.csproj --out c:\temp\snitch.json
+```
+
+That json file can be also used e.g. to auto-uninstall the detected packages in package manager console in Visual Studio:
+```
+function Uninstall-FromSnitch { param($filename) (Get-Content $filename | ConvertFrom-Json) | %{ $p = $_.Project; foreach ($c in $_.CanBeRemoved) { Uninstall-Package $c.PackageName -ProjectName $p } } }
+Uninstall-FromSnitch C:\temp\snitch.json
+```
+
+
 ## Building Snitch from source
 
 ```

src/Snitch.Tests.Fixtures/FooBar/Class1.cs

@@ -0,0 +1,8 @@
+using System;
+
+namespace Baz
+{
+    public class Class1
+    {
+    }
+}

src/Snitch.Tests.Fixtures/FooBar/FooBar.csproj

@@ -0,0 +1,8 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+  <PropertyGroup>
+    <TargetFrameworkIdentifier>Silverlight</TargetFrameworkIdentifier>
+    <TargetFrameworkVersion>v5.0</TargetFrameworkVersion>
+  </PropertyGroup>
+
+</Project>

src/Snitch.Tests/Expectations/Baz.Json.verified.json

@@ -0,0 +1,14 @@
+[
+  {
+    "Project": "Baz",
+    "CanBeRemoved": [
+      {
+        "PackageName": "Autofac",
+        "PackageVersion": "4.9.4",
+        "ReferencedBy": "Foo"
+      }
+    ],
+    "MightBeRemoved": [],
+    "PreRelease": []
+  }
+]

src/Snitch.Tests/Expectations/FooBar.Default.verified.txt

@@ -0,0 +1,6 @@
+Analyzing...
+Analyzing FooBar.csproj
+Analyzing FooBar...
+  ERROR: Value cannot be null. (Parameter 'folderName')
+
+Everything looks good!

src/Snitch.Tests/Expectations/FooBar.Strict.verified.txt

@@ -0,0 +1,4 @@
+Analyzing...
+Analyzing FooBar.csproj
+Analyzing FooBar...
+  ERROR: Value cannot be null. (Parameter 'folderName')

src/Snitch.Tests/Expectations/Solution.Json.verified.json

@@ -0,0 +1,87 @@
+[
+  {
+    "Project": "Bar",
+    "CanBeRemoved": [
+      {
+        "PackageName": "Autofac",
+        "PackageVersion": "4.9.4",
+        "ReferencedBy": "Foo"
+      }
+    ],
+    "MightBeRemoved": [],
+    "PreRelease": []
+  },
+  {
+    "Project": "Baz",
+    "CanBeRemoved": [
+      {
+        "PackageName": "Autofac",
+        "PackageVersion": "4.9.4",
+        "ReferencedBy": "Foo"
+      }
+    ],
+    "MightBeRemoved": [],
+    "PreRelease": []
+  },
+  {
+    "Project": "Qux",
+    "CanBeRemoved": [],
+    "MightBeRemoved": [
+      {
+        "PackageName": "Autofac",
+        "PackageVersion": "4.9.3",
+        "ReferencedBy": "Foo",
+        "ReferencePackageVersion": "4.9.4"
+      }
+    ],
+    "PreRelease": []
+  },
+  {
+    "Project": "Zap",
+    "CanBeRemoved": [],
+    "MightBeRemoved": [
+      {
+        "PackageName": "Newtonsoft.Json",
+        "PackageVersion": "12.0.3",
+        "ReferencedBy": "Foo",
+        "ReferencePackageVersion": "12.0.1"
+      },
+      {
+        "PackageName": "Autofac",
+        "PackageVersion": "4.9.3",
+        "ReferencedBy": "Foo",
+        "ReferencePackageVersion": "4.9.4"
+      }
+    ],
+    "PreRelease": []
+  },
+  {
+    "Project": "Thud",
+    "CanBeRemoved": [],
+    "MightBeRemoved": [],
+    "PreRelease": [
+      {
+        "PackageName": "Newtonsoft.Json",
+        "PackageVersion": "13.0.2-beta2"
+      }
+    ]
+  },
+  {
+    "Project": "Thuuud",
+    "CanBeRemoved": [],
+    "MightBeRemoved": [
+      {
+        "PackageName": "Newtonsoft.Json",
+        "PackageVersion": "13.0.2-beta2",
+        "ReferencedBy": "Foo",
+        "ReferencePackageVersion": "12.0.1"
+      }
+    ],
+    "PreRelease": [
+      {
+        "PackageName": "Newtonsoft.Json",
+        "PackageVersion": "13.0.2-beta2"
+      }
+    ]
+  }
+]

src/Snitch.Tests/ProgramTests.cs

@@ -1,5 +1,4 @@
 using Shouldly;
-using Snitch;
 using System;
 using System.IO;
 using System.Threading.Tasks;
@@ -9,7 +8,7 @@
 using Xunit;
 using VerifyXunit;
 
-namespace Sntich.Tests
+namespace Snitch.Tests
 {
     [UsesVerify]
     public class ProgramTests
@@ -19,7 +18,6 @@ public class ProgramTests
         public async Task Should_Return_Expected_Result_For_Baz_Not_Specifying_Framework()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Baz/Baz.csproj");
 
             // When
@@ -30,12 +28,27 @@ public async Task Should_Return_Expected_Result_For_Baz_Not_Specifying_Framework
             await Verifier.Verify(output);
         }
 
+        [Fact]
+        [Expectation("Baz", "Json")]
+        public async Task Should_Return_Expected_Json_For_Baz_Not_Specifying_Framework()
+        {
+            // Given
+            var project = Fixture.GetPath("Baz/Baz.csproj");
+
+            // When
+            using var tempFileScope = new TempFileScope();
+            var (exitCode, _) = await Fixture.Run(project, "--out", tempFileScope.FileName);
+
+            // Then
+            exitCode.ShouldBe(0);
+            await Verifier.VerifyFile(tempFileScope.FileName);
+        }
+
         [Fact]
         [Expectation("Solution", "Default")]
         public async Task Should_Return_Expected_Result_For_Solution_Not_Specifying_Framework()
         {
             // Given
-            var fixture = new Fixture();
             var solution = Fixture.GetPath("Snitch.Tests.Fixtures.sln");
 
             // When
@@ -46,12 +59,27 @@ public async Task Should_Return_Expected_Result_For_Solution_Not_Specifying_Fram
             await Verifier.Verify(output);
         }
 
+        [Fact]
+        [Expectation("Solution", "Json")]
+        public async Task Should_Return_Expected_Json_For_Solution_Not_Specifying_Framework()
+        {
+            // Given
+            var solution = Fixture.GetPath("Snitch.Tests.Fixtures.sln");
+
+            // When
+            using var tempFileScope = new TempFileScope();
+            var (exitCode, _) = await Fixture.Run(solution, "--out", tempFileScope.FileName);
+
+            // Then
+            exitCode.ShouldBe(0);
+            await Verifier.VerifyFile(tempFileScope.FileName);
+        }
+
         [Fact]
         [Expectation("Baz", "netstandard2.0")]
         public async Task Should_Return_Expected_Result_For_Baz_Specifying_Framework()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Baz/Baz.csproj");
 
             // When
@@ -67,7 +95,6 @@ public async Task Should_Return_Expected_Result_For_Baz_Specifying_Framework()
         public async Task Should_Return_Non_Zero_Exit_Code_For_Baz_When_Running_With_Strict()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Baz/Baz.csproj");
 
             // When
@@ -83,7 +110,6 @@ public async Task Should_Return_Non_Zero_Exit_Code_For_Baz_When_Running_With_Str
         public async Task Should_Return_Expected_Result_For_Baz_When_Excluding_Library()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Baz/Baz.csproj");
 
             // When
@@ -99,7 +125,6 @@ public async Task Should_Return_Expected_Result_For_Baz_When_Excluding_Library()
         public async Task Should_Return_Expected_Result_For_Baz_When_Skipping_Project()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Baz/Baz.csproj");
 
             // When
@@ -115,7 +140,6 @@ public async Task Should_Return_Expected_Result_For_Baz_When_Skipping_Project()
         public async Task Should_Return_Expected_Result_For_Baz_When_Skipping_Project_And_NoReleases()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Baz/Baz.csproj");
 
             // When
@@ -131,7 +155,6 @@ public async Task Should_Return_Expected_Result_For_Baz_When_Skipping_Project_An
         public async Task Should_Return_Non_Zero_Exit_Code_For_Baz_When_Running_With_Strict_And_NoPreRelease()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Baz/Baz.csproj");
 
             // When
@@ -147,7 +170,6 @@ public async Task Should_Return_Non_Zero_Exit_Code_For_Baz_When_Running_With_Str
         public async Task Should_Return_Non_Zero_Exit_Code_For_Thud_When_Running_With_Strict_And_NoPreRelease()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Thud/Thud.csproj");
 
             // When
@@ -163,7 +185,6 @@ public async Task Should_Return_Non_Zero_Exit_Code_For_Thud_When_Running_With_St
         public async Task Should_Return_Non_Zero_Exit_Code_For_Thuuud_When_Running_With_Strict_And_NoPreRelease()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Thuuud/Thuuud.csproj");
 
             // When
@@ -179,7 +200,6 @@ public async Task Should_Return_Non_Zero_Exit_Code_For_Thuuud_When_Running_With_
         public async Task Should_Return_Zero_Exit_Code_For_Thuuud_When_Running_With_NoPreRelease()
         {
             // Given
-            var fixture = new Fixture();
             var project = Fixture.GetPath("Thuuud/Thuuud.csproj");
 
             // When
@@ -190,7 +210,52 @@ public async Task Should_Return_Zero_Exit_Code_For_Thuuud_When_Running_With_NoPr
             await Verifier.Verify(output);
         }
 
-        public sealed class Fixture
+        [Fact]
+        [Expectation("FooBar", "Default")]
+        public async Task Should_Print_Error_For_FooBar()
+        {
+            // Given
+            var project = Fixture.GetPath("FooBar/FooBar.csproj");
+
+            // When
+            var (exitCode, output) = await Fixture.Run(project);
+
+            // Then
+            exitCode.ShouldBe(0);
+            await Verifier.Verify(output);
+        }
+
+        [Fact]
+        [Expectation("FooBar", "Strict")]
+        public async Task Should_Return_NonZero_Exit_Code_For_FooBar_When_Running_With_Strict()
+        {
+            // Given
+            var project = Fixture.GetPath("FooBar/FooBar.csproj");
+
+            // When
+            var (exitCode, output) = await Fixture.Run(project, "--strict");
+
+            // Then
+            exitCode.ShouldBe(-1);
+            await Verifier.Verify(output);
+        }
+
+        [Fact]
+        [Expectation("FSharp", "Default")]
+        public async Task Should_Return_Expected_Result_For_FSharp_Not_Specifying_Framework()
+        {
+            // Given
+            var project = Fixture.GetPath("FSharp/FSharp.fsproj");
+
+            // When
+            var (exitCode, output) = await Fixture.Run(project);
+
+            // Then
+            exitCode.ShouldBe(0);
+            await Verifier.Verify(output);
+        }
+
+        private static class Fixture
         {
             public static string GetPath(string path)
             {
@@ -207,20 +272,22 @@ public static string GetPath(string path)
             }
         }
 
-        [Fact]
-        [Expectation("FSharp", "Default")]
-        public async Task Should_Return_Expected_Result_For_FSharp_Not_Specifying_Framework()
+        private sealed class TempFileScope : IDisposable
         {
-            // Given
-            var fixture = new Fixture();
-            var project = Fixture.GetPath("FSharp/FSharp.fsproj");
+            public TempFileScope()
+            {
+                this.FileName = Path.GetTempFileName() + ".json";
+            }
 
-            // When
-            var (exitCode, output) = await Fixture.Run(project);
+            public string FileName { get; }
 
-            // Then
-            exitCode.ShouldBe(0);
-            await Verifier.Verify(output);
+            public void Dispose()
+            {
+                if (File.Exists(this.FileName))
+                {
+                    File.Delete(this.FileName);
+                }
+            }
         }
     }
 }

src/Snitch.Tests/VerifyConfiguration.cs

@@ -1,7 +1,7 @@
 using System.Runtime.CompilerServices;
 using VerifyTests;
 
-namespace Sntich.Tests
+namespace Snitch.Tests
 {
     public static class VerifyConfiguration
     {