robotframework-pabot 3.1.0__tar.gz → 4.0.4__tar.gz

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: robotframework-pabot
-Version: 3.1.0
+Version: 4.0.4
 Summary: Parallel test runner for Robot Framework
 Home-page: https://pabot.org
 Download-URL: https://pypi.python.org/pypi/robotframework-pabot

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/README.md

@@ -55,14 +55,14 @@ There are several ways you can help in improving this tool:
 ## Command-line options
 
     pabot [--verbose|--testlevelsplit|--command .. --end-command|
-           --processes num|--pabotlib|--pabotlibhost host|--pabotlibport port|
+           --processes num|--no-pabotlib|--pabotlibhost host|--pabotlibport port|
            --processtimeout num|
            --shard i/n|
            --artifacts extensions|--artifactsinsubfolders|
            --resourcefile file|--argumentfile[num] file|--suitesfrom file] 
           [robot options] [path ...]
 
-Supports all [Robot Framework command line options](https://robotframework.org/robotframework/latest/RobotFrameworkUserGuide.html#all-command-line-options) and also following options (these must be before RF options):
+Supports all [Robot Framework command line options](https://robotframework.org/robotframework/latest/RobotFrameworkUserGuide.html#all-command-line-options) and also following pabot options:
 
 --verbose     
   more output from the parallel execution
@@ -83,13 +83,16 @@ Supports all [Robot Framework command line options](https://robotframework.org/r
   Special option "all" will use as many processes as there are
   executable suites or tests.
 
---pabotlib          
-  Start PabotLib remote server. This enables locking and resource distribution between parallel test executions.
+PabotLib remote server is started by default to enable locking and resource distribution 
+between parallel test executions.
+
+--no-pabotlib
+  Disable the PabotLib remote server if you don't need locking or resource distribution features.
 
 --pabotlibhost   [HOSTNAME]          
-  Host name of the PabotLib remote server (default is 127.0.0.1)
-  If used with --pabotlib option, will change the host listen address of the created remote server (see https://github.com/robotframework/PythonRemoteServer)
-  If used without the --pabotlib option, will connect to already running instance of the PabotLib remote server in the given host. The remote server can be also started and executed separately from pabot instances:
+  Connect to an already running instance of the PabotLib remote server at the given host
+  (disables the local PabotLib server start). 
+  The remote server can be also started and executed separately from pabot instances:
   
       python -m pabot.pabotlib <path_to_resourcefile> <host> <port>
       python -m pabot.pabotlib resource.txt 192.168.1.123 8271
@@ -142,8 +145,9 @@ Example usages:
      pabot --command java -jar robotframework.jar --end-command --include SMOKE tests
      pabot --processes 10 tests     
      pabot --pabotlibhost 192.168.1.123 --pabotlibport 8271 --processes 10 tests
-     pabot --pabotlib --pabotlibhost 192.168.1.111 --pabotlibport 8272 --processes 10 tests
      pabot --artifacts png,mp4,txt --artifactsinsubfolders directory_to_tests
+     # To disable PabotLib:
+     pabot --no-pabotlib tests
 
 ### PabotLib
 

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/pabot/SharedLibrary.py

@@ -11,7 +11,6 @@ from .robotremoteserver import RemoteLibraryFactory
 
 
 class SharedLibrary(object):
-
     ROBOT_LIBRARY_SCOPE = "GLOBAL"
 
     def __init__(self, name, args=None):
@@ -26,8 +25,10 @@ class SharedLibrary(object):
                 "Not currently running pabot. Importing library for this process."
             )
             self._lib = RemoteLibraryFactory(
-                TestLibrary.from_name(name, args=args, variables=None, create_keywords=True).instance 
-                if ROBOT_VERSION >= "7.0" 
+                TestLibrary.from_name(
+                    name, args=args, variables=None, create_keywords=True
+                ).instance
+                if ROBOT_VERSION >= "7.0"
                 else TestLibrary(name, args=args).get_instance()
             )
             return
@@ -36,7 +37,9 @@ class SharedLibrary(object):
         remotelib = Remote(uri) if uri else None
         if remotelib:
             try:
-                port = remotelib.run_keyword("import_shared_library", [name], {"args": args})
+                port = remotelib.run_keyword(
+                    "import_shared_library", [name], {"args": args}
+                )
             except RuntimeError:
                 logger.error("No connection - is pabot called with --pabotlib option?")
                 raise

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/pabot/__init__.py

@@ -1,4 +1,5 @@
 from __future__ import absolute_import
 
 from .pabotlib import PabotLib
-__version__ = "3.1.0"
+
+__version__ = "4.0.4"

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/pabot/arguments.py

@@ -80,7 +80,7 @@ def _parse_pabot_args(args):  # type: (List[str]) -> Tuple[List[str], Dict[str,
         "verbose": False,
         "help": False,
         "testlevelsplit": False,
-        "pabotlib": False,
+        "pabotlib": True,
         "pabotlibhost": "127.0.0.1",
         "pabotlibport": 8270,
         "processes": _processes_count(),
@@ -91,108 +91,108 @@ def _parse_pabot_args(args):  # type: (List[str]) -> Tuple[List[str], Dict[str,
         "shardcount": 1,
         "chunk": False,
     }
+    # Explicitly define argument types for validation
+    flag_args = {
+        "verbose",
+        "help",
+        "testlevelsplit",
+        "pabotlib",
+        "artifactsinsubfolders",
+        "chunk",
+    }
+    value_args = {
+        "hive": str,
+        "processes": lambda x: int(x) if x != "all" else None,
+        "resourcefile": str,
+        "pabotlibhost": str,
+        "pabotlibport": int,
+        "processtimeout": int,
+        "ordering": _parse_ordering,
+        "suitesfrom": str,
+        "artifacts": lambda x: x.split(","),
+        "shard": _parse_shard,
+    }
+
     argumentfiles = []
-    while args and (
-        args[0]
-        in [
-            "--" + param
-            for param in [
-                "hive",
-                "command",
-                "processes",
-                "verbose",
-                "resourcefile",
-                "testlevelsplit",
-                "pabotlib",
-                "pabotlibhost",
-                "pabotlibport",
-                "processtimeout",
-                "ordering",
-                "suitesfrom",
-                "artifacts",
-                "artifactsinsubfolders",
-                "help",
-                "shard",
-                "chunk",
-            ]
-        ]
-        or ARGSMATCHER.match(args[0])
-    ):
-        if args[0] == "--hive":
-            pabot_args["hive"] = args[1]
-            args = args[2:]
-            continue
-        if args[0] == "--command":
-            end_index = args.index("--end-command")
-            pabot_args["command"] = args[1:end_index]
-            args = args[end_index + 1 :]
-            continue
-        if args[0] == "--processes":
-            pabot_args["processes"] = int(args[1]) if args[1] != 'all' else None
-            args = args[2:]
-            continue
-        if args[0] == "--verbose":
-            pabot_args["verbose"] = True
-            args = args[1:]
-            continue
-        if args[0] == "--chunk":
-            pabot_args["chunk"] = True
-            args = args[1:]
-            continue
-        if args[0] == "--resourcefile":
-            pabot_args["resourcefile"] = args[1]
-            args = args[2:]
-            continue
-        if args[0] == "--pabotlib":
-            pabot_args["pabotlib"] = True
-            args = args[1:]
-            continue
-        if args[0] == "--ordering":
-            pabot_args["ordering"] = _parse_ordering(args[1])
-            args = args[2:]
+    remaining_args = []
+    i = 0
+
+    # Track conflicting options during parsing
+    saw_pabotlib_flag = False
+    saw_no_pabotlib = False
+
+    while i < len(args):
+        arg = args[i]
+        if not arg.startswith("--"):
+            remaining_args.append(arg)
+            i += 1
             continue
-        if args[0] == "--testlevelsplit":
-            pabot_args["testlevelsplit"] = True
+
+        arg_name = arg[2:]  # Strip '--'
+
+        if arg_name == "no-pabotlib":
+            saw_no_pabotlib = True
+            pabot_args["pabotlib"] = False  # Just set the main flag
             args = args[1:]
             continue
-        if args[0] == "--pabotlibhost":
-            pabot_args["pabotlibhost"] = args[1]
-            args = args[2:]
-            continue
-        if args[0] == "--pabotlibport":
-            pabot_args["pabotlibport"] = int(args[1])
-            args = args[2:]
-            continue
-        if args[0] == "--processtimeout":
-            pabot_args["processtimeout"] = int(args[1])
-            args = args[2:]
-            continue
-        if args[0] == "--suitesfrom":
-            pabot_args["suitesfrom"] = args[1]
-            args = args[2:]
-            continue
-        if args[0] == "--artifacts":
-            pabot_args["artifacts"] = args[1].split(",")
-            args = args[2:]
-            continue
-        if args[0] == "--artifactsinsubfolders":
-            pabot_args["artifactsinsubfolders"] = True
+        if arg_name == "pabotlib":
+            saw_pabotlib_flag = True
             args = args[1:]
             continue
-        if args[0] == "--shard":
-            pabot_args["shardindex"], pabot_args["shardcount"] = _parse_shard(args[1])
-            args = args[2:]
-            continue
-        match = ARGSMATCHER.match(args[0])
+
+        # Special case for command
+        if arg_name == "command":
+            try:
+                end_index = args.index("--end-command", i)
+                pabot_args["command"] = args[i + 1 : end_index]
+                i = end_index + 1
+                continue
+            except ValueError:
+                raise DataError("--command requires matching --end-command")
+
+        # Handle flag arguments
+        if arg_name in flag_args:
+            pabot_args[arg_name] = True
+            i += 1
+            continue
+
+        # Handle value arguments
+        if arg_name in value_args:
+            if i + 1 >= len(args):
+                raise DataError(f"--{arg_name} requires a value")
+            try:
+                value = value_args[arg_name](args[i + 1])
+                if arg_name == "shard":
+                    pabot_args["shardindex"], pabot_args["shardcount"] = value
+                elif arg_name == "pabotlibhost":
+                    pabot_args["pabotlib"] = False
+                    pabot_args[arg_name] = value
+                else:
+                    pabot_args[arg_name] = value
+                i += 2
+                continue
+            except (ValueError, TypeError) as e:
+                raise DataError(f"Invalid value for --{arg_name}: {args[i + 1]}")
+
+        # Handle argument files
+        match = ARGSMATCHER.match(arg)
         if match:
-            argumentfiles += [(match.group(1), args[1])]
-            args = args[2:]
+            if i + 1 >= len(args):
+                raise DataError(f"{arg} requires a value")
+            argumentfiles.append((match.group(1), args[i + 1]))
+            i += 2
             continue
-        if args and args[0] == "--help":
-            pabot_args["help"] = True
-            args = args[1:]
+
+        # If we get here, it's a non-pabot argument
+        remaining_args.append(arg)
+        i += 1
+
+    if saw_pabotlib_flag and saw_no_pabotlib:
+        raise DataError("Cannot use both --pabotlib and --no-pabotlib options together")
+
     pabot_args["argumentfiles"] = argumentfiles
-    return args, pabot_args
+
+    return remaining_args, pabot_args
 
 
 def _parse_ordering(filename):  # type: (str) -> List[ExecutionItem]

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/pabot/execution_items.py

@@ -8,7 +8,6 @@ from robot.utils import PY2, is_unicode
 
 @total_ordering
 class ExecutionItem(object):
-
     isWait = False
     type = None  # type: str
     name = None  # type: str
@@ -51,7 +50,6 @@ class ExecutionItem(object):
 
 
 class HivedItem(ExecutionItem):
-
     type = "hived"
 
     def __init__(self, item, hive):
@@ -67,7 +65,6 @@ class HivedItem(ExecutionItem):
 
 
 class GroupItem(ExecutionItem):
-
     type = "group"
 
     def __init__(self):
@@ -128,7 +125,6 @@ class RunnableItem(ExecutionItem):
 
 
 class SuiteItem(RunnableItem):
-
     type = "suite"
 
     def __init__(self, name, tests=None, suites=None, dynamictests=None):
@@ -163,9 +159,9 @@ class SuiteItem(RunnableItem):
             return False
         if self.name == other.name:
             return True
-        if other.name.endswith('.'+self.name):
+        if other.name.endswith("." + self.name):
             return True
-        if self.name.endswith('.'+other.name):
+        if self.name.endswith("." + other.name):
             return True
         return False
 
@@ -178,7 +174,6 @@ class SuiteItem(RunnableItem):
 
 
 class TestItem(RunnableItem):
-
     type = "test"
 
     def __init__(self, name):
@@ -229,7 +224,6 @@ class DynamicSuiteItem(SuiteItem):
 
 
 class DynamicTestItem(ExecutionItem):
-
     type = "dynamictest"
 
     def __init__(self, name, suite):
@@ -258,7 +252,6 @@ class DynamicTestItem(ExecutionItem):
 
 
 class WaitItem(ExecutionItem):
-
     type = "wait"
     isWait = True
 
@@ -270,7 +263,6 @@ class WaitItem(ExecutionItem):
 
 
 class GroupStartItem(ExecutionItem):
-
     type = "group"
 
     def __init__(self):
@@ -281,7 +273,6 @@ class GroupStartItem(ExecutionItem):
 
 
 class GroupEndItem(ExecutionItem):
-
     type = "group"
 
     def __init__(self):
@@ -292,7 +283,6 @@ class GroupEndItem(ExecutionItem):
 
 
 class IncludeItem(ExecutionItem):
-
     type = "include"
 
     def __init__(self, tag):
@@ -309,7 +299,6 @@ class IncludeItem(ExecutionItem):
 
 
 class SuiteItems(ExecutionItem):
-
     type = "suite"
 
     def __init__(self, suites):

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/pabot/pabot.py

@@ -44,12 +44,20 @@ options (these must be before normal RF options):
   Indicator for a file that can contain shared variables for
   distributing resources.
 
---pabotlib
-  Start PabotLib remote server. This enables locking and resource
-  distribution between parallel test executions.
+--no-pabotlib
+  Disable the PabotLib remote server if you don't need locking or resource distribution features.
+  PabotLib remote server is started by default.
 
---pabotlibhost [HOSTNAME]
-  Host name of the PabotLib remote server (default is 127.0.0.1)
+--pabotlibhost [HOSTNAME]          
+  Connect to an already running instance of the PabotLib remote server at the given host
+  (disables the local PabotLib server start). For example, to connect to a 
+  remote PabotLib server running on another machine:
+  
+      pabot --pabotlibhost 192.168.1.123 --pabotlibport 8271 tests/
+
+  The remote PabotLib server can be started separately using:
+      python -m pabot.pabotlib <path_to_resourcefile> <host> <port>
+      python -m pabot.pabotlib resource.txt 192.168.1.123 8271
 
 --pabotlibport [PORT]
   Port number of the PabotLib remote server (default is 8270)
@@ -236,7 +244,7 @@ def execute_and_wait_with(item):
                 caller_id,
                 item.index,
                 item.execution_item.type != "test",
-                process_timeout=item.timeout
+                process_timeout=item.timeout,
             )
         outputxml_preprocessing(
             item.options, outs_dir, name, item.verbose, _make_id(), caller_id
@@ -294,7 +302,7 @@ def _try_execute_and_wait(
     caller_id,
     my_index=-1,
     show_stdout_on_failure=False,
-    process_timeout=None
+    process_timeout=None,
 ):
     # type: (List[str], str, str, bool, int, str, int, bool, Optional[int]) -> None
     plib = None
@@ -305,7 +313,15 @@ def _try_execute_and_wait(
         with open(os.path.join(outs_dir, cmd[0] + "_stdout.out"), "w") as stdout:
             with open(os.path.join(outs_dir, cmd[0] + "_stderr.out"), "w") as stderr:
                 process, (rc, elapsed) = _run(
-                    cmd, stderr, stdout, item_name, verbose, pool_id, my_index, outs_dir, process_timeout
+                    cmd,
+                    stderr,
+                    stdout,
+                    item_name,
+                    verbose,
+                    pool_id,
+                    my_index,
+                    outs_dir,
+                    process_timeout,
                 )
     except:
         _write(traceback.format_exc())
@@ -478,7 +494,17 @@ def _increase_completed(plib, my_index):
             )
 
 
-def _run(command, stderr, stdout, item_name, verbose, pool_id, item_index, outs_dir, process_timeout):
+def _run(
+    command,
+    stderr,
+    stdout,
+    item_name,
+    verbose,
+    pool_id,
+    item_index,
+    outs_dir,
+    process_timeout,
+):
     # type: (List[str], IO[Any], IO[Any], str, bool, int, int, str, Optional[int]) -> Tuple[Union[subprocess.Popen[bytes], subprocess.Popen], Tuple[int, float]]
     timestamp = datetime.datetime.now()
     cmd = " ".join(command)
@@ -487,10 +513,14 @@ def _run(command, stderr, stdout, item_name, verbose, pool_id, item_index, outs_
     # avoid hitting https://bugs.python.org/issue10394
     with POPEN_LOCK:
         my_env = os.environ.copy()
-        syslog_file = my_env.get('ROBOT_SYSLOG_FILE', None)
+        syslog_file = my_env.get("ROBOT_SYSLOG_FILE", None)
         if syslog_file:
-            my_env['ROBOT_SYSLOG_FILE'] = os.path.join(outs_dir, os.path.basename(syslog_file))
-        process = subprocess.Popen(cmd, shell=True, stderr=stderr, stdout=stdout, env=my_env)
+            my_env["ROBOT_SYSLOG_FILE"] = os.path.join(
+                outs_dir, os.path.basename(syslog_file)
+            )
+        process = subprocess.Popen(
+            cmd, shell=True, stderr=stderr, stdout=stdout, env=my_env
+        )
     if verbose:
         _write_with_id(
             process,
@@ -507,7 +537,9 @@ def _run(command, stderr, stdout, item_name, verbose, pool_id, item_index, outs_
             "EXECUTING %s" % item_name,
             timestamp=timestamp,
         )
-    return process, _wait_for_return_code(process, item_name, pool_id, item_index, process_timeout)
+    return process, _wait_for_return_code(
+        process, item_name, pool_id, item_index, process_timeout
+    )
 
 
 def _wait_for_return_code(process, item_name, pool_id, item_index, process_timeout):
@@ -522,12 +554,15 @@ def _wait_for_return_code(process, item_name, pool_id, item_index, process_timeo
         if process_timeout and elapsed / 10.0 >= process_timeout:
             process.terminate()
             process.wait()
-            rc = -1  # Set a return code indicating that the process was killed due to timeout
+            rc = (
+                -1
+            )  # Set a return code indicating that the process was killed due to timeout
             _write_with_id(
                 process,
                 pool_id,
                 item_index,
-                "Process %s killed due to exceeding the maximum timeout of %s seconds" % (item_name, process_timeout),
+                "Process %s killed due to exceeding the maximum timeout of %s seconds"
+                % (item_name, process_timeout),
             )
             break
 
@@ -544,7 +579,6 @@ def _wait_for_return_code(process, item_name, pool_id, item_index, process_timeo
     return rc, elapsed / 10.0
 
 
-
 def _read_file(file_handle):
     try:
         with open(file_handle.name, "r") as content_file:
@@ -1121,7 +1155,6 @@ def store_suite_names(hashes, suite_names):
                 for d in suite_lines
             )
     except IOError:
-
         _write(
             "[ "
             + _wrap_with(Color.YELLOW, "WARNING")
@@ -1154,12 +1187,14 @@ def generate_suite_names_with_builder(outs_dir, datasources, options):
     # Note: first argument (included_suites) is deprecated from RobotFramework 6.1
     if ROBOT_VERSION >= "6.1":
         builder = TestSuiteBuilder(
-            included_extensions=settings.extension, rpa=settings.rpa, lang=opts.get("language")
+            included_extensions=settings.extension,
+            rpa=settings.rpa,
+            lang=opts.get("language"),
         )
     else:
         builder = TestSuiteBuilder(
             settings["SuiteNames"], settings.extension, rpa=settings.rpa
-        ) 
+        )
 
     suite = builder.build(*datasources)
     settings.rpa = builder.rpa
@@ -1657,7 +1692,7 @@ class QueueItem(object):
         argfile,
         hive=None,
         processes=0,
-        timeout=None
+        timeout=None,
     ):
         # type: (List[str], str, Dict[str, object], ExecutionItem, List[str], bool, Tuple[str, Optional[str]], Optional[str], int, Optional[int]) -> None
         self.datasources = datasources
@@ -1755,7 +1790,7 @@ def _create_items(datasources, opts_for_run, outs_dir, pabot_args, suite_group):
             argfile,
             pabot_args.get("hive"),
             pabot_args["processes"],
-            pabot_args["processtimeout"]
+            pabot_args["processtimeout"],
         )
         for suite in suite_group
         for argfile in pabot_args["argumentfiles"] or [("", None)]
@@ -1774,9 +1809,7 @@ def _create_execution_items_for_dry_run(
             datasources, opts_for_run, outs_dir, pabot_args, suite_group
         )
         chunk_size = (
-            round(len(items) / processes_count)
-            if len(items) > processes_count
-            else 1
+            round(len(items) / processes_count) if len(items) > processes_count else 1
         )
         chunked_items = list(_chunk_items(items, chunk_size))
         _NUMBER_OF_ITEMS_TO_BE_EXECUTED += len(chunked_items)
@@ -1800,7 +1833,7 @@ def _chunk_items(items, chunk_size):
             base_item.verbose,
             (base_item.argfile_index, base_item.argfile),
             processes=base_item.processes,
-            timeout=base_item.timeout
+            timeout=base_item.timeout,
         )
         yield chunked_item
 
@@ -1880,7 +1913,7 @@ def _get_dynamically_created_execution_items(
             ("", None),
             pabot_args.get("hive"),
             pabot_args["processes"],
-            pabot_args["processtimeout"]
+            pabot_args["processtimeout"],
         )
         for suite in suite_group
     ]
@@ -1982,7 +2015,7 @@ def _group_suites(outs_dir, datasources, options, pabot_args):
     grouped_suites = (
         _chunked_suite_names(shard_suites, pabot_args["processes"])
         if pabot_args["chunk"]
-        else _group_by_wait(_group_by_groups(ordered_suites))
+        else _group_by_wait(_group_by_groups(shard_suites))
     )
     grouped_by_depend = _all_grouped_suites_by_depend(grouped_suites)
     return grouped_by_depend
@@ -2019,17 +2052,23 @@ def _verify_depends(suite_names):
         )
     )
     if suites_with_depends != suites_with_found_dependencies:
-        raise Exception("There are unmet dependencies using #DEPENDS")
+        raise DataError(
+            "Invalid test configuration: Some test suites have dependencies (#DEPENDS) that cannot be found."
+        )
     suites_with_circular_dependencies = list(
         filter(lambda suite: suite.depends == suite.name, suites_with_depends)
     )
     if suites_with_circular_dependencies:
-        raise Exception("There are suites with circular dependencies using #DEPENDS")
+        raise DataError(
+            "Invalid test configuration: Test suites cannot depend on themselves."
+        )
     grouped_suites = list(
         filter(lambda suite: isinstance(suite, GroupItem), suite_names)
     )
     if grouped_suites and suites_with_depends:
-        raise Exception("#DEPENDS and grouped suites are incompatible")
+        raise DataError(
+            "Invalid test configuration: Cannot use both #DEPENDS and grouped suites."
+        )
 
 
 def _group_by_depend(suite_names):
@@ -2057,7 +2096,9 @@ def _group_by_depend(suite_names):
         dependency_tree += [dependent_on_last_stage]
     flattened_dependency_tree = sum(dependency_tree, [])
     if len(flattened_dependency_tree) != len(runnable_suites):
-        raise Exception("There are circular or unmet dependencies using #DEPENDS")
+        raise DataError(
+            "Invalid test configuration: Circular or unmet dependencies detected between test suites. Please check your #DEPENDS definitions."
+        )
     return dependency_tree
 
 

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/pabot/pabotlib.py

