diff options
| author | Joe Gordon <joe.gordon0@gmail.com> | 2013-05-28 06:52:55 +0300 |
|---|---|---|
| committer | Joe Gordon <joe.gordon0@gmail.com> | 2013-05-28 23:04:35 +0200 |
| commit | d9c145c7b86f7cc8807ec4c63fd2cbdad17794ab (patch) | |
| tree | ceb25f6cb5bc4aaa508900d8d65394f0e4ddd03c /nova/tests | |
| parent | a3e6133628310b0f41fad7271b048e30c0401e62 (diff) | |
Fix and enable flake8 F812
F812 list comprehension redefines
List comprehension don't have scope so redefining variables can lead to
unexpected behaviors
Change-Id: I85484dfcd4fa391195ea7be75aadc7897da4739c
Diffstat (limited to 'nova/tests')
| -rw-r--r-- | nova/tests/integrated/test_servers.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/nova/tests/integrated/test_servers.py b/nova/tests/integrated/test_servers.py index b45bd88f7..694577fa0 100644 --- a/nova/tests/integrated/test_servers.py +++ b/nova/tests/integrated/test_servers.py @@ -126,7 +126,7 @@ class ServersTest(integrated_helpers._IntegratedTestBase): # It should also be in the all-servers list servers = self.api.get_servers() - server_ids = [server['id'] for server in servers] + server_ids = [s['id'] for s in servers] self.assertTrue(created_server_id in server_ids) found_server = self._wait_for_state_change(found_server, 'BUILD') |