refactor: add test case for duplicate prefixes & remove redundant check on duplicate project name

Author: corban-beaird

master/internal/api_project.go

@@ -542,9 +542,6 @@ func (a *apiServer) PostProject(
 		if strings.Contains(err.Error(), "projects_key_key") {
 			return nil,
 				status.Errorf(codes.AlreadyExists, "project with key %s already exists", projectKey)
-		} else if strings.Contains(err.Error(), "projects_name_key") {
-			return nil,
-				status.Errorf(codes.AlreadyExists, "project with name %s already exists", req.Name)
 		}
 	}
 

master/internal/api_project_intg_test.go

@@ -435,3 +435,24 @@ func TestCreateProjectWithDuplicateProjectKey(t *testing.T) {
 	require.Error(t, err)
 	require.Equal(t, status.Errorf(codes.AlreadyExists, "project with key %s already exists", projectKey), err)
 }
+
+func TestCreateProjectWithDefaultKeyAndDuplicatePrefix(t *testing.T) {
+	api, _, ctx := setupAPITest(t, nil)
+	wresp, werr := api.PostWorkspace(ctx, &apiv1.PostWorkspaceRequest{Name: uuid.New().String()})
+	require.NoError(t, werr)
+
+	projectName := uuid.New().String()
+	projectKeyPrefix := projectName[:3]
+	resp1, err := api.PostProject(ctx, &apiv1.PostProjectRequest{
+		Name: projectName, WorkspaceId: wresp.Workspace.Id,
+	})
+	require.NoError(t, err)
+	require.Equal(t, (projectKeyPrefix + "1"), resp1.Project.Key)
+
+	resp2, err := api.PostProject(ctx, &apiv1.PostProjectRequest{
+		Name: projectName + "2", WorkspaceId: wresp.Workspace.Id,
+	})
+	require.NoError(t, err)
+	require.NoError(t, err)
+	require.Equal(t, (projectKeyPrefix + "2"), resp2.Project.Key)
+}