Skip to content

feat: add --experiments flag #5767

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 17 commits into from
Jan 18, 2023
Merged

feat: add --experiments flag #5767

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
76c7b91
feat: support multiple values for --experimental
johnstcn Jan 16, 2023
626d81b
develop.sh: allow passing additional args to backend API via -- arg1=…
johnstcn Jan 18, 2023
d93a373
fix swaggerparser test
johnstcn Jan 18, 2023
e82c306
make fmt
johnstcn Jan 18, 2023
047771c
require AuthN on /api/v2/experiments
johnstcn Jan 18, 2023
3ba27ea
fixup! require AuthN on /api/v2/experiments
johnstcn Jan 18, 2023
3ebb778
fix nits
johnstcn Jan 18, 2023
2aeb93e
add type alias for codersdk.Experiment
johnstcn Jan 18, 2023
53d92d7
make it clear that Entitlements.Experimental is deprecated
johnstcn Jan 18, 2023
ef0cbc3
remove unnecessary types as they are now autogenerated
johnstcn Jan 18, 2023
8fdb21b
apply suggestions from code review
johnstcn Jan 18, 2023
0cb9a6a
fixup! apply suggestions from code review
johnstcn Jan 18, 2023
44e9638
warn about dragons
johnstcn Jan 18, 2023
6c32aa5
fix apidoc type
johnstcn Jan 18, 2023
1cbaaf7
codersdk.Experiments no longer needs be flaggable or special-cased in…
johnstcn Jan 18, 2023
67d07f3
lower-case when ingesting and comparing experiments
johnstcn Jan 18, 2023
196b263
update apitypings unit tests
johnstcn Jan 18, 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
fix nits
  • Loading branch information
@johnstcn
johnstcn committed Jan 18, 2023
commit 3ebb778a145f12495b62cfa2db14bd5c85541f46
2 changes: 2 additions & 0 deletions coderd/experiments_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -90,6 +90,8 @@ func Test_Experiments(t *testing.T) {
t.Run("Unauthorized", func(t *testing.T) {
t.Setenv("CODER_EXPERIMENTAL", "foo,bar")
client := coderdtest.New(t, nil)
// Explicitly omit creating a user so we're unauthorized.
// _ = coderdtest.CreateFirstUser(t, client)

ctx, cancel := context.WithTimeout(context.Background(), testutil.WaitLong)
defer cancel()
Expand Down
4 changes: 2 additions & 2 deletions codersdk/experiments.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,11 +37,11 @@ func (e Experiments) Enabled(ex string) bool {
func (c *Client) Experiments(ctx context.Context) (Experiments, error) {
res, err := c.Request(ctx, http.MethodGet, "/api/v2/experiments", nil)
if err != nil {
return []string{}, err
return nil, err
}
defer res.Body.Close()
if res.StatusCode != http.StatusOK {
return []string{}, readBodyAsError(res)
return nil, readBodyAsError(res)
}
var exp []string
return exp, json.NewDecoder(res.Body).Decode(&exp)
Expand Down