Birim testi Anti-pattern kataloğu

anti-patern : gerçek bir paterni basit bir kötü alışkanlıktan, kötü uygulamadan veya kötü fikirden resmen ayırmak için en az iki anahtar unsur bulunmalıdır:

• Başlangıçta yararlı gibi görünen, ancak sonuçta faydalı sonuçlardan daha kötü sonuçlar doğuran, tekrarlanan bazı eylem, süreç veya yapı paternleri ve
• Açıkça belgelenmiş, gerçek uygulamada kanıtlanmış ve tekrarlanabilir yeniden düzenlenmiş bir çözüm.

Bir kerede "vahşi doğada" gördüğünüz TDD anti-paternine oy verin.
James Carr tarafından yazılan blog yazısı ve testdrivendevelopment yahoogroup ile ilgili tartışma

'Adsız' bir tane bulduysanız .. onları da gönderin. Anti-desen başına bir gönderi , oyların bir şey için sayılmasını sağlamak için lütfen .

Yaklaştığım ilgi top-n alt kümesini bulmak, böylece yakın gelecekte bir beslenme çantası buluşmasında onları tartışabiliyorum.

İkinci Sınıf Vatandaşlar - test kodu, çok sayıda yinelenen kod içeren üretim kodu kadar iyi bir şekilde yeniden düzenlenmez, bu da testleri sürdürmeyi zorlaştırır.

