D3vl0per / clean-code-solid-bootcamp-2022-06-24

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Előkészítés:

Lokális fejlesztőkörnyezet telepítése

ng serve --open
ng test

Postman letöltése: https://www.postman.com/downloads/


Story:

  • HTML-t és az általa hívott függvényeket a UX csapat meghatározta, így kell működnie, ahogy most látjuk.
  • A Nominatim API használatát (backend) a CTO meghatározta, így kell hívni, ahogy most látjuk.
  • A CIB bankra a Nominatim API nem tud Szlovákiában keresni, ezért az előző fejlesztő hozzáadott egy if CIB és if Bratislava kivételt, hogy ilyenkor a VUB bankra keressünk.

Amire a jövőben számítani lehet, hogy változni fog:

  • A UX csapat bármikor meggondolhatja magát, és akkor másképp kell majd a kinézetet alakítani, az API-tól kapott mezőket lehet, hogy másképp kell majd publikálni a HTML-ben. Amit mindenképp meg akarnak tartani, az a terminál és a felette lévő kikeresett banki branchet mutató UI widget mint különálló UI komponensek. Ezekre akár egymástól függetlenül is jöhetnek változtatási kérések. Ezért kérték, hogy mindenképp két model legyen hozzájuk, ezt az előző fejlesztő létre is hozta.
  • Új országra, városra nem kell számítani, viszont új bankok jöhetnek a rendszerbe. Őket ugyanúgy fogjuk keresni, ahogy most az OTP-t és a CIB-et, azaz a kisbetűs magyar banknévvel és a kisbetűs városnévvel.
  • Elképzelhető, hogy a Nominatim API adataiban változás lesz, és a bank nevét az amenity mezőben másképp fogja visszaadni, tehát több variáció is lehet még az OTP-re és CIB-re is, mint ami most le van kódolva.
  1. Feladat - Funkcionális teszt lefedettség
  • A viewModel architektúrának köszönhetően elég az app viewModelljeinek állapotát vizsgálni, de jelenleg csak 1 search van lefedve teszttel.
  • A további 5 search-öt is érdemes lefedni integrációs teszttel, ezzel biztosítható lesz, hogy a refaktoroknál nem fognak eltörni.

About


Languages

Language:TypeScript 69.9%Language:CSS 17.4%Language:HTML 8.0%Language:JavaScript 4.7%