Fixes project names being validated as IDs

CHANGELOG.md

@@ -1,7 +1,9 @@
-# Unreleased
+# v1.15.1
 
 ## Bug Fixes
 
+* Project names were being incorrectly validated as ID's @brandonc [#608](https://github.com/hashicorp/go-tfe/pull/608)
+
 ## Enhancements
 
 # v1.15.0

project.go

@@ -203,16 +203,9 @@ func (o ProjectCreateOptions) valid() error {
 	if !validString(&o.Name) {
 		return ErrRequiredName
 	}
-
-	if !validStringID(&o.Name) {
-		return ErrInvalidName
-	}
 	return nil
 }
 
 func (o ProjectUpdateOptions) valid() error {
-	if o.Name != nil && !validStringID(o.Name) {
-		return ErrInvalidName
-	}
 	return nil
 }

projects_integration_test.go

@@ -123,7 +123,7 @@ func TestProjectsCreate(t *testing.T) {
 			Name: badIdentifier,
 		})
 		assert.Nil(t, w)
-		assert.EqualError(t, err, ErrInvalidName.Error())
+		assert.Contains(t, err.Error(), "invalid attribute\n\nName may only contain")
 	})
 
 	t.Run("when options has an invalid organization", func(t *testing.T) {
@@ -149,7 +149,7 @@ func TestProjectsUpdate(t *testing.T) {
 		defer kTestCleanup()
 
 		kAfter, err := client.Projects.Update(ctx, kBefore.ID, ProjectUpdateOptions{
-			Name: String("new_project_name"),
+			Name: String("new project name!"),
 		})
 		require.NoError(t, err)
 
@@ -165,7 +165,7 @@ func TestProjectsUpdate(t *testing.T) {
 			Name: String(badIdentifier),
 		})
 		assert.Nil(t, kAfter)
-		assert.EqualError(t, err, ErrInvalidName.Error())
+		assert.Contains(t, err.Error(), "invalid attribute\n\nName may only contain")
 	})
 
 	t.Run("without a valid projects ID", func(t *testing.T) {