Index: /branches/ipp-259_genericise_backups/tools/backups/backup_atlassian_applications.py
===================================================================
--- /branches/ipp-259_genericise_backups/tools/backups/backup_atlassian_applications.py	(revision 40912)
+++ /branches/ipp-259_genericise_backups/tools/backups/backup_atlassian_applications.py	(revision 40913)
@@ -79,5 +79,5 @@
                ]
 
-    def perform_atlassian_backups(self):
+    def perform_backups(self):
 
         # Check all paths are ok first:
@@ -92,18 +92,7 @@
         gen_utils.move_dirs(latest_paths, tmp_paths, self.verbose)
 
-        return_status = []
-
-        # copy jira
-        return_status.append(self._copy_jira_backup(self.verbose))
-
-        # copy confluence
-        return_status.append(self._copy_confluence_backup(self.verbose))
-
-        # tarfiles:
-        return_status.append(self._tar_jira_attachments())
-        return_status.append(self._tar_conf_attachments())
-
-        # mysqldump
-        return_status.append(self.make_mysql_dump(self.mysqldump_password))
+        return_status = self._jira_backups()
+        return_status = return_status + self._conf_backups()
+        return_status = return_status + self._db_dump()
 
         # If successful: move tmp to old
@@ -119,4 +108,19 @@
 
         return 0
+
+    def _jira_backups(self):
+        return_status = []
+        return_status.append(self._copy_jira_backup(self.verbose))
+        return_status.append(self._tar_jira_attachments())
+        return return_status
+
+    def _conf_backups(self):
+        return_status = []
+        return_status.append(self._copy_confluence_backup(self.verbose))
+        return_status.append(self._tar_conf_attachments())
+        return return_status
+
+    def _db_dump(self):
+        return [self.make_mysql_dump(self.mysqldump_password)]
 
     def get_jira_backup_date_format(self, custom_date=None):
@@ -197,5 +201,4 @@
         return 0
 
-
     def _tar_conf_attachments(self):
         latest_paths = self.target_paths("latest")
@@ -261,5 +264,5 @@
 
     def run(self):
-        return self.perform_atlassian_backups()
+        return self.perform_backups()
 
 
Index: /branches/ipp-259_genericise_backups/tools/backups/backup_atlassian_applications_test.py
===================================================================
--- /branches/ipp-259_genericise_backups/tools/backups/backup_atlassian_applications_test.py	(revision 40912)
+++ /branches/ipp-259_genericise_backups/tools/backups/backup_atlassian_applications_test.py	(revision 40913)
@@ -335,5 +335,5 @@
         ab = fab_test.create_all_paths_and_folders(ab, tmpdir)
 
-        result = ab.perform_atlassian_backups()
+        result = ab.perform_backups()
 
         assert result == 0
@@ -353,5 +353,5 @@
         assert not os.path.exists(ab.backup_2_path)
 
-        result = ab.perform_atlassian_backups()
+        result = ab.perform_backups()
         assert result == 1
 
Index: /branches/ipp-259_genericise_backups/tools/backups/full_atlassian_backup_test.py
===================================================================
--- /branches/ipp-259_genericise_backups/tools/backups/full_atlassian_backup_test.py	(revision 40912)
+++ /branches/ipp-259_genericise_backups/tools/backups/full_atlassian_backup_test.py	(revision 40913)
@@ -142,5 +142,5 @@
         ab = create_all_paths_and_folders(ab, tmpdir)
 
-        result = ab.perform_atlassian_backups()
+        result = ab.perform_backups()
 
         assert result == 0
Index: /branches/ipp-259_genericise_backups/tools/backups/utils/subprocess_utils.py
===================================================================
--- /branches/ipp-259_genericise_backups/tools/backups/utils/subprocess_utils.py	(revision 40912)
+++ /branches/ipp-259_genericise_backups/tools/backups/utils/subprocess_utils.py	(revision 40913)
@@ -42,8 +42,8 @@
 
     try:
-        subprocess.check_call(command_and_args)
+        subprocess_exitcode = subprocess.check_call(command_and_args)
     except (subprocess.CalledProcessError) as cpe:
         raise SubprocessError(f"{type(cpe)} in python simple_unix_wrapper") from cpe
     except (ValidationError) as ve:
         raise ValidationError(f"{type(ve)} in python simple_unix_wrapper") from ve
-    return 0
+    return subprocess_exitcode