MVC3 で単体テストを行うことを考えています。合理的な解決策は、アクションをマークして「B」ビューを返し、他のアクションをマークして結果を記録できるようにすることだと思いました。
おそらく、コントローラーは次のようになります。
[AB(ABModes.View)]
public ActionResult SignUp()
{
return View();
}
[HttpPost]
public ActionResult SignUp(int id)
{
return RedirectToAction("Confirmation");
return View();
}
[AB(ABModes.Result)]
public ActionResult Confirmation()
{
return View();
}
サインアップは A または B ビューを返し、確認はどのビューが使用されたかを記録します。
属性は次のようになります。
using System;
using System.Web.Mvc;
namespace ABTesting.lib
{
public class ABAttribute : ActionFilterAttribute
{
private ABModes mode;
private Abstract.IABChooser abChooser;
private Abstract.IABLogMessenger abMessenger;
public ABAttribute(ABModes mode) : this(mode, new Concrete.ABChooser(), null)
{
}
public ABAttribute(ABModes mode, Abstract.IABChooser abChooser, Abstract.IABLogMessenger abMessenger)
{
this.mode = mode;
this.abChooser = abChooser;
this.abMessenger = abMessenger;
}
public override void OnActionExecuted(ActionExecutedContext filterContext)
{
var result = filterContext.Result as ViewResultBase;
var action = filterContext.Controller.ControllerContext.RouteData.Values["action"].ToString();
var actionName = String.IsNullOrEmpty(result.ViewName) ? action : result.ViewName;
if(mode == ABModes.View)
result.ViewName = String.Format("{0}{1}", actionName, abChooser.UseB()? "_B" : String.Empty);
else{
var controller = filterContext.Controller.ControllerContext.RouteData.Values["controller"].ToString();
if (abMessenger != null)
abMessenger.Write(new Entities.ABLogMessage
{
DateCreated = DateTime.Now,
ControllerName = controller,
ActionName = actionName,
IsB = abChooser.UseB()
});
}
base.OnActionExecuted(filterContext);
}
}
}
と
public interface IABChooser
{
bool UseB();
}
と
public interface IABLogMessenger
{
void Write(ABLogMessage message);
}
これは、最小限のコード変更でこれを達成する合理的な方法のように思えますか?