Merge pull request #36 from PavelBansky/master

Fixed issue with file writer
microsoft · Dec 1, 2017 · cc8530a · cc8530a
2 parents d6ab2c0 + e19a859
commit cc8530a
Show file tree

Hide file tree

Showing 4 changed files with 26 additions and 13 deletions.
diff --git a/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Commands/AnalyzeCommand.cs b/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Commands/AnalyzeCommand.cs
@@ -140,7 +140,7 @@ public int Run()
             Writer outputWriter = WriterFactory.GetWriter(_fileFormat, _outputFormat);            
             if (string.IsNullOrEmpty(_outputFile))
                 outputWriter.TextWriter= Console.Out;
-            else            
+            else 
                 outputWriter.TextWriter = File.CreateText(_outputFile);            
 
             int filesAnalyzed = 0;

diff --git a/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Writers/DummyWriter.cs b/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Writers/DummyWriter.cs
@@ -0,0 +1,19 @@
+// Copyright (C) Microsoft. All rights reserved.
+// Licensed under the MIT License. See LICENSE.txt in the project root for license information.
+
+namespace Microsoft.DevSkim.CLI.Writers
+{
+    public class DummyWriter : Writer
+    {
+        public override void WriteIssue(IssueRecord issue)
+        {
+            // This is intentionaly empty
+        }
+
+        public override void FlushAndClose()
+        {
+            // This is intentionaly empty
+        }
+
+    }
+}
diff --git a/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Writers/IWritter.cs b/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Writers/IWritter.cs
diff --git a/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Writers/WriterFactory.cs b/src/Microsoft.DevSkim/Microsoft.DevSkim.CLI/Writers/WriterFactory.cs
@@ -9,8 +9,13 @@ public class WriterFactory
     {
         public static Writer GetWriter(string writerName, string format = null)
         {
+            if (string.IsNullOrEmpty(writerName))
+                writerName = "_dummy";
+
             switch (writerName.ToLowerInvariant())
-            {
+            {  
+                case "_dummy":
+                    return new DummyWriter();
                 case "json":
                     return new JsonWriter(format);                    
                 case "text":