Conversation
…eam1 into black_box
…eam1 into black_box
…eam1 into black_box
…eam1 into black_box
client/client_daemon.py
Outdated
There was a problem hiding this comment.
non è un po ambiguo chiamarla share_path ? sembra la path della cartella dello share e non della cartella di rawbox no?
There was a problem hiding this comment.
non ho capito la differenza tra cartella dello share e quella di rawbox...
ho usato il termine share_path per definire la cartella con i file dell'utente che viene sincronizzata col server. Bene o male ho seguito questa terminologia per tutto il codice
There was a problem hiding this comment.
share_path sembra "RawBox/shares" a meno a me XD l'importante è essere coerenti, nel resto del demone non compare mai share_path, la path di rawbox viene chiamata dir_path, se vuoi cambiarla rinominala ovunque, però secondo me a quel punto è meglio rawbox_dir_path perche abbiamo anche una cartella shares e share_path può essere forviante per quello.
There was a problem hiding this comment.
cambiato nomenclatura
systemtest/testenvironment.py
Outdated
There was a problem hiding this comment.
è solo una questione stilistica, ma personalmente penso che era più bello (perché più conciso):
return PasswordChecker(password) == password
There was a problem hiding this comment.
lo stavo riguardando prima ed ero arrivato alla stessa conclusione. applico la modifica
|
Instance e non istance :) |
System test environment and related system test
Fixed some minor bug