openstack
/
manila-tempest-plugin
Code Issues Proposed changes

Fix pep8 issues 

flake8 was updated to 3.8.0, and it
caught a couple of omissions including
E741 (ambiguous variable name) and
E225 (missing whitespace around operator).

Change-Id: I375e948684bb4a6b35297689f837056b0cb48fb1
Signed-off-by: Goutham Pacha Ravi <gouthampravi@gmail.com>
This commit is contained in:
Goutham Pacha Ravi 2020-05-11 16:55:39 -07:00
parent 7a8f5efb88
commit 5e0152b212
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manila_tempest_tests/tests/api/admin/test_quotas.py
View File

@ -118,7 +118,7 @@ class SharesAdminQuotasTest(base.BaseSharesAdminTest):
self.tenant_id, self.user_id,
version=PRE_SHARE_REPLICA_QUOTAS_MICROVERSION)
for key in('share_replicas', 'replica_gigabytes'):
for key in ('share_replicas', 'replica_gigabytes'):
self.assertNotIn(key, quotas)
@ddt.data(

6
manila_tempest_tests/tests/api/test_shares_actions.py
View File

@ -545,10 +545,10 @@ class SharesActionsTest(base.BaseSharesMixedTest):
@testtools.skipUnless(CONF.share.run_snapshot_tests,
"Snapshot tests are disabled.")
def test_list_snapshots_with_detail_use_limit(self):
for l, o in [('1', '1'), ('0', '1')]:
for limit, offset in [('1', '1'), ('0', '1')]:
filters = {
'limit': l,
'offset': o,
'limit': limit,
'offset': offset,
'share_id': self.shares[0]['id'],
}