@@ -43,7 +43,6 @@ PABOT_MIN_QUEUE_INDEX_EXECUTING_PARALLEL_VALUE = "pabot_min_queue_index_executin
 
 
 class _PabotLib(object):
-
     _TAGS_KEY = "tags"
 
     def __init__(self, resourcefile=None):  # type: (Optional[str]) -> None
@@ -155,17 +154,19 @@ class _PabotLib(object):
             content[self._TAGS_KEY] = []
         self._values[name] = content
 
-    def import_shared_library(self, name, args=None):  # type: (str, Iterable[Any]|None) -> int
+    def import_shared_library(
+        self, name, args=None
+    ):  # type: (str, Iterable[Any]|None) -> int
         if name in self._remote_libraries:
             return self._remote_libraries[name][0]
         if name in STDLIBS:
-            import_name = 'robot.libraries.' + name
+            import_name = "robot.libraries." + name
         else:
             import_name = name
-        imported = Importer('library').import_class_or_module(name_or_path=import_name, instantiate_with_args=args)
-        server = RobotRemoteServer(
-            imported, port=0, serve=False, allow_stop=True
+        imported = Importer("library").import_class_or_module(
+            name_or_path=import_name, instantiate_with_args=args
         )
+        server = RobotRemoteServer(imported, port=0, serve=False, allow_stop=True)
         server_thread = threading.Thread(target=server.serve)
         server_thread.start()
         time.sleep(1)
@@ -197,7 +198,6 @@ class _PabotLib(object):
 
 
 class PabotLib(_PabotLib):
-
     __version__ = 0.67
     ROBOT_LIBRARY_SCOPE = "GLOBAL"
     ROBOT_LISTENER_API_VERSION = 2

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/pabot/py3/messages.py

@@ -34,7 +34,6 @@ def get(sock) -> str:
 
 
 class Message:
-
     _type: Optional[int]
     _length: int
     _data: Optional[str]

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/src/robotframework_pabot.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: robotframework-pabot
-Version: 3.1.0
+Version: 4.0.4
 Summary: Parallel test runner for Robot Framework
 Home-page: https://pabot.org
 Download-URL: https://pypi.python.org/pypi/robotframework-pabot

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/tests/test_depends.py

@@ -96,7 +96,10 @@ class DependsTest(unittest.TestCase):
         --test Test.The Test S1Test 08
         """,
         )
-        self.assertIn(b"circular or unmet dependencies", stderr)
+        self.assertIn(
+            b"Invalid test configuration: Circular or unmet dependencies detected between test suites",
+            stdout,
+        )
 
     def test_unmet_dependency(self):
         stdout, stderr = self._run_tests_with(
@@ -107,7 +110,10 @@ class DependsTest(unittest.TestCase):
         --test Test.The Test S1Test 08
         """,
         )
