openstack
/
manila-tempest-plugin
Code Issues Proposed changes

Remove the max api microversion assertion 

max api microversion assertion should be removed here, because
microversion change patches for manila need to trigger this
plugging, but this plugging still test old version microversion
tests, this will cause manila-tempest-minimal-dsvm-dummy job
never passed.

Change-Id: Ib691a0925e29f90bf6d5b0a22fa8b03c9fe176cf
This commit is contained in:
junboli 2017-12-31 15:14:04 +08:00
parent a6a639fcf9
commit be1134c700
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
manila_tempest_tests/tests/api/test_microversions.py
View File

@ -53,7 +53,7 @@ class MicroversionsTest(base.BaseSharesTest):
v2 = [v for v in version_list if v['id'] == 'v2.0'][0]
self.assertEqual(_MIN_API_VERSION, v2.get('min_version'))
self.assertEqual(_MAX_API_VERSION, v2.get('version'))
self.assertNotIn(v2.get('version'), [None, ''])
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_microversions_v1_no_version(self):
@ -105,7 +105,7 @@ class MicroversionsTest(base.BaseSharesTest):
resp.get(API_MICROVERSIONS_HEADER_LOWER))
self.assertEqual(API_MICROVERSIONS_HEADER, resp.get('vary'))
self.assertEqual(_MIN_API_VERSION, version_list[0].get('min_version'))
self.assertEqual(_MAX_API_VERSION, version_list[0].get('version'))
self.assertNotIn(version_list[0].get('version'), [None, ''])
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_microversions_v2_min_version(self):
@ -123,7 +123,7 @@ class MicroversionsTest(base.BaseSharesTest):
resp.get(API_MICROVERSIONS_HEADER_LOWER))
self.assertEqual(API_MICROVERSIONS_HEADER, resp.get('vary'))
self.assertEqual(_MIN_API_VERSION, version_list[0].get('min_version'))
self.assertEqual(_MAX_API_VERSION, version_list[0].get('version'))
self.assertNotIn(version_list[0].get('version'), [None, ''])
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_microversions_v2_max_version(self):
@ -141,7 +141,7 @@ class MicroversionsTest(base.BaseSharesTest):
resp.get(API_MICROVERSIONS_HEADER_LOWER))
self.assertEqual(API_MICROVERSIONS_HEADER, resp.get('vary'))
self.assertEqual(_MIN_API_VERSION, version_list[0].get('min_version'))
self.assertEqual(_MAX_API_VERSION, version_list[0].get('version'))
self.assertNotIn(version_list[0].get('version'), [None, ''])
@tc.attr(base.TAG_POSITIVE, base.TAG_API)
def test_microversions_v2_invalid_version(self):