DBを更新するメソッドのテストの自動化の方針は？

Question

こんにちは。
Windows10でASP.NETのアプリケーションを開発しています。
Visual Studio 2017 Communityを使っています。

###前提・実現したいこと
あるメソッドのテストコードを書きたいです。
コードは下記のようなものです。
それぞれの処理で10-20行ぐらいずつあり、全体では100行くらいのメソッドです。

###該当のソースコード

private void method(){
 //メールの本文をファイルから取得。
 string mailtemplate = GetMailTemplateFromFile(filepath);

 //送信先のメールアドレスをデータベースから取得。
 List<string> mailaddresses = GetMailAddressFromDB();
 foreach(string mail in mailaddresses){
  //メール送信
  SendMail(mail, mailtemplate);
  //送信したかどうかをデータベースに登録。
  UpdateDatabase(mail);
 }
}

###発生している問題・エラーメッセージ
問題は、メールを送信するとデータベースを更新してしまうため、同じメソッドを2回繰り返すと、2回目は本来送るべきメール送信ターゲットが、ターゲットでなくなってしまうことです。
そのため、現在は、
・手動でテスト
・DBをSSMSで手動で編集。送信済みフラグをオフ。
というのを繰り返しています。
これは効率が悪い。

こういう場合、どういうふうにメソッドを構成すれば、自動でテストできるようになるのでしょう？

#If DebugとかでDBに登録しないようにするとかでしょうか？

###補足情報(言語/FW/ツール等のバージョンなど)
Microsoft Visual Studio Community 2017
Version 15.0.26228.9 D15RTWSVC
Microsoft .NET Framework
Version 4.6.01586

です。
よろしくお願いします。

Answer 1

メソッドの問題というよりも、テストの方法の問題の様な気がします。

テスト開始時にマスタも含めて全て自動的に削除＆作成するコードを走らないと前提条件が変わってきます。
一般的なテストスイートですとテストの前処理と後処理が書けるはずですので、リセットする処理を書かないといけません。

Comments

[cancat]

なるほど。テストには、前処理と後処理を書けるのですね。
その後処理で、送信済みフラグをオフにすればよいってことですね。

Answer 2

決めの問題とは思いますが、２回実施することを自動化したいのであれば、
テストコードに送信済みフラグをオフにするコードを挿入すればいいのではないでしょうか。

Comments

[cancat]

なるほど。テストコードで送信済みフラグをオフにすればよかったのですね!
それは目からうろこでした。

Answer 3

手動作業の内容を逐一書き出しして、
それを、コード化する。
自動ならではの、機能も考えておいた方が良いが、
まずは、出来る事から。

１．DBをSSMSで手動で編集。送信済みフラグをオフ。
⇒ストアド１本書いて、手動作業で行っている内容＋実行ログを記録
ストアドでなくても、拡張子：sql のファイルを実行しても良い。
成功時、失敗時のデータベースの変更内容が不要なら、トランザクションで
巻き戻し。

２．手動でテスト
⇒シナリオ化、分類別にコード化

３．5W1H + α
⇒これが出来ていれば、この質問はでないはず。

Comments

[cancat]

ご回答ありがとうございます。
ストアドってなんでしょうか?

[daive]

ストアド：https://www.google.co.jp/search?hl=ja&q=%E3%82%B9%E3%83%88%E3%82%A2%E3%83%89&lr=lang_ja&gws_rd=ssl
便利な世の中になっているのに、道具すら使えないのはねぇぇ。。。