SS-1112 Enforce unique project names by owner

[anondo1969]

Description

Source: https://scilifelab.atlassian.net/browse/SS-1112

The users should only be able to have a single project with a particular name. This should not be applied to admins.

Checklist

If you're unsure about any of the items below, don't hesitate to ask. We're here to help!
This is simply a reminder of what we are going to look for before merging your code.

• This pull request is against develop branch (not applicable for hotfixes)
• I have included a link to the issue on GitHub or JIRA (if any)
• I have included migration files (if there are changes to the model classes)
• I have included, reviewed and executed tests (unit and end2end) to complement my changes
• I have updated the related documentation (if necessary)
• I have added a reviewer for this pull request
• I have added myself as an author for this pull request
• In the case I have modified settings.py, then I have also updated the studio-settings-configmap.yaml file in serve-charts

Further comments

Anything else you think we should know before merging your code!

[churnikov]

Suggested change

	# Ensure no duplicate project name for the common user
	current_user_previous_project_names = list(Project.objects.filter(owner=request.user).values("name", "status"))
	project_name_already_exists = any(
	current_user_project["name"] == name and current_user_project["status"] != "deleted"
	for current_user_project in current_user_previous_project_names
	)
	# Ensure no duplicate project name for the common user
	project_name_already_exists = Project.objects.filter(
	owner=request.user,
	name=name,
	).exclude(status="deleted").exists()

[anondo1969]

Thanks. changed the code according to the suggestion.

[churnikov]

Suggested change

	arr = ProjectTemplate.objects.filter(name=pre_selected_template)
	template = arr[0] if len(arr) > 0 else None
	template = ProjectTemplate.objects.filter(name=pre_selected_template).first()

[anondo1969]

Thanks. changed the code according to the suggestion.

[alfredeen]

I tested it and it works fine. It would be nice to add some end2end tests for this as well.

[akochari]

I tested it and it works fine. It would be nice to add some end2end tests for this as well.

We have this planned in a separate PR.

[anondo1969]

I tested it and it works fine. It would be nice to add some end2end tests for this as well.

Thanks a lot. I will write it in the near future.

[anondo1969]

SS-1112 Enforce unique project names by owner

For details: https://scilifelab.atlassian.net/browse/SS-1112