Контекст выглядит хорошо. Мне нравится, как вы решили конфликтующие It с псевдонимами. Я бы сказал, что псевдоним Moq можно улучшить. Рассмотрим что-то похожее на предложение. Например, Param.Is<T> или Value.Is<T>.

Некоторые примечания с фрагментами кода, затем вся спецификация переписана внизу.

Сценарий — это ваш Subject

Сюжетом может быть Сценарий из рассказа. Кроме того, он отображается вместе с отчетом о выполнении теста (особенно хорошо в отчете HTML).

[Subject("Login Page")]

Не тратьте время на именованные базовые классы With.

Создатель MSpec, Аарон Дженсен, полностью отказался от использования синтаксиса "With". Имена контекстных классов не отображаются ни в одном отчете, поэтому не тратьте время на придумывание значимого имени.

public abstract class MembershipContext

Given - это имя вашего класса спецификации

Назовите конкретный класс спецификаций в честь данного в вашей истории. Тем более, что имя базового класса нигде не сообщается, вы можете потерять половину своего контекста в отчете! Вам также следует избегать включения имени тестируемой системы в имена классов контекста. Это делает ваши контексты более удобными для рефакторинга тестируемой системы.

public class When_an_existing_user_enters_valid_credentials

Базовые классы спецификации должны содержать только общую инициализацию

И часто не нужны. Они приводят к разделению фаз аранжировки и действия. Используйте базовый класс для инициализации общего поля, например, для настройки фиктивных зависимостей. Но вы не должны издеваться над поведением в базовом классе. И вы не должны помещать контекстно-зависимую информацию в базовый класс. В вашем примере имя пользователя/пароль. Таким образом, вы можете создать второй контекст с недействительными учетными данными.

Establish context = () =>
{
    membership = new Mock<ISiteMembership>();
    loginController = new LoginController(membership.Object);
};

Поля в конкретном классе спецификаций должны быть закрытыми.

Это уменьшает «церемонность» языка в вашем тесте. Вы должны поместить их ниже всех конкретных делегатов MSpec, так как эти части спецификации рассказывают большую часть истории.

static ActionResult result;

Капитальный ремонт спецификации

Спецификация здесь является отличным примером установления глобального контекста MembershipContext и его наследования в контексте, специфичном для спецификации (таким образом, дополнительный Establish).

[Subject("Login Page")]
public class When_an_existing_user_enters_valid_credentials : MembershipContext 
{
    Establish context = () =>
    {
        membership
            .Setup<bool>(m => m.Validate(
                Param.Is<string>(s => s == username), 
                Param.Is<string>(s => s == password)))
            .Returns(true);
    };

    Because of = () => result = loginController.Login(username, password);

    It should_log_the_user_in;
    It should_redirect_the_user_to_the_admin_panel;
    It should_show_message_confirming_successful_login;

    static ActionResult result;
    const string username = "username";
    const string password = "password";
}

public abstract class MembershipContext 
{
    Establish context = () =>
    {
        membership = new Mock<ISiteMembership>();
        loginController = new LoginController(membership.Object);
    };

    protected static Mock<ISiteMembership> membership;
    protected static LoginController loginController;
}