Added test for PostgresNode.__repr__() method; removed node name from examples

Author: zilder

testgres/api.py

@@ -7,27 +7,27 @@
 edit configuration files, start/stop cluster, execute queries. The
 typical flow may look like:
 
->>> with get_new_node('test') as node:
+>>> with get_new_node() as node:
 ...     node.init().start()
 ...     result = node.safe_psql('postgres', 'select 1')
 ...     print(result.decode('utf-8').strip())
 ...     node.stop()
-PostgresNode('test', port=..., base_dir=...)
+PostgresNode(name='...', port=..., base_dir='...')
 1
-PostgresNode('test', port=..., base_dir=...)
+PostgresNode(name='...', port=..., base_dir='...')
 
     Or:
 
->>> with get_new_node('master') as master:
+>>> with get_new_node() as master:
 ...     master.init().start()
 ...     with master.backup() as backup:
-...         with backup.spawn_replica('replica') as replica:
+...         with backup.spawn_replica() as replica:
 ...             replica = replica.start()
 ...             master.execute('postgres', 'create table test (val int4)')
 ...             master.execute('postgres', 'insert into test values (0), (1), (2)')
 ...             replica.catchup()  # wait until changes are visible
 ...             print(replica.execute('postgres', 'select count(*) from test'))
-PostgresNode('master', port=..., base_dir=...)
+PostgresNode(name='...', port=..., base_dir='...')
 [(3,)]
 
 Copyright (c) 2016, Postgres Professional

testgres/node.py

@@ -136,7 +136,7 @@ def __exit__(self, type, value, traceback):
             self._try_shutdown(attempts)
 
     def __repr__(self):
-        return "PostgresNode('{}', port={}, base_dir={})".format(
+        return "PostgresNode(name='{}', port={}, base_dir='{}')".format(
             self.name, self.port, self.base_dir)
 
     @property

tests/test_simple.py

@@ -2,6 +2,7 @@
 # coding: utf-8
 
 import os
+import re
 import subprocess
 import tempfile
 import testgres
@@ -70,6 +71,11 @@ def removing(f):
 
 
 class TestgresTests(unittest.TestCase):
+    def test_node_repr(self):
+        with get_new_node() as node:
+            pattern = 'PostgresNode\(name=\'.+\', port=.+, base_dir=\'.+\'\)'
+            self.assertIsNotNone(re.match(pattern, str(node)))
+
     def test_custom_init(self):
         with get_new_node() as node:
             # enable page checksums