Errata for Test-Drive ASP.NET MVC

#45374: It should probably be made more explicit that if using the Express Editions of VS, ReSharper cannot be installed. This is because the Express Editions do not support Add-Ins and Extensions. (per jetbrains.net)--Nate Southerland

#45367: When explaining the new NUnit assertion format, the author makes a subtle, yet important semantic error. The new Assert.That style of assertion places the value under test first. The example should read:

Assert.That(new Welcomer().SayHello(), Is.EqualTo("Hello World"));

The reason it is important is the error message you get when the assertion fails. The second part is what is expected. The new Assert.That format has more descriptive error messages, so they are preferred.

The change was intentional because it was more natural to understand that the first parameter is what is under test. (Can't tell you how many people have reversed these in the legacy assertion format over the years). For newcomers to unit testing, the Assert.That format will likely prove easier to learn, and in the end are far more powerful.--Berin Loritsch

#46348: The figure on that page suggests that we have to implement the Equals() function for the Topic class. But there's another implementation of Equals() that we are missing to pass the unit test: the one for the Topic class. I know it's mentioned in passing on page #95, but it would be nice to emphasis this a little more.
Here's my suggestion for the Equals() in Topic:

public override bool Equals(object obj)
{
if (ReferenceEquals(null, obj)) return false;
if (ReferenceEquals(this, obj)) return true;
if (obj.GetType() != typeof(Todo)) return false;
var other = (Todo)obj;
return other.Completed == Completed &&
other.Title==Title &&
other.Outcome==Outcome && other.Topic==Topic
; ;
}

As a general rule, some bits and pieces are missing to complete the application in the book. This is good, because it forces us to think. But it might be good to emphasise these missing bits more, ore to put them as exercises.
For instance, prompt the readers to figure out how to delete a Topic. Or to display clean error message on the Process view when the list of thoughts is empty.

Otherwise, I really enjoyed the book. Thanks.
--Jean-Rémy duboc

#46156: Page 97, Sec 5.3, Have brought in the MVC Contrib dll and tried out the Test, but it fails to assert. Think there is something missing in this section and more needs to be done to setup the test.--Mickey Puri

#46170: It says that changing Mime type from application/octet-stream to image/jpg would render it as an image. In fact it appears to make no difference and it still comes up with a show/save box.--Mickey Puri

#46187: in Should_Display_Errors_When_Todo_Is_Not_Valid()

Need to create a todoController to access the Create method. so needs to be new todoController().Create(...

--Mickey Puri