[pep8] make test files and config files pep8 valid

Signed-off-by: akoskaaa <akos.hochrein@prezi.com>
This commit is contained in:
akoskaaa 2015-03-25 20:13:01 -07:00
parent 9aab7bc242
commit f9ea5ecf40
13 changed files with 66 additions and 66 deletions

View File

@ -39,14 +39,14 @@ ALLOWED_KEYS = DOCKER_CONFIG_KEYS + [
] ]
DOCKER_CONFIG_HINTS = { DOCKER_CONFIG_HINTS = {
'cpu_share' : 'cpu_shares', 'cpu_share': 'cpu_shares',
'link' : 'links', 'link': 'links',
'port' : 'ports', 'port': 'ports',
'privilege' : 'privileged', 'privilege': 'privileged',
'priviliged': 'privileged', 'priviliged': 'privileged',
'privilige' : 'privileged', 'privilige': 'privileged',
'volume' : 'volumes', 'volume': 'volumes',
'workdir' : 'working_dir', 'workdir': 'working_dir',
} }

View File

@ -35,6 +35,7 @@ install_requires = [
'six >= 1.3.0, < 2', 'six >= 1.3.0, < 2',
] ]
tests_require = [ tests_require = [
'mock >= 1.0.1', 'mock >= 1.0.1',
'nose', 'nose',
@ -54,7 +55,7 @@ setup(
url='https://www.docker.com/', url='https://www.docker.com/',
author='Docker, Inc.', author='Docker, Inc.',
license='Apache License 2.0', license='Apache License 2.0',
packages=find_packages(exclude=[ 'tests.*', 'tests' ]), packages=find_packages(exclude=['tests.*', 'tests']),
include_package_data=True, include_package_data=True,
test_suite='nose.collector', test_suite='nose.collector',
install_requires=install_requires, install_requires=install_requires,

View File

@ -1,7 +1,6 @@
import sys import sys
if sys.version_info >= (2,7): if sys.version_info >= (2, 7):
import unittest import unittest
else: else:
import unittest2 as unittest import unittest2 as unittest

View File

@ -70,7 +70,7 @@ class ProjectTest(DockerClientTestCase):
web = project.get_service('web') web = project.get_service('web')
net = project.get_service('net') net = project.get_service('net')
self.assertEqual(web._get_net(), 'container:'+net.containers()[0].id) self.assertEqual(web._get_net(), 'container:' + net.containers()[0].id)
project.kill() project.kill()
project.remove_stopped() project.remove_stopped()
@ -98,7 +98,7 @@ class ProjectTest(DockerClientTestCase):
project.up() project.up()
web = project.get_service('web') web = project.get_service('web')
self.assertEqual(web._get_net(), 'container:'+net_container.id) self.assertEqual(web._get_net(), 'container:' + net_container.id)
project.kill() project.kill()
project.remove_stopped() project.remove_stopped()
@ -266,7 +266,7 @@ class ProjectTest(DockerClientTestCase):
'image': 'busybox:latest', 'image': 'busybox:latest',
'command': ["/bin/sleep", "300"], 'command': ["/bin/sleep", "300"],
}, },
'data' : { 'data': {
'image': 'busybox:latest', 'image': 'busybox:latest',
'command': ["/bin/sleep", "300"] 'command': ["/bin/sleep", "300"]
}, },
@ -304,7 +304,7 @@ class ProjectTest(DockerClientTestCase):
'image': 'busybox:latest', 'image': 'busybox:latest',
'command': ["/bin/sleep", "300"], 'command': ["/bin/sleep", "300"],
}, },
'data' : { 'data': {
'image': 'busybox:latest', 'image': 'busybox:latest',
'command': ["/bin/sleep", "300"] 'command': ["/bin/sleep", "300"]
}, },

View File

