Fixed warnings

Author: marklagendijk

src/ImageSharp.Web/Commands/PresetRequestParser.cs

@@ -11,18 +11,20 @@
 namespace SixLabors.ImageSharp.Web.Commands
 {
     /// <summary>
-    /// Parses ImageSharp.Web image processing requests to ImageSharp.Web commands based on statically defined presets.
-    /// The advantages of this are:
-    /// 1. Image processing for certain scenarios can be adjusted in a single place, by adjusting the preset definition.
-    /// 2. Security. Only known presets are applied. Limits DOS opportunities.
+    /// Parses preset name from the request querystring and returns the commands configured for that preset.
     /// </summary>
     public class PresetRequestParser : IRequestParser
     {
         private readonly IDictionary<string, IDictionary<string, string>> presets;
 
-        public PresetRequestParser(IOptions<PresetRequestParserOptions> options) =>
-            this.presets = ParsePresets(options.Value.Presets);
+        /// <summary>
+        /// Initializes a new instance of the <see cref="PresetRequestParser"/> class.
+        /// </summary>
+        /// <param name="presetOptions">The preset options.</param>
+        public PresetRequestParser(IOptions<PresetRequestParserOptions> presetOptions) =>
+            this.presets = ParsePresets(presetOptions.Value.Presets);
 
+        /// <inheritdoc/>
         public IDictionary<string, string> ParseRequestCommands(HttpContext context)
         {
             var requestedPreset = context.Request.Query["preset"].ToString();

src/ImageSharp.Web/Commands/PresetRequestParserOptions.cs

@@ -5,8 +5,14 @@
 
 namespace SixLabors.ImageSharp.Web.Commands
 {
+    /// <summary>
+    /// Configuration options for the <see cref="PresetRequestParser"/>.
+    /// </summary>
     public class PresetRequestParserOptions
     {
+        /// <summary>
+        /// Gets or sets the presets, which is a Dictionary of preset names to command query strings.
+        /// </summary>
         public IDictionary<string, string> Presets { get; set; } = new Dictionary<string, string>();
     }
 }

tests/ImageSharp.Web.Tests/Commands/PresetRequestParserTests.cs

@@ -17,15 +17,16 @@ public void PresetRequestParserExtractsCommands()
         {
             IDictionary<string, string> expected = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase)
             {
-                {"width", "400"}, {"height", "200"}
+                { "width", "400" },
+                { "height", "200" }
             };
 
             HttpContext context = CreateHttpContext("?preset=Preset1");
             IDictionary<string, string> actual = new PresetRequestParser(Options.Create(new PresetRequestParserOptions
             {
                 Presets = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase)
                 {
-                    {"Preset1", "width=400&height=200"}
+                    { "Preset1", "width=400&height=200" }
                 }
             })).ParseRequestCommands(context);
 
@@ -42,7 +43,7 @@ public void PresetRequestParserCommandsWithoutPresetParam()
             {
                 Presets = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase)
                 {
-                    {"Preset1", "width=400&height=200"}
+                    { "Preset1", "width=400&height=200" }
                 }
             })).ParseRequestCommands(context);
 
@@ -59,7 +60,7 @@ public void PresetRequestParserCommandsWithoutMatchingPreset()
             {
                 Presets = new Dictionary<string, string>(StringComparer.OrdinalIgnoreCase)
                 {
-                    {"Preset1", "width=400&height=200"}
+                    { "Preset1", "width=400&height=200" }
                 }
             })).ParseRequestCommands(context);