mirror of
https://github.com/outbackdingo/patroni.git
synced 2026-08-25 14:53:37 +00:00
Fix tests on windows (#1303)
and disable junit, it produces a deprecation warning
This commit is contained in:
@@ -34,7 +34,7 @@ KEYWORDS = 'etcd governor patroni postgresql postgres ha haproxy confd' +\
|
|||||||
|
|
||||||
COVERAGE_XML = True
|
COVERAGE_XML = True
|
||||||
COVERAGE_HTML = False
|
COVERAGE_HTML = False
|
||||||
JUNIT_XML = True
|
JUNIT_XML = False
|
||||||
|
|
||||||
# Add here all kinds of additional classifiers as defined under
|
# Add here all kinds of additional classifiers as defined under
|
||||||
# https://pypi.python.org/pypi?%3Aaction=list_classifiers
|
# https://pypi.python.org/pypi?%3Aaction=list_classifiers
|
||||||
|
|||||||
+1
-1
@@ -172,7 +172,7 @@ class PostgresInit(unittest.TestCase):
|
|||||||
@patch.object(ConfigHandler, 'replace_pg_ident', Mock())
|
@patch.object(ConfigHandler, 'replace_pg_ident', Mock())
|
||||||
@patch.object(Postgresql, 'get_postgres_role_from_data_directory', Mock(return_value='master'))
|
@patch.object(Postgresql, 'get_postgres_role_from_data_directory', Mock(return_value='master'))
|
||||||
def setUp(self):
|
def setUp(self):
|
||||||
data_dir = 'data/test0'
|
data_dir = os.path.join('data', 'test0')
|
||||||
self.p = Postgresql({'name': 'postgresql0', 'scope': 'batman', 'data_dir': data_dir,
|
self.p = Postgresql({'name': 'postgresql0', 'scope': 'batman', 'data_dir': data_dir,
|
||||||
'config_dir': data_dir, 'retry_timeout': 10,
|
'config_dir': data_dir, 'retry_timeout': 10,
|
||||||
'krbsrvname': 'postgres', 'pgpass': os.path.join(data_dir, 'pgpass0'),
|
'krbsrvname': 'postgres', 'pgpass': os.path.join(data_dir, 'pgpass0'),
|
||||||
|
|||||||
@@ -1,6 +1,7 @@
|
|||||||
import mock # for the mock.call method, importing it without a namespace breaks python3
|
import mock # for the mock.call method, importing it without a namespace breaks python3
|
||||||
import os
|
import os
|
||||||
import psycopg2
|
import psycopg2
|
||||||
|
import re
|
||||||
import subprocess
|
import subprocess
|
||||||
import time
|
import time
|
||||||
|
|
||||||
@@ -220,7 +221,8 @@ class TestPostgresql(BaseTestPostgresql):
|
|||||||
with patch('patroni.postgresql.config.ConfigHandler.primary_conninfo_params', Mock(return_value=conninfo)):
|
with patch('patroni.postgresql.config.ConfigHandler.primary_conninfo_params', Mock(return_value=conninfo)):
|
||||||
mock_get_pg_settings.return_value['recovery_min_apply_delay'][1] = '1'
|
mock_get_pg_settings.return_value['recovery_min_apply_delay'][1] = '1'
|
||||||
self.assertEqual(self.p.config.check_recovery_conf(None), (True, True))
|
self.assertEqual(self.p.config.check_recovery_conf(None), (True, True))
|
||||||
mock_get_pg_settings.return_value['primary_conninfo'][1] = 'host=1 passfile=' + self.p.config._pgpass
|
mock_get_pg_settings.return_value['primary_conninfo'][1] = 'host=1 passfile='\
|
||||||
|
+ re.sub(r'([\'\\ ])', r'\\\1', self.p.config._pgpass)
|
||||||
mock_get_pg_settings.return_value['recovery_min_apply_delay'][1] = '0'
|
mock_get_pg_settings.return_value['recovery_min_apply_delay'][1] = '0'
|
||||||
self.assertEqual(self.p.config.check_recovery_conf(None), (True, True))
|
self.assertEqual(self.p.config.check_recovery_conf(None), (True, True))
|
||||||
self.p.config.write_recovery_conf({'standby_mode': 'on', 'primary_conninfo': conninfo.copy()})
|
self.p.config.write_recovery_conf({'standby_mode': 'on', 'primary_conninfo': conninfo.copy()})
|
||||||
|
|||||||
Reference in New Issue
Block a user