Skip to content

fix: Fix properly selecting workspace apps by agent #3684

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 12 commits into from
Aug 29, 2022
Merged

fix: Fix properly selecting workspace apps by agent #3684

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
433705d
fix: Fix properly naming workspace agents
Emyrk Aug 24, 2022
7d0bc56
chore: Make workspace and agent extract a middleware
Emyrk Aug 24, 2022
0bb51b4
chore: Write unit tests for agent selector
Emyrk Aug 24, 2022
8b3f46d
Add unit test cases
Emyrk Aug 24, 2022
f44f5c0
Linting
Emyrk Aug 24, 2022
712906e
Fix authorize unit test
Emyrk Aug 24, 2022
c5bacc6
Merge remote-tracking branch 'origin/main' into stevenmasley/select_a…
Emyrk Aug 25, 2022
4058665
Add test cases
Emyrk Aug 25, 2022
5401e70
Merge remote-tracking branch 'origin/main' into stevenmasley/select_a…
Emyrk Aug 25, 2022
48ed5db
Merge remote-tracking branch 'origin/main' into stevenmasley/select_a…
Emyrk Aug 26, 2022
991448d
Fix auth test on all http method types
Emyrk Aug 26, 2022
2a30b23
Rename url variable
Emyrk Aug 29, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Linting
  • Loading branch information
@Emyrk
Emyrk committed Aug 24, 2022
commit f44f5c05eb4e6466a52ce23751db9de437941ddc
23 changes: 12 additions & 11 deletions coderd/httpmw/workspaceparam_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,7 +133,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
Name string
// Agents are mapped to a resource
Agents map[string][]string
UrlParam string
URLParam string
WorkspaceName string
ExpectedAgent string
ExpectedStatusCode int
Expand All @@ -143,7 +143,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
Name: "NoAgents",
WorkspaceName: "dev",
Agents: map[string][]string{},
UrlParam: "dev",
URLParam: "dev",
ExpectedError: "No agents exist",
ExpectedStatusCode: http.StatusBadRequest,
},
Expand All @@ -156,7 +156,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-two",
},
},
UrlParam: "dev",
URLParam: "dev",
ExpectedStatusCode: http.StatusBadRequest,
ExpectedError: "More than one agent exists, but no agent specified",
},
Expand All @@ -171,7 +171,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-two",
},
},
UrlParam: "dev",
URLParam: "dev",
ExpectedStatusCode: http.StatusBadRequest,
ExpectedError: "More than one agent exists, but no agent specified",
},
Expand All @@ -183,7 +183,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-one",
},
},
UrlParam: "dev.not-exists",
URLParam: "dev.not-exists",
ExpectedStatusCode: http.StatusBadRequest,
ExpectedError: "No agent exists with the name",
},
Expand All @@ -201,7 +201,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-three",
},
},
UrlParam: "dev.not-exists",
URLParam: "dev.not-exists",
ExpectedStatusCode: http.StatusBadRequest,
ExpectedError: "No agent exists with the name",
},
Expand All @@ -216,7 +216,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-one",
},
},
UrlParam: "dev",
URLParam: "dev",
ExpectedAgent: "agent-one",
ExpectedStatusCode: http.StatusOK,
},
Expand All @@ -228,7 +228,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-one",
},
},
UrlParam: "dev",
URLParam: "dev",
ExpectedAgent: "agent-one",
ExpectedStatusCode: http.StatusOK,
},
Expand All @@ -242,7 +242,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-selected",
},
},
UrlParam: "dev.agent-selected",
URLParam: "dev.agent-selected",
ExpectedAgent: "agent-selected",
ExpectedStatusCode: http.StatusOK,
},
Expand All @@ -261,7 +261,7 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
"agent-three",
},
},
UrlParam: "dev.agent-selected",
URLParam: "dev.agent-selected",
ExpectedAgent: "agent-selected",
ExpectedStatusCode: http.StatusOK,
},
Expand All @@ -270,12 +270,13 @@ func TestWorkspaceAgentByNameParam(t *testing.T) {
for _, c := range testCases {
c := c
t.Run(c.Name, func(t *testing.T) {
t.Parallel()
db, r := setupWorkspaceWithAgents(t, setupConfig{
WorkspaceName: c.WorkspaceName,
Agents: c.Agents,
})

chi.RouteContext(r.Context()).URLParams.Add("workspacename_and_agent", c.UrlParam)
chi.RouteContext(r.Context()).URLParams.Add("workspacename_and_agent", c.URLParam)

rtr := chi.NewRouter()
rtr.Use(
Expand Down