Xml comments added and overloads removed.

Author: kevinkuszyk

src/FluentAssertions.Mvc3/ViewResultAssertions.cs

@@ -5,18 +5,29 @@
 
 namespace FluentAssertions.Mvc
 {
+    /// <summary>
+    /// Contains a number of methods to assert that a <see cref="ViewResult"/> is in the expected state.
+    /// </summary>
     [DebuggerNonUserCode]
-	public class ViewResultAssertions : ViewResultBaseAssertions<ViewResult>
+    public class ViewResultAssertions : ViewResultBaseAssertions<ViewResult>
 	{
+        /// <summary>
+        /// Initializes a new instance of the <see cref="T:ViewResultAssertions" /> class.
+        /// </summary>
 		public ViewResultAssertions (ViewResult subject) : base(subject) { }
 
-        public ViewResultAssertions WithMasterName(string expectedMasterName)
-        {
-            WithMasterName(expectedMasterName, string.Empty, null);
-            return this;
-        }
-
-        public ViewResultAssertions WithMasterName(string expectedMasterName, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the master name is the expected master name.
+        /// </summary>
+        /// <param name="expectedMasterName">The expected master 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 ViewResultAssertions WithMasterName(string expectedMasterName, string reason = "", params object[] reasonArgs)
         {
             string actualMasterName = (Subject as ViewResult).MasterName;
 

src/FluentAssertions.Mvc3/ViewResultBaseAssertions.cs

@@ -9,22 +9,33 @@
 
 namespace FluentAssertions.Mvc
 {
+    /// <summary>
+    /// Contains a number of methods to assert that a <see cref="ViewResultBase"/> is in the expected state.
+    /// </summary>
     [DebuggerNonUserCode]
     public abstract class ViewResultBaseAssertions<T> : ObjectAssertions
         where T : ViewResultBase
     {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="T:ViewResultBaseAssertions" /> class.
+        /// </summary>
         public ViewResultBaseAssertions(ViewResultBase subject) : base(subject)
         {
             Subject = (T)subject;
         }
 
-        public ViewResultBaseAssertions<T> WithViewName(string expectedViewName)
-        {
-            WithViewName(expectedViewName, string.Empty, null);
-            return this;
-        }
-
-        public ViewResultBaseAssertions<T> WithViewName(string expectedViewName, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the view name is the expected view name.
+        /// </summary>
+        /// <param name="expectedViewName">The view 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 ViewResultBaseAssertions<T> WithViewName(string expectedViewName, string reason = "", params object[] reasonArgs)
         {
             string actualViewName = (Subject as ViewResultBase).ViewName;
 
@@ -35,13 +46,19 @@ public ViewResultBaseAssertions<T> WithViewName(string expectedViewName, string
             return this;
         }
 
-        public ViewResultBaseAssertions<T> WithViewData(string key, object expectedValue)
-        {
-            WithViewData(key, expectedValue, string.Empty, null);
-            return this;
-        }
-
-        public ViewResultBaseAssertions<T> WithViewData(string key, object expectedValue, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the view data contains the expected data.
+        /// </summary>
+        /// <param name="key">The expected view data key.</param>
+        /// <param name="expectedValue">The expected view data.</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 ViewResultBaseAssertions<T> WithViewData(string key, object expectedValue, string reason = "", params object[] reasonArgs)
         {
             ViewDataDictionary actualViewData = (Subject as ViewResultBase).ViewData;
 
@@ -60,13 +77,19 @@ public ViewResultBaseAssertions<T> WithViewData(string key, object expectedValue
             return this;
         }
 
-        public ViewResultBaseAssertions<T> WithTempData(string key, object expectedValue)
-        {
-            WithTempData(key, expectedValue, string.Empty, null);
-            return this;
-        }
-
-        public ViewResultBaseAssertions<T> WithTempData(string key, object expectedValue, string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the temp data contains the expected data.
+        /// </summary>
+        /// <param name="key">The expected temp data key.</param>
+        /// <param name="expectedValue">The expected temp data.</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 ViewResultBaseAssertions<T> WithTempData(string key, object expectedValue, string reason = "", params object[] reasonArgs)
         {
             TempDataDictionary actualTempData = (Subject as ViewResultBase).TempData;
 
@@ -80,6 +103,10 @@ public ViewResultBaseAssertions<T> WithTempData(string key, object expectedValue
             return this;
         }
 
+        
+        /// <summary>
+        /// The model.
+        /// </summary>
         public object Model
         {
             get
@@ -89,6 +116,11 @@ public object Model
             }
         }
 
+        /// <summary>
+        /// Asserts the model is of the expected type.
+        /// </summary>
+        /// <typeparam name="TModel">The expected type.</typeparam>
+        /// <returns>The typed model.</returns>
         public TModel ModelAs<TModel>()
         {
             object model = (Subject as ViewResultBase).Model;
@@ -103,13 +135,17 @@ public TModel ModelAs<TModel>()
             return (TModel)model;
         }
 
-        public ViewResultBaseAssertions<T> WithDefaultViewName()
-        {
-            WithDefaultViewName(string.Empty, null);
-            return this;
-        }
-
-        public ViewResultBaseAssertions<T> WithDefaultViewName(string reason, params object[] reasonArgs)
+        /// <summary>
+        /// Asserts that the default view will be used.
+        /// </summary>
+        /// <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 ViewResultBaseAssertions<T> WithDefaultViewName(string reason = "", params object[] reasonArgs)
         {
             string viewName = (Subject as ViewResultBase).ViewName;