filipfe / BezCV

MIT License
0 stars 0 forks source link

[Maintainability] Unit Tests #12

Open automationdream opened 1 year ago

automationdream commented 1 year ago

Repozytorium powinno zawierać chociaż najważniejsze testy jednostkowe.

filipfe commented 1 year ago

Nie jesteśmy fanami testowania jednostkowego, a o to kilka powodów:

Oprócz tego mamy koło ratunkowe w postaci Typescriptu co bardzo ułatwia identyfikację błędów na etapie użytku przy rozwijaniu. Nie zaprzeczamy natomiast, że takie testy w aplikacji się nie pojawią i nie będą konieczne dla szczególnych wypadków

automationdream commented 1 year ago

Ok, to będę miał parę pytań.. Może rozwiniemy ten temat, jeśli mogę 😃 ?

Bardzo dobrze, że używacie Typescriptu 👍 ale on pokrywa raczej warstwę front :(

Zgadzam się, że wszystko nie musi być testowane. Najlepiej jak są to najważniejsze funkcjonalności. Skoro nie wykluczacie ich na kolejnych etapach, czy powinniśmy pozostawić ten issue? Jaki nadałbyś temu priorytet?

automationdream commented 1 year ago

Tu podsyłam króciutki artykuł o piramidzie testów, czym w skrócie jest i czy istnieje jako standard w branży:

https://automationpanda.com/2018/08/01/the-testing-pyramid/

filipfe commented 1 year ago

Do tej pory, na przestrzeni czasu programowania strony nie stwierdziliśmy, aby któreś z poszczególnych komponentów lub ich współdziałającej grupy wymagały takiego testowania. Chcielibyśmy poznać natomiast praktyczną opinię tzn. w których konkretnie miejscach w naszej aplikacji takie testy mogłyby się pojawić? Chodzi o to, że znamy scenariusze dla większości komponentów i ja sam staram się izolować funkcjonalność do jak najmniejszych elementów przez co takie błędy mają bardzo małą szansę na wystąpienie lub są wręcz niemożliwe. Tutaj z Typescriptem oczywiście w pełni front, chodziło bardziej o to, że jeżeli jakiś błąd już wystąpi to mamy jasny obraz po stronie frontu przez co. Myślę, że issue można wykluczyć lub przesunąć na mniejszy priorytet i późniejszy etap, aby ewentualnie na końcu przetestować te elementy, u których występuje największe ryzyko.