Skip to content

feat: add customizable upgrade message on client/server version mismatch #11587

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 19 commits into from
Jan 30, 2024
Merged

feat: add customizable upgrade message on client/server version mismatch #11587

Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
442125d
feat: add customizable upgrade message on client/server version mismatch
sreya Jan 11, 2024
b2a0b8b
make gen
sreya Jan 12, 2024
d3fb0b8
typo
sreya Jan 12, 2024
d5cf4e1
golden
sreya Jan 12, 2024
8e6385a
delete cruft
sreya Jan 12, 2024
d161287
make gen
sreya Jan 16, 2024
42cae32
Merge branch 'main' into jon/installmsg
sreya Jan 29, 2024
61bef0a
pr comments
sreya Jan 29, 2024
460f8cf
make gen
sreya Jan 29, 2024
790cfc3
update golden files
sreya Jan 29, 2024
1749686
pr comments
sreya Jan 30, 2024
0531bd3
fix default value
sreya Jan 30, 2024
c9921fc
lint
sreya Jan 30, 2024
d880b9e
fmt
sreya Jan 30, 2024
fc2c926
golden
sreya Jan 30, 2024
180212c
Merge branch 'main' into jon/installmsg
sreya Jan 30, 2024
7b1be43
fix version formatting
sreya Jan 30, 2024
423ab8b
move trimming into helper
sreya Jan 30, 2024
bf2dce1
fix merge
sreya Jan 30, 2024
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
delete cruft
  • Loading branch information
@sreya
sreya committed Jan 12, 2024
commit 8e6385a98c74c332bd53f244a1708a8303c5c96a
25 changes: 0 additions & 25 deletions cli/root_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -105,31 +105,6 @@ func TestRoot(t *testing.T) {
require.ErrorContains(t, err, "unexpected status code 410")
require.EqualValues(t, 1, atomic.LoadInt64(&called), "called exactly once")
})

t.Run("VersionMismatchWarning", func(t *testing.T) {
t.Parallel()

var (
expectedUpgradeMessage = "My custom upgrade message"
dv = coderdtest.DeploymentValues(t)
)
dv.CLIUpgradeMessage = clibase.String(expectedUpgradeMessage)

client := coderdtest.New(t, &coderdtest.Options{
DeploymentValues: dv,
})

_ = coderdtest.CreateFirstUser(t, client)

inv, root := clitest.New(t, "ls")
clitest.SetupConfig(t, client, root)

var buf bytes.Buffer
inv.Stderr = &buf
err := inv.Run()
require.NoError(t, err)
fmt.Println("output: ", buf.String())
})
}

// TestDERPHeaders ensures that the client sends the global `--header`s and
Expand Down