Тестирование кода с проверкой ReCaptcha в контроллере?

Вот мой упрощенный контроллер:

[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Register(RegisterModel model)
{
    if (!ReCaptcha.Validate(Constants.ReCaptchaPrivateKey))
        ModelState.AddModelError("recaptcha", "Incorrect value, enter the text again.");

    if (ModelState.IsValid)
    {
        //Code for register 
    }
}

Где следует проверять логику проверки данных?

1 ответ

Решение

Я хотел бы создать интерфейс для проверки ReCaptcha, или для того, что это представляет, что действительно является проверкой Человека, так что-то вроде:

public interface IHumanValidator
{
    ///Checks validates that the currentuser is human and not a bot
    bool Validate();

    /// Returns the text to display if the validation fails
    string ValidationFailText{get;}
}

Вам нужно будет изменить контроллер, чтобы принять IHumanValidator в конструкторе (или свойстве, если необходимо). Затем измените свой метод на:

[HttpPost]
[ValidateAntiForgeryToken]
public ActionResult Register(RegisterModel model)
{
    if (!m_humanValidator.Validate())
        ModelState.AddModelError("recaptcha", m_humanValidator.ValidationFailText);

    if (ModelState.IsValid)
    {
        //Code for register 
    }
}

Затем я вставил бы реализацию, основанную на валидации ReCaptcha, в контроллер и проверил по ней:

public class ReCaptchaHumanValidator : IHumanValidator
{
    public bool Validate()
    {
        ReCaptcha.Validate(Constants.ReCaptchaPrivateKey)
    }

    public string ValidationFailText
    {
        get{return "Incorrect value, enter the text again.";}
    }
}

Затем вы можете ввести фиктивный валидатор для тестирования, который вы можете настроить так, чтобы он возвращал действительный или нет в зависимости от теста.

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

Другие вопросы по тегам