Skip to content

Conversation

hmzzrcs
Copy link

@hmzzrcs hmzzrcs commented Jul 5, 2023

Allow user to send something by cmd.Stdin

hmzzrcs added 2 commits July 5, 2023 16:16
Allow user to send something by cmd.Stdin
@hashicorp-cla
Copy link

hashicorp-cla commented Jul 5, 2023

CLA assistant check
All committers have signed the CLA.

add test case for stdin
Copy link
Contributor

@hanzei hanzei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code style comments

Comment on lines 821 to +822
}
func TestClient_StdinBuffer(t *testing.T) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
}
func TestClient_StdinBuffer(t *testing.T) {
}
func TestClient_StdinBuffer(t *testing.T) {

Comment on lines +849 to 850
}
func TestClient_SecureConfig(t *testing.T) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
}
func TestClient_SecureConfig(t *testing.T) {
}
func TestClient_SecureConfig(t *testing.T) {

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants