Issue46 fixed

[giornaledisistema]

Bug fixato riportando i tempi alle stesse unità di misura.

[coveralls]

Coverage remained the same when pulling 760a15d3091978ca20fbda955a33ddf3260bd4a4 on giornaledisistema:master into f1dc1a092f0d798c877c6e4730eac8e36a3ab328 on corso-python-prato:master.

[k4rl85]

Essendo una costante non si potrebbe modificare direttamente USER_ACTIVATION_TIMESTAMP?

[giornaledisistema]

Se non serve così com'è in altri punti del codice, immagino non ci siano problemi. Verifico ed eventualmente ricommitto.

[GendoIkari]

Si, cerchiamo di mantenere le stesse unita' di misura, ad esempio millisecondi.

[coveralls]

Coverage remained the same when pulling 57846fadc838d5cdc3eca9e8a2aac15ba96e8051 on giornaledisistema:master into 6389d11a538c5be95cedcc8fa9d419160f06a83a on corso-python-prato:master.

[giornaledisistema]

Ho preferito rendere evidente che USER_ACTIVATION_TIMEOUT è in secondi rinominandola come USER_ACTIVATION_TIMEOUT_SECS, e trasformarla in decimillesimi di secondo nel solo punto dove serve espressa in questo modo.

[gvaldambrini]

Come detto a voce: serve un test che dimostri che prima c'era un problema. Quindi se prima i test coprivano quella parte ma passavano, vuol dire che erano deboli o da fixare.

[giornaledisistema]

Come anticipavo ieri a lezione, il bug s'intreccia strettamente con le modifiche da apportare per risolvere l'issue #41 di cui si sta occupando Roxana, per cui sto lavorando con lei alla risoluzione coordinata di entrambi in un'unica pull request.

[k4rl85]

Non so come mai ma la pull request risulta non mergiata, o meglio le modifiche non sono avvenute.

[coveralls]

Coverage remained the same when pulling 7905eb8757485228ca2c170b621c4b1261873dd9 on giornaledisistema:master into 3194b40b5d01509dba50dd9febc98e6168bbd647 on corso-python-prato:master.

[coveralls]

Coverage remained the same when pulling 7905eb8757485228ca2c170b621c4b1261873dd9 on giornaledisistema:master into 3194b40b5d01509dba50dd9febc98e6168bbd647 on corso-python-prato:master.