-        self.assertIn(b"circular or unmet dependencies", stderr)
+        self.assertIn(
+            b"Invalid test configuration: Circular or unmet dependencies detected between test suites. Please check your #DEPENDS definitions.",
+            stdout,
+        )
 
     def test_same_reference(self):
         stdout, stderr = self._run_tests_with(
@@ -118,7 +124,10 @@ class DependsTest(unittest.TestCase):
         --test Test.The Test S1Test 08
         """,
         )
-        self.assertIn(b"circular or unmet dependencies", stderr)
+        self.assertIn(
+            b"Invalid test configuration: Circular or unmet dependencies detected between test suites. Please check your #DEPENDS definitions.",
+            stdout,
+        )
 
     def test_wait(self):
         stdout, stderr = self._run_tests_with(
@@ -130,4 +139,7 @@ class DependsTest(unittest.TestCase):
         --test Test.The Test S1Test 08
         """,
         )
-        self.assertIn(b"circular or unmet dependencies", stderr)
+        self.assertIn(
+            b"Invalid test configuration: Circular or unmet dependencies detected between test suites",
+            stdout,
+        )

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/tests/test_pabot.py

@@ -86,7 +86,6 @@ class PabotTests(unittest.TestCase):
                 "--resourcefile",
                 "resourcefile.ini",
                 "--testlevelsplit",
