diff --git a/plinth/modules/samba/tests/test_views.py b/plinth/modules/samba/tests/test_views.py index ed1919470..f0bc5592e 100644 --- a/plinth/modules/samba/tests/test_views.py +++ b/plinth/modules/samba/tests/test_views.py @@ -55,7 +55,7 @@ SHARES = [ "mount_point": "/media/root/otherdisk", "path": "/media/root/otherdisk/FreedomBox/shares/homes/open_share", "share_type": "open" - } + }, ] @@ -110,14 +110,10 @@ def test_samba_shares_view(rf): '/media/root/otherdisk': ['open'] } assert response.context_data['unavailable_shares'] == [{ - 'mount_point': - '/media/root/otherdisk', - 'name': - 'otherdisk', - 'path': - '/media/root/otherdisk/FreedomBox/shares/homes/open_share', - 'share_type': - 'open' + 'mount_point': '/media/root/otherdisk', + 'name': 'otherdisk', + 'path': '/media/root/otherdisk/FreedomBox/shares/homes/open_share', + 'share_type': 'open' }] assert response.context_data['users'] == USERS assert response.status_code == 200 @@ -127,8 +123,8 @@ def test_enable_samba_share_view(rf): """Test that enabling share sends correct success message.""" form_data = {'filesystem_type': 'ext4', 'open_share': 'enable'} mount_point = urllib.parse.quote('/') - response, messages = make_request( - rf.post('', data=form_data), views.share, mount_point=mount_point) + response, messages = make_request(rf.post('', data=form_data), views.share, + mount_point=mount_point) assert list(messages)[0].message == 'Share enabled.' assert response.status_code == 302 @@ -142,8 +138,8 @@ def test_enable_samba_share_failed_view(rf): error_message = 'Sharing failed' with patch('plinth.modules.samba.add_share', side_effect=ActionError(error_message)): - response, messages = make_request( - rf.post('', data=form_data), views.share, mount_point=mount_point) + response, messages = make_request(rf.post('', data=form_data), + views.share, mount_point=mount_point) assert list(messages)[0].message == 'Error enabling share: {0}'.format( error_message) @@ -155,8 +151,8 @@ def test_disable_samba_share(rf): """Test that enabling share sends correct success message.""" form_data = {'filesystem_type': 'ext4', 'open_share': 'disable'} mount_point = urllib.parse.quote('/') - response, messages = make_request( - rf.post('', data=form_data), views.share, mount_point=mount_point) + response, messages = make_request(rf.post('', data=form_data), views.share, + mount_point=mount_point) assert list(messages)[0].message == 'Share disabled.' assert response.status_code == 302 @@ -170,10 +166,11 @@ def test_disable_samba_share_failed_view(rf): error_message = 'Unsharing failed' with patch('plinth.modules.samba.delete_share', side_effect=ActionError(error_message)): - response, messages = make_request( - rf.post('', data=form_data), views.share, mount_point=mount_point) + response, messages = make_request(rf.post('', data=form_data), + views.share, mount_point=mount_point) - assert list(messages)[ - 0].message == 'Error disabling share: {0}'.format(error_message) + assert list( + messages)[0].message == 'Error disabling share: {0}'.format( + error_message) assert response.status_code == 302 assert response.url == urls.reverse('samba:index')