Tek mantığın koruyucusu olduğu yerde test yöntemleri uygulamak yararlı mıdır?

Diyelim ki böyle bir yöntem var:

public void OrderNewWidget(Widget widget)
{
   if ((widget.PartNumber > 0) && (widget.PartAvailable))
   {
        WigdetOrderingService.OrderNewWidgetAsync(widget.PartNumber);
   }
}

Benim kodu (bir async Web Servis çağrısı için ön yarısı) bu tür birkaç yöntem var.

Onları birim testlere tabi tutmanın faydalı olup olmadığını tartışıyorum. Evet burada bir mantık var, ama sadece koruma mantığı. (Yani web servis çağrısının gerçekleşmesine izin vermeden önce ihtiyacım olan şeylere sahip olduğumdan eminim.)

Bir parçam "Onları birim test edebileceğinizden emin olabilirsiniz, ama zaman ayırmaya değmez" (zaten programın gerisinde olan bir projedeyim) diyor.

Ama diğer tarafım, eğer onları birim olarak test etmezseniz ve birisi Muhafızları değiştirirse, o zaman problemler olabilir.

Ama ilk bölümüm diyor ki, eğer birisi gardiyanları değiştirirse, onlar için daha fazla iş yapıyorsunuz (çünkü şimdi gardiyanları ve gardiyanlar için birim testlerini değiştirmek zorundalar).

Örneğin, hizmetim Widget kullanılabilirliğini kontrol etme sorumluluğunu üstlenirse, o korumayı daha fazla istemeyebilirim. Birim testi altındaysa, şimdi iki yeri değiştirmek zorundayım.

Artıları ve eksileri her iki şekilde de görüyorum. Bu yüzden başkalarının ne yaptığını soracağımı düşündüm.

Bir parçam "Onları birim test edebileceğinizden emin olabilirsiniz, ama zaman ayırmaya değmez" (zaten programın gerisinde olan bir projedeyim) diyor.

Üç çok kısa test. Kendinize soruyu sormak için çok zaman harcadınız.

Ama diğer tarafım, eğer onları birim olarak test etmezseniz ve birisi Muhafızları değiştirirse, o zaman problemler olabilir.

Bu tarafı dinleyin.

Ama ilk bölümüm diyor ki, eğer birisi gardiyanları değiştirirse, onlar için daha fazla iş yapıyorsunuz (çünkü şimdi gardiyanları ve gardiyanlar için birim testlerini değiştirmek zorundalar).

Eğer bakıcınız bir TDD somunu ise, onlar için daha zor hale getirirsiniz. İlgili bir değişiklik veya test eklenmesi olmadan yaptığım herhangi bir değişiklik, çok düşünmeme yol açıyor. Aslında, devam etmeden ve değişikliği yapmadan önce muhtemelen testleri eklerdim.

İlk kısmı sadece yanlış. İkinci bölüme arkada bir pat verin ve düşünmeyi bırakın.

Koruma mantığı ve gerçek sıralama ayrı yöntemler olsaydı, birim testini basitleştirecekti.

In Widgetsınıfında

public bool IsReadyForOrdering { get { return PartNumber > 0 && PartAvailable; } }

veya başka bir yerde eşdeğer bir yöntem

public bool IsWidgetReadyForOrdering(Widget widget)
{
    return widget.PartNumber > 0 && widget.PartAvailable;
}

Sipariş yöntemi

public void OrderNewWidget(Widget widget)
{
   if (IsWidgetReadyForOrdering(widget)) {
        WigdetOrderingService.OrderNewWidgetAsync(widget.PartNumber);
   }
}

Artık testler IsWidgetReadyForOrderingkolaylaştı. Artık bu konuda uzun süre düşünmeyin. Dene!

Birim testi için programınızda zamanınız yoksa ancak sağlam KG kullanımı için bir kenara ayırdığınız zaman varsa, birim testleri yazmak için bu KG süresinin bir kısmını çalabilecek misiniz veya KG döneminin bir kısmını harcayabileceğinizi sorun birim testleri yapmak, ya da belki sadece birim test edilmemiş kod ile uğraşmak. Ne yazık ki taşınmaz olan programlar size taviz vermeye ya da kendinizi ölüme zorlamak için, genellikle ilk seçeneği öneriyorum çünkü ikincisi sizi destekleyemez / sistemin süresi boyunca doğru şekilde bakımını yapmak.

Bununla birlikte, koruma ifadelerini test etme genel sorunuza; Evet! Koruma ifadelerini kesinlikle test edin! Bunlar, bu yöntemin davranışının önemli parçalarıdır, birisinin hata düzeltmesi yapan bir şeyi yanlış anladığını ve korumalarınızı çıkardığını veya değiştirdiğinizi &&bulmak ||istemez miydiniz? Birim testleri, a) korumalarınızdaki mantığı doğru bir şekilde aldığınızdan ve b) birim testlerini gerçekleştirdiklerinde hiç kimsenin şikâyet etmeden bu mantığı kırmadığını ve bunun bir sebepten ötürü böyle olması gerektiğini söyleyecektir.

Yukarıda bazı mükemmel cevaplar var ve yaptıkları noktalar çok önemli. Ancak kaçırılmış gibi görünen bir tanesi, kapsamlı bir dizi birim testinizin olması, kodun son derece ayrıntılı bir spesifikasyonu gibi okuyorlar. Test doğrulamasını yalnızca kod çok kolay olduğu için atlarsanız, nasıl yanlış gidebileceğini görmek zor, belirtiminizin bir kısmı eksik. Bu testlerin kaçırıldığı bir takıma girsem, aslında hizmetinizin argümanlarını doğrulamadığını varsayarım.

Kod koddur. Test sırasında% 100 kapsama almaya çalışmalısınız. Eğer önemli olmasaydı orada olmazdı.