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.

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

services endpoints were started #464

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

adastreamer
Copy link

Hi guys!
Just want to add a few important endpoints which are available in docker-engine since the version 1.12
Let me know any thoughts.

Also I have one question: why all the tests are so massive? They're running too long and I can't realize why even the docker is required! So, this app is just a simple REST API client, it should not testing the docker functionality! Imho strictly should not! Why not to write simple mock-objects of supposed responses? In this case it's possible to track the Docker API versions. Apologize if I'm missing something, just trying to understand what's going on.

@hammady
Copy link

hammady commented Oct 29, 2017

@adastreamer thanks for that, any plans to write the tests?

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.

2 participants