mirror of https://github.com/docker/docker-py.git
Rename DockerClientTest to BaseAPIClientTest
Signed-off-by: Ben Firshman <ben@firshman.co.uk>
This commit is contained in:
parent
dac7174ff2
commit
e055a1c813
|
@ -4,10 +4,10 @@ import io
|
||||||
import docker
|
import docker
|
||||||
from docker import auth
|
from docker import auth
|
||||||
|
|
||||||
from .api_test import DockerClientTest, fake_request, url_prefix
|
from .api_test import BaseAPIClientTest, fake_request, url_prefix
|
||||||
|
|
||||||
|
|
||||||
class BuildTest(DockerClientTest):
|
class BuildTest(BaseAPIClientTest):
|
||||||
def test_build_container(self):
|
def test_build_container(self):
|
||||||
script = io.BytesIO('\n'.join([
|
script = io.BytesIO('\n'.join([
|
||||||
'FROM busybox',
|
'FROM busybox',
|
||||||
|
|
|
@ -11,7 +11,7 @@ import six
|
||||||
from . import fake_api
|
from . import fake_api
|
||||||
from ..helpers import requires_api_version
|
from ..helpers import requires_api_version
|
||||||
from .api_test import (
|
from .api_test import (
|
||||||
DockerClientTest, url_prefix, fake_request, DEFAULT_TIMEOUT_SECONDS,
|
BaseAPIClientTest, url_prefix, fake_request, DEFAULT_TIMEOUT_SECONDS,
|
||||||
fake_inspect_container
|
fake_inspect_container
|
||||||
)
|
)
|
||||||
|
|
||||||
|
@ -25,7 +25,7 @@ def fake_inspect_container_tty(self, container):
|
||||||
return fake_inspect_container(self, container, tty=True)
|
return fake_inspect_container(self, container, tty=True)
|
||||||
|
|
||||||
|
|
||||||
class StartContainerTest(DockerClientTest):
|
class StartContainerTest(BaseAPIClientTest):
|
||||||
def test_start_container(self):
|
def test_start_container(self):
|
||||||
self.client.start(fake_api.FAKE_CONTAINER_ID)
|
self.client.start(fake_api.FAKE_CONTAINER_ID)
|
||||||
|
|
||||||
|
@ -168,7 +168,7 @@ class StartContainerTest(DockerClientTest):
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
class CreateContainerTest(DockerClientTest):
|
class CreateContainerTest(BaseAPIClientTest):
|
||||||
def test_create_container(self):
|
def test_create_container(self):
|
||||||
self.client.create_container('busybox', 'true')
|
self.client.create_container('busybox', 'true')
|
||||||
|
|
||||||
|
@ -1180,7 +1180,7 @@ class CreateContainerTest(DockerClientTest):
|
||||||
self.assertEqual(json.loads(args[1]['data'])['Env'], expected)
|
self.assertEqual(json.loads(args[1]['data'])['Env'], expected)
|
||||||
|
|
||||||
|
|
||||||
class ContainerTest(DockerClientTest):
|
class ContainerTest(BaseAPIClientTest):
|
||||||
def test_list_containers(self):
|
def test_list_containers(self):
|
||||||
self.client.containers(all=True)
|
self.client.containers(all=True)
|
||||||
|
|
||||||
|
|
|
@ -2,11 +2,11 @@ import json
|
||||||
|
|
||||||
from . import fake_api
|
from . import fake_api
|
||||||
from .api_test import (
|
from .api_test import (
|
||||||
DockerClientTest, url_prefix, fake_request, DEFAULT_TIMEOUT_SECONDS,
|
BaseAPIClientTest, url_prefix, fake_request, DEFAULT_TIMEOUT_SECONDS,
|
||||||
)
|
)
|
||||||
|
|
||||||
|
|
||||||
class ExecTest(DockerClientTest):
|
class ExecTest(BaseAPIClientTest):
|
||||||
def test_exec_create(self):
|
def test_exec_create(self):
|
||||||
self.client.exec_create(fake_api.FAKE_CONTAINER_ID, ['ls', '-1'])
|
self.client.exec_create(fake_api.FAKE_CONTAINER_ID, ['ls', '-1'])
|
||||||
|
|
||||||
|
|
|
@ -4,7 +4,7 @@ import pytest
|
||||||
from . import fake_api
|
from . import fake_api
|
||||||
from docker import auth
|
from docker import auth
|
||||||
from .api_test import (
|
from .api_test import (
|
||||||
DockerClientTest, fake_request, DEFAULT_TIMEOUT_SECONDS, url_prefix,
|
BaseAPIClientTest, fake_request, DEFAULT_TIMEOUT_SECONDS, url_prefix,
|
||||||
fake_resolve_authconfig
|
fake_resolve_authconfig
|
||||||
)
|
)
|
||||||
|
|
||||||
|
@ -14,7 +14,7 @@ except ImportError:
|
||||||
import mock
|
import mock
|
||||||
|
|
||||||
|
|
||||||
class ImageTest(DockerClientTest):
|
class ImageTest(BaseAPIClientTest):
|
||||||
def test_image_viz(self):
|
def test_image_viz(self):
|
||||||
with pytest.raises(Exception):
|
with pytest.raises(Exception):
|
||||||
self.client.images('busybox', viz=True)
|
self.client.images('busybox', viz=True)
|
||||||
|
|
|
@ -2,7 +2,7 @@ import json
|
||||||
|
|
||||||
import six
|
import six
|
||||||
|
|
||||||
from .api_test import DockerClientTest, url_prefix, response
|
from .api_test import BaseAPIClientTest, url_prefix, response
|
||||||
from ..helpers import requires_api_version
|
from ..helpers import requires_api_version
|
||||||
from docker.utils import create_ipam_config, create_ipam_pool
|
from docker.utils import create_ipam_config, create_ipam_pool
|
||||||
|
|
||||||
|
@ -12,7 +12,7 @@ except ImportError:
|
||||||
import mock
|
import mock
|
||||||
|
|
||||||
|
|
||||||
class NetworkTest(DockerClientTest):
|
class NetworkTest(BaseAPIClientTest):
|
||||||
@requires_api_version('1.21')
|
@requires_api_version('1.21')
|
||||||
def test_list_networks(self):
|
def test_list_networks(self):
|
||||||
networks = [
|
networks = [
|
||||||
|
|
|
@ -93,7 +93,7 @@ url_prefix = '{0}v{1}/'.format(
|
||||||
docker.constants.DEFAULT_DOCKER_API_VERSION)
|
docker.constants.DEFAULT_DOCKER_API_VERSION)
|
||||||
|
|
||||||
|
|
||||||
class DockerClientTest(unittest.TestCase):
|
class BaseAPIClientTest(unittest.TestCase):
|
||||||
def setUp(self):
|
def setUp(self):
|
||||||
self.patcher = mock.patch.multiple(
|
self.patcher = mock.patch.multiple(
|
||||||
'docker.api.client.APIClient',
|
'docker.api.client.APIClient',
|
||||||
|
@ -123,7 +123,7 @@ class DockerClientTest(unittest.TestCase):
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
||||||
class DockerApiTest(DockerClientTest):
|
class DockerApiTest(BaseAPIClientTest):
|
||||||
def test_ctor(self):
|
def test_ctor(self):
|
||||||
with pytest.raises(docker.errors.DockerException) as excinfo:
|
with pytest.raises(docker.errors.DockerException) as excinfo:
|
||||||
APIClient(version=1.12)
|
APIClient(version=1.12)
|
||||||
|
|
|
@ -3,10 +3,10 @@ import json
|
||||||
import pytest
|
import pytest
|
||||||
|
|
||||||
from ..helpers import requires_api_version
|
from ..helpers import requires_api_version
|
||||||
from .api_test import DockerClientTest, url_prefix, fake_request
|
from .api_test import BaseAPIClientTest, url_prefix, fake_request
|
||||||
|
|
||||||
|
|
||||||
class VolumeTest(DockerClientTest):
|
class VolumeTest(BaseAPIClientTest):
|
||||||
@requires_api_version('1.21')
|
@requires_api_version('1.21')
|
||||||
def test_list_volumes(self):
|
def test_list_volumes(self):
|
||||||
volumes = self.client.volumes()
|
volumes = self.client.volumes()
|
||||||
|
|
|
@ -4,10 +4,10 @@ import json
|
||||||
|
|
||||||
from . import fake_api
|
from . import fake_api
|
||||||
from ..helpers import requires_api_version
|
from ..helpers import requires_api_version
|
||||||
from .api_test import (DockerClientTest, url_prefix, fake_request)
|
from .api_test import BaseAPIClientTest, url_prefix, fake_request
|
||||||
|
|
||||||
|
|
||||||
class SwarmTest(DockerClientTest):
|
class SwarmTest(BaseAPIClientTest):
|
||||||
@requires_api_version('1.24')
|
@requires_api_version('1.24')
|
||||||
def test_node_update(self):
|
def test_node_update(self):
|
||||||
node_spec = {
|
node_spec = {
|
||||||
|
|
Loading…
Reference in New Issue