Remaining days should be renewed on NYE

[lucyhilltop]

Only 28 vacation days can be transferred from the year before, rest are expired and should be deleted from the day-count

[lucyhilltop]

• [x] Iga aasta iga kasutaja puhkuste jääk kirjutatakse eraldi tabelisse
• [ ] Tabelit uuendatakse kõigil 1x aastas
• [x] Kõigi kasutajate jaoks tabeli uuendamiseks on nupp admin kasutajal
• [x] Nupu vajutamisel peab kindlasti mingi teavitus olema kas update õnnestus/ei õnnestunud
• [x] User management all peab jäägi väli alles jääma
• [x] * Peab olema selge, mis aasta jääki kuvatakse
• [x] * Kui jääki käsitsi uuendatakse, tuleb jäägi tabelit 'save' peale update'ida. Sealjuures käsitsi sisestatud jääk tuleb arvestada, et on täpsem, kui arvutatud jääk
• [x] * Kui 1. tööpäev puudub, jääb arvutamine nii, nagu praegu on. Jäägi arvutamise protsess arvestab jäägiks sel juhul 0 või ""
• [x] * Kui lisatakse 1. tööpäev (ja jääki mitte), siis peaks 'save' vajutamisel arvutama eelmise aasta jäägi kasutajal

[lucyhilltop]

Potentsiaalsed test case'id:

• [ ] 2017 aasta jäägi lisamine tabelisse aastal 2018, et kirje on kirjutatud
• [ ] 2017 aasta jäägi muutmine tabelis, kui jääk on lisatud user management alt käsitsi
• [ ] Kui lisatakse kasutajale esimene tööpäev (aastasse 2017) ja jääk, peaks salvestatama tabelisse kirjutatud jääk
• [ ] Kui lisatakse kasutajale esimene tööpäev (aastasse 2017) ja jääki ei lisata: -- Kui kasutajal on 2017 aastal puhkuseid, siis arvutada, kui palju puhkusepäevi kasutaja teenis tööl oldud aja jooksul ja lahutada puhatud päevad (siin silmas pidada, et kuna kasutajal polnud esimest tööpäeva, on tal kindlasti puhatud päevade arv negatiivne. Ei tohi kogemata lahutada negatiivset arvu) -- Kui kasutajal pole puhkuseid, siis arvutada, palju puhkusepäevi kasutaja teenis tööl oldud aja jooksul
• [ ] Kui tabeli uuendamise nuppu vajutatakse pärast seda, kui seda on juba vajutatud (ja vahepeal pole keegi käsitsi midagi muutnud), peaks kõigi jääk samaks jääma
• [ ] Kui kasutajal on 2016 jääk olemas ning nuppu vajutatakse aastal 2018, peaks tabelis ja user managementi all uut jääki kuvatama. Tabelis peaks olema salvestatud ka 2016 jääk
• [ ] Kontrollida, et 2018 aastal jooksvat jääki arvutatakse 2017 jäägiga
• [ ] Paar lihtsamat juhtu: kontrollida, et kui 2017 jooksev puhkuste jääk on 15, on 2018 eelmise aasta jääk 15, kui 0 siis 0, kui -5 siis -5 jne.

[lucyhilltop]

Was tested in demo environment, but unit tests are still missing.

[HallikiM]

Nende unit testide kirjutamisega tekib see probleem, et enamus testcase-ide jaoks oleks vaja aega kontrollida. Otsisin, kaalusin ja katsetasin erinevaid variante, kuidas neid teste kirjutada nii, et oleks võimalik aega kontrollid või poleks vaja aega kontrollida. Viimase puhul eriti häid teste ikkagi ei tule, sest testid ei ole korratavad. Näiteks kui jooksutad eelmisel päeval ebaõnnestunud testi uuesti, siis see võib ebaõnnestuda ainutl selle tõttu, et kuupäev on teine.

Mõningad ideed, mida saaks teha, et neid teste ikkagi teha. Kahe esimese variandi puhul oleks vist mõistlik, et kasutatakse üle terve projekti kas java.time APIt või Joda Time-i. VacationCalculationUtils klassis on näiteks mõlemaid kasutatud. Aga üldiselt tahaks need variandid kellegagi koos läbi arutada, et mis on kõige parem variant. Võibolla on veel mõni idee.

1. java.time API Clock-i kasutades https://stackoverflow.com/a/2001696 https://docs.oracle.com/javase/8/docs/api/java/time/Clock.html See eeldaks, et kõik .now meetodid saavad endale Clock-i parameetriks.
2. Kui kasutada joda-time, siis seal on DateTimeUtils klassis setCurrentMillisFixed meetod https://www.joda.org/joda-time/apidocs/org/joda/time/DateTimeUtils.html ja Interface DateTimeUtils.MillisProvider https://www.joda.org/joda-time/apidocs/org/joda/time/DateTimeUtils.MillisProvider.html
3. Lisada aja parameetrid alati meetodi väljakutsesse.
4. Powermockiga peaks olema võimalik ka klassimeetodeid mockida. See muudaks testid väga implementatsioonispetsiifiliseks.

Linke lisainfoga: java 8 java.time API ja Joda-Time võrdlus https://stackoverflow.com/questions/24631909/differences-between-java-8-date-time-api-java-time-and-joda-time https://softwareengineering.stackexchange.com/questions/235145/real-time-unit-testing-or-how-to-mock-now https://www.baeldung.com/java-override-system-time