tests: added backup.BackupTest.test_backup_concurrent_drop_table and compatibility.CompatibilityTest.test_backup_concurrent_drop_table

gsmolk · gsmolk · commit 0493eabfd618 · 2019-05-21T17:54:23.000+03:00
diff --git a/tests/backup.py b/tests/backup.py
@@ -1002,6 +1002,51 @@ def test_temp_slot_for_stream_backup(self):
         # Clean after yourself
         self.del_test_dir(module_name, fname)
 
+    # @unittest.skip("skip")
+    def test_backup_concurrent_drop_table(self):
+        """"""
+        fname = self.id().split('.')[3]
+        backup_dir = os.path.join(self.tmp_path, module_name, fname, 'backup')
+        node = self.make_simple_node(
+            base_dir=os.path.join(module_name, fname, 'node'),
+            set_replication=True,
+            initdb_params=['--data-checksums'])
+
+        self.init_pb(backup_dir)
+        self.add_instance(backup_dir, 'node', node)
+        self.set_archiving(backup_dir, 'node', node)
+        node.slow_start()
+
+        node.pgbench_init(scale=5)
+
+        # FULL backup
+        gdb = self.backup_node(
+            backup_dir, 'node', node,
+            options=['--stream', '--compress', '--log-level-file=VERBOSE'],
+            gdb=True)
+
+        gdb.set_breakpoint('backup_data_file')
+        gdb.run_until_break()
+
+        node.safe_psql(
+            'postgres',
+            'DROP TABLE pgbench_accounts')
+
+        # do checkpoint to guarantee filenode removal
+        node.safe_psql(
+            'postgres',
+            'CHECKPOINT')
+
+        gdb.remove_all_breakpoints()
+        gdb.continue_execution_until_exit()
+
+        show_backup = self.show_pb(backup_dir, 'node')[0]
+
+        self.assertEqual(show_backup['status'], "OK")
+
+        # Clean after yourself
+        self.del_test_dir(module_name, fname)
+
     # @unittest.skip("skip")
     def test_pg_11_adjusted_wal_segment_size(self):
         """"""
diff --git a/tests/compatibility.py b/tests/compatibility.py
@@ -537,3 +537,49 @@ def test_backward_compatibility_merge(self):
 
         # Clean after yourself
         self.del_test_dir(module_name, fname)
+
+    # @unittest.skip("skip")
+    def test_backup_concurrent_drop_table(self):
+        """"""
+        fname = self.id().split('.')[3]
+        backup_dir = os.path.join(self.tmp_path, module_name, fname, 'backup')
+        node = self.make_simple_node(
+            base_dir=os.path.join(module_name, fname, 'node'),
+            set_replication=True,
+            initdb_params=['--data-checksums'])
+
+        self.init_pb(backup_dir)
+        self.add_instance(backup_dir, 'node', node, old_binary=True)
+        node.slow_start()
+
+        node.pgbench_init(scale=5)
+
+        # FULL backup
+        gdb = self.backup_node(
+            backup_dir, 'node', node,
+            options=['--stream', '--compress', '--log-level-file=VERBOSE'],
+            gdb=True, old_binary=True)
+
+        gdb.set_breakpoint('backup_data_file')
+        gdb.run_until_break()
+
+        node.safe_psql(
+            'postgres',
+            'DROP TABLE pgbench_accounts')
+
+        # do checkpoint to guarantee filenode removal
+        node.safe_psql(
+            'postgres',
+            'CHECKPOINT')
+
+        gdb.remove_all_breakpoints()
+        gdb.continue_execution_until_exit()
+
+        # show_backup = self.show_pb(backup_dir, 'node')[0]
+        # self.assertEqual(show_backup['status'], "OK")
+
+        # validate with fresh binary, it MUST be successful
+        self.validate_pb(backup_dir)
+
+        # Clean after yourself
+        self.del_test_dir(module_name, fname)