@ -121,7 +121,7 @@ class ServiceTest(DockerClientTestCase):
# Match the last component ("host-path"), because boot2docker symlinks /tmp # Match the last component ("host-path"), because boot2docker symlinks /tmp
actual_host_path = volumes[container_path] actual_host_path = volumes[container_path]
self.assertTrue(path.basename(actual_host_path) == path.basename(host_path), self.assertTrue(path.basename(actual_host_path) == path.basename(host_path),
msg=("Last component differs: %s, %s" % (actual_host_path, host_path))) msg=("Last component differs: %s, %s" % (actual_host_path, host_path)))
def test_create_container_with_volumes_from(self): def test_create_container_with_volumes_from(self):
volume_service = self.create_service('data') volume_service = self.create_service('data')
@ -187,7 +187,6 @@ class ServiceTest(DockerClientTestCase):
service.recreate_containers() service.recreate_containers()
self.assertEqual(len(service.containers(stopped=True)), 1) self.assertEqual(len(service.containers(stopped=True)), 1)
def test_recreate_containers_with_image_declared_volume(self): def test_recreate_containers_with_image_declared_volume(self):
service = Service( service = Service(
project='composetest', project='composetest',
@ -229,8 +228,7 @@ class ServiceTest(DockerClientTestCase):
set([ set([
'composetest_db_1', 'db_1', 'composetest_db_1', 'db_1',
'composetest_db_2', 'db_2', 'composetest_db_2', 'db_2',
'db', 'db'])
]),
) )
def test_start_container_creates_links_with_names(self): def test_start_container_creates_links_with_names(self):
@ -246,8 +244,7 @@ class ServiceTest(DockerClientTestCase):
set([ set([
'composetest_db_1', 'db_1', 'composetest_db_1', 'db_1',
'composetest_db_2', 'db_2', 'composetest_db_2', 'db_2',
'custom_link_name', 'custom_link_name'])
]),
) )
def test_start_container_with_external_links(self): def test_start_container_with_external_links(self):
@ -291,8 +288,7 @@ class ServiceTest(DockerClientTestCase):
set([ set([
'composetest_db_1', 'db_1', 'composetest_db_1', 'db_1',
'composetest_db_2', 'db_2', 'composetest_db_2', 'db_2',
'db', 'db'])
]),
) )
def test_start_container_builds_images(self): def test_start_container_builds_images(self):
@ -331,7 +327,7 @@ class ServiceTest(DockerClientTestCase):
self.assertEqual(container['HostConfig']['Privileged'], False) self.assertEqual(container['HostConfig']['Privileged'], False)
def test_start_container_becomes_priviliged(self): def test_start_container_becomes_priviliged(self):
service = self.create_service('web', privileged = True) service = self.create_service('web', privileged=True)
container = create_and_start_container(service).inspect() container = create_and_start_container(service).inspect()
self.assertEqual(container['HostConfig']['Privileged'], True) self.assertEqual(container['HostConfig']['Privileged'], True)
@ -483,13 +479,13 @@ class ServiceTest(DockerClientTestCase):
def test_split_env(self): def test_split_env(self):
service = self.create_service('web', environment=['NORMAL=F1', 'CONTAINS_EQUALS=F=2', 'TRAILING_EQUALS=']) service = self.create_service('web', environment=['NORMAL=F1', 'CONTAINS_EQUALS=F=2', 'TRAILING_EQUALS='])
env = create_and_start_container(service).environment env = create_and_start_container(service).environment
for k,v in {'NORMAL': 'F1', 'CONTAINS_EQUALS': 'F=2', 'TRAILING_EQUALS': ''}.items(): for k, v in {'NORMAL': 'F1', 'CONTAINS_EQUALS': 'F=2', 'TRAILING_EQUALS': ''}.items():
self.assertEqual(env[k], v) self.assertEqual(env[k], v)
def test_env_from_file_combined_with_env(self): def test_env_from_file_combined_with_env(self):
service = self.create_service('web', environment=['ONE=1', 'TWO=2', 'THREE=3'], env_file=['tests/fixtures/env/one.env', 'tests/fixtures/env/two.env']) service = self.create_service('web', environment=['ONE=1', 'TWO=2', 'THREE=3'], env_file=['tests/fixtures/env/one.env', 'tests/fixtures/env/two.env'])
env = create_and_start_container(service).environment env = create_and_start_container(service).environment
for k,v in {'ONE': '1', 'TWO': '2', 'THREE': '3', 'FOO': 'baz', 'DOO': 'dah'}.items(): for k, v in {'ONE': '1', 'TWO': '2', 'THREE': '3', 'FOO': 'baz', 'DOO': 'dah'}.items():
self.assertEqual(env[k], v) self.assertEqual(env[k], v)
@mock.patch.dict(os.environ) @mock.patch.dict(os.environ)
@ -499,5 +495,5 @@ class ServiceTest(DockerClientTestCase):
os.environ['ENV_DEF'] = 'E3' os.environ['ENV_DEF'] = 'E3'
service = self.create_service('web', environment={'FILE_DEF': 'F1', 'FILE_DEF_EMPTY': '', 'ENV_DEF': None, 'NO_DEF': None}) service = self.create_service('web', environment={'FILE_DEF': 'F1', 'FILE_DEF_EMPTY': '', 'ENV_DEF': None, 'NO_DEF': None})
env = create_and_start_container(service).environment env = create_and_start_container(service).environment
for k,v in {'FILE_DEF': 'F1', 'FILE_DEF_EMPTY': '', 'ENV_DEF': 'E3', 'NO_DEF': ''}.items(): for k, v in {'FILE_DEF': 'F1', 'FILE_DEF_EMPTY': '', 'ENV_DEF': 'E3', 'NO_DEF': ''}.items():
self.assertEqual(env[k], v) self.assertEqual(env[k], v)

