revitdevtool-pytest 0.0.2__tar.gz → 0.0.3__tar.gz

{revitdevtool_pytest-0.0.2 → revitdevtool_pytest-0.0.3}/CHANGELOG.md

@@ -2,6 +2,18 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.0.3] - 2026-04-10
+
+### Fixed
+
+- Fixed Revit auto-launch dialog handling by starting the startup dialog resolver immediately after spawning Revit, before waiting for the named pipe.
+- Fixed startup dialog button selection to prefer safe allow-list actions such as `Always Load` and `Load Once`.
+- Prevented accidental clicks on destructive dialog actions such as `Do Not Load`, `Cancel`, and `No`.
+
+### Notes
+
+- This release focuses on reliable auto-launch behavior when Revit shows unsigned add-in security dialogs during startup.
+
 ## [0.0.2] - 2026-04-10
 
 ### Fixed

{revitdevtool_pytest-0.0.2 → revitdevtool_pytest-0.0.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: revitdevtool_pytest
-Version: 0.0.2
+Version: 0.0.3
 Summary: pytest plugin for testing Revit API code via RevitDevTool Named Pipe bridge
 Project-URL: Homepage, https://github.com/trgiangv/RevitDevTool.PyTest
 Project-URL: Repository, https://github.com/trgiangv/RevitDevTool.PyTest

{revitdevtool_pytest-0.0.2 → revitdevtool_pytest-0.0.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "revitdevtool_pytest"
-version = "0.0.2"
+version = "0.0.3"
 description = "pytest plugin for testing Revit API code via RevitDevTool Named Pipe bridge"
 readme = "README.md"
 license = "MIT"

{revitdevtool_pytest-0.0.2 → revitdevtool_pytest-0.0.3}/src/revitdevtool_pytest/dialog_resolver.py

@@ -59,7 +59,10 @@ class DialogResolverOptions:
         "Autodesk", "Revit", "Load", "Security", "Warning", "Add-in", "Addin",
     ])
     preferred_button_keywords: list[str] = field(default_factory=lambda: [
-        "Always Load", "Load", "OK", "Yes", "Accept", "Continue", "Close",
+        "Always Load", "Load Once", "Load", "OK", "Yes", "Accept", "Continue", "Close",
+    ])
+    blocked_button_keywords: list[str] = field(default_factory=lambda: [
+        "Do Not Load", "Cancel", "No",
     ])
 
 
@@ -106,17 +109,25 @@ class StartupDialogResolver:
         return any(kw.lower() in lower for kw in self._opts.dialog_title_keywords)
 
     def _find_button(self, parent: int) -> int | None:
-        result: list[int] = []
+        result: list[tuple[int, int]] = []
 
         @_EnumWindowsProc
         def _cb(child: int, _: int) -> bool:
-            if _is_preferred_button(child, self._opts.preferred_button_keywords):
-                result.append(child)
-                return False
+            score = _get_button_score(
+                child,
+                self._opts.preferred_button_keywords,
+                self._opts.blocked_button_keywords,
+            )
+            if score is not None:
+                result.append((score, child))
             return True
 
         _user32.EnumChildWindows(parent, _cb, 0)
-        return result[0] if result else None
+        if not result:
+            return None
+
+        result.sort(key=lambda item: item[0])
+        return result[0][1]
 
     def _enum_dialog_windows(self) -> list[int]:
         windows: list[int] = []
@@ -131,14 +142,30 @@ class StartupDialogResolver:
         return windows
 
 
-def _is_preferred_button(hwnd: int, keywords: list[str]) -> bool:
+def _get_button_score(hwnd: int, keywords: list[str], blocked_keywords: list[str]) -> int | None:
     if _get_class_name(hwnd).lower() != _BUTTON_CLASS:
-        return False
+        return None
+
     text = _get_window_text(hwnd)
     if not text:
-        return False
+        return None
+
     text_lower = text.lower()
-    return any(kw.lower() in text_lower for kw in keywords)
+
+    if any(kw.lower() in text_lower for kw in blocked_keywords):
+        return None
+
+    for index, keyword in enumerate(keywords):
+        keyword_lower = keyword.lower()
+        if text_lower == keyword_lower:
+            return index
+
+    for index, keyword in enumerate(keywords):
+        keyword_lower = keyword.lower()
+        if keyword_lower in text_lower:
+            return index + len(keywords)
+
+    return None
 
 
 def _is_target_dialog(hwnd: int, target_pid: int) -> bool:

{revitdevtool_pytest-0.0.2 → revitdevtool_pytest-0.0.3}/src/revitdevtool_pytest/discovery.py

@@ -87,6 +87,19 @@ def launch_revit(
     return wait_for_revit_pipe(version, timeout_s=wait_timeout_s)
 
 
+def start_revit(version: int) -> int:
+    """Start Revit with ``/nosplash`` and return the spawned process id."""
+    exe_path = find_revit_path(version)
+    if exe_path is None:
+        raise FileNotFoundError(f"Revit {version} installation not found.")
+
+    process = subprocess.Popen(  # noqa: S603
+        [exe_path, REVIT_NOSPLASH],
+        creationflags=subprocess.DETACHED_PROCESS,
+    )
+    return int(process.pid)
+
+
 def wait_for_revit_pipe(
     version: int | None = None,
     timeout_s: float = DEFAULT_LAUNCH_TIMEOUT_S,

{revitdevtool_pytest-0.0.2 → revitdevtool_pytest-0.0.3}/src/revitdevtool_pytest/plugin.py

@@ -22,7 +22,7 @@ from .constants import (
     OPT_VERSION,
     PLUGIN_NAME,
 )
-from .discovery import find_revit_path, find_revit_pipes, launch_revit, select_instance
+from .discovery import find_revit_path, find_revit_pipes, select_instance, start_revit, wait_for_revit_pipe
 from .models import TestOutcome
 from .serializer import serialize_test
 
@@ -152,22 +152,25 @@ def _auto_launch(version: int, config: pytest.Config) -> RevitInstance:
         )
 
     print(f"{PLUGIN_NAME}: Launching Revit {version}...")
-    instance = launch_revit(version, wait_timeout_s=timeout)
 
-    if instance is None:
-        pytest.exit(
-            f"{PLUGIN_NAME}: Revit {version} launched but Named Pipe did not appear within {timeout}s.",
-            returncode=EXIT_CODE_CONFIG_ERROR,
-        )
+    process_id = start_revit(version)
 
     try:
         from .dialog_resolver import StartupDialogResolver as _Resolver
 
-        _dialog_resolver = _Resolver(instance.process_id)
+        _dialog_resolver = _Resolver(process_id)
         _dialog_resolver.start()
     except ImportError:
         pass
 
+    instance = wait_for_revit_pipe(version, timeout_s=timeout)
+
+    if instance is None:
+        pytest.exit(
+            f"{PLUGIN_NAME}: Revit {version} launched but Named Pipe did not appear within {timeout}s.",
+            returncode=EXIT_CODE_CONFIG_ERROR,
+        )
+
     print(f"{PLUGIN_NAME}: Connected to Revit {instance.version} (pid={instance.process_id})")
     return instance