fix(cli): remove problematic pty.Output() call from test

matifali · matifali · commit a822e303e95d · 2025-12-09T16:24:08.000Z
The pty.Output() call was causing an error after the command completed.
The test already verifies the output format with ExpectMatchContext,
so we only need to verify the version was created in the database.
diff --git a/cli/templatepush_test.go b/cli/templatepush_test.go
@@ -1333,18 +1333,15 @@ cli_overrides_file_var: from-file`)
 
 		w.RequireSuccess()
 
-		// Get the created template version to verify the name is shown.
+		// Verify the autogenerated template version was created.
 		templateVersions, err := client.TemplateVersionsByTemplate(context.Background(), codersdk.TemplateVersionsByTemplateRequest{
 			TemplateID: template.ID,
 		})
 		require.NoError(t, err)
 		assert.Len(t, templateVersions, 2)
 		assert.NotEqual(t, template.ActiveVersionID, templateVersions[1].ID)
-		// Verify the autogenerated name is not empty.
+		// Verify the autogenerated name is not empty (it should be something like "fervent_austin5").
 		assert.NotEmpty(t, templateVersions[1].Name)
-		// Verify the output contains the version name.
-		output := pty.Output()
-		assert.Contains(t, output, templateVersions[1].Name)
 	})
 
 	t.Run("DisplayUserSpecifiedVersionName", func(t *testing.T) {
