Merge branch 'main' into fix/export-facets

Author: RobPasMue

doc/changelog.d/2260.test.md

@@ -0,0 +1 @@
+Adding test coverage for nurbs, design, and geometry commands

doc/changelog.d/2293.maintenance.md

@@ -0,0 +1 @@
+Prevent invalid kwargs from being passed to launcher methods

src/ansys/geometry/core/connection/launcher.py

@@ -35,7 +35,11 @@
 )
 from ansys.geometry.core.connection.product_instance import prepare_and_start_backend
 from ansys.geometry.core.logger import LOG
-from ansys.geometry.core.misc.checks import check_type, deprecated_argument
+from ansys.geometry.core.misc.checks import (
+    check_type,
+    deprecated_argument,
+    kwargs_passed_not_accepted,
+)
 
 try:
     import ansys.platform.instancemanagement as pypim
@@ -233,6 +237,7 @@ def _launch_with_automatic_detection(**kwargs: dict | None) -> "Modeler":
     raise NotImplementedError("Geometry service cannot be initialized.")
 
 
+@kwargs_passed_not_accepted
 def launch_remote_modeler(
     platform: str = "windows",
     version: str | None = None,
@@ -287,6 +292,7 @@ def launch_remote_modeler(
     )
 
 
+@kwargs_passed_not_accepted
 def launch_docker_modeler(
     port: int = pygeom_defaults.DEFAULT_PORT,
     connect_to_existing_service: bool = True,
@@ -362,6 +368,7 @@ def launch_docker_modeler(
     )
 
 
+@kwargs_passed_not_accepted
 def launch_modeler_with_discovery_and_pimlight(
     version: str | None = None,
     client_log_level: int = logging.INFO,
@@ -404,6 +411,7 @@ def launch_modeler_with_discovery_and_pimlight(
     )
 
 
+@kwargs_passed_not_accepted
 def launch_modeler_with_geometry_service_and_pimlight(
     version: str | None = None,
     client_log_level: int = logging.INFO,
@@ -446,6 +454,7 @@ def launch_modeler_with_geometry_service_and_pimlight(
     )
 
 
+@kwargs_passed_not_accepted
 def launch_modeler_with_spaceclaim_and_pimlight(
     version: str | None = None,
     client_log_level: int = logging.INFO,
@@ -488,6 +497,7 @@ def launch_modeler_with_spaceclaim_and_pimlight(
     )
 
 
+@kwargs_passed_not_accepted
 @deprecated_argument("product_version", "version", version="0.10.8", remove="0.13.0")
 def launch_modeler_with_geometry_service(
     version: str | int | None = None,
@@ -587,13 +597,6 @@ def launch_modeler_with_geometry_service(
         timeout=300,
         server_log_level=0)
     """
-    # if api_version is passed, throw a warning saying that it is not used
-    if "api_version" in kwargs:
-        LOG.warning(
-            "The 'api_version' parameter is not used in 'launch_modeler_with_geometry_service'. "
-            "Please remove it from the arguments."
-        )
-
     # If we are in a Windows environment, we are going to write down the server
     # logs in the %PUBLIC%/Documents/Ansys/GeometryService folder.
     if os.name == "nt" and server_logs_folder is None:
@@ -617,6 +620,7 @@ def launch_modeler_with_geometry_service(
     )
 
 
+@kwargs_passed_not_accepted
 @deprecated_argument("product_version", "version", version="0.10.8", remove="0.13.0")
 def launch_modeler_with_discovery(
     version: str | int | None = None,
@@ -720,13 +724,6 @@ def launch_modeler_with_discovery(
         timeout=300,
         server_log_level=0)
     """
-    for unused_var in ["server_logs_folder", "logs_folder"]:
-        if unused_var in kwargs:
-            LOG.warning(
-                f"The '{unused_var}' parameter is not used in 'launch_modeler_with_discovery'. "
-                "Please remove it from the arguments."
-            )
-
     return prepare_and_start_backend(
         BackendType.DISCOVERY,
         version=version,
@@ -744,6 +741,7 @@ def launch_modeler_with_discovery(
     )
 
 
+@kwargs_passed_not_accepted
 @deprecated_argument("product_version", "version", version="0.10.8", remove="0.13.0")
 def launch_modeler_with_spaceclaim(
     version: str | int | None = None,
@@ -847,13 +845,6 @@ def launch_modeler_with_spaceclaim(
         timeout=300,
         server_log_level=0)
     """
-    for unused_var in ["server_logs_folder", "logs_folder"]:
-        if unused_var in kwargs:
-            LOG.warning(
-                f"The '{unused_var}' parameter is not used in 'launch_modeler_with_spaceclaim'. "
-                "Please remove it from the arguments."
-            )
-
     return prepare_and_start_backend(
         BackendType.SPACECLAIM,
         version=version,
@@ -871,6 +862,7 @@ def launch_modeler_with_spaceclaim(
     )
 
 
+@kwargs_passed_not_accepted
 @deprecated_argument("product_version", "version", version="0.10.8", remove="0.13.0")
 def launch_modeler_with_core_service(
     version: str | int | None = None,
@@ -970,13 +962,6 @@ def launch_modeler_with_core_service(
         timeout=300,
         server_log_level=0)
     """
-    # if api_version is passed, throw a warning saying that it is not used
-    if "api_version" in kwargs:
-        LOG.warning(
-            "The 'api_version' parameter is not used in 'launch_modeler_with_core_service'. "
-            "Please remove it from the arguments."
-        )
-
     # If we are in a Windows environment, we are going to write down the server
     # logs in the %PUBLIC%/Documents/Ansys/GeometryService folder.
     if os.name == "nt" and server_logs_folder is None:

src/ansys/geometry/core/misc/__init__.py

@@ -45,6 +45,7 @@
     deprecated_argument,
     deprecated_method,
     graphics_required,
+    kwargs_passed_not_accepted,
     min_backend_version,
     run_if_graphics_required,
 )

src/ansys/geometry/core/misc/checks.py

@@ -478,3 +478,79 @@ def wrapper(*args, **kwargs):
         return method(*args, **kwargs)
 
     return wrapper
+
+
+def kwargs_passed_not_accepted(method):
+    """Check that no unexpected kwargs are passed to the method.
+
+    This decorator will raise a TypeError if any keyword arguments are passed
+    to the decorated method that don't correspond to the method's parameters.
+    If the method has ``**kwargs`` in its signature, this decorator will raise an
+    error for any kwargs passed to it (that are not explicitly accepted).
+
+    Parameters
+    ----------
+    method : callable
+        The method to decorate.
+
+    Returns
+    -------
+    callable
+        Decorated method that raises TypeError if unexpected kwargs are passed.
+
+    Raises
+    ------
+    TypeError
+        If unexpected keyword arguments are passed to the decorated method.
+
+    Examples
+    --------
+    >>> @kwargs_passed_not_accepted
+    ... def my_method(arg1, arg2):
+    ...     return arg1 + arg2
+    >>> my_method(1, 2)  # Works fine
+    3
+    >>> my_method(arg1=1, arg2=2)  # Works fine
+    3
+    >>> my_method(1, 2, invalid_arg=3)  # Raises TypeError
+    TypeError: The following keyword arguments are not accepted
+    in the method 'my_method': invalid_arg.
+    >>> @kwargs_passed_not_accepted
+    ... def my_method_with_kwargs(arg1, arg2, **kwargs):
+    ...     return arg1 + arg2
+    >>> my_method_with_kwargs(1, 2, invalid_arg=3)  # Raises TypeError
+    TypeError: The following keyword arguments are not accepted
+    in the method 'my_method_with_kwargs': invalid_arg.
+    """
+    import inspect
+
+    def wrapper(*args, **kwargs):
+        # Get the method signature
+        sig = inspect.signature(method)
+
+        # Check if method has **kwargs parameter
+        has_var_keyword = any(
+            param.kind == inspect.Parameter.VAR_KEYWORD for param in sig.parameters.values()
+        )
+
+        # If method has **kwargs and kwargs are passed...
+        if has_var_keyword and len(kwargs) > 0:
+            # Retrieve a list of all parameter names excluding **kwargs
+            param_names = {
+                name
+                for name, param in sig.parameters.items()
+                if param.kind != inspect.Parameter.VAR_KEYWORD
+            }
+
+            # Identify unexpected kwargs
+            unexpected_kwargs = [key for key in kwargs.keys() if key not in param_names]
+
+            if unexpected_kwargs:
+                raise TypeError(
+                    "The following keyword arguments are not accepted in the"
+                    + f" method '{method.__name__}': {', '.join(unexpected_kwargs)}."
+                )
+
+        return method(*args, **kwargs)
+
+    return wrapper

tests/_incompatible_tests.yml

@@ -58,6 +58,9 @@ backends:
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_combine_example
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_pull_example
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_intersect_example
+      - tests/integration/test_design.py::test_legacy_export_download
+      - tests/integration/test_design.py::test_updating_design_from_tracker
+      - tests/integration/test_design.py::test_failure_for_export
       # Opening large files through streaming is only available from 25.2 onwards
       - tests/integration/test_design.py::test_modeler_open_files
       # Named selections elements are only consistent from 25.2 onwards
@@ -83,6 +86,8 @@ backends:
       - tests/integration/test_design_import.py::test_design_import_solid_edge2025
       - tests/integration/test_design_import.py::test_design_import_nx2412
       - tests/integration/test_design_import.py::test_design_import_inventor2026
+      - tests/integration/test_design.py::test_legacy_export_download
+      - tests/integration/test_design.py::test_failure_for_export
       # Bug fix included from 25.2 onwards
       - tests/integration/test_issues.py::test_issue_1192_temp_body_on_empty_intersect
       - tests/integration/test_issues.py::test_issue_2251_double_import_crash
@@ -169,6 +174,9 @@ backends:
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_combine_example
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_pull_example
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_intersect_example
+      - tests/integration/test_design.py::test_legacy_export_download
+      - tests/integration/test_design.py::test_updating_design_from_tracker
+      - tests/integration/test_design.py::test_failure_for_export
       # Opening large files through streaming is only available from 25.2 onwards
       - tests/integration/test_design.py::test_modeler_open_files
       # Named selections elements are only consistent from 25.2 onwards
@@ -203,6 +211,8 @@ backends:
       - tests/integration/test_design_import.py::test_design_import_solid_edge2025
       - tests/integration/test_design_import.py::test_design_import_nx2412
       - tests/integration/test_design_import.py::test_design_import_inventor2026
+      - tests/integration/test_design.py::test_legacy_export_download
+      - tests/integration/test_design.py::test_failure_for_export
       # Insert file naming convention changed in 26.1
       - tests/integration/test_design_import.py::test_design_insert
       # Bug fix included from 25.2 onwards
@@ -230,6 +240,9 @@ backends:
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_combine_example
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_pull_example
       - tests/integration/test_spaceclaim_tutorial_examples.py::test_intersect_example
+      - tests/integration/test_design.py::test_legacy_export_download
+      - tests/integration/test_design.py::test_updating_design_from_tracker
+      - tests/integration/test_design.py::test_failure_for_export
       # Opening large files through streaming is only available from 25.2 onwards
       - tests/integration/test_design.py::test_modeler_open_files
       # Named selections elements are only consistent from 25.2 onwards
@@ -264,6 +277,8 @@ backends:
       - tests/integration/test_design_import.py::test_design_import_jt
       - tests/integration/test_design_import.py::test_design_import_solid_edge2025
       - tests/integration/test_design_import.py::test_design_import_nx2412
+      - tests/integration/test_design.py::test_legacy_export_download
+      - tests/integration/test_design.py::test_failure_for_export
       # Insert file naming convention changed in 26.1
       - tests/integration/test_design_import.py::test_design_insert
       # Importing named selections from design was not available prior to 26.1
@@ -315,6 +330,8 @@ backends:
       - tests/integration/test_design_export.py::test_import_export_open_file_design[PARASOLID_TEXT-x_t-rci_std.x_t-2-1]
       - tests/integration/test_design_export.py::test_import_export_open_file_design[SCDOCX-scdocx-reactorWNS.scdocx-1-3]
       - tests/integration/test_design_import.py::test_design_insert_id_bug
+      - tests/integration/test_design.py::test_legacy_export_download
+      - tests/integration/test_design.py::test_failure_for_export
       # Insert file naming convention changed in 26.1
       - tests/integration/test_design_import.py::test_design_insert
       # Importing named selections from design was not available prior to 26.1