mirror of
https://github.com/saltstack/salt.git
synced 2025-04-17 10:10:20 +00:00
Don't fail the test because of order
This commit is contained in:
parent
9a31be7271
commit
0281fc5b50
6 changed files with 14 additions and 29 deletions
|
@ -222,7 +222,7 @@ class SupervisordModuleTest(ModuleCase):
|
|||
ret = self.run_function(
|
||||
'supervisord.status', [], conf_file=self.supervisor_conf,
|
||||
bin_env=self.venv_dir)
|
||||
self.assertEqual(list(ret.keys()), ['sleep_service', 'sleep_service2'])
|
||||
self.assertEqual(sorted(ret), ['sleep_service', 'sleep_service2'])
|
||||
|
||||
def test_status_one(self):
|
||||
'''
|
||||
|
|
|
@ -118,9 +118,12 @@ class CassandraTestCase(TestCase, LoaderModuleMockMixin):
|
|||
self.assertCountEqual(cassandra.column_families(),
|
||||
{'A': ['a', 'b'], 'B': ['c', 'd']})
|
||||
else:
|
||||
self.assertEqual(cassandra.column_families('A'),
|
||||
self.assertEqual(sorted(cassandra.column_families('A')),
|
||||
['a', 'b'])
|
||||
self.assertEqual(cassandra.column_families(),
|
||||
column_families = cassandra.column_families()
|
||||
for key in ('A', 'B'):
|
||||
column_families[key] = sorted(column_families[key])
|
||||
self.assertEqual(column_families,
|
||||
{'A': ['a', 'b'], 'B': ['c', 'd']})
|
||||
|
||||
def test_column_family_definition(self):
|
||||
|
|
|
@ -135,26 +135,9 @@ class InspectorFSDBTestCase(TestCase):
|
|||
csvdb.open()
|
||||
csvdb.create_table_from_object(FoobarEntity())
|
||||
|
||||
if six.PY2:
|
||||
assert writable.data[0].strip() == "foo:int,bar:str,spam:float"
|
||||
else:
|
||||
# Order in PY3 is not the same for every run
|
||||
writable_data = writable.data[0].strip()
|
||||
assert_order_options = ['bar:str,foo:int,spam:float',
|
||||
'bar:str,spam:float,foo:int',
|
||||
'foo:int,spam:float,bar:str',
|
||||
'foo:int,bar:str,spam:float',
|
||||
'spam:float,foo:int,bar:str',
|
||||
'spam:float,bar:str,foo:int']
|
||||
while assert_order_options:
|
||||
assert_option = assert_order_options.pop()
|
||||
try:
|
||||
assert writable_data == assert_option
|
||||
break
|
||||
except AssertionError:
|
||||
if not assert_order_options:
|
||||
raise
|
||||
continue
|
||||
sorted_writable_data = sorted(writable.data[0].strip().split(','))
|
||||
sorted_expected_data = sorted("foo:int,bar:str,spam:float".split(','))
|
||||
self.assertEqual(sorted_writable_data, sorted_expected_data)
|
||||
|
||||
def test_list_databases(self):
|
||||
'''
|
||||
|
|
|
@ -54,10 +54,11 @@ class PillarModuleTestCase(TestCase, LoaderModuleMockMixin):
|
|||
@skipIf(NO_MOCK, NO_MOCK_REASON)
|
||||
def test_ls(self):
|
||||
with patch('salt.modules.pillar.items', MagicMock(return_value=pillar_value_1)):
|
||||
ls = sorted(pillarmod.ls())
|
||||
if six.PY3:
|
||||
self.assertCountEqual(pillarmod.ls(), ['a', 'b'])
|
||||
self.assertCountEqual(ls, ['a', 'b'])
|
||||
else:
|
||||
self.assertEqual(pillarmod.ls(), ['a', 'b'])
|
||||
self.assertEqual(ls, ['a', 'b'])
|
||||
|
||||
@skipIf(NO_MOCK, NO_MOCK_REASON)
|
||||
def test_pillar_get_default_merge(self):
|
||||
|
|
|
@ -66,6 +66,6 @@ class JsonTestCase(TestCase, LoaderModuleMockMixin):
|
|||
'"error": "Unable to serialize output to json"}')
|
||||
ret = json_out.output(data)
|
||||
if six.PY2:
|
||||
self.assertEqual(expect, ret)
|
||||
self.assertEqual(sorted(expect), sorted(ret))
|
||||
else:
|
||||
self.assertEqual(json.loads(ret), data)
|
||||
|
|
|
@ -70,9 +70,7 @@ class ProxyTestCase(TestCase, LoaderModuleMockMixin):
|
|||
with patch.dict(proxy.__salt__, patches):
|
||||
out = proxy.managed('192.168.0.1', '3128', user='frank', password='passw0rd',
|
||||
bypass_domains=['salt.com', 'test.com'])
|
||||
if six.PY3:
|
||||
# Sorting is different in Py3
|
||||
out['changes']['new'][-1]['bypass_domains'] = sorted(out['changes']['new'][-1]['bypass_domains'])
|
||||
out['changes']['new'][-1]['bypass_domains'] = sorted(out['changes']['new'][-1]['bypass_domains'])
|
||||
|
||||
calls = [
|
||||
call('192.168.0.1', '3128', 'frank', 'passw0rd', 'Ethernet'),
|
||||
|
|
Loading…
Add table
Reference in a new issue