Add system test #66
Add system test #66
Conversation
…eam1 into black_box
…eam1 into black_box
…eam1 into black_box
…eam1 into black_box
| @@ -798,11 +798,13 @@ def on_modified(self, event): | |||
|
|
|||
|
|
|||
| class DirSnapshotManager(object): | |||
| def __init__(self, snapshot_file_path): | |||
|
|
|||
| def __init__(self, snapshot_file_path, share_path): | |||
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
| if PasswordChecker(password) == password: | ||
| return True | ||
| else: | ||
| return False |
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