View File

@ -5,7 +5,7 @@ import os
import mock import mock
from tests import unittest from tests import unittest
from compose.cli import docker_client from compose.cli import docker_client
class DockerClientTestCase(unittest.TestCase): class DockerClientTestCase(unittest.TestCase):

View File

@ -151,4 +151,3 @@ def make_files(dirname, filenames):
for fname in filenames: for fname in filenames:
with open(os.path.join(dirname, fname), 'w') as f: with open(os.path.join(dirname, fname), 'w') as f:
f.write('') f.write('')

View File

@ -4,6 +4,7 @@ from .. import unittest
from compose import config from compose import config
class ConfigTest(unittest.TestCase): class ConfigTest(unittest.TestCase):
def test_from_dictionary(self): def test_from_dictionary(self):
service_dicts = config.from_dictionary({ service_dicts = config.from_dictionary({
@ -82,7 +83,7 @@ class MergeTest(unittest.TestCase):
class EnvTest(unittest.TestCase): class EnvTest(unittest.TestCase):
def test_parse_environment_as_list(self): def test_parse_environment_as_list(self):
environment =[ environment = [
'NORMAL=F1', 'NORMAL=F1',
'CONTAINS_EQUALS=F=2', 'CONTAINS_EQUALS=F=2',
'TRAILING_EQUALS=', 'TRAILING_EQUALS=',
@ -114,9 +115,8 @@ class EnvTest(unittest.TestCase):
os.environ['ENV_DEF'] = 'E3' os.environ['ENV_DEF'] = 'E3'
service_dict = config.make_service_dict( service_dict = config.make_service_dict(
'foo', 'foo', {
{ 'environment': {
'environment': {
'FILE_DEF': 'F1', 'FILE_DEF': 'F1',
'FILE_DEF_EMPTY': '', 'FILE_DEF_EMPTY': '',
'ENV_DEF': None, 'ENV_DEF': None,
@ -174,6 +174,7 @@ class EnvTest(unittest.TestCase):
{'FILE_DEF': 'F1', 'FILE_DEF_EMPTY': '', 'ENV_DEF': 'E3', 'NO_DEF': ''}, {'FILE_DEF': 'F1', 'FILE_DEF_EMPTY': '', 'ENV_DEF': 'E3', 'NO_DEF': ''},
) )
class ExtendsTest(unittest.TestCase): class ExtendsTest(unittest.TestCase):
def test_extends(self): def test_extends(self):
service_dicts = config.load('tests/fixtures/extends/docker-compose.yml') service_dicts = config.load('tests/fixtures/extends/docker-compose.yml')
@ -231,7 +232,6 @@ class ExtendsTest(unittest.TestCase):
], ],
) )
def test_extends_validation(self): def test_extends_validation(self):
dictionary = {'extends': None} dictionary = {'extends': None}
load_config = lambda: config.make_service_dict('myweb', dictionary, working_dir='tests/fixtures/extends') load_config = lambda: config.make_service_dict('myweb', dictionary, working_dir='tests/fixtures/extends')

View File

@ -9,7 +9,6 @@ from compose.container import Container
class ContainerTest(unittest.TestCase): class ContainerTest(unittest.TestCase):
def setUp(self): def setUp(self):
self.container_dict = { self.container_dict = {
"Id": "abc", "Id": "abc",
@ -30,11 +29,13 @@ class ContainerTest(unittest.TestCase):
container = Container.from_ps(None, container = Container.from_ps(None,
self.container_dict, self.container_dict,
has_been_inspected=True) has_been_inspected=True)
self.assertEqual(container.dictionary, { self.assertEqual(
"Id": "abc", container.dictionary,
"Image":"busybox:latest", {
"Name": "/composetest_db_1", "Id": "abc",
}) "Image": "busybox:latest",
"Name": "/composetest_db_1",
})
def test_from_ps_prefixed(self): def test_from_ps_prefixed(self):
self.container_dict['Names'] = ['/swarm-host-1' + n for n in self.container_dict['Names']] self.container_dict['Names'] = ['/swarm-host-1' + n for n in self.container_dict['Names']]
@ -44,7 +45,7 @@ class ContainerTest(unittest.TestCase):
has_been_inspected=True) has_been_inspected=True)
self.assertEqual(container.dictionary, { self.assertEqual(container.dictionary, {
"Id": "abc", "Id": "abc",
"Image":"busybox:latest", "Image": "busybox:latest",
"Name": "/composetest_db_1", "Name": "/composetest_db_1",
}) })
@ -100,7 +101,7 @@ class ContainerTest(unittest.TestCase):
def test_human_readable_ports_public_and_private(self): def test_human_readable_ports_public_and_private(self):
self.container_dict['NetworkSettings']['Ports'].update({ self.container_dict['NetworkSettings']['Ports'].update({
"45454/tcp": [ { "HostIp": "0.0.0.0", "HostPort": "49197" } ], "45454/tcp": [{"HostIp": "0.0.0.0", "HostPort": "49197"}],
"45453/tcp": [], "45453/tcp": [],
}) })
container = Container(None, self.container_dict, has_been_inspected=True) container = Container(None, self.container_dict, has_been_inspected=True)
@ -110,7 +111,7 @@ class ContainerTest(unittest.TestCase):
def test_get_local_port(self): def test_get_local_port(self):
self.container_dict['NetworkSettings']['Ports'].update({ self.container_dict['NetworkSettings']['Ports'].update({
"45454/tcp": [ { "HostIp": "0.0.0.0", "HostPort": "49197" } ], "45454/tcp": [{"HostIp": "0.0.0.0", "HostPort": "49197"}],
}) })
container = Container(None, self.container_dict, has_been_inspected=True) container = Container(None, self.container_dict, has_been_inspected=True)
@ -120,12 +121,12 @@ class ContainerTest(unittest.TestCase):
def test_get(self): def test_get(self):
container = Container(None, { container = Container(None, {
"Status":"Up 8 seconds", "Status": "Up 8 seconds",
"HostConfig": { "HostConfig": {
"VolumesFrom": ["volume_id",] "VolumesFrom": ["volume_id"]
}, },
}, has_been_inspected=True) }, has_been_inspected=True)
self.assertEqual(container.get('Status'), "Up 8 seconds") self.assertEqual(container.get('Status'), "Up 8 seconds")
self.assertEqual(container.get('HostConfig.VolumesFrom'), ["volume_id",]) self.assertEqual(container.get('HostConfig.VolumesFrom'), ["volume_id"])
self.assertEqual(container.get('Foo.Bar.DoesNotExist'), None) self.assertEqual(container.get('Foo.Bar.DoesNotExist'), None)

View File

@ -5,7 +5,7 @@ from tests import unittest
import mock import mock
from six import StringIO from six import StringIO
from compose import progress_stream from compose import progress_stream
class ProgressStreamTestCase(unittest.TestCase): class ProgressStreamTestCase(unittest.TestCase):

View File

@ -8,6 +8,7 @@ from compose import config
import mock import mock
import docker import docker
class ProjectTest(unittest.TestCase): class ProjectTest(unittest.TestCase):
def test_from_dict(self): def test_from_dict(self):
project = Project.from_dicts('composetest', [ project = Project.from_dicts('composetest', [
@ -211,7 +212,7 @@ class ProjectTest(unittest.TestCase):
} }
], mock_client) ], mock_client)
service = project.get_service('test') service = project.get_service('test')
self.assertEqual(service._get_net(), 'container:'+container_id) self.assertEqual(service._get_net(), 'container:' + container_id)
def test_use_net_from_service(self): def test_use_net_from_service(self):
container_name = 'test_aaa_1' container_name = 'test_aaa_1'
@ -237,4 +238,4 @@ class ProjectTest(unittest.TestCase):
], mock_client) ], mock_client)
service = project.get_service('test') service = project.get_service('test')
self.assertEqual(service._get_net(), 'container:'+container_name) self.assertEqual(service._get_net(), 'container:' + container_name)