-                "--pabotlib",
                 "--pabotlibhost",
                 "123.123.233.123",
                 "--pabotlibport",
@@ -105,7 +104,7 @@ class PabotTests(unittest.TestCase):
         self.assertEqual(pabot_args["command"], ["my_own_command.sh"])
         self.assertEqual(pabot_args["processes"], 12)
         self.assertEqual(pabot_args["resourcefile"], "resourcefile.ini")
-        self.assertEqual(pabot_args["pabotlib"], True)
+        self.assertEqual(pabot_args["pabotlib"], False)
         self.assertEqual(pabot_args["pabotlibhost"], "123.123.233.123")
         self.assertEqual(pabot_args["pabotlibport"], 4562)
         self.assertEqual(pabot_args["suitesfrom"], "some.xml")
@@ -1226,16 +1225,17 @@ class PabotTests(unittest.TestCase):
             file_path = os.path.join(_opts["outputdir"], f)
             self.assertTrue(os.path.isfile(file_path), "file not copied: {}".format(f))
             os.remove(file_path)  # clean up
-    
+
     def test_merge_one_run_with_and_without_legacyoutput(self):
         dtemp = tempfile.mkdtemp()
         # Create the same directory structure as pabot
-        test_outputs = os.path.join(dtemp, 'outputs')
+        test_outputs = os.path.join(dtemp, "outputs")
         os.makedirs(test_outputs)
