Apakah ada nilai nyata dalam pengujian unit controller di ASP.NET MVC?

Saya harap pertanyaan ini memberikan beberapa jawaban menarik karena itu salah satu yang mengganggu saya untuk sementara waktu.

Apakah ada nilai nyata dalam pengujian unit controller di ASP.NET MVC?

Yang saya maksud dengan itu adalah, sebagian besar waktu, (dan saya bukan jenius), metode pengontrol saya, bahkan pada sesuatu yang paling kompleks seperti ini:

public ActionResult Create(MyModel model)
{
    // start error list
    var errors = new List<string>();

    // check model state based on data annotations
    if(ModelState.IsValid)
    {
        // call a service method
        if(this._myService.CreateNew(model, Request.UserHostAddress, ref errors))
        {
            // all is well, data is saved, 
            // so tell the user they are brilliant
            return View("_Success");
        }
    }

    // add errors to model state
    errors.ForEach(e => ModelState.AddModelError("", e));

    // return view
    return View(model);
}

Sebagian besar pekerjaan berat dilakukan oleh pipa MVC atau perpustakaan layanan saya.

Jadi mungkin pertanyaan yang akan diajukan:

• berapakah nilai unit yang menguji metode ini?
• tidakkah akan putus Request.UserHostAddressdan ModelStatedengan NullReferenceException? Haruskah saya mencoba mengejek ini?
• jika saya refractor metode ini menjadi "helper" yang dapat digunakan kembali (yang mungkin saya harus, mengingat berapa kali saya melakukannya!), akan menguji yang bahkan berharga ketika semua saya benar-benar menguji sebagian besar "pipa" yang, mungkin, telah diuji dalam satu inci dari kehidupannya oleh Microsoft?

Saya pikir maksud saya sebenarnya , melakukan hal-hal berikut ini tampaknya sama sekali tidak ada gunanya dan salah

[TestMethod]
public void Test_Home_Index()
{
    var controller = new HomeController();
    var expected = "Index";
    var actual = ((ViewResult)controller.Index()).ViewName;
    Assert.AreEqual(expected, actual);
}

Jelas saya menjadi tumpul dengan contoh berlebihan yang tidak ada gunanya ini, tetapi apakah ada yang punya kebijaksanaan untuk menambahkan di sini?

Menantikan itu ... Terima kasih.

Bahkan untuk sesuatu yang sangat sederhana, unit test akan melayani berbagai keperluan

1. Percaya diri, apa yang ditulis sesuai dengan hasil yang diharapkan. Tampaknya sepele untuk memverifikasi bahwa itu mengembalikan pandangan yang benar, tetapi hasilnya adalah bukti objektif bahwa persyaratan dipenuhi
2. Pengujian regresi. Jika metode Buat perlu diubah, Anda masih memiliki tes unit untuk output yang diharapkan. Ya, output bisa berubah sepanjang dan itu menghasilkan tes rapuh tetapi masih merupakan pemeriksaan terhadap kontrol perubahan yang tidak dikelola

Untuk tindakan tertentu saya akan menguji untuk yang berikut

1. Apa yang terjadi jika _myService adalah nol?
2. Apa yang terjadi jika _myService.Create melempar Exception, apakah itu melemparkan yang spesifik untuk ditangani?
3. Apakah _myService.Create yang berhasil mengembalikan tampilan _Success?
4. Apakah kesalahan disebarkan hingga ModelState?

Anda menunjukkan memeriksa Permintaan dan Model untuk NullReferenceException dan saya pikir ModelState.IsValid akan menangani penanganan NullReference untuk Model.

Mengejek Permintaan memungkinkan Anda untuk menjaga terhadap Permintaan Null yang secara umum tidak mungkin dalam produksi saya pikir, tetapi dapat terjadi dalam Tes Unit. Dalam Tes Integrasi itu akan memungkinkan Anda untuk memberikan nilai UserHostAddress berbeda (Permintaan masih input pengguna sejauh kontrol yang bersangkutan dan harus diuji sesuai)

Kontroler saya juga sangat kecil. Sebagian besar "logika" di controller ditangani menggunakan atribut filter (built-in dan tulisan tangan). Jadi pengontrol saya biasanya hanya memiliki beberapa pekerjaan:

• Buat model dari string kueri HTTP, nilai formulir, dll.
• Lakukan beberapa validasi dasar
• Panggil ke lapisan data atau bisnis saya
• Hasilkan a ActionResult

Sebagian besar model mengikat dilakukan secara otomatis oleh ASP.NET MVC. DataAnnotations menangani sebagian besar validasi untuk saya juga.

Bahkan dengan begitu sedikit untuk diuji, saya masih biasanya menulisnya. Pada dasarnya, saya menguji apakah repositori saya dipanggil dan ActionResultjenis yang benar dikembalikan. Saya memiliki metode kenyamanan untuk ViewResultmemastikan jalur tampilan yang benar dikembalikan dan model tampilan terlihat seperti yang saya harapkan. Saya punya yang lain untuk memeriksa controller / action yang tepat diatur untuk RedirectToActionResult. Saya punya tes lain untuk JsonResult, dll. Dll

Hasil yang tidak menguntungkan dari sub- Controllerkelas kelas adalah bahwa ia menyediakan banyak metode kenyamanan yang menggunakan HttpContextinternal. Ini membuatnya sulit untuk menguji unit controller. Untuk alasan ini, saya biasanya meletakkan HttpContextpanggilan -dependen di belakang sebuah antarmuka dan meneruskan antarmuka itu ke konstruktor pengontrol (saya menggunakan ekstensi web Ninject untuk membuat pengontrol saya untuk saya). Antarmuka ini biasanya di mana saya menempel properti pembantu untuk mengakses sesi, pengaturan konfigurasi, IPrinciple dan URL helpers.

Ini membutuhkan banyak uji tuntas, tetapi saya pikir itu sangat berharga.

Jelas beberapa pengendali jauh lebih kompleks dari itu tetapi berdasarkan murni pada contoh Anda:

Apa yang terjadi jika myService melempar pengecualian?

Sebagai catatan.

Juga, saya akan mempertanyakan kebijaksanaan melewati daftar dengan referensi (toh tidak perlu karena c # lolos dengan referensi, tetapi meskipun tidak) - meneruskan tindakan errorAction (Aksi) yang kemudian dapat digunakan oleh layanan untuk memompa pesan kesalahan ke yang kemudian dapat ditangani sesuai keinginan Anda (mungkin Anda ingin menambahkannya ke daftar, mungkin Anda ingin menambahkan kesalahan model, mungkin Anda ingin mencatatnya).

Dalam contoh Anda:

bukannya kesalahan ref, lakukan (string s) => ModelState.AddModelError ("", s) misalnya.