テストの書き方「いろは」の「ろ」を教えてください。

Question

前提・実現したいこと

Visual Studio 2015 Proの内蔵MSTestでテストを書いています。
テストの書き方「いろは」の「ろ」を教えてください。

		public static bool IsNumber(string text)
		{
			if (text == null) return false;
			else if (Regex.Match(text, "^[0-9]+$").Success)			return true;
			else return false;
		}

	があるとき、

		[TestMethod()]
		public void IsNumberTest1()
		{
			string text = "012";
			Assert.AreEqual(true, Validation.IsNumber(text));
		}

		[TestMethod()]
		public void IsNumberTest2()
		{
			string text = "012a";
			Assert.AreEqual(false, Validation.IsNumber(text));
		}

		[TestMethod()]
		public void IsNumberTest3()
		{
			string text = "abc";
			Assert.AreEqual(false, Validation.IsNumber(text));
		}

をテストしました。
returnがあるときは、expectedに予定する値を入れればよいのですが、returnがvoidのとき、たとえば下記のMethodをTestするにはどういうテストコードを書けばよいですか？

		static string BaseText = string.Empty;
		public static void SetText(string text)
		{
			BaseText = text;
		}

補足情報（FW/ツールのバージョンなど）

Visual Studio 2015 Pro

Answer 1

public static string GetText() {
  return BaseText;
}

を追加し、SetText("ほげほげ") ののち GetText() から "ほげほげ" が得られることをテストします。
こんなメソッド要らない/あってはダメなら #if FOR_TEST ～ #endif で囲んでおいて、テスト時のみこのコードを有効にすればいいス。

[追記] protected な例

using System;

public class foo {
  protected static string text;
  public static void set(string str) { text = str; }
}

// ↑納品物

// ↓テスト・コード

public class for_test : foo {
  public static string get() { return text; }
}

public class Program {
  public static void Main() {
    foo.set("hoge");
    Console.WriteLine(for_test.get());
  }
}

Comments

[meshkit]

ありがとうございます。なるぼど。
方法はわかりました。
メソッドはご推察のとおり、あってはだめと思います。
このメソッドじたいをテストプロジェクトに書くことはできますか？
```
[TestMethod()]
public void voidTest1()
{
string original = "あいう";
Validation.SetText(original);
string text = Validation.BaseText;
Assert.AreEqual(text, original);
}
```
とかでよいのでしょうか。

[episteme]

同様に #if ～ #endif を使って テストのときだけ BaseText を public にすればいいのでわ?
リフレクション使ってprivateなメソッド/フィールドを参照することもできますが、「ろ」の範囲からは大きく外れますよね。

[meshkit]

#if #endif
でコード中に書くと、納品時に全部消すことになります。
それは避けたいです。．．．。

[episteme]

納品時に全部消す? なんで?

[episteme]

あとは...そうねぇ、BaseTextをprotectedにしておいて、テスト用にこしらえた導出クラスから読み出しますか。

[meshkit]

なんでといわれても、客がいらないものを書くなと。

[episteme]

...ってお客様がおっしゃるですか。
んじゃ protected が次善の策かな(追記しといた

# あ、リフレクション使った回答が上がってますﾖ

[meshkit]

ありがとうございます。お客様には勝てないです。

[meshkit]

こまごまフォローありがとうございます！

Answer 2

PrivateType を使ってください。

class Target
{
	static string BaseText = string.Empty;
	public static void SetText(string text)
	{
		BaseText = text;
	}
}

[TestClass]
public class UnitTest1
{
	[TestMethod]
	public void TestMethod1()
	{
		string testValue = "Foo";
		Target.SetText(testValue);
		var target = new PrivateType(typeof(Target));
		var baseText = (string)target.GetStaticField("BaseText");
		Assert.AreEqual(testValue, baseText);
	}
}

Comments

[meshkit]

ありがとうございます。使いやすいコードです。活用します！

Answer 3

PrivateType（staticで無い場合はPrivateObject）を使いましょう。

        [TestMethod()]
        public void SetTextTest()
        {
            string original = "あいう";
            Validation.SetText(original);

            var pt = new PrivateType(typeof(Validation));
            var text = pt.GetStaticField("BaseText") as string;

            Assert.AreEqual(original, text);
        }
    }

Comments

[meshkit]

ありがとうございます。わかりやすいコードです！

Answer 4

SetTextを呼び出すことで、そのメソッドを持つクラスが外部に見える形で何らかの変化を起こすはずです。
その変化が期待通りであるかをテストします。

Comments

[episteme]

テストのためであれテスト対象に指一本触れてはならぬなら、リフレクションよりコチラを(僕は)推します。
何の変化も起こさないなら、そもそもそんなメソッド(and/or フィールド)は不要なのだから。

Answer 5

SetText を呼び出した後に、BaseText の内容が書き換わっていることを確認すればよいだけでは？

Comments

[meshkit]

ありがとうございます。なるぼど。
方法はわかりました。
いまいちコードにならず。