Standardised all export repository methods across clients.

Signed-off-by: Leander Stephen D'Souza <[email protected]>

Author: leander-dsouza

test/test_bzr.py

@@ -65,7 +65,7 @@ def test_checkout_existing_directory_fails(self):
 
 @unittest.skipIf(not bzr, '`bzr` was not found')
 class TestExportRepository(unittest.TestCase):
-    """Integration tests for BzrClient _export_repository functionality."""
+    """Integration tests for BzrClient export_repository functionality."""
 
     def setUp(self):
         self.test_dir = tempfile.mkdtemp()
@@ -89,7 +89,7 @@ def test_export_repository(self):
             os.chdir(self.repo_path)
 
             # Test export with a specific revision
-            result = self.client._export_repository(None, self.export_path)
+            result = self.client.export_repository(None, self.export_path)
             self.assertTrue(result, 'Export should return True on success')
 
             archive_path = self.export_path + '.tar.gz'
@@ -115,7 +115,7 @@ def test_export_repository_git_version_unsupported(self):
             os.chdir(self.repo_path)
 
             # Test export with invalid version
-            result = self.client._export_repository('999999999', self.export_path)
+            result = self.client.export_repository('999999999', self.export_path)
             self.assertFalse(result, 'Version is not supported for git repositories.')
 
             archive_path = self.export_path + '.tar.gz'

test/test_hg.py

@@ -64,7 +64,7 @@ def test_checkout_existing_directory(self):
 
 @unittest.skipIf(not hg, '`hg` was not found')
 class TestExportRepository(unittest.TestCase):
-    """Integration tests for HgClient _export_repository functionality."""
+    """Integration tests for HgClient export_repository functionality."""
 
     def setUp(self):
         """Set up test fixtures for each test"""
@@ -97,7 +97,7 @@ def _ensure_repo_cloned(self):
 
     def test_export_repository_specific_revision(self):
         """Test exporting a specific revision"""
-        result = self.hg_client._export_repository('1', self.export_base_path)
+        result = self.hg_client.export_repository('1', self.export_base_path)
         self.assertTrue(result, "Export should succeed for revision '1'")
 
         # Verify files were created correctly
@@ -108,7 +108,7 @@ def test_export_repository_invalid_revision(self):
         """Test exporting with an invalid revision"""
         invalid_revision = 'nonexistent123456789'
 
-        result = self.hg_client._export_repository(
+        result = self.hg_client.export_repository(
             invalid_revision, self.export_base_path
         )
         self.assertFalse(result, 'Export should fail for invalid revision')

test/test_svn.py

@@ -88,7 +88,7 @@ def test_existing_non_empty_dir_should_fail(self):
 
 @unittest.skipIf(not svn, '`svn` was not found')
 class TestSvnExportRepository(unittest.TestCase):
-    """Integration tests for SvnClient _export_repository functionality."""
+    """Integration tests for SvnClient export_repository functionality."""
 
     def setUp(self):
         self.test_dir = tempfile.mkdtemp()
@@ -110,7 +110,7 @@ def test_export_repository_with_version(self):
         try:
             os.chdir(self.repo_path)
 
-            result = client._export_repository('1928014', self.export_path)
+            result = client.export_repository('1928014', self.export_path)
             self.assertTrue(result, 'Export should return True on success')
 
             # Verify tar.gz file was created
@@ -136,7 +136,7 @@ def test_export_repository_invalid_version(self):
         try:
             os.chdir(self.repo_path)
 
-            result = client._export_repository('999999999', self.export_path)
+            result = client.export_repository('999999999', self.export_path)
             self.assertFalse(result, 'Export should return False for invalid version')
 
             archive_path = self.export_path + '.tar.gz'

vcs2l/clients/bzr.py

@@ -194,7 +194,7 @@ def _get_parent_branch(self):
         result['output'] = branch
         return result
 
-    def _export_repository(self, version, basepath):
+    def export_repository(self, version, basepath):
         """Export the bzr repository at a given version to a tar.gz file."""
         self._check_executable()
 

vcs2l/clients/hg.py

@@ -340,7 +340,7 @@ def _check_color(self, cmd):
         if HgClient._config_color:
             cmd[1:1] = '--color', 'always'
 
-    def _export_repository(self, version, basepath):
+    def export_repository(self, version, basepath):
         """Export the hg repository at a given version to a tar.gz file."""
         self._check_executable()
 

vcs2l/clients/svn.py

@@ -261,7 +261,7 @@ def validate(self, command):
 
         return {'cmd': cmd, 'cwd': self.path, 'output': output, 'returncode': None}
 
-    def _export_repository(self, version, basepath):
+    def export_repository(self, version, basepath):
         """Export the svn repository at a given version to a tar.gz file."""
         self._check_executable()