Skip to content

[WIP] add e2e case for ssh with password #1487

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.

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

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