C＃unit testing a class

Question

クラスライブラリで単体テストを行っていますが、メソッドのテスト方法が残っています。私は8文字以下のパスワードを受け入れることができないかどうかを確認する、8文字以上のパスワードを受け入れることができるかどうかをチェックし、正面にスペースがあるパスワードを受け入れることができないかどうかを確認するなどのシナリオをテストする必要があります。

以下のコードは、クラスライブラリのコードです。

public class PasswordChecker 
{ 
    public bool CheckPassword(string pwd) 
    { 
     if (pwd.Length >= 8 && !pwd.StartsWith("")) 
     { 
      return true; 
     } 
     else 
     { 
      return false; 
     } 

以下のコードはテストプロジェクトのコードです。

[TestClass] 
public class PasswordCheckerTest 

{ 
    [TestMethod] 
    public void CheckPassword8CharsLong() 
    { 
     string validPassword = "12345678"; 
     string invalidPassword = "abc"; 

     PasswordChecker checker = new PasswordChecker(); 

     Assert.IsTrue(checker.CheckPassword(validPassword)); 
     Assert.IsFalse(checker.CheckPassword(invalidPassword)); 

    } 

Answer 1

1回のテストでこれら2つの条件をテストしないでください。また、CheckPasswordメソッドを静的にすることで、副作用がないようにすることができます。

[TestClass] 
public class PasswordCheckerTests 
{ 
    [TestMethod] 
    public void CheckPassword_PasswordLessThanEightCharacters_ReturnsFalse() 
    { 
     string invalidPassword = "1234567"; 

     PasswordChecker checker = new PasswordChecker(); 

     Assert.IsFalse(checker.CheckPassword(invalidPassword)); 
    } 

    [TestMethod] 
    public void CheckPassword_PasswordLongerThanSevenCharacters_ReturnsTrue() 
    { 
     string validPassword= "12345678"; 

     PasswordChecker checker = new PasswordChecker(); 

     Assert.IsTrue(checker.CheckPassword(validPassword)); 
    } 
} 

Answer 2

if(pwd.Length < 8 || pwd.Contains(" ") == true){ //VALID } 

Answer 3

私はあなたのコードをこのように書くでしょう。パスワードがnullでなく、いないスペースと8文字以上

public static bool CheckPassword(string pwd) 
{ 
    return pwd != null && !pwd.StartsWith(" ") && pwd.Length >= 8; 
} 

Answer 4

で始まる場合、私はAAAの原則使用を言うだろう、それは次のようになります。

A：あなたが呼び出すために必要なものを初期化する場所を手配しますテストされるメソッド。

A：実際に電話をかける場所。

A：あなたが期待していたアサーションをどこにアサートしますか？

Answer 5

おそらくエラーが原因pwd.StartsWith("")ステートメントに、しかし、あなたのテストを見て、一つの試験方法では、あまりにも多くのテストがあると、それは3つのメソッドに分割しなければならない、

[TestMethod] 
    public void GivenValidPassword_WhenCheckPassword_ThenReturnTrue(){ 
     var password= "12345678"; 

     var sut = new PasswordChecker(); 
     var result = sut.CheckPassword(password); 

      Assert.IsTrue(result); 
    } 

    [TestMethod] 
    public void GivenPasswordLessThan8Characters_WhenCheckPassword_ThenReturnFalse(){ 
     var password= "1278"; 

     var sut = new PasswordChecker(); 
     var result = sut.CheckPassword(password); 

      Assert.IsFalse(result); 
    } 

    [TestMethod] 
    public void GivenPasswordStartWithSpace_WhenCheckPassword_ThenReturnFalse(){ 
     var password= " 12345678"; 

     var sut = new PasswordChecker(); 
     var result = sut.CheckPassword(password); 

      Assert.IsFalse(result); 
    }