Xml comments added and overloaded removed from RouteDataAssertions.

Author: kevinkuszyk

src/FluentAssertions.Mvc3/RouteDataAssertions.cs

@@ -11,45 +11,67 @@
 namespace FluentAssertions.Mvc
 {
     [DebuggerNonUserCode]
+    /// <summary>
+    /// Contains a number of methods to assert that a <see cref="RouteData"/> is in the expected state.
+    /// </summary>
     public class RouteDataAssertions : ObjectAssertions
     {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="T:RouteDataAssertions" /> class.
+        /// </summary>
         public RouteDataAssertions(RouteData subject)
             : base(subject)
         {
             Subject = subject;
         }
 
-        public RouteDataAssertions HaveController(string expectedControllerName)
-        {
-            HaveController(expectedControllerName, string.Empty, null);
-            return this;
-        }
-
-        public RouteDataAssertions HaveController(string expectedControllerName, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the route data has the expected controller name.
+        /// </summary>
+        /// <param name="expectedControllerName">The expected controller name.</param>
+        /// <param name="reason">
+        /// A formatted phrase as is supported by <see cref="string.Format(string,object[])" /> explaining why the assertion 
+        /// is needed. If the phrase does not start with the word <i>because</i>, it is prepended automatically.
+        /// </param>
+        /// <param name="reasonArgs">
+        /// Zero or more objects to format using the placeholders in <see cref="reason" />.
+        /// </param>
+        public RouteDataAssertions HaveController(string expectedControllerName, string reason = "", params object[] reasonArgs)
         {
             HaveValue("controller", expectedControllerName, reason, reasonArgs);
             return this;
         }
 
-        public RouteDataAssertions HaveAction(string expectedActionName)
-        {
-            HaveAction(expectedActionName, string.Empty, null);
-            return this;
-        }
-
-        public RouteDataAssertions HaveAction(string expectedActionName, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the route data has the expected action name.
+        /// </summary>
+        /// <param name="expectedActionName">The expected action name.</param>
+        /// <param name="reason">
+        /// A formatted phrase as is supported by <see cref="string.Format(string,object[])" /> explaining why the assertion 
+        /// is needed. If the phrase does not start with the word <i>because</i>, it is prepended automatically.
+        /// </param>
+        /// <param name="reasonArgs">
+        /// Zero or more objects to format using the placeholders in <see cref="reason" />.
+        /// </param>
+        public RouteDataAssertions HaveAction(string expectedActionName, string reason = "", params object[] reasonArgs)
         {
             HaveValue("action", expectedActionName, reason, reasonArgs);
             return this;
         }
 
-        public RouteDataAssertions HaveDataToken(string key, object expectedValue)
-        {
-            HaveDataToken(key, expectedValue, string.Empty, null);
-            return this;
-        }
-
-        public RouteDataAssertions HaveDataToken(string key, object expectedValue, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the route data has the expected data token.
+        /// </summary>
+        /// <param name="key">The expected data token key value.</param>
+        /// <param name="expectedValue">The expected data token value.</param>
+        /// <param name="reason">
+        /// A formatted phrase as is supported by <see cref="string.Format(string,object[])" /> explaining why the assertion 
+        /// is needed. If the phrase does not start with the word <i>because</i>, it is prepended automatically.
+        /// </param>
+        /// <param name="reasonArgs">
+        /// Zero or more objects to format using the placeholders in <see cref="reason" />.
+        /// </param>
+        public RouteDataAssertions HaveDataToken(string key, object expectedValue, string reason = "", params object[] reasonArgs)
         {
             var subjectTyped = Subject as RouteData;
 
@@ -68,13 +90,19 @@ public RouteDataAssertions HaveDataToken(string key, object expectedValue, strin
             return this;
         }
 
-        public RouteDataAssertions HaveValue(string key, object expectedValue)
-        {
-            HaveValue(key, expectedValue, string.Empty, null);
-            return this;
-        }
-
-        public RouteDataAssertions HaveValue(string key, object expectedValue, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the route data has the expected value.
+        /// </summary>
+        /// <param name="key">The expected key.</param>
+        /// <param name="expectedValue">The expected value.</param>
+        /// <param name="reason">
+        /// A formatted phrase as is supported by <see cref="string.Format(string,object[])" /> explaining why the assertion 
+        /// is needed. If the phrase does not start with the word <i>because</i>, it is prepended automatically.
+        /// </param>
+        /// <param name="reasonArgs">
+        /// Zero or more objects to format using the placeholders in <see cref="reason" />.
+        /// </param>
+        public RouteDataAssertions HaveValue(string key, object expectedValue, string reason = "", params object[] reasonArgs)
         {
             var subjectTyped = Subject as RouteData;