User activation

definitely a clean pull request
[non usa niente della classe per cui è definito] - effettivamente avrei potuto usare le variabili di classe che definisco poco sopra, non l'ho fatto per rendere piu' chiara la chiamata "self.inject_user(TEST_USER_DATA, UserActions.user, UserActions.psw)"

[fa cose differenti a seconda del valore di inject_dest] - concettualmente non fa cose differenti, sta sempre iniettando un utente, solo che  a seconda della destinazione cambia di poco la struttura e i dati, si potrebbe spezzare, ma l'ho lasciata unica perche' i controlli che faccio sul file di destinazione prima di lavorarci sopra sono uguali e creare una funzione che fa un check su un file e che poi ne richiama due a seconda di un flag mi sembra eccessivo, potrei estraniare la funzione del check del file e chiamarla dentro le due nuove, ma ha senso fare 3 funzioni per risparmiare due blocchi if? Magari con una piccola documentazione risulta piu' chiara e si risolve prima.
[Non potrebbe essere spezzato in due o più funzioni (attenzione: funzioni, non metodi) che fanno un'unica cosa?] - non si dovrebbe rispondere a una domanda con un'altra domanda, ma perche' fare delle funzioni? Queste sono operazioni che svolgo esclusivamente in questa classe di test separarle completamente non creerebbe confusione?