Skip to content
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

Jump to bottom

Improve e2e test coverage #9149

Closed
gyuho opened this issue Jan 16, 2018 · 4 comments
Closed

Improve e2e test coverage #9149

gyuho opened this issue Jan 16, 2018 · 4 comments
Assignees
@hexfusion
Labels
area/testing

Comments

@gyuho
Copy link
Contributor

gyuho commented Jan 16, 2018 •
edited
Loading

While etcd integration and clientv3/integration test its functionalities to ensure its correctness, e2e verifies etcdctl output on each command with various user inputs.

Most of the commands are already covered, but not all possible cases (e.g., endpoint health with auth enabled, TLS enabled).

Improve e2e test coverage by:

  1. check what's missing/tested in e2e
  2. add more tests to cover those missing cases
@gyuho
Copy link
Contributor Author

gyuho commented Jan 17, 2018

@hexfusion Do you want to work on this?

@hexfusion
Copy link
Contributor

@gyuho I do.

@hexfusion hexfusion self-assigned this Jan 17, 2018
@hexfusion
Copy link
Contributor

started a doc to try and flush out coverage. Anyone can feel free to update.

This was referenced Jan 20, 2018
Closed
Merged
@gyuho
Copy link
Contributor Author

gyuho commented Jan 22, 2018

Partially related to #9150.

@hexfusion hexfusion mentioned this issue Jan 24, 2018
Closed
This was referenced Feb 9, 2018
Merged
Merged
@hexfusion hexfusion mentioned this issue Mar 10, 2018
Closed
@gyuho gyuho closed this as completed Sep 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hexfusion hexfusion

Labels
area/testing
Milestone
No milestone
Development

No branches or pull requests
3 participants
@hexfusion @xiang90 @gyuho