chore: rename 'first-organization' to 'coder' (coder#14808)

Emyrk · web-flow · commit 5c977c6be799 · 2024-09-26T13:20:44.000-05:00
Rename the first-organization original name. Users can change from the
original name.
diff --git a/cli/testdata/coder_list_--output_json.golden b/cli/testdata/coder_list_--output_json.golden
@@ -7,7 +7,7 @@
     "owner_name": "testuser",
     "owner_avatar_url": "",
     "organization_id": "[first org ID]",
-    "organization_name": "first-organization",
+    "organization_name": "coder",
     "template_id": "[template ID]",
     "template_name": "test-template",
     "template_display_name": "",
diff --git a/coderd/database/dbmem/dbmem.go b/coderd/database/dbmem/dbmem.go
@@ -92,8 +92,8 @@ func New() database.Store {
 	// Always start with a default org. Matching migration 198.
 	defaultOrg, err := q.InsertOrganization(context.Background(), database.InsertOrganizationParams{
 		ID:          uuid.New(),
-		Name:        "first-organization",
-		DisplayName: "first-organization",
+		Name:        "coder",
+		DisplayName: "Coder",
 		Description: "Builtin default organization.",
 		Icon:        "",
 		CreatedAt:   dbtime.Now(),
diff --git a/coderd/database/migrations/000259_rename_first_organization.down.sql b/coderd/database/migrations/000259_rename_first_organization.down.sql
@@ -0,0 +1,3 @@
+-- Leave the name as 'coder', there is no downside.
+-- The old name 'first-organization' is not used anywhere, just the
+-- is_default property.
diff --git a/coderd/database/migrations/000259_rename_first_organization.up.sql b/coderd/database/migrations/000259_rename_first_organization.up.sql
@@ -0,0 +1,10 @@
+UPDATE
+	organizations
+SET
+	name = 'coder',
+	display_name = 'Coder'
+WHERE
+	-- The old name was too long.
+	name = 'first-organization'
+	AND is_default = true
+;
diff --git a/enterprise/cli/create_test.go b/enterprise/cli/create_test.go
@@ -202,6 +202,6 @@ func TestEnterpriseCreate(t *testing.T) {
 		err := inv.Run()
 		require.Error(t, err)
 		// The error message should indicate the flag to fix the issue.
-		require.ErrorContains(t, err, fmt.Sprintf("--org=%q", "first-organization"))
+		require.ErrorContains(t, err, fmt.Sprintf("--org=%q", "coder"))
 	})
 }

Original file line number	Diff line number	Diff line change
`@@ -0,0 +1,3 @@`
	`1`	`+-- Leave the name as 'coder', there is no downside.`
	`2`	`+-- The old name 'first-organization' is not used anywhere, just the`
	`3`	`+-- is_default property.`
Original file line number	Diff line number	Diff line change
`@@ -202,6 +202,6 @@ func TestEnterpriseCreate(t *testing.T) {`
`202`	`202`	`err := inv.Run()`
`203`	`203`	`require.Error(t, err)`
`204`	`204`	`// The error message should indicate the flag to fix the issue.`
`205`		`- require.ErrorContains(t, err, fmt.Sprintf("--org=%q", "first-organization"))`
	`205`	`+ require.ErrorContains(t, err, fmt.Sprintf("--org=%q", "coder"))`
`206`	`206`	`})`
`207`	`207`	`}`