Merge pull request #17 from red-gate/NoneOutputType

Display None for void output type.

Author: ChrisLambrou

XmlDoc2CmdletDoc.Core/Engine.cs

@@ -500,7 +500,7 @@ private XElement GenerateReturnValuesElement(ICommentReader commentReader, Comma
             var returnValueElement = new XElement(commandNs + "returnValues");
             foreach (var type in command.OutputTypes)
             {
-                returnValueElement.Add(GenerateComment("OutputType: " + type.Name));
+                returnValueElement.Add(GenerateComment("OutputType: " + (type == typeof(void) ? "None" : type.Name)));
                 var returnValueDescription = commentReader.GetOutputTypeDescriptionElement(command, type, reportWarning);
                 returnValueElement.Add(new XElement(commandNs + "returnValue",
                                                     GenerateTypeElement(commentReader, type, returnValueDescription == null, reportWarning),
@@ -558,7 +558,7 @@ private XElement GenerateRelatedLinksElement(ICommentReader commentReader, Comma
         private XElement GenerateTypeElement(ICommentReader commentReader, Type type, bool includeMamlDescription, ReportWarning reportWarning)
         {
             return new XElement(devNs + "type",
-                                new XElement(mamlNs + "name", type.FullName),
+                                new XElement(mamlNs + "name", type == typeof(void) ? "None" : type.FullName),
                                 new XElement(mamlNs + "uri"),
                                 includeMamlDescription ? commentReader.GetTypeDescriptionElement(type, reportWarning) : null);
         }

XmlDoc2CmdletDoc.TestModule/Manual/TestManualElementsCommand.cs

@@ -54,6 +54,7 @@ namespace XmlDoc2CmdletDoc.TestModule.Manual
     [Cmdlet(VerbsDiagnostic.Test, "ManualElements")]
     [OutputType(typeof(ManualClass))]
     [OutputType(typeof(string))]
+    [OutputType(typeof(void))]
     public class TestManualElementsCommand : Cmdlet
     {
         public enum Importance

XmlDoc2CmdletDoc.Tests/AcceptanceTests.cs

@@ -606,16 +606,22 @@ public void Command_ReturnValues_ForTestManualElements()
                 .XPathSelectElements("command:returnValues/command:returnValue", resolver)
                 .ToList();
             Assert.That(returnValues, Is.Not.Empty);
-            Assert.That(returnValues.Count, Is.EqualTo(2));
+            Assert.That(returnValues.Count, Is.EqualTo(3));
 
             {
-                var returnValue = returnValues.First();
+                var returnValue = returnValues[0];
                 var name = returnValue.XPathSelectElement("dev:type/maml:name", resolver);
                 Assert.That(name.Value, Is.EqualTo(typeof(string).FullName));
             }
 
             {
-                var returnValue = returnValues.Last();
+                var returnValue = returnValues[1];
+                var name = returnValue.XPathSelectElement("dev:type/maml:name", resolver);
+                Assert.That(name.Value, Is.EqualTo("None"));
+            }
+
+            {
+                var returnValue = returnValues[2];
                 var type = returnValue.XPathSelectElement("dev:type", resolver);
                 CheckManualClassType(type, false);