odoo-dev 0.2.3__tar.gz → 0.2.4__tar.gz

{odoo_dev-0.2.3 → odoo_dev-0.2.4}/PKG-INFO

@@ -1,10 +1,11 @@
 Metadata-Version: 2.4
 Name: odoo-dev
-Version: 0.2.3
+Version: 0.2.4
 Summary: Odoo Development Environment Helper
 Project-URL: Homepage, https://git.bemade.org/bemade/odoo-dev
 Project-URL: Repository, https://git.bemade.org/bemade/odoo-dev
 Project-URL: Issues, https://git.bemade.org/bemade/odoo-dev/-/issues
+License-Expression: LGPL-3.0
 Requires-Python: >=3.12
 Requires-Dist: rich>=13.0.0
 Requires-Dist: typer>=0.9.0

{odoo_dev-0.2.3 → odoo_dev-0.2.4}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "odoo-dev"
-version = "0.2.3"
+version = "0.2.4"
 description = "Odoo Development Environment Helper"
 readme = "README.md"
 requires-python = ">=3.12"
@@ -8,6 +8,7 @@ dependencies = [
     "typer>=0.9.0",
     "rich>=13.0.0",
 ]
+license = "LGPL-3.0"
 
 [project.urls]
 Homepage = "https://git.bemade.org/bemade/odoo-dev"

{odoo_dev-0.2.3 → odoo_dev-0.2.4}/src/odoo_dev/__init__.py

@@ -1,3 +1,3 @@
 """Odoo Development Environment Helper."""
 
-__version__ = "0.2.3"
+__version__ = "0.2.4"

{odoo_dev-0.2.3 → odoo_dev-0.2.4}/src/odoo_dev/commands/setup.py

@@ -1,6 +1,7 @@
 """Setup commands for initializing Odoo development environment."""
 
 import os
+import shutil
 import subprocess
 from pathlib import Path
 
@@ -75,12 +76,27 @@ def _update_env_file(env_file: Path, updates: dict[str, str]) -> None:
     env_file.write_text(content + "\n")
 
 
+def _clean():
+    cfg = load_config()
+    success("Cleaning up current environment...")
+    for name in ["odoo", "enterprise", "design-themes", "industry", ".venv"]:
+        path = cfg.project_dir / name
+        if path.exists():
+            shutil.rmtree(path)
+    success("Clean up complete.")
+
+
 def setup(
     community: bool = typer.Option(
         False,
         "--community",
         help="Set up Community edition only (skip Enterprise repos)",
     ),
+    clean: bool = typer.Option(
+        False,
+        "--clean",
+        help="Clean up before setup (remove existing venv, etc.)",
+    ),
 ) -> None:
     """Complete setup: clone Odoo repos, configure VSCode, build image."""
     # Prompt for versions if not configured
@@ -88,6 +104,8 @@ def setup(
 
     cfg = load_config()
 
+    if clean:
+        _clean()
     success("Setting up complete Odoo development environment...")
 
     # Initialize/update git submodules first
@@ -305,22 +323,25 @@ def _clone_odoo_repos(cfg, community_only: bool = False) -> None:
     if community_only:
         warning("Community edition mode: Enterprise repositories will be skipped")
 
+    # Use HTTPS for odoo (public), SSH for all others (private)
     repos = [
-        (f"git@github.com:odoo/odoo.git", "odoo", cfg.odoo_version),
-        (f"git@github.com:odoo/design-themes.git", "design-themes", cfg.odoo_version),
+        ("https://github.com/odoo/odoo.git", "odoo", cfg.odoo_version),
     ]
 
-    # Add enterprise if not community only
+    # Private repos require SSH
+    repos.append(
+        ("git@github.com:odoo/design-themes.git", "design-themes", cfg.odoo_version)
+    )
+
+    # Add enterprise if not community only (private repo, needs SSH)
     if not community_only:
         repos.append(
-            (f"git@github.com:odoo/enterprise.git", "enterprise", cfg.odoo_version)
+            ("git@github.com:odoo/enterprise.git", "enterprise", cfg.odoo_version)
         )
 
-    # Add industry for Odoo 18+
+    # Add industry for Odoo 18+ (private repo)
     if cfg.odoo_version.startswith("18") or cfg.odoo_version.startswith("19"):
-        repos.append(
-            (f"git@github.com:odoo/industry.git", "industry", cfg.odoo_version)
-        )
+        repos.append(("git@github.com:odoo/industry.git", "industry", cfg.odoo_version))
 
     for repo_url, repo_dir, branch in repos:
         repo_path = cfg.project_dir / repo_dir
@@ -355,6 +376,15 @@ def _clone_odoo_repos(cfg, community_only: bool = False) -> None:
     success("Odoo repositories setup complete.")
 
 
+def _can_sudo_without_password() -> bool:
+    """Check if sudo can run without prompting for password."""
+    result = subprocess.run(
+        ["sudo", "-n", "true"],
+        capture_output=True,
+    )
+    return result.returncode == 0
+
+
 def _install_system_dependencies() -> None:
     """Install system dependencies based on OS."""
     import platform
@@ -376,6 +406,13 @@ def _install_system_dependencies() -> None:
             check=False,
         )
     elif system == "Linux":