View File

@ -146,16 +146,16 @@ class ServiceTest(unittest.TestCase):
def test_build_port_bindings_with_one_port(self): def test_build_port_bindings_with_one_port(self):
port_bindings = build_port_bindings(["127.0.0.1:1000:1000"]) port_bindings = build_port_bindings(["127.0.0.1:1000:1000"])
self.assertEqual(port_bindings["1000"],[("127.0.0.1","1000")]) self.assertEqual(port_bindings["1000"], [("127.0.0.1", "1000")])
def test_build_port_bindings_with_matching_internal_ports(self): def test_build_port_bindings_with_matching_internal_ports(self):
port_bindings = build_port_bindings(["127.0.0.1:1000:1000","127.0.0.1:2000:1000"]) port_bindings = build_port_bindings(["127.0.0.1:1000:1000", "127.0.0.1:2000:1000"])
self.assertEqual(port_bindings["1000"],[("127.0.0.1","1000"),("127.0.0.1","2000")]) self.assertEqual(port_bindings["1000"], [("127.0.0.1", "1000"), ("127.0.0.1", "2000")])
def test_build_port_bindings_with_nonmatching_internal_ports(self): def test_build_port_bindings_with_nonmatching_internal_ports(self):
port_bindings = build_port_bindings(["127.0.0.1:1000:1000","127.0.0.1:2000:2000"]) port_bindings = build_port_bindings(["127.0.0.1:1000:1000", "127.0.0.1:2000:2000"])
self.assertEqual(port_bindings["1000"],[("127.0.0.1","1000")]) self.assertEqual(port_bindings["1000"], [("127.0.0.1", "1000")])
self.assertEqual(port_bindings["2000"],[("127.0.0.1","2000")]) self.assertEqual(port_bindings["2000"], [("127.0.0.1", "2000")])
def test_split_domainname_none(self): def test_split_domainname_none(self):
service = Service('foo', hostname='name', client=self.mock_client) service = Service('foo', hostname='name', client=self.mock_client)
@ -165,29 +165,32 @@ class ServiceTest(unittest.TestCase):
self.assertFalse('domainname' in opts, 'domainname') self.assertFalse('domainname' in opts, 'domainname')
def test_split_domainname_fqdn(self): def test_split_domainname_fqdn(self):
service = Service('foo', service = Service(
hostname='name.domain.tld', 'foo',
client=self.mock_client) hostname='name.domain.tld',
client=self.mock_client)
self.mock_client.containers.return_value = [] self.mock_client.containers.return_value = []
opts = service._get_container_create_options({'image': 'foo'}) opts = service._get_container_create_options({'image': 'foo'})
self.assertEqual(opts['hostname'], 'name', 'hostname') self.assertEqual(opts['hostname'], 'name', 'hostname')
self.assertEqual(opts['domainname'], 'domain.tld', 'domainname') self.assertEqual(opts['domainname'], 'domain.tld', 'domainname')
def test_split_domainname_both(self): def test_split_domainname_both(self):
service = Service('foo', service = Service(
hostname='name', 'foo',
domainname='domain.tld', hostname='name',
client=self.mock_client) domainname='domain.tld',
client=self.mock_client)
self.mock_client.containers.return_value = [] self.mock_client.containers.return_value = []
opts = service._get_container_create_options({'image': 'foo'}) opts = service._get_container_create_options({'image': 'foo'})
self.assertEqual(opts['hostname'], 'name', 'hostname') self.assertEqual(opts['hostname'], 'name', 'hostname')
self.assertEqual(opts['domainname'], 'domain.tld', 'domainname') self.assertEqual(opts['domainname'], 'domain.tld', 'domainname')
def test_split_domainname_weird(self): def test_split_domainname_weird(self):
service = Service('foo', service = Service(
hostname='name.sub', 'foo',
domainname='domain.tld', hostname='name.sub',
client=self.mock_client) domainname='domain.tld',
client=self.mock_client)
self.mock_client.containers.return_value = [] self.mock_client.containers.return_value = []
opts = service._get_container_create_options({'image': 'foo'}) opts = service._get_container_create_options({'image': 'foo'})
self.assertEqual(opts['hostname'], 'name.sub', 'hostname') self.assertEqual(opts['hostname'], 'name.sub', 'hostname')
@ -315,4 +318,3 @@ class ServiceVolumesTest(unittest.TestCase):
self.assertEqual( self.assertEqual(
binding, binding,
('/home/user', dict(bind='/home/user', ro=False))) ('/home/user', dict(bind='/home/user', ro=False)))

View File

@ -3,6 +3,7 @@ from __future__ import absolute_import
from compose.cli.utils import split_buffer from compose.cli.utils import split_buffer
from .. import unittest from .. import unittest
class SplitBufferTest(unittest.TestCase): class SplitBufferTest(unittest.TestCase):
def test_single_line_chunks(self): def test_single_line_chunks(self):
def reader(): def reader():