Skip to content

Conversation

@lifubang
Copy link
Contributor

Signed-off-by: Lifubang lifubang@acmcoder.com

- What I did
As mentioned in #1482 and #1476 ,
add e2e case for ssh with password.

- How I did it
Add a mock docker cli to deal with ssh password input

- Description for the changelog
add e2e case for ssh with password

- A picture of a cute animal (not mandatory but encouraged)
e 67 tt1 dx go79pqk 2dk

@codecov-io
Copy link

Codecov Report

Merging #1487 into master will decrease coverage by 0.95%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1487      +/-   ##
==========================================
- Coverage   55.14%   54.19%   -0.96%     
==========================================
  Files         289      289              
  Lines       19371    19377       +6     
==========================================
- Hits        10683    10502     -181     
- Misses       7997     8199     +202     
+ Partials      691      676      -15

@lifubang lifubang force-pushed the runusesshtest branch 16 times, most recently from 7afffda to 0b9f3dd Compare October 28, 2018 09:45
Signed-off-by: Lifubang <lifubang@acmcoder.com>

add a mock cli to handle ssh password input

Signed-off-by: Lifubang <lifubang@acmcoder.com>
@AkihiroSuda
Copy link
Collaborator

What's current status?

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants