gOdoo-cli 1.0.0__tar.gz → 1.0.1__tar.gz

{godoo_cli-1.0.0 → godoo_cli-1.0.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: gOdoo-cli
-Version: 1.0.0
+Version: 1.0.1
 Summary: Odoo lifecycle, Python dependency preflight, and shared Git workspaces.
 Project-URL: repository, https://github.com/OpenJKSoftware/gOdoo
 Author-email: Joshua Kreuder <Joshua_Kreuder@outlook.com>

{godoo_cli-1.0.0 → godoo_cli-1.0.1}/docs/lifecycle.md

@@ -178,6 +178,12 @@ sequenceDiagram
     Maintainer->>Maintainer: Review Odoo's report before restoring into a test runtime
 ```
 
+Before downloading the official client, `godoo upgrade` logs the modification date and absolute path of the dump
+selected by `-i` / `--dump` and the adjacent `odoo_filestore` directory. Dates include the local timezone offset. The
+filestore stays local for restoring the result. Its directory timestamp reflects changes to immediate entries, not
+changes throughout the attachment tree or proof that the dump and filestore were captured together. Missing metadata
+produces a warning; the official client still owns input validation. Commands without a dump do not log source dates.
+
 HTTPS certificate and hostname verification is enabled by default. `GODOO_UPGRADE_SSL=0` disables verification for both
 the download and client while retaining HTTPS encryption; use it only for an endpoint whose certificate the host cannot
 verify.

{godoo_cli-1.0.0 → godoo_cli-1.0.1}/src/godoo_cli/__about__.py

@@ -1,3 +1,3 @@
 """Version information."""
 
-__version__ = "1.0.0"
+__version__ = "1.0.1"

{godoo_cli-1.0.0 → godoo_cli-1.0.1}/src/godoo_cli/commands/upgrade.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import argparse
 import logging
 import os
 import signal
@@ -10,6 +11,7 @@ import subprocess
 import sys
 import tempfile
 from contextlib import suppress
+from datetime import UTC, datetime
 from pathlib import Path
 from typing import Annotated, cast
 from urllib.error import URLError
@@ -24,8 +26,35 @@ LOGGER = logging.getLogger(__name__)
 UPGRADE_CLIENT_URL = "https://upgrade.odoo.com/upgrade"
 
 
+def _log_upgrade_sources(arguments: list[str]) -> None:
+    """Log the selected dump and paired local filestore before the client starts."""
+    parser = argparse.ArgumentParser(add_help=False, allow_abbrev=False, exit_on_error=False)
+    parser.add_argument("-i", "--dump", type=Path)
+    try:
+        options, _ = parser.parse_known_args(arguments)
+    except argparse.ArgumentError:
+        # Odoo's client owns argument validation and its error messages.
+        return
+    if options.dump is None:
+        return
+
+    dump = options.dump.absolute()
+    sources = (
+        ("Dump change date (upload)", dump),
+        ("Filestore change date (local restore)", dump.parent / "odoo_filestore"),
+    )
+    for label, path in sources:
+        try:
+            modified_at = datetime.fromtimestamp(path.stat().st_mtime, tz=UTC).astimezone()
+        except OSError as error:
+            LOGGER.warning("Could not read %s (%s): %s", label, path, error)
+            continue
+        LOGGER.info("%s: %s (%s)", label, modified_at.isoformat(timespec="seconds"), path)
+
+
 def _run_upgrade_client(arguments: list[str]) -> int:
     """Download and run Odoo's official upgrade client with its arguments."""
+    _log_upgrade_sources(arguments)
     ssl_disabled = os.getenv("GODOO_UPGRADE_SSL") == "0"
     context = ssl._create_unverified_context() if ssl_disabled else None
     try:

{godoo_cli-1.0.0 → godoo_cli-1.0.1}/tests/commands/test_upgrade.py

@@ -3,10 +3,12 @@
 from __future__ import annotations
 
 import importlib
+import logging
 import os
 import signal
 import ssl
 import sys
+from datetime import UTC, datetime
 from io import BytesIO
 from pathlib import Path
 from unittest.mock import Mock
@@ -194,3 +196,70 @@ def test_upgrade_cli_preserves_nonzero_client_status(monkeypatch: pytest.MonkeyP
     result = CliRunner().invoke(main_cli(), ["upgrade", "status", "token"])
 
     assert result.exit_code == 23, result.output
+
+
+@pytest.mark.parametrize("dump_option", ["-i", "--dump", "--dump=", "-i-attached"])
+def test_upgrade_logs_source_dates_before_download_and_client(
+    monkeypatch: pytest.MonkeyPatch,
+    tmp_path: Path,
+    caplog: pytest.LogCaptureFixture,
+    dump_option: str,
+) -> None:
+    """The actual selected file and paired directory are logged before network work."""
+    dataset = tmp_path / "dataset with spaces"
+    dataset.mkdir()
+    dump = dataset / "source.dump"
+    dump.touch()
+    filestore = dataset / "odoo_filestore"
+    filestore.mkdir()
+    os.utime(dump, (1700000000, 1700000000))
+    os.utime(filestore, (1600000000, 1600000000))
+    monkeypatch.chdir(tmp_path)
+    relative_dump = str(dump.relative_to(tmp_path))
+    if dump_option == "--dump=":
+        selection = [f"--dump={relative_dump}"]
+    elif dump_option == "-i-attached":
+        selection = [f"-i{relative_dump}"]
+    else:
+        selection = [dump_option, relative_dump]
+    arguments = ["test", *selection, "--future-official-option", "value", "-x"]
+    caplog.set_level(logging.INFO, logger=upgrade_command.LOGGER.name)
+
+    def download(*_args: object, **_kwargs: object) -> BytesIO:
+        assert (
+            f"Dump change date (upload): {datetime.fromtimestamp(1700000000, UTC).astimezone().isoformat()} ({dump})"
+            in caplog.text
+        )
+        assert (
+            f"Filestore change date (local restore): {datetime.fromtimestamp(1600000000, UTC).astimezone().isoformat()} ({filestore})"
+            in caplog.text
+        )
+        return BytesIO(b"official client")
+
+    child = Mock(return_value=Mock(wait=lambda: 0, pid=42))
+    monkeypatch.setattr(upgrade_command, "urlopen", download)
+    monkeypatch.setattr(upgrade_command.subprocess, "Popen", child)
+
+    assert upgrade_command._run_upgrade_client(arguments) == 0
+    assert child.call_args.args[0][2:] == arguments
+
+
+def test_upgrade_missing_source_metadata_does_not_replace_client_validation(
+    monkeypatch: pytest.MonkeyPatch, tmp_path: Path, caplog: pytest.LogCaptureFixture
+) -> None:
+    """Missing timestamps produce warnings while the official client keeps ownership."""
+    dump = tmp_path / "missing.dump"
+    monkeypatch.setattr(upgrade_command, "urlopen", lambda *_args, **_kwargs: BytesIO(b""))
+    monkeypatch.setattr(upgrade_command, "_run_process", lambda *_args, **_kwargs: 23)
+
+    assert upgrade_command._run_upgrade_client(["test", "-i", str(dump)]) == 23
+    assert f"Could not read Dump change date (upload) ({dump})" in caplog.text
+    assert f"Could not read Filestore change date (local restore) ({tmp_path / 'odoo_filestore'})" in caplog.text
+
+
+@pytest.mark.parametrize("arguments", [["status", "token"], ["test", "--dbname", "database"], ["test", "-i"]])
+def test_upgrade_skips_source_logging_without_a_dump(arguments: list[str], caplog: pytest.LogCaptureFixture) -> None:
+    """Database mode, status, and malformed dump options stay with the official client."""
+    caplog.set_level(logging.INFO, logger=upgrade_command.LOGGER.name)
+    upgrade_command._log_upgrade_sources(arguments)
+    assert not caplog.records