Commit improvements

MakSl · MakSl · commit 0dc237d9798f · 2024-05-23T02:35:28.000+03:00
diff --git a/testgres/performance_test.py b/testgres/performance_test.py
@@ -1,5 +1,6 @@
 from .plugins.pg_probackup2.pg_probackup2.tests.basic_test import ProbackupTest
 
+
 class PerformanceTest(ProbackupTest):
 
     def test_full_backup_perf_basic(self):
diff --git a/testgres/plugins/pg_probackup2/pg_probackup2/app.py b/testgres/plugins/pg_probackup2/pg_probackup2/app.py
@@ -59,7 +59,7 @@ def __init__(self, test_class: unittest.TestCase,
         self.execution_time = execution_time
 
     def run(self, command, gdb=False, old_binary=False, return_id=True, env=None,
-            skip_log_directory=False, expect_error=False, use_backup_dir=True, execution_time=False):
+            skip_log_directory=False, expect_error=False, use_backup_dir=True):
         """
         Run pg_probackup
         backup_dir: target directory for making backup
@@ -114,7 +114,7 @@ def run(self, command, gdb=False, old_binary=False, return_id=True, env=None,
                 cmdline = ['gdbserver'] + ['localhost:' + str(gdb_port)] + cmdline
                 print("pg_probackup gdb suspended, waiting gdb connection on localhost:{0}".format(gdb_port))
 
-            if execution_time:
+            if self.execution_time:
                 start_time = time.time()
                 self.test_class.output = subprocess.check_output(
                     cmdline,
@@ -225,8 +225,7 @@ def backup_node(
             old_binary=False, return_id=True, no_remote=False,
             env=None,
             expect_error=False,
-            sync=False,
-            execution_time=False
+            sync=False
     ):
         if options is None:
             options = []
@@ -266,7 +265,7 @@ def backup_node(
             cmd_list += ['--no-sync']
 
         return self.run(cmd_list + options, gdb, old_binary, return_id, env=env,
-                        expect_error=expect_error, execution_time=execution_time)
+                        expect_error=expect_error)
 
     def backup_replica_node(self, instance, node, data_dir=False, *,
                             master, backup_type='full', datname=False,
