Update workspace generator function name

This generates agent models, not workspaces.  Renaming opens the way for
generating random workspaces.

Author: code-asher

src/test/groovy/CoderCLIManagerTest.groovy

@@ -413,7 +413,7 @@ class CoderCLIManagerTest extends Specification {
                 .replace("/tmp/coder-gateway/test.coder.invalid/coder-linux-amd64", CoderCLIManager.escape(ccm.localBinaryPath.toString()))
 
         when:
-        ccm.configSsh(workspaces.collect { DataGen.workspace(it) }, headerCommand)
+        ccm.configSsh(workspaces.collect { DataGen.workspaceAgentModel(it) }, headerCommand)
 
         then:
         sshConfigPath.toFile().text == expectedConf
@@ -473,7 +473,7 @@ class CoderCLIManagerTest extends Specification {
         def ccm = new CoderCLIManager(new URL("https://test.coder.invalid"), tmpdir)
 
         when:
-        ccm.configSsh(["foo", "bar"].collect { DataGen.workspace(it) }, headerCommand)
+        ccm.configSsh(["foo", "bar"].collect { DataGen.workspaceAgentModel(it) }, headerCommand)
 
         then:
         thrown(Exception)

src/test/groovy/CoderWorkspacesStepViewTest.groovy

@@ -9,46 +9,46 @@ class CoderWorkspacesStepViewTest extends Specification {
         def table = new WorkspacesTable()
         table.listTableModel.items = List.of(
                 // An off workspace.
-                DataGen.workspace("ws1", "ws1"),
+                DataGen.workspaceAgentModel("ws1", "ws1"),
 
                 // On workspaces.
-                DataGen.workspace("agent1", "ws2"),
-                DataGen.workspace("agent2", "ws2"),
-                DataGen.workspace("agent3", "ws3"),
+                DataGen.workspaceAgentModel("agent1", "ws2"),
+                DataGen.workspaceAgentModel("agent2", "ws2"),
+                DataGen.workspaceAgentModel("agent3", "ws3"),
 
                 // Another off workspace.
-                DataGen.workspace("ws4", "ws4"),
+                DataGen.workspaceAgentModel("ws4", "ws4"),
 
                 // In practice we do not list both agents and workspaces
                 // together but here test that anyway with an agent first and
                 // then with a workspace first.
-                DataGen.workspace("agent2", "ws5"),
-                DataGen.workspace("ws5", "ws5"),
-                DataGen.workspace("ws6", "ws6"),
-                DataGen.workspace("agent3", "ws6"),
+                DataGen.workspaceAgentModel("agent2", "ws5"),
+                DataGen.workspaceAgentModel("ws5", "ws5"),
+                DataGen.workspaceAgentModel("ws6", "ws6"),
+                DataGen.workspaceAgentModel("agent3", "ws6"),
         )
 
         expect:
         table.getNewSelection(selected) == expected
 
         where:
-        selected                           | expected
-        null                               | -1 // No selection.
-        DataGen.workspace("gone", "gone")  | -1 // No workspace that matches.
-        DataGen.workspace("ws1", "ws1")    | 0  // Workspace exact match.
-        DataGen.workspace("gone", "ws1")   | 0  // Agent gone, select workspace.
-        DataGen.workspace("ws2", "ws2")    | 1  // Workspace gone, select first agent.
-        DataGen.workspace("agent1", "ws2") | 1  // Agent exact match.
-        DataGen.workspace("agent2", "ws2") | 2  // Agent exact match.
-        DataGen.workspace("ws3", "ws3")    | 3  // Workspace gone, select first agent.
-        DataGen.workspace("agent3", "ws3") | 3  // Agent exact match.
-        DataGen.workspace("gone", "ws4")   | 4  // Agent gone, select workspace.
-        DataGen.workspace("ws4", "ws4")    | 4  // Workspace exact match.
-        DataGen.workspace("agent2", "ws5") | 5  // Agent exact match.
-        DataGen.workspace("gone", "ws5")   | 5  // Agent gone, another agent comes first.
-        DataGen.workspace("ws5", "ws5")    | 6  // Workspace exact match.
-        DataGen.workspace("ws6", "ws6")    | 7  // Workspace exact match.
-        DataGen.workspace("gone", "ws6")   | 7  // Agent gone, workspace comes first.
-        DataGen.workspace("agent3", "ws6") | 8  // Agent exact match.
+        selected                                     | expected
+        null                                         | -1 // No selection.
+        DataGen.workspaceAgentModel("gone", "gone")  | -1 // No workspace that matches.
+        DataGen.workspaceAgentModel("ws1", "ws1")    | 0  // Workspace exact match.
+        DataGen.workspaceAgentModel("gone", "ws1")   | 0  // Agent gone, select workspace.
+        DataGen.workspaceAgentModel("ws2", "ws2")    | 1  // Workspace gone, select first agent.
+        DataGen.workspaceAgentModel("agent1", "ws2") | 1  // Agent exact match.
+        DataGen.workspaceAgentModel("agent2", "ws2") | 2  // Agent exact match.
+        DataGen.workspaceAgentModel("ws3", "ws3")    | 3  // Workspace gone, select first agent.
+        DataGen.workspaceAgentModel("agent3", "ws3") | 3  // Agent exact match.
+        DataGen.workspaceAgentModel("gone", "ws4")   | 4  // Agent gone, select workspace.
+        DataGen.workspaceAgentModel("ws4", "ws4")    | 4  // Workspace exact match.
+        DataGen.workspaceAgentModel("agent2", "ws5") | 5  // Agent exact match.
+        DataGen.workspaceAgentModel("gone", "ws5")   | 5  // Agent gone, another agent comes first.
+        DataGen.workspaceAgentModel("ws5", "ws5")    | 6  // Workspace exact match.
+        DataGen.workspaceAgentModel("ws6", "ws6")    | 7  // Workspace exact match.
+        DataGen.workspaceAgentModel("gone", "ws6")   | 7  // Agent gone, workspace comes first.
+        DataGen.workspaceAgentModel("agent3", "ws6") | 8  // Agent exact match.
     }
 }

src/test/groovy/DataGen.groovy

@@ -12,7 +12,7 @@ class DataGen {
     //       toAgentModels() on it.  Also the way an "agent" model can have no
     //       agent in it seems weird; can we refactor to remove
     //       WorkspaceAgentModel and use the original structs from the API?
-    static WorkspaceAgentModel workspace(String name, String workspaceName = "", UUID agentId = UUID.randomUUID()) {
+    static WorkspaceAgentModel workspaceAgentModel(String name, String workspaceName = "", UUID agentId = UUID.randomUUID()) {
         return new WorkspaceAgentModel(
                 workspaceName == "" ? null : agentId,
                 UUID.randomUUID(),