diff options
Diffstat (limited to 'tests/integration')
-rw-r--r-- | tests/integration/api_build_test.py | 16 | ||||
-rw-r--r-- | tests/integration/api_container_test.py | 46 | ||||
-rw-r--r-- | tests/integration/api_network_test.py | 9 | ||||
-rw-r--r-- | tests/integration/api_volume_test.py | 1 |
4 files changed, 19 insertions, 53 deletions
diff --git a/tests/integration/api_build_test.py b/tests/integration/api_build_test.py index ee9b68a..4c2b992 100644 --- a/tests/integration/api_build_test.py +++ b/tests/integration/api_build_test.py @@ -21,7 +21,7 @@ class BuildTest(BaseAPIIntegrationTest): 'ADD https://dl.dropboxusercontent.com/u/20637798/silence.tar.gz' ' /tmp/silence.tar.gz' ]).encode('ascii')) - stream = self.client.build(fileobj=script, stream=True, decode=True) + stream = self.client.build(fileobj=script, decode=True) logs = [] for chunk in stream: logs.append(chunk) @@ -37,7 +37,7 @@ class BuildTest(BaseAPIIntegrationTest): 'ADD https://dl.dropboxusercontent.com/u/20637798/silence.tar.gz' ' /tmp/silence.tar.gz' ])) - stream = self.client.build(fileobj=script, stream=True) + stream = self.client.build(fileobj=script) logs = '' for chunk in stream: if six.PY3: @@ -45,7 +45,6 @@ class BuildTest(BaseAPIIntegrationTest): logs += chunk assert logs != '' - @requires_api_version('1.8') def test_build_with_dockerignore(self): base_dir = tempfile.mkdtemp() self.addCleanup(shutil.rmtree, base_dir) @@ -97,7 +96,6 @@ class BuildTest(BaseAPIIntegrationTest): '/test/not-ignored' ]) - @requires_api_version('1.21') def test_build_with_buildargs(self): script = io.BytesIO('\n'.join([ 'FROM scratch', @@ -320,7 +318,7 @@ class BuildTest(BaseAPIIntegrationTest): ])) stream = self.client.build( - fileobj=script, stream=True, decode=True, nocache=True + fileobj=script, decode=True, nocache=True ) lines = [] for chunk in stream: @@ -341,7 +339,7 @@ class BuildTest(BaseAPIIntegrationTest): ])) stream = self.client.build( - path=base_dir, stream=True, decode=True, nocache=True, + path=base_dir, decode=True, nocache=True, gzip=True ) @@ -365,7 +363,7 @@ class BuildTest(BaseAPIIntegrationTest): ])) stream = self.client.build( - path=base_dir, stream=True, decode=True, nocache=True + path=base_dir, decode=True, nocache=True ) lines = [] @@ -383,9 +381,7 @@ class BuildTest(BaseAPIIntegrationTest): script = io.BytesIO('FROM busybox\n'.encode('ascii')) with pytest.raises(errors.APIError) as excinfo: - stream = self.client.build( - fileobj=script, stream=True, platform='foobar' - ) + stream = self.client.build(fileobj=script, platform='foobar') for _ in stream: pass diff --git a/tests/integration/api_container_test.py b/tests/integration/api_container_test.py index e5d7943..0925352 100644 --- a/tests/integration/api_container_test.py +++ b/tests/integration/api_container_test.py @@ -145,22 +145,18 @@ class CreateContainerTest(BaseAPIIntegrationTest): container2_id = res1['Id'] self.tmp_containers.append(container2_id) self.client.start(container2_id) - with pytest.raises(docker.errors.DockerException): - self.client.create_container( - BUSYBOX, 'cat', detach=True, stdin_open=True, - volumes_from=vol_names - ) - res2 = self.client.create_container( + + res = self.client.create_container( BUSYBOX, 'cat', detach=True, stdin_open=True, host_config=self.client.create_host_config( volumes_from=vol_names, network_mode='none' ) ) - container3_id = res2['Id'] + container3_id = res['Id'] self.tmp_containers.append(container3_id) self.client.start(container3_id) - info = self.client.inspect_container(res2['Id']) + info = self.client.inspect_container(res['Id']) assert len(info['HostConfig']['VolumesFrom']) == len(vol_names) def create_container_readonly_fs(self): @@ -222,7 +218,6 @@ class CreateContainerTest(BaseAPIIntegrationTest): self.client.kill(id) - @requires_api_version('1.20') def test_group_id_ints(self): container = self.client.create_container( BUSYBOX, 'id -G', @@ -239,7 +234,6 @@ class CreateContainerTest(BaseAPIIntegrationTest): assert '1000' in groups assert '1001' in groups - @requires_api_version('1.20') def test_group_id_strings(self): container = self.client.create_container( BUSYBOX, 'id -G', host_config=self.client.create_host_config( @@ -604,24 +598,15 @@ class VolumeBindTest(BaseAPIIntegrationTest): assert mount_data['RW'] is True def check_container_data(self, inspect_data, rw): - if docker.utils.compare_version('1.20', self.client._version) < 0: - assert 'Volumes' in inspect_data - assert self.mount_dest in inspect_data['Volumes'] - assert ( - self.mount_origin == inspect_data['Volumes'][self.mount_dest] - ) - assert self.mount_dest in inspect_data['VolumesRW'] - assert not inspect_data['VolumesRW'][self.mount_dest] - else: - assert 'Mounts' in inspect_data - filtered = list(filter( - lambda x: x['Destination'] == self.mount_dest, - inspect_data['Mounts'] - )) - assert len(filtered) == 1 - mount_data = filtered[0] - assert mount_data['Source'] == self.mount_origin - assert mount_data['RW'] == rw + assert 'Mounts' in inspect_data + filtered = list(filter( + lambda x: x['Destination'] == self.mount_dest, + inspect_data['Mounts'] + )) + assert len(filtered) == 1 + mount_data = filtered[0] + assert mount_data['Source'] == self.mount_origin + assert mount_data['RW'] == rw def run_with_volume(self, ro, *args, **kwargs): return self.run_container( @@ -640,7 +625,6 @@ class VolumeBindTest(BaseAPIIntegrationTest): ) -@requires_api_version('1.20') class ArchiveTest(BaseAPIIntegrationTest): def test_get_file_archive_from_container(self): data = 'The Maid and the Pocket Watch of Blood' @@ -1323,7 +1307,6 @@ class PruneTest(BaseAPIIntegrationTest): class GetContainerStatsTest(BaseAPIIntegrationTest): - @requires_api_version('1.19') def test_get_container_stats_no_stream(self): container = self.client.create_container( BUSYBOX, ['sleep', '60'], @@ -1338,7 +1321,6 @@ class GetContainerStatsTest(BaseAPIIntegrationTest): 'memory_stats', 'blkio_stats']: assert key in response - @requires_api_version('1.17') def test_get_container_stats_stream(self): container = self.client.create_container( BUSYBOX, ['sleep', '60'], @@ -1401,7 +1383,6 @@ class ContainerUpdateTest(BaseAPIIntegrationTest): class ContainerCPUTest(BaseAPIIntegrationTest): - @requires_api_version('1.18') def test_container_cpu_shares(self): cpu_shares = 512 container = self.client.create_container( @@ -1414,7 +1395,6 @@ class ContainerCPUTest(BaseAPIIntegrationTest): inspect_data = self.client.inspect_container(container) assert inspect_data['HostConfig']['CpuShares'] == 512 - @requires_api_version('1.18') def test_container_cpuset(self): cpuset_cpus = "0,1" container = self.client.create_container( diff --git a/tests/integration/api_network_test.py b/tests/integration/api_network_test.py index ec92bd7..b6726d0 100644 --- a/tests/integration/api_network_test.py +++ b/tests/integration/api_network_test.py @@ -17,7 +17,6 @@ class TestNetworks(BaseAPIIntegrationTest): self.tmp_networks.append(net_id) return (net_name, net_id) - @requires_api_version('1.21') def test_list_networks(self): networks = self.client.networks() @@ -32,7 +31,6 @@ class TestNetworks(BaseAPIIntegrationTest): networks_by_partial_id = self.client.networks(ids=[net_id[:8]]) assert [n['Id'] for n in networks_by_partial_id] == [net_id] - @requires_api_version('1.21') def test_inspect_network(self): net_name, net_id = self.create_network() @@ -43,7 +41,6 @@ class TestNetworks(BaseAPIIntegrationTest): assert net['Scope'] == 'local' assert net['IPAM']['Driver'] == 'default' - @requires_api_version('1.21') def test_create_network_with_ipam_config(self): _, net_id = self.create_network( ipam=IPAMConfig( @@ -81,12 +78,10 @@ class TestNetworks(BaseAPIIntegrationTest): }, }] - @requires_api_version('1.21') def test_create_network_with_host_driver_fails(self): with pytest.raises(docker.errors.APIError): self.client.create_network(random_name(), driver='host') - @requires_api_version('1.21') def test_remove_network(self): net_name, net_id = self.create_network() assert net_name in [n['Name'] for n in self.client.networks()] @@ -94,7 +89,6 @@ class TestNetworks(BaseAPIIntegrationTest): self.client.remove_network(net_id) assert net_name not in [n['Name'] for n in self.client.networks()] - @requires_api_version('1.21') def test_connect_and_disconnect_container(self): net_name, net_id = self.create_network() @@ -163,7 +157,6 @@ class TestNetworks(BaseAPIIntegrationTest): assert 'foo' in aliases assert 'bar' in aliases - @requires_api_version('1.21') def test_connect_on_container_create(self): net_name, net_id = self.create_network() @@ -309,7 +302,6 @@ class TestNetworks(BaseAPIIntegrationTest): self.execute(container, ['nslookup', 'bar']) - @requires_api_version('1.21') def test_create_check_duplicate(self): net_name, net_id = self.create_network() with pytest.raises(docker.errors.APIError): @@ -475,7 +467,6 @@ class TestNetworks(BaseAPIIntegrationTest): with pytest.raises(docker.errors.NotFound): self.client.inspect_network(net_name_swarm, scope='local') - @requires_api_version('1.21') def test_create_remove_network_with_space_in_name(self): net_id = self.client.create_network('test 01') self.tmp_networks.append(net_id) diff --git a/tests/integration/api_volume_test.py b/tests/integration/api_volume_test.py index e09f12a..8e7dd3a 100644 --- a/tests/integration/api_volume_test.py +++ b/tests/integration/api_volume_test.py @@ -5,7 +5,6 @@ from ..helpers import requires_api_version from .base import BaseAPIIntegrationTest -@requires_api_version('1.21') class TestVolumes(BaseAPIIntegrationTest): def test_create_volume(self): name = 'perfectcherryblossom' |