Currency ve unit test iyileştirmeleri

[edigu]

Currency bileşeninde kullanılan para birimi bilgisinin kütüphane içinde filtre ve validasyondan geçirilmeden payu tarafına aktarılmasının beklenmedik sonuçlara sebep olabileceğini düşündüğüm için basit bir filtre-validasyon mekanizması ekledim. Bununla beraber payu dökümanlarında birimin ISO 4217 standardında olması gerektiği belirtildiğinden ilgili para birimleri listesini sınıfa ekleyerek validasyonu bunun üzerinden yaptım.

Composer.json require-dev içinde phpunit bağımlılık olarak gelse de ne yaptıysam vendor/bin içindeki phpunit ile testleri çalıştıramadım. tests dizinine eski usül phpunit.xml ile basit bir Boostrap.php ekledim. Namespacing/autoloading mekanizması testler sırasında da composer'ın autoloader'ı üzerinden çalışıyor. Readme.md dosyasına konuyla ilgili döküman ekledim.

Yaptığım diğer birkaç değişikliğin özeti:

• Test sonuç raporlarını ve mac'in .DS_Store'unu .gitignore'a ekledim.
• Kur ismini almaya yarayan yardımcı statik getNameByCode() metodunu ekledim.
• phpunit.xml direktifleriyle test sonuçlarının html raporu haline getirilebilmesini sağladım.

[ibrahimgunduz34]

Yılmaz selamlar, Ellerine sağlık. Çok doğru düşünmüşsün. Yalnız bu validasyonun yeri componentlerin içi olmamalı. Componentler, istek build edilirken serialize edilip doğrudan provider a iletilmesi amaçlanan yapılar. Bu noktada validasyonu validator sınıflarıyla halledebilirsin. Mesela bunun için bir OrderValidator oluşturup payment işlemi sırasında Order component ini doğrulayacak bir validator sınıfı çağırttırabilirsin. Halihazırda böyle bir yapı mevcut. Şayet beklediğin gibi çalışmıyorsa mümkünse burayı revize edelim. Tekrardan çok teşekkür ediyorum, ellerine sağlık.