More robusts tests

modules/pymol/commanding.py

@@ -619,12 +619,13 @@ def inner(*args, **kwargs):
                     elif funcs[arg] == List[float]:
                         funcs[arg] = _parse_list_float
                     else:
-                        raise NotImplemented(f"{funcs[arg]} argument is not supported.")
+                        # Assumi it's a literal supported type
+                        pass
                     # Convert the argument to the correct type
                     kwargs[arg] = funcs[arg](kwargs[arg])
                 return function(**kwargs)
 
-            # It was called by Python, so pass the arguments as is
+            # It was called from Python, so pass the arguments as is
             else:
                 return function(*args, **kwargs)
 

testing/tests/api/commanding.py

@@ -190,13 +190,15 @@ def func(a: int, b: Path):
     cmd.do('func 1, /tmp')
 
 
-def test_declare_command_default():
+def test_declare_command_default(capsys):
     from pymol.commanding import Selection
     @cmd.declare_command
     def func(a: Selection = "sele"):
         assert a == "a"
     func("a")
     cmd.do("func a")
+    out, err = capsys.readouterr()
+    assert out == ''
 
 def test_declare_command_docstring():
     @cmd.declare_command
@@ -214,12 +216,14 @@ def func():
     assert func.__doc__ == "docstring\nTest:\n    --foo"
 
 
-def test_declare_command_type_return():
+def test_declare_command_type_return(capsys):
     @cmd.declare_command
     def func() -> int:
         return 1
 
     assert func() == 1
+    out, err = capsys.readouterr()
+    assert out == ''
 
     @cmd.declare_command
     def func():