このテストユニットが合格するのはなぜですか？

Question

私はこれで2時間以上を費やします...なぜこのテストがパスしているのか分かりません。私は..それは "Completed"ビューを返すべきではありませんが、テストでそれはしない！それは、 "string.Empty"と期待しているが、クレジットカードを渡すために "Completed"を返す。セキュリティコードはテスト中では全く "テスト"である必要がある。したがって、デフォルトのビュー（view（）;のようになります）を返します。

私は間違っていますか？それは間違っている私のテストですか？またはコントローラのロジック？

ありがとうございます。

---

[Test] 
public void Cannot_Check_Out_If_Credit_Card_Failed_To_Process() 
{ 
var mockOrderSubmitter = new Mock<IOrderSubmitter>(); 
var mockCreditCardProcessor = new Mock<ICreditCardProcessor>(); 

// Arrange: Given a user has a non-empty cart 
var cart = new Cart(); 
cart.AddItem(new Product(), 1); 

// Arrange: ... but the credit card failed to process 
var cartController = new CartController(null, mockOrderSubmitter.Object, mockCreditCardProcessor.Object); 
var result = cartController.CheckOut(cart, new ShippingDetails(), new CreditCard() { SecurityCode = "123" }); 

// Assert 
result.ShouldBeDefaultView(); 
} 

---

[HttpPost] 
public ActionResult CheckOut(Cart cart, ShippingDetails shippingDetails, CreditCard creditCard) 
{ 
    // Empty carts can't be checked out 
    if (cart.Lines.Count == 0) 
     ModelState.AddModelError("Cart", "Sorry, your cart is empty!"); 

    // Everything is valid 
    if (ModelState.IsValid) 
    { 
     // Effectue le paiement. 
     TransactionResult result = creditcardProcessor.TakePayment(creditCard, cart.ComputeTotalValue()); 
     if (result == TransactionResult.Success) 
     { 
      // Envoi la commande 
      orderSubmitter.SubmitOrder(cart, shippingDetails); 
      cart.Clear(); 
      return View("Completed"); 
     } 
     else 
     { 
      ModelState.AddModelError("CreditCard", "Sorry, we couldn't process your credit card, please verify your credit card details and retry."); 
      return View(new CheckOutViewModel()); 
     } 

    } 
    else // Something was invalid 
     return View(new CheckOutViewModel()); 
} 

---

public class MainCreditCardProcessor : ICreditCardProcessor 
{ 
    public TransactionResult TakePayment(CreditCard card, decimal amount) 
    { 
     if (card.SecurityCode == "test") 
      return TransactionResult.Success; 
     else 
      return TransactionResult.TransactionDeclined; 
    } 
} 

---

Answer 1

ShouldBeDefaultView()がスローした場合、テストは失敗します。

Answer 2

私の問題の解決策が見つかりました。他の人が同じ問題を抱えているのを助けるために、私はそれを説明します。

問題は私のテストでした。 mocksオブジェクトは内部にメソッドを持たない空のオブジェクトなので、ロジックはゼロです。つまり、テストでは実際にMainCreditCarDProcessorのインスタンスは使用されません。

// Arrange: Given we have a creditcard processor that return TransactionDeclined. 
mockCreditCardProcessor.SetReturnsDefault(TransactionResult.TransactionDeclined); 

をこのを通じて,, TransactionDeclinedを返すために、それを強制的にので、「強制」されていません。

を（私はそのことについて知らなかった）ので、私は追加することによって、この特定のテストのためのモックをセットアップするために持っていましたそれはあなたがちょうどその値を返すように設定しています。そうでなければ、それは見つけたデフォルト値を返します。この場合、enumの最初の要素だったか、他の種類のメソッドの他の種類の値。

私が言ったことが間違っている場合は、私を修正してください。

ありがとうございました。