add more tests and raise exception if double quotes in args

Author: klmcadams

src/ansys/mechanical/core/run.py

@@ -115,6 +115,10 @@ def _cli_impl(
     if not input_script and script_args:
         raise Exception("Cannot add script arguments without an input script.")
 
+    if script_args:
+        if '"' in script_args:
+            raise Exception("Cannot have double quotes in the arguments.")
+
     # If the input_script and port are missing in batch mode, raise an exception
     if (not graphical) and (input_script is None) and (not port):
         raise Exception("An input script, -i, or port, --port, are required in batch mode.")
@@ -148,10 +152,7 @@ def _cli_impl(
 
     if script_args:
         args.append("-ScriptArgs")
-        if '"' in script_args:
-            args.append(f"'{script_args}'")
-        else:
-            args.append(f'"{script_args}"')
+        args.append(f'"{script_args}"')
 
     if (not graphical) and input_script:
         exit = True

tests/test_cli.py

@@ -127,6 +127,33 @@ def test_cli_scriptargs(disable_cli):
     assert "foo.py" in args
 
 
+@pytest.mark.cli
+def test_cli_scriptargs_singlequote(disable_cli):
+    args, _ = _cli_impl(
+        exe="AnsysWBU.exe",
+        version=241,
+        input_script="foo.py",
+        script_args="arg1,arg2,'arg3'",
+        graphical=True,
+    )
+    assert "-ScriptArgs" in args
+    assert "\"arg1,arg2,'arg3'\"" in args
+    assert "-script" in args
+    assert "foo.py" in args
+
+
+@pytest.mark.cli
+def test_cli_scriptargs_doublequote(disable_cli):
+    with pytest.raises(Exception):
+        _cli_impl(
+            exe="AnsysWBU.exe",
+            version=241,
+            input_script="foo.py",
+            script_args='arg1,"arg2",arg3',
+            graphical=True,
+        )
+
+
 @pytest.mark.cli
 def test_cli_features(disable_cli):
     with pytest.warns(UserWarning):