mirror of
https://github.com/GSA/notifications-admin.git
synced 2026-07-17 13:09:49 -04:00
Check if folder is empty before calling the api
This commit is contained in:
@@ -408,6 +408,14 @@ def delete_template_folder(service_id, template_folder_id):
|
||||
form = TemplateFolderForm()
|
||||
template_folder_path = current_service.get_template_folder_path(template_folder_id)
|
||||
template_folder_name = template_folder_path[-1]["name"]
|
||||
if len(current_service.get_template_folders_and_templates(
|
||||
template_type="all", template_folder_id=template_folder_id
|
||||
)) > 0:
|
||||
flash("'{}' folder is not empty".format(template_folder_name), 'info')
|
||||
return redirect(url_for(
|
||||
'.manage_template_folder', service_id, template_folder_id
|
||||
))
|
||||
|
||||
if request.method == 'POST':
|
||||
try:
|
||||
template_folder_api_client.delete_template_folder(current_service.id, template_folder_id)
|
||||
|
||||
@@ -381,9 +381,13 @@ def test_rename_folder(client_request, service_one, mock_get_template_folders, m
|
||||
def test_delete_folder(client_request, service_one, mock_get_template_folders, mocker):
|
||||
mock_delete = mocker.patch('app.template_folder_api_client.delete_template_folder')
|
||||
folder_id = str(uuid.uuid4())
|
||||
mock_get_template_folders.return_value = [
|
||||
mock_get_template_folders.side_effect = [[
|
||||
{'id': folder_id, 'name': 'sacrifice', 'parent_id': None},
|
||||
]
|
||||
], []]
|
||||
mocker.patch(
|
||||
'app.models.service.Service.get_templates',
|
||||
return_value=[],
|
||||
)
|
||||
service_one['permissions'] += ['edit_folders']
|
||||
|
||||
client_request.post(
|
||||
|
||||
Reference in New Issue
Block a user