Skip to content

feat: Add confirm prompts to some cli actions #1591

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 18 commits into from
May 20, 2022
Merged

feat: Add confirm prompts to some cli actions #1591

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
a44b1fc
feat: Add confirm prompts to some cli actions
Emyrk May 19, 2022
664ea38
Add confirm prompt to delete workspace
Emyrk May 19, 2022
373cb23
Add unit test for prompt skipping
Emyrk May 19, 2022
3b6cea6
Fix test flakes
Emyrk May 19, 2022
843dd21
Add skip test for workspace create
Emyrk May 19, 2022
9c09a08
Make test parallel
Emyrk May 19, 2022
15acead
Merge remote-tracking branch 'origin/main' into stevenmasley/skip_prompt
Emyrk May 19, 2022
6a4682a
Allow -y to template update
Emyrk May 20, 2022
031134a
Merge remote-tracking branch 'origin/main' into stevenmasley/skip_prompt
Emyrk May 20, 2022
e334b22
Fix delete test
Emyrk May 20, 2022
2ecf664
Do not use deprecated function
Emyrk May 20, 2022
e1b6bfb
Add timeout to cmd execute
Emyrk May 20, 2022
0101e6d
revert test back to pre-skip
Emyrk May 20, 2022
d31fbc2
Remove set input
Emyrk May 20, 2022
bed1d44
Do not use ptty
Emyrk May 20, 2022
9d527bd
use ctx over don channel
Emyrk May 20, 2022
d3f0c6d
Merge remote-tracking branch 'origin/main' into stevenmasley/skip_prompt
Emyrk May 20, 2022
7cdb06e
Remove unused flag
Emyrk May 20, 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
Make test parallel
  • Loading branch information
@Emyrk
Emyrk committed May 19, 2022
commit 9c09a084761a0032fab977d40671e82cabc88f0e
3 changes: 2 additions & 1 deletion cli/cliui/prompt_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,7 @@ func TestPrompt(t *testing.T) {
})

t.Run("Skip", func(t *testing.T) {
t.Parallel()
ptty := ptytest.New(t)
var buf bytes.Buffer

Expand Down Expand Up @@ -89,7 +90,7 @@ func TestPrompt(t *testing.T) {
// Wait for the IO copy to finish
<-dataRead.Done()
// Timeout error means the output was hanging
require.ErrorIs(t, dataRead.Err(), context.Canceled, "should be cancelled")
require.ErrorIs(t, dataRead.Err(), context.Canceled, "should be canceled")
require.Len(t, buf.Bytes(), 0, "expect no output")
})
t.Run("JSON", func(t *testing.T) {
Expand Down