Saya tidak merekomendasikan menggunakan atribut ExpectedException (karena terlalu membatasi dan rawan kesalahan) atau untuk menulis blok try / catch di setiap tes (karena terlalu rumit dan rawan kesalahan). Gunakan metode penegasan yang dirancang dengan baik - baik yang disediakan oleh kerangka pengujian Anda atau tulis sendiri. Inilah yang saya tulis dan gunakan.
public static class ExceptionAssert
{
private static T GetException<T>(Action action, string message="") where T : Exception
{
try
{
action();
}
catch (T exception)
{
return exception;
}
throw new AssertFailedException("Expected exception " + typeof(T).FullName + ", but none was propagated. " + message);
}
public static void Propagates<T>(Action action) where T : Exception
{
Propagates<T>(action, "");
}
public static void Propagates<T>(Action action, string message) where T : Exception
{
GetException<T>(action, message);
}
public static void Propagates<T>(Action action, Action<T> validation) where T : Exception
{
Propagates(action, validation, "");
}
public static void Propagates<T>(Action action, Action<T> validation, string message) where T : Exception
{
validation(GetException<T>(action, message));
}
}
Contoh menggunakan:
[TestMethod]
public void Run_PropagatesWin32Exception_ForInvalidExeFile()
{
(test setup that might propagate Win32Exception)
ExceptionAssert.Propagates<Win32Exception>(
() => CommandExecutionUtil.Run(Assembly.GetExecutingAssembly().Location, new string[0]));
(more asserts or something)
}
[TestMethod]
public void Run_PropagatesFileNotFoundException_ForExecutableNotFound()
{
(test setup that might propagate FileNotFoundException)
ExceptionAssert.Propagates<FileNotFoundException>(
() => CommandExecutionUtil.Run("NotThere.exe", new string[0]),
e => StringAssert.Contains(e.Message, "NotThere.exe"));
(more asserts or something)
}
CATATAN
Mengembalikan pengecualian alih-alih mendukung validasi panggilan balik adalah ide yang masuk akal kecuali bahwa melakukan hal itu membuat sintaks panggilan pernyataan ini sangat berbeda dari pernyataan lain yang saya gunakan.
Tidak seperti yang lain, saya menggunakan 'propagasi' bukan 'melempar' karena kami hanya dapat menguji apakah pengecualian merambat dari panggilan. Kami tidak dapat menguji secara langsung bahwa pengecualian dilemparkan. Tapi saya kira Anda bisa membayangkan melempar artinya: dibuang dan tidak tertangkap.
PIKIRAN FINAL
Sebelum beralih ke pendekatan semacam ini, saya mempertimbangkan untuk menggunakan atribut ExpectedException ketika tes hanya memverifikasi tipe pengecualian dan menggunakan blok coba / tangkap jika diperlukan lebih banyak validasi. Tetapi, saya tidak hanya harus memikirkan teknik mana yang akan digunakan untuk setiap tes, tetapi mengubah kode dari satu teknik ke teknik lainnya karena kebutuhan berubah bukanlah usaha yang sepele. Menggunakan satu pendekatan yang konsisten menghemat upaya mental.
Jadi secara ringkas, pendekatan ini olahraga: kemudahan penggunaan, fleksibilitas dan ketahanan (sulit untuk melakukannya salah).