-        test_output = os.path.join(test_outputs, 'output.xml')
+        test_output = os.path.join(test_outputs, "output.xml")
         # Create a minimal but valid output.xml
-        with open(test_output, 'w') as f:
-            f.write("""<?xml version="1.0" encoding="UTF-8"?>
+        with open(test_output, "w") as f:
+            f.write(
+                """<?xml version="1.0" encoding="UTF-8"?>
 <robot generator="Rebot 7.1.1 (Python 3.11.7 on darwin)" generated="20241130 11:19:45.235" rpa="false" schemaversion="4">
 <suite id="s1" name="Suites">
 <suite id="s1-s1" name="Test" source="/Users/mkorpela/workspace/pabot/test.robot">
@@ -1281,16 +1281,17 @@ class PabotTests(unittest.TestCase):
 <msg timestamp="20241130 11:19:44.910" level="ERROR">Error in file '/Users/mkorpela/workspace/pabot/test.robot' on line 2: Library 'Easter' expected 0 arguments, got 1.</msg>
 <msg timestamp="20241130 11:19:44.913" level="ERROR">Error in file '/Users/mkorpela/workspace/pabot/test.robot' on line 2: Library 'Easter' expected 0 arguments, got 1.</msg>
 </errors>
-</robot>""")
-            
-        self._options['outputdir'] = dtemp
+</robot>"""
+            )
+
+        self._options["outputdir"] = dtemp
         if ROBOT_VERSION >= "7.0":
