mirror of
https://github.com/kevinveenbirkenbach/docker-volume-backup.git
synced 2026-08-20 13:12:48 +00:00
Kevin Veen-Birkenbach
94637c32aa
A finished generation cannot show whether a volume held a database, nor whether a dump was produced for it: under --only-sql a failed dump falls back to a file copy, and the resulting files/ tree looks like any other copy. The run knows both and threw the knowledge away as a printed warning, leaving every reader to guess from file names. Each generation now carries a manifest.json stating its layout and, per volume, database / dumped / engine. baudolo.generation is the single place those names are spelled; restore/paths.py, backup/db.py and backup/volume.py stop repeating them. It is deliberately import-free so a consumer can read the manifest with nothing but json, on hosts where this package is not installed. BREAKING CHANGE: BackupException is renamed BackupError. The rename is atomic across the ten modules that define or import it, three of which also carry the manifest change, so it lands in this commit rather than a separate one that could not import. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
54 lines
2.0 KiB
Python
54 lines
2.0 KiB
Python
"""Contract of where a backup run puts its directories."""
|
|
|
|
from __future__ import annotations
|
|
|
|
import tempfile
|
|
import unittest
|
|
from pathlib import Path
|
|
from unittest import mock
|
|
|
|
from baudolo.backup import layout as mod
|
|
from baudolo.backup.shell import BackupError
|
|
|
|
|
|
class TestVersionDirectory(unittest.TestCase):
|
|
def test_it_creates_the_generation_directory(self) -> None:
|
|
with tempfile.TemporaryDirectory() as tmp:
|
|
created = mod.create_version_directory(tmp, "20260731020304")
|
|
self.assertTrue(Path(created).is_dir())
|
|
self.assertEqual(Path(created).name, "20260731020304")
|
|
|
|
def test_it_refuses_a_generation_another_run_already_claimed(self) -> None:
|
|
with tempfile.TemporaryDirectory() as tmp:
|
|
mod.create_version_directory(tmp, "20260731")
|
|
with self.assertRaises(BackupError) as caught:
|
|
mod.create_version_directory(tmp, "20260731")
|
|
self.assertIn("20260731", str(caught.exception))
|
|
|
|
def test_it_creates_missing_parents(self) -> None:
|
|
with tempfile.TemporaryDirectory() as tmp:
|
|
nested = str(Path(tmp) / "machine" / "repo")
|
|
created = mod.create_version_directory(nested, "20260731")
|
|
self.assertTrue(Path(created).is_dir())
|
|
|
|
|
|
class TestVolumeDirectory(unittest.TestCase):
|
|
def test_it_nests_the_volume_under_the_generation(self) -> None:
|
|
with tempfile.TemporaryDirectory() as tmp:
|
|
created = mod.create_volume_directory(tmp, "postgres_data")
|
|
self.assertEqual(Path(created).parent, Path(tmp))
|
|
self.assertTrue(Path(created).is_dir())
|
|
|
|
|
|
class TestMachineId(unittest.TestCase):
|
|
def test_it_takes_the_hash_without_the_filename(self) -> None:
|
|
digest = "a" * 64
|
|
with mock.patch.object(
|
|
mod, "execute_shell_command", return_value=[f"{digest} /etc/machine-id"]
|
|
):
|
|
self.assertEqual(mod.get_machine_id(), digest)
|
|
|
|
|
|
if __name__ == "__main__":
|
|
unittest.main()
|