Free Ride / Piggyback - James Carr, Tim Ottinger Başka bir / farklı özelliği / işlevselliği
test etmek için yeni bir test senaryosu yöntemi yazmak yerine , yeni bir iddia (ve ilgili eylemleri, yani AAA'dan adımlar) mevcut bir test senaryosunda ilerliyor .

Mutlu Yol

Test, sınırları ve istisnaları test etmeden mutlu yollarda (yani beklenen sonuçlar) kalır.

JUnit Antipatterns

Yerel Kahraman

Çalıştırmak için yazıldığı geliştirme ortamına özgü bir şeye bağlı olan bir test durumu. Sonuç, test geliştirme kutularına geçer, ancak birisi başka bir yerde çalıştırmayı denediğinde başarısız olur.

Gizli Bağımlılık

Yerel kahramanla yakından ilişkili olan, test çalıştırılmadan önce mevcut verilerin bir kısmının doldurulmasını gerektiren bir birim testi. Eğer bu veriler doldurulmamışsa, test başarısız olacak ve geliştiriciye ne istediğini ya da nedenini kullanamayacağı verinin nereden geleceğini bulmak için onları bir dönümlük kod kazmaya zorlayacak.

Ne yazık ki, bu dll'lerden sorumlu üç geliştiriciyle kapsamlı bir danışma olmaksızın, makinenizde tek başına durmaksızın, herhangi bir üretim sisteminde sürekli olarak senkronize olmayan nebulous ve çeşitli .ini dosyalarına bağımlı olan eski .dll'lerle çok fazla kez görüldü. İç çekmek.

Zincir Çetesi

Belirli bir sırayla çalışması gereken birkaç test, yani bir test sistemin global durumunu (global değişkenler, veritabanındaki veriler) değiştirir ve sonraki testler buna bağlıdır.

Bunu genellikle veritabanı testlerinde görürsünüz. teardown()Testler geri alma yerine , veritabanındaki değişikliklerini yapar. Diğer bir yaygın neden, genel durumdaki değişikliklerin, test başarısız olursa temizlenecek olan try / nihayet bloklarına sarılmamasıdır.

Alay
Bazen alay etmek iyi ve kullanışlı olabilir. Ancak bazen geliştiriciler kendilerini kaybedebilir ve test edilmeyen şeyleri alay etme çabalarında. Bu durumda, bir birim test, test edilen sistemin hiç test edilmediği kadar çok sahte, taslak ve / veya sahte içerir, bunun yerine alaylardan döndürülen veriler test edilir.

Kaynak: James Carr'ın gönderisi.

Sessiz Alıcı - Kelly?
Bir istisna atılırsa geçen bir test .. gerçekte meydana gelen istisna bile geliştiricinin amaçladığından farklıdır.
Ayrıca Bakınız: Gizli Yakalayıcı

[Test]
[ExpectedException(typeof(Exception))]
public void ItShouldThrowDivideByZeroException()
{
   // some code that throws another exception yet passes the test
}

Müfettiş
% 100 kod kapsamı elde etmek amacıyla kapsüllemeyi ihlal eden, ancak nesnede neler olup bittiğini çok iyi bilen bir birim testi, refactor için yapılan herhangi bir girişim mevcut testi kıracak ve herhangi bir değişikliğin birime yansıtılmasını gerektirecektir Ölçek.

'üye değişkenlerimi herkese açık hale getirmeden nasıl test ederim ... sadece birim test için?'

Aşırı Kurulum - James Carr
Teste başlamak için büyük bir kurulum gerektiren bir test. Bazen yüzlerce kod satırı, bir teste hazırlanmak için birkaç nesne kullanılır ve bu da devam eden tüm kurulumun “gürültüsü” nedeniyle neyin test edildiğini gerçekten zorlaştırır. (Src: James Carr'ın gönderisi )

Anal Prob

Görevini yerine getirmek için çılgın, yasadışı veya sağlıksız yollar kullanmak zorunda olan bir test: Java setAccessible (true) kullanarak özel alanları okuma veya korumalı alanlara / yöntemlere erişmek için bir sınıfı genişletme veya sınava erişmek için belirli bir pakete koyma zorunluluğu paket global alanları / yöntemleri.

Bu modeli görürseniz, test altındaki sınıflar çok fazla veri gizleme kullanır.

Bu ve Müfettiş arasındaki fark, test edilen sınıfın test etmeniz gereken şeyleri bile gizlemeye çalışmasıdır. Yani amacınız% 100 test kapsamı elde etmek değil, hiçbir şeyi test edebilmektir. Yalnızca özel alanları, run()argümanları olmayan ve hiç alıcısı olmayan bir yöntemi düşünün . Kuralları ihlal etmeden bunu test etmenin bir yolu yoktur.

Michael Borgwardt'ın yorumu: Bu gerçekten bir test antipatterni değil, test edilen koddaki eksikliklerle uğraşmak pragmatizm. Elbette bu eksikliklerin giderilmesi daha iyidir, ancak 3. taraf kütüphaneler için bu mümkün olmayabilir.

Aaron Digulla: Katılıyorum. Belki de bu giriş bir antipattern için değil, "JUnit HOWTO" wiki için gerçekten daha uygundur. Yorumlar?

İsimsiz Test - Nick Pellow

Hata izleyicide belirli bir hatayı yeniden oluşturmak için eklenen ve yazarı kendi düşünen bir isim garanti etmeyen test. Mevcut olmayan bir testi geliştirmek yerine testForBUG123 adlı yeni bir test oluşturulur.

İki yıl sonra, bu test başarısız olduğunda, öncelikle testin amacını bulmak için hata izleyicinizde BUG-123'ü bulmaya çalışmanız gerekebilir.

Yavaş Poke

Son derece yavaş çalışan bir birim testi. Geliştiriciler onu başlattığında, tuvalete gitmek, sigara içmek veya daha da kötüsü, günün sonunda eve gitmeden önce testi başlatmak için zamanları var. (Src: James Carr'ın gönderisi )

diğer bir deyişle, gerektiği kadar sık ​​çalıştırılmayacak testler

Kelebek

Güncel tarihi içeren bir yapı gibi her zaman değişen verileri içeren bir şeyi test etmeniz gerekir ve sonucu sabit bir değere sabitlemenin bir yolu yoktur. Çirkin kısım, bu değeri hiç umursamamanızdır. Herhangi bir değer katmadan testinizi daha karmaşık hale getirir.

Kanatının yarasası dünyanın diğer tarafında bir kasırgaya neden olabilir. - Edward Lorenz, Kelebek Etkisi

Titreşen Test (Kaynak: Romilly Cocking)

Belirli zamanlarda değil, nadiren başarısız olan ve genellikle test içindeki yarış koşullarından kaynaklanan bir test. Genellikle JMS gibi eşzamansız bir şeyi test ederken oluşur.

Muhtemelen ' Bekle ve Bak ' anti-desen ve ' Uyuyan ' anti-desen için bir süper set .

Yapı başarısız oldu, ah, sadece tekrar çalıştırın. - Anonim Geliştirici

Bekle ve gör

Bazı kurulum kodunu çalıştıran ve ardından test edilen kodun beklendiği gibi çalışıp çalışmadığını 'görebilmesi için belirli bir süre' beklemesi 'gereken bir test. Thread.sleep () veya eşdeğerini kullanan bir testMethod kesinlikle bir "Bekle ve Gör" testidir.

Genellikle, sınama, e-posta, http isteği veya sisteme bir dosya diske yazan sistem dışında bir olay oluşturan kodu test ediyorsa bunu görebilirsiniz.

Böyle bir test aynı zamanda bir Yerel Kahraman olabilir, çünkü daha yavaş bir kutuda veya aşırı yüklenmiş bir CI sunucusunda çalıştırıldığında BAŞARISIZ olacaktır.

Bekle ve Gör anti deseniyle karıştırılmamalıdır Uyuyan .

Uygunsuz Paylaşılan Armatür - Tim Ottinger
Test armatüründeki birkaç test durumu, kurulum / sökme işlemlerini bile kullanmaz veya buna ihtiyaç duymaz. Kısmen yeni bir test fikstürü oluşturmak için geliştirici ataletinden dolayı ... kazığa bir tane daha test durumu eklemek daha kolay

Dev

Test edilen nesneyi geçerli bir şekilde test etmesine rağmen, binlerce satıra yayılabilen ve çok sayıda test durumu içerebilen birim test. Bu, test edilen sistemin bir Tanrı Nesnesi (James Carr'ın gönderisi) .

Bunun kesin bir işareti bir kod satırından daha fazlasını kapsayan bir testtir. Genellikle, test o kadar karmaşıktır ki, kendi veya pul pul davranışı olan böcekleri içermeye başlar.

Bazı yanıp sönen GUI'leri gördüğümde buna inanacağım
Uygulamanın GUI'si üzerinden 'gerçek bir kullanıcı gibi' test edilmesiyle sağlıksız bir sabitleme / takıntı

İş kurallarını GUI aracılığıyla test etmek korkunç bir bağlantı şeklidir. GUI aracılığıyla binlerce test yazıp GUI'nizi değiştirirseniz, binlerce test kesilir.
Bunun yerine, GUI aracılığıyla yalnızca GUI şeylerini test edin ve bu testleri çalıştırdığınızda GUI'yi gerçek sistem yerine sahte bir sistemle birleştirin. GUI içermeyen bir API aracılığıyla iş kurallarını test edin. - Bob Martin

“Görmenin inanmakta olduğunu anlamalı, aynı zamanda inanmanın görüyor olduğunu da bilmelisin.” - Denis Waitley

Uyuyan, yani Vezüv Yanardağı - Nick Pellow

Gelecekte belirli bir saat ve tarihte FAIL'a giden bir test. Bunun nedeni, bir Date veya Calendar nesnesi kullanan kodun test edilmesinde hatalı sınırların kontrol edilmesinden kaynaklanır. Bazen, gece yarısı gibi günün belirli bir saatinde çalıştırılırsa test başarısız olabilir.

'Uyuyan', ' Bekle ve Bak ' anti-deseniyle karıştırılmamalıdır.

Bu kod 2000 yılından çok önce değiştirilmiş olacak - 1960 yılında birçok geliştirici

Ölü Ağaç

Bir saplamanın oluşturulduğu, ancak testin aslında yazılmadığı bir test.

Bunu aslında üretim kodumuzda gördüm:

class TD_SomeClass {
  public void testAdd() {
    assertEquals(1+1, 2);
  }
}

Bunun hakkında ne düşüneceğimi bile bilmiyorum.

Bugün bunu biraz bitirdim:

Islak Zemin :
Test, bir yerde kalıcı olan veriler oluşturur, ancak test tamamlandığında temizlenmez. Bu, testlerin (aynı test veya muhtemelen diğer testler) daha sonra başarısız olmasına neden olur test işlemlerinde .

Bizim durumumuzda, test, "temp" dizininde bulunan ve dosyayı ilk kez çalıştıran kullanıcının izinleriyle bir dosya bıraktı. Farklı bir kullanıcı aynı makinede test etmeye çalıştığında: bom. James Carr'ın sitesindeki yorumlarda Joakim Ohlrogge buna "Özensiz İşçi" olarak atıfta bulundu ve "Cömert Artıklar" ın ilham kaynağının bir parçasıydı. Adımı daha iyi seviyorum (daha az hakaret, daha tanıdık).

Cuckoo - Frank Carver
Diğerleriyle bir test senaryosunda oturan ve test durumundaki diğer testlerle aynı (potansiyel olarak uzun) kurulum sürecine sahip olan, ancak kurulumdan bazı veya tüm parçaları atan bir birim testi ve kendi
Gelişmiş Belirti: Uygunsuz Paylaşılan Fikstür

The Secret Catcher - Frank Carver
İddiaların olmaması nedeniyle ilk bakışta test yapmıyor gibi görünen bir test. Ancak "şeytan ayrıntıda gizlidir" .. test gerçekten atılacak bir istisnayı temel alıyor ve test çerçevesinin istisnayı yakalamasını ve hata olarak kullanıcıya raporlamasını bekliyor.

[Test]
public void ShouldNotThrow()
{
   DoSomethingThatShouldNotThrowAnException();
}

Çevresel Vandal

Çeşitli 'gereksinimler' için ortam değişkenlerini / bağlantı noktalarını kullanarak ve ayarlayarak ortamına yayılmaya başlayan bir 'birim' testi. Bu testlerden ikisinin aynı anda yapılması 'kullanılamayan bağlantı noktası' istisnalarına vb. Neden olacaktır.

Bu testler aralıklı olacak ve geliştiricileri 'sadece tekrar çalıştır' gibi şeyler söyleyecek.

Gördüğüm bir çözüm, kullanılacak bir bağlantı noktası numarasını rastgele seçmektir. Bu, çatışma olasılığını azaltır, ancak sorunu açıkça çözmez. Bu yüzden eğer yapabiliyorsanız, kodu her zaman taklit edilemeyecek bir kaynak ayırmayacak şekilde alay edin.

Turing Testi

Çok zeki bir veri akışı analizi kullanılarak test edilen sınıftan toplanan birçok, çok sayıda iddia içeren bazı pahalı araçlar tarafından otomatik olarak oluşturulan bir test çantası. Geliştiricileri, kodlarının iyi test edildiğine dair yanlış bir güven duygusuna sokar ve onları yüksek kaliteli testler tasarlama ve sürdürme sorumluluğundan kurtarır. Makine testleri sizin için yazabiliyorsa, neden parmağını dışarı çekip uygulamanın kendisini yazamıyor?

Merhaba aptal. - Dünyanın en akıllı bilgisayarı yeni çıraklara (eski bir Amiga çizgi romanından).

Kırk Ayak Direği Testi

Test etmeye çalıştıkları sınıfa çok yaklaşmaktan korkan bu testler, sayısız soyutlama katmanı ve kontrol ettikleri mantıktan binlerce kod satırı ile ayrılmış bir mesafede hareket eder. Bu nedenle, son derece kırılgandırlar ve ilgi sınıfına gidip gelen destansı yolculukta meydana gelen her türlü yan etkiye duyarlıdırlar.

Doppelgänger

Bir şeyi test etmek için, test edilen kodun parçalarını aynı ad ve pakete sahip yeni bir sınıfa kopyalamanız ve önce görünür olduğundan emin olmak için sınıf yolu büyüsünü veya özel bir sınıf yükleyiciyi kullanmanız gerekir (böylece kopyanız seçilir) kadar).

Bu kalıp, bir testten kontrol edemediğiniz sağlıksız miktarda gizli bağımlılığı gösterir.

Yüzüne baktım ... yüzüme! Ayna gibiydi ama kanımı dondurdu.

Anne Hen - Frank Carver
Gerçek test senaryolarının gerektirdiğinden çok daha fazlasını yapan ortak bir kurulum. Örneğin, testler sadece bir şeyin varlığı ya da yokluğu için öne sürüldüğünde, görünüşte önemli ve benzersiz değerlerle dolu her türlü karmaşık veri yapısının oluşturulması.
Gelişmiş Belirti: Uygunsuz Paylaşılan Fikstür

Ne yaptığını bilmiyorum ... Yine de ekliyorum, her ihtimale karşı. - Anonim Geliştirici

Hepsini Test Et

Bunun şimdiye kadar bahsedilmediğine inanamıyorum, ancak testler Tek Sorumluluk İlkesini ihlal etmemelidir .

Bu kadar çok kez karşılaştım, bu kuralı ihlal eden testler tanımı gereği sürdürmek için bir kabus.

Hat vurucu

İlk bakışta testler her şeyi kapsar ve kod kapsamı araçları% 100 ile onaylar, ancak gerçekte testler herhangi bir çıktı analizi olmadan sadece koda çarptı.

kapsama-vs-ulaşılabilir-kodu