karhatsu
commented
13 years ago Mietin ensin sitä, että käyttöliittymän ohjeet monimutkaistuvat tässä ratkaisussa mutta sitten tajusin, ettei kälissä itse asiassa tarvitakaan mitään muutoksia. Eli tämä on oikeastaan aika näppärä ratkaisu käytettävyyden näkökulmasta.
Specit ovat tosiaan aika monimutkaiset. Näin usein käy, kun tulee iso hierarkia eri tapauksia. Ratkaisu olisi varmaankin se, että pitäisi eriyttää koko tallennusketju yksittäisistä tilanteista. Esim. niin, että testataan erikseen sallitut string-formaatit ja sitten muissa metodeissa stubataan kyseinen metodi palauttamaan true. Nythän tuo on vähän integraatiomaista testausta mutta toisaalta voidaan myös argumentoida, että kyllä siinä public-metodeja testataan ihan siististi eikä sisäyseen toteutukseen pitäisikään ottaa testien puolella kantaa. Joka tapauksessa nyt ei tarvitse tehdä mitään muutoksia ellet itse halua.
Tänään ja huomenna on tosi kiireistä mutta yritän ehtiä tekemään mergen jossain vaiheessa.
jkpj
Pikasyötössä estä päällekirjoitus paitsi jos ++ ennen numeroa. Spec voisi olla selkeämminkin tehty.