fix assertions so non-deterministic writes can be asserted

Signed-off-by: Alfredo Deza <alfredo.deza@inktank.com>

diff --git a/ceph_deploy/tests/unit/test_mon.py b/ceph_deploy/tests/unit/test_mon.py
index 871d7e04a8b0b5a17babdae70873ea755a08c49b..0cfccbad097be79f979b913f85894083df64f128 100644 (file)
--- a/ceph_deploy/tests/unit/test_mon.py
+++ b/ceph_deploy/tests/unit/test_mon.py
@@ -199,17 +199,13 @@ class TestConcatenateKeyrings(object):
         self.make_keyring(tmpdir, 'bar.keyring', '[mon.2]\nkey = value\n')
         self.make_keyring(tmpdir, 'fez.keyring', '[mon.3]\nkey = value\n')
         self.args.keyrings = tmpdir.strpath
-        expected = '\n'.join([
-            '[mon.2]',
-            'key = value',
-            '[mon.3]',
-            'key = value',
-            '[mon.1]',
-            'key = value',
-            '',
-        ])
-
-        assert mon.concatenate_keyrings(self.args) == expected
+        result = mon.concatenate_keyrings(self.args).split('\n')
+        assert '[mon.2]' in result
+        assert 'key = value' in result
+        assert '[mon.3]' in result
+        assert 'key = value' in result
+        assert '[mon.1]' in result
+        assert 'key = value' in result
 
     def test_skips_duplicate_content(self, tmpdir):
         self.make_keyring(tmpdir, 'foo.keyring', '[mon.1]\nkey = value\n')
@@ -217,17 +213,10 @@ class TestConcatenateKeyrings(object):
         self.make_keyring(tmpdir, 'fez.keyring', '[mon.3]\nkey = value\n')
         self.make_keyring(tmpdir, 'dupe.keyring', '[mon.3]\nkey = value\n')
         self.args.keyrings = tmpdir.strpath
-        expected = '\n'.join([
-            '[mon.2]',
-            'key = value',
-            '[mon.3]',
-            'key = value',
-            '[mon.1]',
-            'key = value',
-            '',
-        ])
-
-        assert mon.concatenate_keyrings(self.args) == expected
+        result = mon.concatenate_keyrings(self.args).split('\n')
+        assert result.count('[mon.3]') == 1
+        assert result.count('[mon.2]') == 1
+        assert result.count('[mon.1]') == 1
 
     def test_errors_when_no_keyrings(self, tmpdir):
         self.args.keyrings = tmpdir.strpath