fix tests

Author: egegunes

internal/pgbackrest/postgres.go

@@ -36,7 +36,14 @@ func PostgreSQL(
 	archive += ` if [ ! -z ${timestamp} ]; then echo ${timestamp} > /pgdata/latest_commit_timestamp.txt; fi`
 
 	outParameters.Mandatory.Add("archive_mode", "on")
-	outParameters.Mandatory.Add("archive_command", archive)
+
+	if backupsEnabled {
+		outParameters.Mandatory.Add("archive_command", archive)
+	} else {
+		// If backups are disabled, keep archive_mode on (to avoid a Postgres restart)
+		// and throw away WAL.
+		outParameters.Mandatory.Add("archive_command", `true`)
+	}
 
 	// K8SPG-518: This parameter is required to ensure that the commit timestamp is
 	// included in the WAL file. This is necessary for the WAL watcher to

internal/pgbackrest/postgres_test.go

@@ -51,7 +51,7 @@ func TestPostgreSQLParameters(t *testing.T) {
 	}
 	cluster.Spec.Patroni.DynamicConfiguration = dynamic
 
-	PostgreSQL(cluster, parameters)
+	PostgreSQL(cluster, parameters, true)
 	assert.DeepEqual(t, parameters.Mandatory.AsMap(), map[string]string{
 		"archive_mode": "on",
 		"archive_command": strings.Join([]string{