Fix: primary_slot_name never actually was set

Author: zilder

testgres/backup.py

@@ -171,7 +171,7 @@ def spawn_replica(self, name=None, destroy=True, slot_name=None):
 
         # Assign it a master and a recovery file (private magic)
         node._assign_master(self.original_node)
-        node._create_recovery_conf(username=self.username)
+        node._create_recovery_conf(username=self.username, slot_name=slot_name)
 
         return node
 

testgres/node.py

@@ -209,7 +209,7 @@ def _create_recovery_conf(self, username, slot_name=None):
         ).format(conninfo)
 
         if slot_name:
-            line += "primary_slot_name={}\n".format()
+            line += "primary_slot_name={}\n".format(slot_name)
 
         self.append_conf(RECOVERY_CONF_FILE, line)
 
@@ -872,7 +872,9 @@ def create_replication_slot(self, slot_name, dbname=None, username=None):
             dbname: database name
             username: database user name
         """
-        query = "select pg_create_physical_replication_slot('{}')".format(slot_name)
+        query = (
+            "select pg_create_physical_replication_slot('{}')"
+        ).format(slot_name)
 
         self.execute(query=query,
                      dbname=dbname or default_dbname(),