+        if not _can_sudo_without_password():
+            warning(
+                "Cannot run sudo without password. "
+                "Please install system dependencies manually or run with sudo."
+            )
+            return
+
         success("Installing dependencies for Linux...")
         subprocess.run(
             [

{odoo_dev-0.2.3 → odoo_dev-0.2.4}/tests/test_integration.py

@@ -10,6 +10,7 @@ from typer.testing import CliRunner
 
 from odoo_dev.cli import app
 from odoo_dev.config import load_config
+from odoo_dev.commands import setup
 
 runner = CliRunner()
 
@@ -20,7 +21,9 @@ FIXTURE_DIR = Path(__file__).parent / "fixtures" / "odoo-empty"
 def fixture_dir() -> Path:
     """Return the path to the odoo-empty fixture."""
     if not FIXTURE_DIR.exists():
-        pytest.skip("odoo-empty fixture not available (run: git submodule update --init)")
+        pytest.skip(
+            "odoo-empty fixture not available (run: git submodule update --init)"
+        )
     return FIXTURE_DIR
 
 
@@ -109,11 +112,14 @@ def setup_fixture_env():
         result = runner.invoke(
             app,
             ["setup", "--community"],
-            input="n\n",  # Say no to Docker setup prompt
+            # Odoo 19.0, Python 3.12, no save to .env, no docker
+            input="19.0\n3.12\nn\nn\n",
         )
 
         if result.exit_code != 0:
-            pytest.skip(f"Setup failed: {result.output}")
+            pytest.fail(
+                f"Setup failed (exit code {result.exit_code}):\n{result.output}"
+            )
 
         yield FIXTURE_DIR
 
@@ -156,7 +162,10 @@ class TestRunCommandsWithoutPrerequisites:
     def test_run_fails_without_venv(self, in_clean_fixture: Path):
         result = runner.invoke(app, ["run"])
         assert result.exit_code != 0
-        assert "Virtual environment not found" in result.output or "not found" in result.output.lower()
+        assert (
+            "Virtual environment not found" in result.output
+            or "not found" in result.output.lower()
+        )
 
     def test_shell_fails_without_venv(self, in_clean_fixture: Path):
         result = runner.invoke(app, ["shell", "testdb"])

{odoo_dev-0.2.3 → odoo_dev-0.2.4}/tests/test_setup.py

@@ -1,8 +1,9 @@
 """Tests for setup command utilities."""
 
 from pathlib import Path
+from unittest.mock import patch
 
-from odoo_dev.commands.setup import _update_env_file
+from odoo_dev.commands.setup import _clean, _update_env_file
 
 
 class TestUpdateEnvFile:
@@ -51,3 +52,33 @@ class TestUpdateEnvFile:
         assert "NEW_VAR=new" in content
         assert "VAR=value" in content
         # Comments are not preserved (simplification)
+
+
+class TestClean:
+    """Test _clean function."""
+
+    def test_clean_removes_directories(self, tmp_path: Path):
+        """Verify _clean removes expected directories."""
+        # Create directories that _clean should remove
+        for name in ["odoo", "enterprise", "design-themes", "industry", ".venv"]:
+            (tmp_path / name).mkdir()
+
+        # Mock load_config to return our tmp_path as project_dir
+        mock_cfg = type("Config", (), {"project_dir": tmp_path})()
+        with patch("odoo_dev.commands.setup.load_config", return_value=mock_cfg):
+            _clean()
+
+        # Verify all directories were removed
+        for name in ["odoo", "enterprise", "design-themes", "industry", ".venv"]:
+            assert not (tmp_path / name).exists(), f"{name} should be removed"
+
+    def test_clean_ignores_missing_directories(self, tmp_path: Path):
+        """Verify _clean doesn't fail if directories don't exist."""
+        # Only create some directories
+        (tmp_path / "odoo").mkdir()
+
+        mock_cfg = type("Config", (), {"project_dir": tmp_path})()
+        with patch("odoo_dev.commands.setup.load_config", return_value=mock_cfg):
+            _clean()  # Should not raise
+
+        assert not (tmp_path / "odoo").exists()

{odoo_dev-0.2.3 → odoo_dev-0.2.4}/uv.lock

@@ -55,7 +55,7 @@ wheels = [
 
 [[package]]
 name = "odoo-dev"
-version = "0.2.0"
+version = "0.2.4"
 source = { editable = "." }
 dependencies = [
     { name = "rich" },