Tweak to prevent confusing NRE from surfacing.

Author: AlexArchive

TestStack.FluentMVCTesting.Tests/ControllerResultTestTests.cs

@@ -844,6 +844,14 @@ public void Check_for_content_result_and_check_invalid_content_and_check_invalid
             Assert.That(exception.Message.Contains("content type"));
         }
 
+        [Test]
+        public void Emit_readable_error_message_when_the_actual_content_encoding_has_not_been_specified()
+        {
+            var exception = Assert.Throws<ActionResultAssertionException>(() => _controller.WithCallTo(c => c.ContentWithoutEncodingSpecified()).ShouldReturnContent(encoding: ControllerResultTestController.TextualContentEncoding));
+
+            Assert.That(exception.Message, Is.EqualTo(string.Format("Expected encoding to be equal to {0}, but instead was null.", ControllerResultTestController.TextualContentEncoding.EncodingName)));
+        }
+
         #endregion
     }
 }

TestStack.FluentMVCTesting.Tests/TestControllers/ControllerResultTestController.cs

@@ -237,6 +237,11 @@ public ActionResult Content()
             return Content(TextualContent, ContentType, TextualContentEncoding);
         }
 
+        public ActionResult ContentWithoutEncodingSpecified()
+        {
+            return Content(TextualContent, ContentType);
+        }
+
         #endregion
     }
 

TestStack.FluentMvcTesting/ControllerResultTest.cs

@@ -425,7 +425,7 @@ public ContentResult ShouldReturnContent(string content = null, string contentTy
                 throw new ActionResultAssertionException(string.Format(
                     "Expected encoding to be equal to {0}, but instead was {1}.",
                     encoding.EncodingName,
-                    contentResult.ContentEncoding.EncodingName));
+                    contentResult.ContentEncoding != null ? contentResult.ContentEncoding.EncodingName : @"null"));
             }
 
             return contentResult;