airship
/
armada
Code Issues Proposed changes

Fix pep8 errors 

This addresses the pep8 errors that are causing gate failures

Change-Id: Id92dbbf527af1953026f17ddb3f2d79f0a635284
This commit is contained in:
Michael Beaver 2019-02-14 20:52:38 -06:00
parent aa90e5c17e
commit 7f26bbcd59
3 changed files with 14 additions and 22 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
armada/api/controller/versions.py
View File

@ -27,10 +27,9 @@ class Versions(api.BaseResource):
def on_get(self, req, resp): def on_get(self, req, resp):
resp.status = falcon.HTTP_200 resp.status = falcon.HTTP_200
resp.body = json.dumps({ resp.body = json.dumps(
'v1.0': { {'v1.0': {
'path': '/api/v1.0', 'path': '/api/v1.0',
'status': 'stable' 'status': 'stable'
} }})
})
resp.content_type = 'application/json' resp.content_type = 'application/json'

6
armada/tests/unit/handlers/test_armada.py
View File

@ -356,8 +356,10 @@ class ArmadaHandlerTestCase(base.ArmadaTestCase):
if test_failure_to_run: if test_failure_to_run:
def fail(tiller, release, timeout=None, cleanup=False): def fail(tiller, release, timeout=None, cleanup=False):
status = AttrDict( status = AttrDict(**{
**{'info': AttrDict(**{'Description': 'Failed'})}) 'info':
AttrDict(**{'Description': 'Failed'})
})
raise tiller_exceptions.ReleaseException( raise tiller_exceptions.ReleaseException(
release, status, 'Test') release, status, 'Test')

23
armada/tests/unit/handlers/test_wait.py
View File

@ -170,15 +170,14 @@ class ChartWaitTestCase(base.ArmadaTestCase):
get_resource_wait.side_effect = return_mock get_resource_wait.side_effect = return_mock
unit = self.get_unit({ unit = self.get_unit(
'wait': { {'wait': {
'resources': [{ 'resources': [{
'type': 'foo' 'type': 'foo'
}, { }, {
'type': 'bar' 'type': 'bar'
}] }]
} }})
})
unit.wait(10) unit.wait(10)
@ -205,20 +204,12 @@ class PodWaitTestCase(base.ArmadaTestCase):
'key': 'value', 'key': 'value',
'helm.sh/hook': 'test-success' 'helm.sh/hook': 'test-success'
}), }),
mock_resource({ mock_resource({'helm.sh/hook': 'test-failure'}),
'helm.sh/hook': 'test-failure' mock_resource({'helm.sh/hook': 'test-success,pre-install'})
}),
mock_resource({
'helm.sh/hook': 'test-success,pre-install'
})
] ]
non_test_resources = [ non_test_resources = [
mock_resource({ mock_resource({'helm.sh/hook': 'pre-install'}),
'helm.sh/hook': 'pre-install' mock_resource({'key': 'value'}),
}),
mock_resource({
'key': 'value'
}),
mock_resource({}) mock_resource({})
] ]