-            self._options['legacyoutput'] = True
+            self._options["legacyoutput"] = True
         try:
             output = pabot._merge_one_run(
                 outs_dir=dtemp,
                 options=self._options,
-                tests_root_name='Test',  # Should match suite name in XML
+                tests_root_name="Test",  # Should match suite name in XML
                 stats={
                     "total": 0,
                     "passed": 0,
@@ -1298,9 +1299,12 @@ class PabotTests(unittest.TestCase):
                     "skipped": 0,
                 },
                 copied_artifacts=[],
-                outputfile='merged_output.xml')  # Use different name to avoid confusion
-            self.assertTrue(output, "merge_one_run returned empty string")  # Verify we got output path
-            with open(output, 'r') as f:
+                outputfile="merged_output.xml",
+            )  # Use different name to avoid confusion
+            self.assertTrue(
+                output, "merge_one_run returned empty string"
+            )  # Verify we got output path
+            with open(output, "r") as f:
                 content = f.read()
                 if ROBOT_VERSION >= "6.1":
                     self.assertIn('schemaversion="4"', content)
@@ -1309,11 +1313,11 @@ class PabotTests(unittest.TestCase):
                 elif ROBOT_VERSION >= "4.0":
                     self.assertIn('schemaversion="2"', content)
             if ROBOT_VERSION >= "7.0":
-                del self._options['legacyoutput']
+                del self._options["legacyoutput"]
                 output = pabot._merge_one_run(
                     outs_dir=dtemp,
                     options=self._options,
-                    tests_root_name='Test',  # Should match suite name in XML
+                    tests_root_name="Test",  # Should match suite name in XML
                     stats={
                         "total": 0,
                         "passed": 0,
@@ -1321,14 +1325,92 @@ class PabotTests(unittest.TestCase):
                         "skipped": 0,
                     },
                     copied_artifacts=[],
-                    outputfile='merged_2_output.xml')  # Use different name to avoid confusion
-                self.assertTrue(output, "merge_one_run returned empty string")  # Verify we got output path
-                with open(output, 'r') as f:
+                    outputfile="merged_2_output.xml",
+                )  # Use different name to avoid confusion
+                self.assertTrue(
+                    output, "merge_one_run returned empty string"
+                )  # Verify we got output path
+                with open(output, "r") as f:
                     content = f.read()
                 self.assertIn('schemaversion="5"', content)
                 self.assertNotIn('schemaversion="4"', content)
         finally:
