Тестирование кода с проверкой ReCaptcha в контроллере? - PullRequest
2 голосов
/ 11 октября 2011

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

[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 Ответ

0 голосов
/ 11 октября 2011

Я бы создал интерфейс для проверки 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 и не нужно ничего менять в своем коде.

...