Ajust conversarion visibility deletion test to properly change user

Controller visibility test that test if a foreign user can't delete visibility does not properly change user before try to delete record.

Currently, that test only check if current user can delete same record twice.
You test external library behavior there,  the`sign_in` method from Devise. Please remove this, you don't do this in unit tests.