-            shutil.rmtree(dtemp) 
+            shutil.rmtree(dtemp)
+
+    def test_parse_args_mixed_order(self):
+        (
+            options,
+            datasources,
+            pabot_args,
+            options_for_subprocesses,
+        ) = arguments.parse_args(
+            [
+                "--exitonfailure",
+                "--processes",
+                "12",
+                "--outputdir",
+                "mydir",
+                "--verbose",
+                "--pabotlib",
+                "suite",
+            ]
+        )
+        self.assertEqual(pabot_args["processes"], 12)
+        self.assertEqual(pabot_args["verbose"], True)
+        self.assertEqual(pabot_args["pabotlib"], True)
+        self.assertEqual(options["outputdir"], "mydir")
+        self.assertEqual(options["exitonfailure"], True)
+        self.assertEqual(datasources, ["suite"])
+
+    def test_parse_args_error_handling(self):
+        with self.assertRaises(DataError) as cm:
+            arguments.parse_args(["--processes"])
+        self.assertIn("requires a value", str(cm.exception))
+
+        with self.assertRaises(DataError) as cm:
+            arguments.parse_args(["--processes", "invalid"])
+        self.assertIn("Invalid value for --processes", str(cm.exception))
+
+        with self.assertRaises(DataError) as cm:
+            arguments.parse_args(["--command", "echo", "hello"])
+        self.assertIn("requires matching --end-command", str(cm.exception))
+
+    def test_parse_args_command_with_pabot_args(self):
+        options, datasources, pabot_args, _ = arguments.parse_args(
+            [
+                "--command",
+                "script.sh",
+                "--processes",
+                "5",
+                "--end-command",
+                "--verbose",
+                "suite",
+            ]
+        )
+        self.assertEqual(pabot_args["command"], ["script.sh", "--processes", "5"])
+        self.assertEqual(pabot_args["verbose"], True)
+
+    def test_pabotlib_defaults_to_enabled(self):
+        options, _, pabot_args, _ = arguments.parse_args(["suite"])
+        self.assertTrue(pabot_args["pabotlib"])
+        self.assertFalse("no_pabotlib" in pabot_args)  # Ensure internal flag not leaked
+
+    def test_no_pabotlib_disables_pabotlib(self):
+        options, _, pabot_args, _ = arguments.parse_args(["--no-pabotlib", "suite"])
+        self.assertFalse(pabot_args["pabotlib"])
+        self.assertFalse("no_pabotlib" in pabot_args)  # Ensure internal flag not leaked
+
+    def test_pabotlib_option_shows_warning(self):
+        options, _, pabot_args, _ = arguments.parse_args(["--pabotlib", "suite"])
+        self.assertTrue(pabot_args["pabotlib"])
+        self.assertFalse("no_pabotlib" in pabot_args)  # Ensure internal flag not leaked
+
+    def test_conflicting_pabotlib_options_raise_error(self):
+        with self.assertRaises(DataError) as context:
+            arguments.parse_args(["--pabotlib", "--no-pabotlib", "suite"])
+        self.assertIn(
+            "Cannot use both --pabotlib and --no-pabotlib", str(context.exception)
+        )
 
 
 if __name__ == "__main__":

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/tests/test_pabotlib.py

@@ -28,7 +28,7 @@ class PabotLibTests(unittest.TestCase):
     def test_shared_library_with_args(self):
         try:
             self._create_ctx()  # Set up Robot Framework context
-            lib = SharedLibrary("mylib", ["2"]) 
+            lib = SharedLibrary("mylib", ["2"])
             self.assertIsNotNone(lib)
             lib._remote = None
             lib._lib.run_keyword("mykeyword", ["arg"], {})

{robotframework_pabot-3.1.0 → robotframework_pabot-4.0.4}/tests/test_testlevelsplit_include.py

@@ -7,14 +7,14 @@ import shutil
 import subprocess
 
 
-
 class PabotPassJsonUsingVariableOptionTests(unittest.TestCase):
     def setUp(self):
         self.tmpdir = tempfile.mkdtemp()
         file_path = f"{self.tmpdir}/test.robot"
         with open(file_path, "w") as robot_file:
             robot_file.write(
-                textwrap.dedent("""
+                textwrap.dedent(
+                    """
 *** Test Cases ***
 Testing 1
    [Tags]  tag
@@ -23,8 +23,10 @@ Testing 1
 Testing 2
    [Tags]  tag
    Log  world
-"""))
-        
+"""
+                )
+            )
+
         process = subprocess.Popen(
             [
                 sys.executable,