Skip to content

chore: make default workspace proxy editable #7903

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 13 commits into from
Jun 8, 2023
Merged

chore: make default workspace proxy editable #7903

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
9cb6421
chore: add database methods for editing the default workspace proxy
Emyrk Jun 7, 2023
9db95ae
Unit test to fetch primary with httpmw
Emyrk Jun 7, 2023
e2356b2
Show default proxy in proxy list
Emyrk Jun 7, 2023
d4f00ce
fixup! Show default proxy in proxy list
Emyrk Jun 7, 2023
9a6e5e1
Import order
Emyrk Jun 7, 2023
55b4527
Lint
Emyrk Jun 7, 2023
d4a8b81
fix unit tests
Emyrk Jun 7, 2023
28d80ad
Make sure unit tests have deployment ids
Emyrk Jun 7, 2023
904ce34
Unit test deployment id stuff
Emyrk Jun 7, 2023
bf0e1ce
Fix data race'
Emyrk Jun 7, 2023
881895b
sql spacing
Emyrk Jun 8, 2023
356dc3a
factor out patch primary
Emyrk Jun 8, 2023
79e2637
Make gen
Emyrk Jun 8, 2023
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
Make sure unit tests have deployment ids
  • Loading branch information
@Emyrk
Emyrk committed Jun 7, 2023
commit 28d80adf8c9de5466647eb1a65e853de61d7879d
4 changes: 4 additions & 0 deletions coderd/coderdtest/coderdtest.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -206,6 +206,10 @@ func NewOptions(t testing.TB, options *Options) (func(http.Handler), context.Can
options.Database = dbauthz.New(options.Database, options.Authorizer, slogtest.Make(t, nil).Leveled(slog.LevelDebug))
}

// Some routes expect a deployment ID
err := options.Database.InsertDeploymentID(dbauthz.AsSystemRestricted(context.Background()), uuid.NewString())
require.NoError(t, err, "insert a deployment id")

if options.DeploymentValues == nil {
options.DeploymentValues = DeploymentValues(t)
}
Expand Down